Bug 16461 - Add barcode and item type in fine description for overdue
Summary: Add barcode and item type in fine description for overdue
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Bugs List
QA Contact: Alex Arnaud
URL:
Keywords:
: 16639 (view as bug list)
Depends on: 19066
Blocks:
  Show dependency treegraph
 
Reported: 2016-05-06 12:41 UTC by Sophie MEYNIEUX
Modified: 2023-09-09 20:10 UTC (History)
13 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 16461 - Add branch and barcode in fine description for overdue (8.48 KB, patch)
2016-05-11 14:58 UTC, Alex Arnaud
Details | Diff | Splinter Review
Bug 16461 - Add branch and barcode in fine description for overdue (8.37 KB, patch)
2016-06-04 13:05 UTC, Joy Nelson
Details | Diff | Splinter Review
Bug 16461 - Add branch and barcode in accountline table for overdues (15.05 KB, patch)
2017-03-13 16:21 UTC, Alex Arnaud
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Sophie MEYNIEUX 2016-05-06 12:41:38 UTC
Some libraries need to display item's branch code and barcode number in each fine line, set for overdues.
This way librarian avoid making a mistake and collect a fine for an item that belongs to another branch.
Comment 1 Alex Arnaud 2016-05-11 14:58:04 UTC Comment hidden (obsolete)
Comment 2 Theodoros Theodoropoulos 2016-05-12 11:06:54 UTC
Thank you Alex and Sophie, this would indeed come very handy to our Koha installation.

However, in the long term, (especially if this is to be used the whole community), I think it would be far better if separate columns were created in the accountlines table to hold (at least) the branchcode and/or the item barcode[1]. 

In the same context, an ideal situation would be if branchcode/barcode would be displayed in separate columns in the fines table (and not concatenated in the description).
This way description text would not be unnecessarily long and would not include hardcoded "Branch" and "Barcode" strings (which instead would be shown as translatable table headers in the fines table).

Just as food for thought :)

[1] if you don't want to keep barcodes both in items and accountlines table (also useful if barcode was changed in the meantime), it could be dynamically read during fines display (using itemnumber as key)
Comment 3 Theodoros Theodoropoulos 2016-05-12 11:38:40 UTC
Another advantage of the 'separate columns for Branch/Barcode in fines' approach would be that instead of branch CODE, the branch NAME could be displayed in the separate "Branch" column (as it happens with other item-related tables) which is more clear to staff librarians.

Of course, similarly due date could be a separate column in the fines table!

So to wrap the whole idea up: My proposal for the long run would be something like:

Fines & charges | Description | Barcode | Due Date | Branch | Payment note | Account type | Notify id | Level | Amount | Amount outstanding

(The table seems to hold too much information to fit into the screen, but description is no longer required to be that long!)


Having said that, the disadvantage of this approach is that not ALL fines are related to items, in which case Barcode and Due Date columns will be empty. If this is acceptable and you're willing to make the change in the database schema of the accountlines table, I think it would be fairly easy to implement and would worth the time and effort...
Comment 4 Theodoros Theodoropoulos 2016-06-02 10:50:44 UTC
Based on the case description, the proposed fix does the trick.

However, in order to keep track of the 'more advanced' solution that we would like to have, I created separate RFE:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16639
Comment 5 Joy Nelson 2016-06-04 13:05:43 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2016-06-07 16:27:18 UTC
QA comments:
1/ Please add a test plan
2/ This patch adds 3 strings which are not translatable "Due", "Barcode" and "Branch".
Note that "Branch" should be renamed with "Library".
Comment 7 Alex Arnaud 2017-03-13 16:21:18 UTC
Created attachment 61042 [details] [review]
Bug 16461 - Add branch and barcode in accountline table for overdues
Comment 8 David Bourgault 2017-11-03 20:54:21 UTC
Missing a test plan
Comment 9 Simon Pouchol 2017-11-07 16:05:06 UTC
Looks like this patch doesn't apply :
Applying: Bug 16461 - Add branch and barcode in accountline table for overdues
Using index info to reconstruct a base tree...
M	C4/Overdues.pm
M	installer/data/mysql/kohastructure.sql
M	koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
M	t/db_dependent/Circulation.t
Falling back to patching base and 3-way merge...
Auto-merging t/db_dependent/Circulation.t
CONFLICT (content): Merge conflict in t/db_dependent/Circulation.t
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/pay.tt
Auto-merging installer/data/mysql/kohastructure.sql
Auto-merging C4/Overdues.pm
error: Failed to merge in the changes.
Patch failed at 0001 Bug 16461 - Add branch and barcode in accountline table for overdues
Comment 10 Martin Renvoize 2018-12-17 09:38:03 UTC
This is a partial duplicate of bug 19066 (which adds branchcode to the accountlines table)
Comment 11 Martin Renvoize 2018-12-17 14:01:46 UTC
QA Comment: The SQL side of this is now basically covered by bug 19066 for branchcode and I would prefer not to add barcode from the items table alongside the existing item_id.. you can just use a simple join to get the barcode ;).

Please can this be updated to depend upon 19066 if we still want the included UI changes here.
Comment 12 Wendy Sharkey 2019-08-13 13:54:53 UTC
It would be useful if this table contained more information in general. We have a job identifying what type of item has a fine. Quite often we will have book, audio and movie of the same item. Rather than have to explore the system to get that information by including the item type and call number all the information would be collected together when you need it.
We are part of a consortium so knowing which library the item belongs to would help us to know if a payment needs forwarding to a different library.
Comment 13 Katrin Fischer 2020-01-11 12:38:08 UTC
Updated to reflect changes on master and comments. Setting back to NEW for now, as I think this will require a different/new approach than the existing patch.
Comment 14 Katrin Fischer 2020-01-11 13:31:26 UTC
*** Bug 16639 has been marked as a duplicate of this bug. ***
Comment 15 Katrin Fischer 2023-09-09 20:10:34 UTC
This is possible now using OVERDUE_FINE_DESC in 'notices and slips'.