Bug 31117 - Cloning standard circulation rules for all libraries show up as from '*'
Summary: Cloning standard circulation rules for all libraries show up as from '*'
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: System Administration (show other bugs)
Version: master
Hardware: All All
: P5 - low trivial (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-07 15:22 UTC by Florian
Modified: 2023-06-08 22:26 UTC (History)
5 users (show)

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


Attachments
Picture for exemple (3.66 KB, image/png)
2022-07-07 15:22 UTC, Florian
Details
Bug 31117: New message for cloning standard rules (1.92 KB, patch)
2022-07-07 15:35 UTC, Florian
Details | Diff | Splinter Review
Bug 31117: New message for cloning standard rules (2.06 KB, patch)
2022-07-07 16:22 UTC, Biblibre Sandboxes
Details | Diff | Splinter Review
Bug 31117: New message for cloning standard rules (2.12 KB, patch)
2022-07-08 11:19 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Florian 2022-07-07 15:22:28 UTC
Created attachment 137327 [details]
Picture for exemple

When cloning circulation and fine rules using the "Standard rules for all libraries" default option to another library, the confirmation title after wards shows up as :

Cloning circulation and fine rules from "" to "Centerville"

Because the default option has no real label, it doesn't show up. This doesn't affect the cloning feature, this is purely an text issue. You can see an exemple on the attached picture.
Comment 1 Florian 2022-07-07 15:35:39 UTC
Created attachment 137328 [details] [review]
Bug 31117: New message for cloning standard rules

This commit adds a different message when cloning circulation and fine rules if you use the Standard rules for all libraries option.

Test plan :
1. Go to the Circulation and fine rules page in the Admin tab.
2. Check that the 'Select a library' menu is on 'Standard rules for all libraries'.
3. Check that you also have some random rules to clone, or create some.
4. Clone these rules to the library of your choice.
5. Confirm that the message 'Clone circulation and fine rules from "" to (library of your choice)' appears.
6. Now clone rules from a specific library to another one, and repeat steps 3-4.
7. Confirm that this time, the message will displays both libraries properly.
8. Apply patch.
9. Repeat steps 2 to 4 and 6 to 7. Confirm that this time, the message in the first case will be 'Cloning circulation and fine rules to "(library of your choice)"'.
10. Kindly sign off.
Comment 2 Biblibre Sandboxes 2022-07-07 16:22:55 UTC
Created attachment 137329 [details] [review]
Bug 31117: New message for cloning standard rules

This commit adds a different message when cloning circulation and fine rules if you use the Standard rules for all libraries option.

Test plan :
1. Go to the Circulation and fine rules page in the Admin tab.
2. Check that the 'Select a library' menu is on 'Standard rules for all libraries'.
3. Check that you also have some random rules to clone, or create some.
4. Clone these rules to the library of your choice.
5. Confirm that the message 'Clone circulation and fine rules from "" to (library of your choice)' appears.
6. Now clone rules from a specific library to another one, and repeat steps 3-4.
7. Confirm that this time, the message will displays both libraries properly.
8. Apply patch.
9. Repeat steps 2 to 4 and 6 to 7. Confirm that this time, the message in the first case will be 'Cloning circulation and fine rules to "(library of your choice)"'.
10. Kindly sign off.

Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com>
Comment 3 Emmanuel Bétemps 2022-07-07 16:26:31 UTC
Actually, the message for the 1st case AFTER applying the patch is : Cloning circulation and fine rules from standard rules to "(library of your choice)"

Your testplan says : Cloning circulation and fine rules to "(library of your choice)"

But I think the typo is in the test plan because the sentence displayed (when patch applies) really fits.
Comment 4 Katrin Fischer 2022-07-08 11:19:01 UTC
Hi Emmanuel, I think you are right, the sentence fits, we don't need to have the exact same wording.
Comment 5 Katrin Fischer 2022-07-08 11:19:57 UTC
Created attachment 137404 [details] [review]
Bug 31117: New message for cloning standard rules

This commit adds a different message when cloning circulation and fine rules if you use the Standard rules for all libraries option.

Test plan :
1. Go to the Circulation and fine rules page in the Admin tab.
2. Check that the 'Select a library' menu is on 'Standard rules for all libraries'.
3. Check that you also have some random rules to clone, or create some.
4. Clone these rules to the library of your choice.
5. Confirm that the message 'Clone circulation and fine rules from "" to (library of your choice)' appears.
6. Now clone rules from a specific library to another one, and repeat steps 3-4.
7. Confirm that this time, the message will displays both libraries properly.
8. Apply patch.
9. Repeat steps 2 to 4 and 6 to 7. Confirm that this time, the message in the first case will be 'Cloning circulation and fine rules to "(library of your choice)"'.
10. Kindly sign off.

Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Comment 6 Tomás Cohen Arazi 2022-07-08 18:26:44 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 7 Lucas Gass 2022-08-12 16:51:11 UTC
Backported to 22.05.x for 22.05.04
Comment 8 Arthur Suzuki 2022-08-16 12:59:03 UTC
Salut Florian,
Bravo pour ce patch, il rejoint la 21.11 en version 21.11.11!

backported to 21.11.x for 21.11.11

thx!
Comment 9 Victor Grousset/tuxayo 2022-08-19 23:12:54 UTC
Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed.

Nothing to document, marking resolved.