Bug 24592

Summary: Clarify LOST_RETURN process by using FOUND over RETURNED
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: Fines and feesAssignee: Martin Renvoize <martin.renvoize>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: 1joynelson, david, nengard
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00
Bug Depends on:    
Bug Blocks: 24474    
Attachments: Bug 24592: Reword LOST_RETURN to LOST_FOUND
Bug 24592: Database Update
Bug 24592: Reword LOST_RETURN to LOST_FOUND
Bug 24592: Database Update
Bug 24592: Database Update
Bug 24592: Reword LOST_RETURN to LOST_FOUND
Bug 24592: Database Update
Bug 24592: Reword LOST_RETURN to LOST_FOUND
Bug 24592: Database Update
Bug 24592: Reword LOST_RETURN to LOST_FOUND

Description Martin Renvoize 2020-02-05 11:45:24 UTC
The phrasing of LOST_RETURN is confusing, the item is not always immediately returned, sometimes it is renewed for example.. I think it would be clearer to reword this to use 'FOUND' to more clearly reflect the states..

I intend to do this in the accounts handling as well as rename the _FixAccountForLostAndReturned method in C4::Circulation.
Comment 1 Martin Renvoize 2020-02-05 12:20:18 UTC
Created attachment 98457 [details] [review]
Bug 24592: Reword LOST_RETURN to LOST_FOUND

This patch updates the wording in the 'lost and found' process to more
closely reflect what the process is achieving by replacing 'RETURNED'
with 'FOUND'
Comment 2 Martin Renvoize 2020-02-05 12:20:21 UTC
Created attachment 98458 [details] [review]
Bug 24592: Database Update
Comment 3 Martin Renvoize 2020-02-05 13:46:09 UTC
Created attachment 98481 [details] [review]
Bug 24592: Reword LOST_RETURN to LOST_FOUND

This patch updates the wording in the 'lost and found' process to more
closely reflect what the process is achieving by replacing 'RETURNED'
with 'FOUND'

Test plan:
1) Grep codebase for _FixAccountForLostAndReturned and note there are no
   longer any instanced of it.
2) Run t/db_dependent/Circulation.t and note it passes
3) Test returning/renewing an item that has been marked as lost and note
   the updated values in the accountlines now use LOST_FOUND as
   credit_type_code and 'FOUND' as the status for the 'LOST' fee
   (debit_type_code 'LOST')
Comment 4 Martin Renvoize 2020-02-05 13:46:11 UTC
Created attachment 98482 [details] [review]
Bug 24592: Database Update
Comment 5 Martin Renvoize 2020-02-10 11:22:08 UTC
Created attachment 98643 [details] [review]
Bug 24592: Database Update
Comment 6 Martin Renvoize 2020-02-10 11:22:11 UTC
Created attachment 98644 [details] [review]
Bug 24592: Reword LOST_RETURN to LOST_FOUND

This patch updates the wording in the 'lost and found' process to more
closely reflect what the process is achieving by replacing 'RETURNED'
with 'FOUND'

Test plan:
1) Grep codebase for _FixAccountForLostAndReturned and note there are no
   longer any instanced of it.
2) Run t/db_dependent/Circulation.t and note it passes
3) Test returning/renewing an item that has been marked as lost and note
   the updated values in the accountlines now use LOST_FOUND as
   credit_type_code and 'FOUND' as the status for the 'LOST' fee
   (debit_type_code 'LOST')
Comment 7 David Nind 2020-02-10 18:45:36 UTC
Created attachment 98680 [details] [review]
Bug 24592: Database Update

Signed-off-by: David Nind <david@davidnind.com>
Comment 8 David Nind 2020-02-10 18:45:40 UTC
Created attachment 98681 [details] [review]
Bug 24592: Reword LOST_RETURN to LOST_FOUND

This patch updates the wording in the 'lost and found' process to more
closely reflect what the process is achieving by replacing 'RETURNED'
with 'FOUND'

Test plan:
1) Grep codebase for _FixAccountForLostAndReturned and note there are no
   longer any instanced of it.
2) Run t/db_dependent/Circulation.t and note it passes
3) Test returning/renewing an item that has been marked as lost and note
   the updated values in the accountlines now use LOST_FOUND as
   credit_type_code and 'FOUND' as the status for the 'LOST' fee
   (debit_type_code 'LOST')

Signed-off-by: David Nind <david@davidnind.com>
Comment 9 David Nind 2020-02-10 18:48:38 UTC
Here is an updated test plan (I had to work out how to do some of the things, so I hope it is useful to others and what was intended to check):

1) Apply the patch and run the database update
   (perl installer/data/mysql/updatedatabase.pl)
2) Grep th codebase for _FixAccountForLostAndReturned and note there are no
   longer any instanced of it (grep -r1 "_FixAccountForLostAndReturned" koha)
3) Run t/db_dependent/Circulation.t and note it passes
4) For a patron:
   - Check out an item (Circulation > Check out)
   - Create a manual invoice for the item using the 'Lost item' type
     (Patrons > [Account for patron] > Accounting > Create manual invoice > 
     Enter the barcode and amount)
   - Edit the record for the item and change the 'Lost status' to 'Lost'
     (From the list of transactions  click on the barcode, and change the 
     'Lost status' to 'Lost' and then 'clcik 'Set status')
   - Check in the lost item (Circulation > Check in)
   - A message should pop up saying 'Item was lost, now found.....'
   - Check the transactions for the patron: there should now be a transaction
     with a refund, the account type will say 'Lost item fee refund', and 
     in the description it will say 'Item found...'
5) Check the database to see the changes:
   - koha-mysql kohadev
   - Use the query 'select * from accountlines'
   - For the original manual invoice the debit_type_code will show as 'LOST'
     and the status will show as 'FOUND'
   - For the transactions to reverse the lost item fee the credit_type_code
     will show as "LOST_FOUND'
Comment 10 Martin Renvoize 2020-02-11 09:18:14 UTC
Thanks for testing David :)
Comment 11 Martin Renvoize 2020-02-11 12:57:21 UTC
*** Bug 6383 has been marked as a duplicate of this bug. ***
Comment 12 Joonas Kylmälä 2020-02-13 15:01:13 UTC
Created attachment 98852 [details] [review]
Bug 24592: Database Update

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>
Comment 13 Joonas Kylmälä 2020-02-13 15:01:16 UTC
Created attachment 98853 [details] [review]
Bug 24592: Reword LOST_RETURN to LOST_FOUND

This patch updates the wording in the 'lost and found' process to more
closely reflect what the process is achieving by replacing 'RETURNED'
with 'FOUND'

Test plan:
1) Grep codebase for _FixAccountForLostAndReturned and note there are no
   longer any instanced of it.
2) Run t/db_dependent/Circulation.t and note it passes
3) Test returning/renewing an item that has been marked as lost and note
   the updated values in the accountlines now use LOST_FOUND as
   credit_type_code and 'FOUND' as the status for the 'LOST' fee
   (debit_type_code 'LOST')

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>
Comment 14 Martin Renvoize 2020-02-13 16:02:00 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 15 Joy Nelson 2020-03-05 01:02:21 UTC
enhancement not backported to 19.11.x