[Bug 4065] New: Confusing behavior of patron extended attribute entry leads to error
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4065 Summary: Confusing behavior of patron extended attribute entry leads to error Product: Koha Version: HEAD Platform: All OS/Version: All Status: NEW Severity: normal Priority: P5 Component: Patrons AssignedTo: gmcharlt@gmail.com ReportedBy: oleonard@myacpl.org Estimated Hours: 0.0 Change sponsored?: --- Example sequence which might lead to the error: - A librarian loads the "Additional attributes and identifiers" edit screen. - They choose an attribute from the drop-down, decide it's not correct. - They click the 'Clear' link. - They choose a different attribute and click save. They will see an error: The following fields are wrong. Please fix them. * The attribute value /[attribute value] is already is use by another patron record. Where [attribute value] is their custom attribute value. When the 'Clear' link is clicked, a hidden input field is cleared: <input type="hidden" id="patron_attr_2_code" name="patron_attr_2_code" value="school" /> If you submit the form with an empty value for that input combined with non-empty data in the accompanying text field the error is triggered. -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4065 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> 2010-01-26 21:06:46 --- Created an attachment (id=1488) --> (http://bugs.koha.org/cgi-bin/bugzilla3/attachment.cgi?id=1488) Proposed fix The javascript triggered by the 'Clear' link cleared the "value" attribute of any inputs in the table row when the intention was to only clear inputs of type 'text.' This patch modifies the javascript to target only text inputs. -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4065 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|gmcharlt@gmail.com |oleonard@myacpl.org Priority|P5 |PATCH-Sent -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4065 Galen Charlton <gmcharlt@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gmcharlt@gmail.com Status|NEW |ASSIGNED --- Comment #2 from Galen Charlton <gmcharlt@gmail.com> 2010-01-26 21:57:58 --- Patch pushed to HEAD for inclusion in 3.2. -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4065 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED -- Configure bugmail: http://bugs.koha.org/cgi-bin/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
participants (1)
-
bugzilla-daemon@kohaorg.ec2.liblime.com