Bug 11216: fix bug where editing patron creates indefinite manual restriction
commit165e8750b8e6b57050f11884e58ce85c2888f243
authorDobrica Pavlinusic <dpavlin@rot13.org>
Fri, 8 Nov 2013 13:50:38 +0000 (8 14:50 +0100)
committerGalen Charlton <gmc@esilibrary.com>
Thu, 14 Nov 2013 15:39:39 +0000 (14 15:39 +0000)
treeed54f3aadb0ae3e168494457951ed873c5a65fb2
parenta1d4af3d73bacd552fa7411e29b4963c9350ea20
Bug 11216: fix bug where editing patron creates indefinite manual restriction

Editing partron data results in unintended adding of indefinite manual
restriction to it. Reason for it is hidden field add_debarment which
is wrongly initialized to 1 instead of 0. Also JavaScript code seems
to want to toggle checkbox, and since this field is hidden correct
approach would be to change its value to 1.

Test scenario:

1. edit patron data
2. verify that debarrment is wrongly added
3. remove debarrment
3. apply this patch
4. edit patron again
5. verify that debarrment wasn't added
6. intentionally add debarrment and verify that it's saved
7. add debarment, enter something in comment but press cancel
   and verify that debarment isn't added

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This fixes this blocker, however, with or without the patch, the date
is not saved correctly. I will file a new bug for that

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt