Bug 24516

Summary: Column Configuration does not hide Return Date
Product: Koha Reporter: Kelly McElligott <kelly>
Component: Staff interfaceAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: 1joynelson, david, gmcharlt, jonathan.druart, katrin.fischer, lucas
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24891
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes an issue where hiding the return date column for the "Pay Fines" and "Account Fines" screens does not work.
Version(s) released in:
20.05.00, 19.11.04. 19.05.09
Bug Depends on: 12166    
Bug Blocks:    
Attachments: Bug 24516: Add account_type to the columns settings on boraccount
Bug 24516: Add account_type to the columns settings on boraccount
Bug 24516: Add account_type to the columns settings on boraccount

Description Kelly McElligott 2020-01-25 14:49:10 UTC
There are two tables within the Patron module (in column configuration) that work with the Accounting portion of Koha. "Pay Fines" and "Account Fines" - both of these tables present the user with hiding a specific column found in these areas within the Accounting screens, however, attempting to hide "Return Date" in either/both of these tables does not work. 

To recreate:t

1. Go to Column Configuration.
2. Choose Patrons
3. Choose either Pay Fines or Account Fines table
4. Hide Return Date
5. Save
6. Check on patron's screen - boraccount.pl or pay.pl depending on which or both tables you attempted to hide the return date- see that the return date is still appearing. 

I have filed another bug similar to this 24515 dealing with these same tables.
Comment 1 Lucas Gass 2020-01-27 19:21:46 UTC
I cannot reproduce this in master now. I believe the problem was solved by bz24515.
Comment 2 Jonathan Druart 2020-02-03 16:32:39 UTC
Created attachment 98351 [details] [review]
Bug 24516: Add account_type to the columns settings on boraccount

The column was missing from the yml file.
Comment 3 David Nind 2020-02-03 20:31:12 UTC
Created attachment 98357 [details] [review]
Bug 24516: Add account_type to the columns settings on boraccount

The column was missing from the yml file.

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 David Nind 2020-02-03 20:32:28 UTC
(In reply to Lucas Gass from comment #1)
> I cannot reproduce this in master now. I believe the problem was solved by
> bz24515.

I was able to reproduce on master.
Comment 5 Katrin Fischer 2020-02-09 10:41:11 UTC
(In reply to Lucas Gass from comment #1)
> I cannot reproduce this in master now. I believe the problem was solved by
> bz24515.

I think the problem here is that it works on one tab, but not the other:
- Make a payment = OK
- Transaction = Not OK
Comment 6 Katrin Fischer 2020-02-09 10:42:46 UTC
Created attachment 98614 [details] [review]
Bug 24516: Add account_type to the columns settings on boraccount

The column was missing from the yml file.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Martin Renvoize 2020-02-10 09:23:41 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 8 Joy Nelson 2020-03-05 00:39:01 UTC
Pushed to 19.11.x branch for 19.11.04
Comment 9 Lucas Gass 2020-03-09 19:18:38 UTC
backported to 19.05.x for 19.05.09