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

#772 closed bug (fixed)

getModifiedControls don't work for checkbox

Reported by: bastnic Owned by:
Priority: normal Milestone: jelix 1.1
Component: jelix:forms Version: 1.1 beta 1
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

We should use array_diff_assoc.

Attachments (1)

jforms.modifiedcontrols.html_cli.php​ (4.1 KB) - added by bastnic 12 years ago.
TU which fails with array_diff and work with assoc

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by bastnic

TU which fails with array_diff and work with assoc

comment:1 Changed 12 years ago by laurentj

The final test file shouldn't have this getModifiedControlsWithAssoc. It just should show that the current trunk fails on this tests, and if we change the content of getModifiedControls, it is ok.

If you could add some tests with controls which use datasources (listbox, checkboxes...), it would be better.

And then It will be ok to use array_diff_assoc :-)

comment:2 Changed 12 years ago by laurentj

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

Landed in the trunk + 1.1.x branch, with some modifications I talked about. r1238

Note: See TracTickets for help on using tickets.