Ticket #10 (closed defect: fixed)

Opened 2 years ago

Last modified 1 month ago

Reactor should support checking records exist

Reported by: tom.chiverton@… Owned by: dhughes
Type: defect Priority: lowest
Milestone: Component: Reactor - Gateway
Version: Severity: trivial
Keywords: Cc:

Description

(code is based on SVN from some time last week).
As per mailing list post from Doug:
"I'm not suggesting adding the call to exists() in the general save() method
though (that would generate a pointless extra DB call as you say), just
always having it in the DAO and exposing it via the Record as a matter of
course for people who want to use it."

Modifications I've made here follow, but if you ask for exists(primarKeyField=<defaultValueOfField) it always says yes, even if there is no such field. The fix is porbably not to load() before calling exists().

Attachments

dao.project.xsl (14.4 kB) - added by tom.chiverton@… 2 years ago.
record.project.xsl (14.6 kB) - added by tom.chiverton@… 2 years ago.

Change History

Changed 2 years ago by tom.chiverton@…

Changed 2 years ago by tom.chiverton@…

Changed 2 years ago by tom.chiverton@…

This also makes the API consistant betwen auto generating PK Records and non-auto generating

Changed 2 years ago by tom.chiverton@…

Why is this a documentation issue ?

Changed 2 years ago by dhughes

  • status changed from new to closed
  • resolution set to fixed
  • component changed from Documentation to Reactor - General

This was fixed in changeset 295.

Changed 2 years ago by dhughes

  • milestone changed from Reactor 2.0 Alpha to Beta Candidate 2 (BC2)

Changed 1 month ago by anonymous

  • milestone deleted

Milestone Beta Candidate 2 (BC2) deleted

Note: See TracTickets for help on using tickets.