Bug 40256 - System defined MANUAL debit type description is hard coded but the description can be changed
Summary: System defined MANUAL debit type description is hard coded but the descriptio...
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Fines and fees (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Andrew Fuerste-Henry
QA Contact: Testopia
URL:
Keywords:
Depends on: 40836
Blocks:
  Show dependency treegraph
 
Reported: 2025-06-26 19:36 UTC by Lucas Gass (lukeg)
Modified: 2025-09-26 18:35 UTC (History)
3 users (show)

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


Attachments
Bug 40256: Use debit_type.description for display of manual fees (3.52 KB, patch)
2025-09-17 20:54 UTC, Andrew Fuerste-Henry
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Gass (lukeg) 2025-06-26 19:36:05 UTC
To recreate:

1. Go to Admin -> Debit types 
2. See MANUAL debit type with the description of 'Manual fee'
3. Click edit and edit the description.
4. Go back to the Debit types admin page. The description still reads "Manual fee"
Comment 1 Lucas Gass (lukeg) 2025-06-26 19:37:25 UTC
kohadev-koha@kohadevbox:koha(main)$ git grep 'Manual fee'
koha-tmpl/intranet-tmpl/prog/en/includes/accounts.inc:        <span>Manual fee</span>
koha-tmpl/opac-tmpl/bootstrap/en/includes/accounts.inc:        <span>Manual fee</span>
t/mock_templates/intranet-tmpl/prog/en/includes/accounts.inc:        <span>Manual fee</span>
Comment 2 Andrew Fuerste-Henry 2025-09-17 20:54:29 UTC
Created attachment 186549 [details] [review]
Bug 40256: Use debit_type.description for display of manual fees

To test:
1 - edit the description of your MANUAL debit type to read "TEST" instead of "Manual fee"
2 - create a new debit type with code "BOO" and description "Broken"
3 - find a patron, go to accounting then to apply manual invoice
4 - note that the debit types presented for use show "Manual fee" instead of "TEST"
5 - create a debit on your patron with type MANAUAL and a second debit using the type created in step 2
6 - in your patron account, confirm your manual fee shows "Manual fee" in the Type column
7 - confirm the debit created in step 5 shows "broken" in the type column
8 - log into the OPAC as your patron, go to their account
9 - confirm your MANUAL debit shows "Manual fee" in the type column
10 - confirm your other debit shows no value in the type column -- for some reason the OPAC isn't showing non-system type values here, which seems to be a new issue

APPLY PATCH

11 - reload patron account in staff client, see it now shows "TEST" in the type column for your MANUAL debit
12 - reload the debit types page in admin, confirm it now shows "TEST" in the description column for the MANUAL debit type
13 - reload the patron account on the OPAC, see it now shows no value in the Type column for either of your debits. Whatever's stopping the type from showing for the custom debit type is also stopping it from showing for the MANUAL debit type
Comment 3 Andrew Fuerste-Henry 2025-09-17 20:56:22 UTC
This patch fixes the issue for the staff client, but makes clear that there's something else going wrong on the OPAC. In the patron accounts page on the OPAC, it's for some reason failing to display type values for any debit that's not part of the standard install. I didn't tease out why that's failing.
Comment 4 Andrew Fuerste-Henry 2025-09-18 13:31:42 UTC
Filed bug 40836 for this missing Type values.