https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43437

David Nind <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
             Status|Needs Signoff               |Failed QA

--- Comment #3 from David Nind <[email protected]> ---
I've tested, but have not signed off as I think there are too many issues here.

On the plus side, it fixes the problem with the 500 error when the layout
contains bibliographic data.

However, I've Failed QA because:
1. The instructions in the test plan don't match what is in main 
   (the label printing options have been under Cataloging > Tools for 
   a while now, but I'm not sure for how long).
2. There are JavaScript console errors at various steps (although they exist 
   before the patch) - so maybe another bug is required for these.
3. Batch export: I can only seem to get the barcode to print, even though the
   layout includes bibliograhic data (my testing steps 10 and 11).
4. Unrelated, and I'll comment on bug 42627, but the change to the 
   "Number of barcodes to print" for the "Print barcode range" is misleading.
   It is actually only printing any barcodes that exist from the starting
barcode
   entered plus the number entered in the sequence, not the next X barcodes
   (my testing step 9). Example, if 500 is entered, and the starting barcode is
   3999900000001, then it is only going to print the barcodes that exist up
until
   3999900000500 - not the next 500 barcodes after 3999900000001.

Testing notes (using KTD on main):
1. Steps 1 and 2: Cataloging > Tools > Label creator > New > Label layout
2. Step 3:
   - Layout name: BZ43437
   - Select "List fields" (title us already included)
   - Remove title field and add 245a
3. Step 6: Cataloging > Tools > Label creator > Print barcode range
4. Step 7: Range for barcodes in KTD sample data
   - From: 3999900000001
   - To: 3999900000500
5. On label export page, I get this console error
   (/cgi-bin/koha/labels/label-print.pl?from=3999900000001&to=3999900000500):
    Uncaught TypeError: Cannot read properties of null (reading 'parentNode')
        at label-print.pl?from=3999900000001&to=3999900000500:1049:27
6. Click "Export"
7. I get this console error on the page with the download links 
   (/cgi-bin/koha/labels/label-print.pl):
    Uncaught TypeError: Cannot read properties of null (reading 'parentNode')
        at label-print.pl:1011:27
8. When clicking Download, get the "Can't call method "field" on an undefined 
   value at /kohadevbox/koha/C4/Labels/Label.pm line 180" error trace.
9. Step 13: After the patch (and dependent bug):
   - For the barcode range I selected:
     . Starting barcode: 3999900000001
     . Number of barcodes to print: 500
   - Still get the same console errors as above
   - Only get 7 barcodes printed:
        3999900000001
        3999900000002
        3999900000017
        3999900000018
        3999900000019
        3999900000020
        3999900000021
   - After this barcode (21) the next in the sequence using item search is
     39999000000238
   - If I use this setting I get 26 barcodes (39999000000238 to
39999000000737): 
     . Starting barcode: 39999000000238
     . Number of barcodes to print: 500 
     . The PDF includes barcodes 39999000000238 to 39999000000733
   ==> So the label "Number of barcodes to print" for the "Print barcode range"
        is misleading.
        In this case it is actually only printing any barcodes that exist 
        after the starting barcode + 500 in the sequence (up 39999000000738)
10. Step 16: 
    - If you select a barcode from a batch to export, the "Cancel" button on 
      the label export page does not work - that is clicking "Cancel" does
      nothing (/cgi-bin/koha/labels/label-print.pl?batch_id=1&1#)
    - When I choose a label layout that should include the title, and so on,
      it just prints the barcode.
11. Step 17: Get same results as test plan 16 - only prints barcode and not 
    the bibliographic data.
12. Step 18: No 500 error
13. Tests: these pass
14. Step 20: Not sure if this is asking the tester to check the actual test
    code and make sure it is correct, or to use the -v option:

prove -v t/db_dependent/Labels/t_Label.t
t/db_dependent/Labels/t_Label.t .. 
1..9
ok 1 - use C4::Labels::Label;
ok 2 - Barcode is resolved to the correct itemnumber for label generation
ok 3 - Nonexistent barcode does not return an itemnumber
ok 4 - Label Text Value defined.
ok 5
ok 6
ok 7
ok 8
ok 9 - no warnings
ok
All tests successful.
Files=1, Tests=9,  2 wallclock secs ( 0.02 usr  0.00 sys +  1.76 cusr  0.25
csys =  2.03 CPU)
Result: PASS

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to