developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#911 closed bug (fixed)

record modifications in updatebefore and insertbefore dao events not taken in account

Reported by: pulsation Owned by:
Priority: normal Milestone: Jelix 1.0.9
Component: jelix:dao Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

DAO record modifications triggered by updatebefore and insertbefore events are not taken in account when data is inserted or updated in database.

Calling these events before the query is generated fixes the problem.

Attachments (3)

dao_record_events.diff (2.4 KB) - added by pulsation 12 years ago.
dao_record_events_1.1.x.diff (2.2 KB) - added by pulsation 12 years ago.
Patch for the 1.1.x branch
dao_record_events_1.0.x.diff (2.4 KB) - added by pulsation 12 years ago.
Patch for the 1.0.x branch. Not well tested as we don't use 1.0.x here.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by pulsation

comment:1 Changed 12 years ago by bballizlife

Need to be reviewed ?

comment:2 Changed 12 years ago by laurentj

  • Milestone changed from Jelix 1.1.3 to Jelix 1.0.9
  • review set to review+

Ok for me. However this patch could be apply only on the trunk since some things have been changed in this file in the trunk since the release of 1.1.x. You should provide an other patch for 1.1.x and 1.0.x branches.

Changed 12 years ago by pulsation

Patch for the 1.1.x branch

Changed 12 years ago by pulsation

Patch for the 1.0.x branch. Not well tested as we don't use 1.0.x here.

comment:3 Changed 12 years ago by laurentj

ok pour moi. à intégrer dans le svn.

comment:4 Changed 12 years ago by bballizlife

  • Resolution set to fixed
  • Status changed from new to closed

commited in trunk in r1422 commited in branch 1.0.x in r1423 commited in branch 1.1.x in r1424

Thanks pulsation !

Next could you please provide patch from the root of the working copy ? For example here, from trunk/ and branches/1.0.x/ and branches/1.1.x/. Else never mind but that ease to work of the commiter.

Thanks again.

Note: See TracTickets for help on using tickets.