[Bug 6079] New: Single parentheses
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6079 Bug #: 6079 Summary: Single parentheses Classification: Unclassified Change sponsored?: --- Product: Koha Version: rel_3_4 Platform: All OS/Version: All Status: NEW Severity: trivial Priority: P5 Component: Acquisitions AssignedTo: henridamien@koha-fr.org ReportedBy: mark.gavillet@ptfs-europe.com QAContact: koha-bugs@lists.koha-community.org -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6079 Mark Gavillet <mark.gavillet@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Single parentheses |Single parentheses in | |currency causes order to | |fail when updating --- Comment #1 from Mark Gavillet <mark.gavillet@ptfs-europe.com> 2011-04-06 08:47:14 UTC --- To reproduce add a currency with a single parenthesis - either left or right - and try to update an order. Adding both left and right parentheses to the currency will not cause the order update to fail. Originally discovered because the length of field truncated to leave a single left parenthesis. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6079 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Version|3.4 |master -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6079 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@biblibre.co | |m --- Comment #2 from Jonathan Druart <jonathan.druart@biblibre.com> --- Should be fixed with the following patch: diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm index 0bb34c6..320169a 100644 --- a/C4/Acquisition.pm +++ b/C4/Acquisition.pm @@ -1269,7 +1269,7 @@ sub ModOrder { foreach my $orderinfokey (grep(!/ordernumber/, keys %$orderinfo)){ # ... and skip hash entries that are not in the aqorders table # FIXME : probably not the best way to do it (would be better to have a correct hash) - next unless grep(/^$orderinfokey$/, @$colnames); + next unless grep(/^\Q$orderinfokey\E$/, @$colnames); $query .= "$orderinfokey=?, "; push(@params, $orderinfo->{$orderinfokey}); } But not sure of the consequences... -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org