One instance in this file
Created attachment 30301 [details] [review] Bug 12673: Remove CGI::scrolling_list from C4::Items.pm This patch removes only instance in this file To test: 1. Apply the patch 2. (Esay way) Put on staff URL /cgi-bin/koha/services/itemrecorddisplay.pl?biblionumber=N&itemnumber=M 3. Code replaced corresponds to pulldowns, verify information is correct (check with another item edit view) PrepareItemrecordDisplay() in Items.pm is filled with chunks of HTML code that must be put on TT file. For now scrolling_list is the only removed.
Created attachment 30407 [details] [review] Bug 12673: Remove CGI::scrolling_list from C4::Items.pm This patch removes only instance in this file To test: 1. Apply the patch 2. (Esay way) Put on staff URL /cgi-bin/koha/services/itemrecorddisplay.pl?biblionumber=N&itemnumber=M 3. Code replaced corresponds to pulldowns, verify information is correct (check with another item edit view) PrepareItemrecordDisplay() in Items.pm is filled with chunks of HTML code that must be put on TT file. For now scrolling_list is the only removed.
Created attachment 30434 [details] [review] [SIGNED-OFF] Bug 12673: Remove CGI::scrolling_list from C4::Items.pm This patch removes only instance in this file To test: 1. Apply the patch 2. (Esay way) Put on staff URL /cgi-bin/koha/services/itemrecorddisplay.pl?biblionumber=N&itemnumber=M 3. Code replaced corresponds to pulldowns, verify information is correct (check with another item edit view) PrepareItemrecordDisplay() in Items.pm is filled with chunks of HTML code that must be put on TT file. For now scrolling_list is the only removed. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 30500 [details] [review] [ALREADY PUSHED] Bug 12673: Remove CGI::scrolling_list from C4::Items.pm This patch removes only instance in this file To test: 1. Apply the patch 2. (Esay way) Put on staff URL /cgi-bin/koha/services/itemrecorddisplay.pl?biblionumber=N&itemnumber=M 3. Code replaced corresponds to pulldowns, verify information is correct (check with another item edit view) PrepareItemrecordDisplay() in Items.pm is filled with chunks of HTML code that must be put on TT file. For now scrolling_list is the only removed. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Tested different item edit displays and the URL given above, no regressions found. Also passes all tests and QA script.
Patch pushed to master. Thanks Bernardo!
Created attachment 30765 [details] [review] Bug 12673: Followup - fix display in serials-edit This patch fix the display of items information following changes on previous patch. To test: 1. Go to serials 2. Add a subscription 3. Receive an issue 4. Change status to Arrived Item information display correct labels, but no pulldowns, that's the bug 5. Apply the patch 6. Reload page, change to Arrived again, this time pulldowns must be right
Created attachment 30769 [details] [review] [SIGNED-OFF] Bug 12673: Followup - fix display in serials-edit This patch fix the display of items information following changes on previous patch. To test: 1. Go to serials 2. Add a subscription 3. Receive an issue 4. Change status to Arrived Item information display correct labels, but no pulldowns, that's the bug 5. Apply the patch 6. Reload page, change to Arrived again, this time pulldowns must be right Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 30776 [details] [review] [SIGNED OFF] Bug 12673: Followup - fix display in serials-edit This patch fix the display of items information following changes on previous patch. To test: 1. Go to serials 2. Add a subscription 3. Receive an issue 4. Change status to Arrived Item information display correct labels, but no pulldowns, that's the bug 5. Apply the patch 6. Reload page, change to Arrived again, this time pulldowns must be right Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com> Works as expected.
Created attachment 30777 [details] [review] Bug 12673: (followup) fix Supplemental issue too The same fix needs to be used for the 'Supplemental issue' part. To test, use the same test steps used for the previous followup, but choose "Arrived" in the 'Supplemental issue' row. Regards To+ Sponsored-by: Universidad Nacional de Cordoba
Followups pushed to master. Thanks again, Bernardo!
This patch introduce a regression, I used git bisect, it pointed me: 4e278782f156a9d7452e6e9ddcefee9542b964fc is the first bad commit commit 4e278782f156a9d7452e6e9ddcefee9542b964fc Author: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Date: Tue Jul 29 15:00:56 2014 -0300 Bug 12673: Remove CGI::scrolling_list from C4::Items.pm This patch removes only instance in this file To reproduce, go on acqui/addorderiso2709.pl?import_batch_id=3&basketno=7&booksellerid=5 (change ids), tab 'Item information'. You will get: b - Propriétaire HASH(0xd220cb0) Required c - Dépositaire HASH(0xd44a3ec) Required e - Localisation HASH(0xd44ab30) Required The dropdown list are replaced by the hashref address.