Bug 25583 - When ClaimReturnedLostValue is not set, the claim returned tab doesn't appear
Summary: When ClaimReturnedLostValue is not set, the claim returned tab doesn't appear
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: master
Hardware: All All
: P5 - low minor (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 14697
Blocks:
  Show dependency treegraph
 
Reported: 2020-05-22 11:06 UTC by Katrin Fischer
Modified: 2020-11-20 11:56 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Who signed the patch off:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 25583: Disable the "claim returned" button if the feature is disabled (1.85 KB, patch)
2020-11-20 08:20 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25583: Disable the "claim returned" button if the feature is disabled (1.92 KB, patch)
2020-11-20 11:56 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2020-05-22 11:06:40 UTC
When you add a 'claimed returned' to an item, but ClaimReturnedLostValue is not set, the "Claims" tab won't display and cause some confusion.
Comment 1 Katrin Fischer 2020-05-22 22:53:29 UTC
*** Bug 25518 has been marked as a duplicate of this bug. ***
Comment 2 Jonathan Druart 2020-10-15 09:10:00 UTC
In master at least, you cannot click "claim returned" if the pref is not set. I am not sure this is (still) valid.

Can you close or clarify?
Comment 3 Katrin Fischer 2020-10-15 23:36:28 UTC
I just retested on master, this is what I see:

- Make sure ClaimReturnedLostValue is empty
- Go to a patron account
- Check out an item
- Make the claims returned column visible in the table of checkouts
- Click "Claim returned"
- Modal opens
- Make claim (submit)
- The item is now "claimed returned" in the checkouts tab, but the claims tab is not showing
Comment 4 Andrew Fuerste-Henry 2020-11-18 21:43:51 UTC
This is sort of at odds with bug 24767, in which we hid the claims tab if the value is not set.

Katrin, your test plan sort of sneaks up on Claims, by using the Columns button to show the Return Claims column that would otherwise be hidden. Is the answer here to just not allow one to reveal the claims column at all when ClaimReturnedLostValue is not set?
Comment 5 Jonathan Druart 2020-11-20 08:20:52 UTC
Created attachment 113867 [details] [review]
Bug 25583: Disable the "claim returned" button if the feature is disabled

It's possible to display the "Return claims" column on the checkout
list, even if the feature is disabled.
We should disable the link and add an explanation.

Test plan:
- Make sure ClaimReturnedLostValue is empty
- Go to a patron account
- Check out an item
- Make the claims returned column visible in the table of checkouts
=> The button is disabled and a tooltip explains that the feature is
disabled
Comment 6 Kyle M Hall 2020-11-20 11:56:33 UTC
Created attachment 113878 [details] [review]
Bug 25583: Disable the "claim returned" button if the feature is disabled

It's possible to display the "Return claims" column on the checkout
list, even if the feature is disabled.
We should disable the link and add an explanation.

Test plan:
- Make sure ClaimReturnedLostValue is empty
- Go to a patron account
- Check out an item
- Make the claims returned column visible in the table of checkouts
=> The button is disabled and a tooltip explains that the feature is
disabled

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>