http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10625
--- Comment #15 from M. Tompsett <[email protected]> --- Comment on attachment 37848 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=37848 [Signed-off] Bug 10625: Inventory/Stocktaking tool cannot handle windows file uploads Review of attachment 37848: --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=10625&attachment=37848) ----------------------------------------------------------------- ::: tools/inventory.pl @@ +174,5 @@ > + $barcode =~ s/\r/\n/g; > + $barcode =~ s/\n\n/\n/g; > + $barcode =~ s/\n$//; > + $barcode =~ s/^\n//; > + my @data = split(/\n/,$barcode); Because then I couldn't split the barcodes. I agree the logic might be more optimizable, but the idea is that if $barcode = "foo\nbar"; it needs to split into two barcodes: foo and bar. And given that some OS's may use \r and \n and \r\n in ways different than how $barcode=<$uploadbarcodes> works, I wanted to be certain that all middle combinations get split, and that there are no beginning or end strangeness. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
