August 2017 - Koha-bugs - lists.koha-community.org

[Bug 15239] Add multiple items fail on barcode
by bugzilla-daemon＠bugs.koha-community.org 23 Aug '17

23 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15239 Ruben Norori <kachomanic(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15239] Add multiple items fail on barcode
by bugzilla-daemon＠bugs.koha-community.org 23 Aug '17

23 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15239 --- Comment #9 from Ruben Norori <kachomanic(a)gmail.com> --- I have the definitive solution: First we must add two new procedures, functions, methods, which consult in the database the registry homebranch. #Original function sub get_item_from_barcode { my ($barcode)=@_; my $dbh=C4::Context->dbh; my $result; my $rq=$dbh->prepare("SELECT itemnumber from items where items.barcode=?"); $rq->execute($barcode); ($result)=$rq->fetchrow; return($result); } #New function sub get_home_from_barcode { my ($barcode)=@_; my $dbh=C4::Context->dbh; my $result; my $rq=$dbh->prepare("SELECT homebranch from items where items.barcode=?"); $rq->execute($barcode); ($result)=$rq->fetchrow; return($result); } #New function sub get_home_from_item { my ($barcode)=@_; my $dbh=C4::Context->dbh; my $result; my $rq=$dbh->prepare("SELECT homebranch from items where items.itemnumber=?"); $rq->execute($barcode); ($result)=$rq->fetchrow; return($result); } Then call the functions and save the result in a variable that we concate at the final result, also add an "if" to use one function in the first iteration and the other function executes it in the "else". for (my $i = 0; $i < $number_of_copies;) { # If there is a barcode if ($barcodevalue) { # Getting a new barcode (if it is not the first iteration or the barcode we tried already exists) # For execute only in the first iteration if($i<1){ # Call new function my $newv = get_home_from_item($exist_itemnumber); $barcodevalue = $newv.$barcodeobj->next_value($oldbarcode) if ($i > 0 || $exist_itemnumber); }else{ # Call new function my $exin = get_home_from_barcode($barcodevalue); $barcodevalue = $exin.$barcodeobj->next_value($oldbarcode) if ($i > 0 || $exist_itemnumber); } # Putting it into the record if ($barcodevalue) { $record->field($tagfield)->update($tagsubfield => $barcodevalue); } -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11897] Stock Rotation for Koha
by bugzilla-daemon＠bugs.koha-community.org 23 Aug '17

23 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11897 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia(a)bugs.koha-communit |tomascohen(a)gmail.com |y.org | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 11897] Stock Rotation for Koha
by bugzilla-daemon＠bugs.koha-community.org 23 Aug '17

23 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11897 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |tomascohen(a)gmail.com --- Comment #150 from Tomás Cohen Arazi <tomascohen(a)gmail.com> --- Hi guys, I've been looking at this patchset without trying it yet. Everything looks as expected :-D The only thing I would like to mention is that the auth chain added a check for maintenance mode (usually on upgrades) that requires some more error codes to be added to the spec. Can you please take a look at (say) patrons.json from bug 18137? It's pretty straightforward. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14967] update_dbix_class_files.pl should have a default DB name
by bugzilla-daemon＠bugs.koha-community.org 22 Aug '17

22 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14967 Tomás Cohen Arazi <tomascohen(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WONTFIX Status|NEW |RESOLVED -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14385] Extend OpacHiddenItems to allow specifying exempt borrower categories
by bugzilla-daemon＠bugs.koha-community.org 22 Aug '17

22 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14385 --- Comment #108 from M. Tompsett <mtompset(a)hotmail.com> --- Created attachment 66361 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66361&action=ed… Bug 14385: Improve C4::Record::marcrecord2csv test cases -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 22 Aug '17

22 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #150 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66359 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66359&action=ed… Bug 12598: Re-add bug 18555: Create patron list from patron import -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 22 Aug '17

22 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #149 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66358 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66358&action=ed… Bug 12598 Allow matched patron to be updated When a test was introduced to not add duplicate userids it overlooked that if you were matching on userid you do not want the update to be rejected. Add a test to overcome this logical flaw if updating by matching userid -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 22 Aug '17

22 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #148 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66357&action=ed… Bug 12598: Re-add some missing stuffs This patch will have to be squashed with previous ones, but for readability I prefer to let it alone for now. It reintroduces 2 missing changes: commit fd123903651435c56ca14d88897e9ce4d7a38bb5 Bug 15010: Import patron tool creates 'duplicate' restrictions ( debarments ) commit 011d8e9855ffeb1779f5c1aadc9f4367c298cee7 Bug 12543 - Add userid as matchpoint for "Import patrons" tool Signed-off-by: Colin Campbell <colin.campbell(a)ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12598] New misc/import_borrowers.pl command line tool
by bugzilla-daemon＠bugs.koha-community.org 22 Aug '17

22 Aug '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12598 --- Comment #147 from Jonathan Druart <jonathan.druart(a)bugs.koha-community.org> --- Created attachment 66356 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66356&action=ed… Bug 12598: Fix conflict with 17829 - GetMember Signed-off-by: Colin Campbell <colin.campbell(a)ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.

1 0