September 2014 - Koha-bugs - lists.koha-community.org

[Bug 12899] New: Row grouping in checkouts table is alphabetical and layout depends on words chosen in translation
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12899 Bug ID: 12899 Summary: Row grouping in checkouts table is alphabetical and layout depends on words chosen in translation Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: viktor.sarge(a)regionhalland.se QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com In checkouts.js the checkouts table is being built and the rows are grouped ascending alphabetically into "Today's checkouts" and "Previous checkouts". The problem is that in translations where the words are different sorting "asc" can change the layout so that previous checkouts are on top and hides todays. Very impractical. .rowGrouping is the culprit. Thanks to Owen and Kyle for troubleshooting. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 18

[Bug 12729] New: Overdue items won't show as overdue in red in circulation
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12729 Bug ID: 12729 Summary: Overdue items won't show as overdue in red in circulation Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Created attachment 30577 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30577&action=edit screenshot of list of checkouts In master overdue items won't show in red when overdue. At the top of the screen, the red message indicating the overdues is still there, but in the checkout's table all items show the same. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 18

[Bug 12883] New: Holds tab on checkout screen never stops processing
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12883 Bug ID: 12883 Summary: Holds tab on checkout screen never stops processing Change sponsored?: --- Product: Koha Version: 3.16 Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Circulation Assignee: gmc(a)esilibrary.com Reporter: magnus(a)enger.priv.no QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Depends on: 11703 After upgrading to 3.16.3, a customer noticed that on the check out page (/cgi-bin/koha/circ/circulation.pl), the "x Checkout(s)" tab was working fine, but the "x Hold(s)" tab just showed the "Processing" message, and never got anywhere. This is caused by an AJAX call to /cgi-bin/koha/svc/holds.pl, which should be a call to /cgi-bin/koha/svc/holds (without the .pl). It looks like the problem was introduced by on bug 11703, attachment 29493, which renamed svc/holds.pl to svc/holds, without fixing the call to this script in koha-tmpl/intranet-tmpl/prog/en/js/holds.js This has since been fixed for master by this commit: http://git.koha-community.org/gitweb/?p=koha.git;a=commit;h=9657a2c6dc92516… So as far as I can tell, the solution to this problem is to pick 9657a2c6dc92516963ffa960b7d561cdcc2dc5e8 for the 3.16.x branch. -- You are receiving this mail because: You are watching all bug changes.

1 7

[Bug 11633] New: Cannot use the override function AllowFineOverride on the Staff client because this stops the pay function working on the SelfCheck (SC) machines
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11633 Bug ID: 11633 Summary: Cannot use the override function AllowFineOverride on the Staff client because this stops the pay function working on the SelfCheck (SC) machines Change sponsored?: --- Product: Koha Version: 3.14 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: SIP2 Assignee: koha-bugs(a)lists.koha-community.org Reporter: jonathan.field(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org CC: colin.campbell(a)ptfs-europe.com You cannot use the override function (AllowFineOverride) on “Fines outstanding” on the Staff client because this stops the pay function working on the SelfCheck (SC) machines. The SIP2 devices share this system preference. Test Plan ========= 1) Set System Preference AllowFineOverride to "Allow". 2) On SIP2 device (e.g. 3M SelfCheck) try paying a fine. We see:- Borrower XXXXXXXXX had existing fines of 50p (2 x 25p). Attempted to borrow book YYYYYYYYYYYYY. Chose 'Pay Fines Later' and issue was allowed even with an outstanding (payable) fine. Tried to pay fines SelfCheck Machine asked for correct amount took money but then returned money stating it could not charge account and to enquire at desk. In the SIP2 response, character position 1 & 2 on the 64 response need to be set to Y for the SIP2 device to properly handle the block. 64YY Y 00020110614 ................. In Koha these are only set in two scenarios 1) If the user has expired and 2) if the user is debarred Needs a change in the Members logic the status in the ILS::Patron is setup on the basis of the return from GetMemberDetails it does not set the noissues flag if staff can override (AllowFineOverride option). In the staff client the noissuescharge is applied as part of the checkout transaction in 'CanBookBeIssued'. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 18

[Bug 11672] New: Untranslatable report areas in step 1 of Guided Reports
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11672 Bug ID: 11672 Summary: Untranslatable report areas in step 1 of Guided Reports Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: I18N/L10N Assignee: koha-bugs(a)lists.koha-community.org Reporter: pasi.kallinen(a)pttk.fi QA Contact: testopia(a)bugs.koha-community.org CC: frederic(a)tamil.fr The guided reports UI shows a drop-down box with "Circulation", "Catalogue", etc. These are hard-coded in C4/Reports/Guided.pm (the @REPORT_AREA hash) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 22

[Bug 643] Allow override of 'debarred' status
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=643 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #31564|0 |1 is obsolete| | --- Comment #16 from M. Tompsett <mtompset(a)hotmail.com> --- Created attachment 31837 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31837&action=edit Bug 643: QA Followup - server-side verification of permissions Only allow $force_allow=1 if the logged in user has permissions. TEST PLAN --------- Attempt to intentionally override the checkout by passing an appropriately handcrafted URL. -- Regardless of the force_allow value, it should be not allowed for those lacking the force_checkout permission. NOTE: I didn't test this. I figured Marc Veron could do that. :) (Sorry, couldn't easily get git bz to work with the accent) -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 643] Allow override of 'debarred' status
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=643 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 643] Allow override of 'debarred' status
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=643 --- Comment #15 from Jonathan Druart <jonathan.druart(a)biblibre.com> --- You can use C4::Auth::haspermission to check if the user has this permission. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2889] templates should use [% IF ( loop.odd ) %]
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2889 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #30790|0 |1 is obsolete| | --- Comment #13 from Owen Leonard <oleonard(a)myacpl.org> --- Created attachment 31834 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=31834&action=edit Bug 2889 - templates should use [% IF ( loop.odd ) %] - Administration Some scripts pass a template variable to facilitate an alternate table row class for styling. Other use the 'IF (loop.odd)' construction. I think we're at the point where the CSS3 :nth-child() selector is widely-supported enough that we can do without template-based solutions: http://caniuse.com/#feat=css-sel3 This patch adds such a selector to the staff client CSS and removes the corresponding template markup from Administration pages. The last in this series of patches will remove the redundant CSS. Also in this patch: a few minor markup corrections. To test, apply the patch and clear your browser cache if necessary. View the following pages and confirm that alternate table row highlighting works as before: - Administration -> Budgets - Acquisitions -> Vendor -> Contracts - Administration -> Budgets -> Budget -> Planning - Administration -> Authority types - Administration -> Authority types -> MARC structure - Administration -> Authority types -> MARC structure -> subfields - Administration -> MARC bibliographic framework - Administration -> MARC bibliographic framework -> MARC structure - Administration -> MARC bibliographic framework -> MARC structure -> Subfields - Administration -> Libraries and groups - Administration -> Cities and towns - Administration -> Classification sources - Administration -> Circulation and fines rules - Administration -> Currencies and exchange rates - Administration -> Item types - Administration -> Koha to MARC mapping - Administration -> System preferences -> Local use - Administration -> Z39.50 client targets -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 2889] templates should use [% IF ( loop.odd ) %]
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '14

23 Sep '14

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=2889 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0