Bug 31724 - MARC framework subfield deletion - 'i' added to end of the breadcrumb on confirm deletion page
Summary: MARC framework subfield deletion - 'i' added to end of the breadcrumb on conf...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: David Nind
QA Contact: Testopia
URL:
Keywords:
Depends on: 27846
Blocks:
  Show dependency treegraph
 
Reported: 2022-10-09 21:37 UTC by David Nind
Modified: 2023-06-08 22:29 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: String patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This fixes the breadcrumb when deleting a subfield for a framework. An 'i' was incorrectly added to the end of the breadcrumb on the deletion confirmation page, for example: ... > Confirm deletion of subfield bi
Version(s) released in:
22.11.00


Attachments
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield (1.72 KB, patch)
2022-10-09 21:59 UTC, David Nind
Details | Diff | Splinter Review
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield (1.78 KB, patch)
2022-10-10 21:10 UTC, Fridolin Somers
Details | Diff | Splinter Review
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield (1.78 KB, patch)
2022-10-10 21:10 UTC, Fridolin Somers
Details | Diff | Splinter Review
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield (1.85 KB, patch)
2022-10-14 06:49 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description David Nind 2022-10-09 21:37:14 UTC
When deleting a subfield in a MARC framework, there is an 'i' added to the breadcrumb on the confirm deletion page, for example:

- Home > Administration > MARC frameworks > BKS framework structure > Tag 245 Subfield structure > Confirm deletion of subfield ci

- This should be ... Confirm deletion of subfield c

It looks like this was added in bug 27846 (released in 21.05.00).
Comment 1 David Nind 2022-10-09 21:59:00 UTC
Created attachment 141560 [details] [review]
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield

Remove 'i' added to the end of the breadcrumb on the confirmation page
when deleteing a MARC framework subfield.

Test plan:
1. Go to Administration > MARC bibliographic frameworks.
2. View the subfields for a tag for a framework, for example:
   . BKS framework > Actions > MARC structure
   . Search for tag 245
   . Select Actions > View subfields.
3. Select any subfield to delete and select the Delete button.
4. Note that there is an 'i' at the end of the breadcrumb, for example:
   . ... > Confirm deletion of subfield bi
5. Apply the patch.
6. Refresh the page and note that that 'i' is removed.
7. Sign off.

Signed-off-by: David Nind <david@davidnind.com>
Comment 2 Fridolin Somers 2022-10-10 21:03:11 UTC
Looks like a VIM residue ;)
Comment 3 David Nind 2022-10-10 21:09:02 UTC
(In reply to Fridolin Somers from comment #2)
> Looks like a VIM residue ;)

That was my thought as well.

The other alternative was that it should be a ? to match the dialogue box text.

But I think it makes sense not to have the question mark.
Comment 4 Fridolin Somers 2022-10-10 21:10:12 UTC
Created attachment 141591 [details] [review]
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield

Remove 'i' added to the end of the breadcrumb on the confirmation page
when deleteing a MARC framework subfield.

Test plan:
1. Go to Administration > MARC bibliographic frameworks.
2. View the subfields for a tag for a framework, for example:
   . BKS framework > Actions > MARC structure
   . Search for tag 245
   . Select Actions > View subfields.
3. Select any subfield to delete and select the Delete button.
4. Note that there is an 'i' at the end of the breadcrumb, for example:
   . ... > Confirm deletion of subfield bi
5. Apply the patch.
6. Refresh the page and note that that 'i' is removed.
7. Sign off.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 5 Fridolin Somers 2022-10-10 21:10:37 UTC
Created attachment 141592 [details] [review]
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield

Remove 'i' added to the end of the breadcrumb on the confirmation page
when deleteing a MARC framework subfield.

Test plan:
1. Go to Administration > MARC bibliographic frameworks.
2. View the subfields for a tag for a framework, for example:
   . BKS framework > Actions > MARC structure
   . Search for tag 245
   . Select Actions > View subfields.
3. Select any subfield to delete and select the Delete button.
4. Note that there is an 'i' at the end of the breadcrumb, for example:
   . ... > Confirm deletion of subfield bi
5. Apply the patch.
6. Refresh the page and note that that 'i' is removed.
7. Sign off.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 6 David Nind 2022-10-10 21:55:54 UTC
Thanks Fridolin for signing off!
Comment 7 Jonathan Druart 2022-10-14 06:49:45 UTC
Created attachment 141854 [details] [review]
Bug 31724: Remove 'i' from breadcrumb when deleting a MARC framework subfield

Remove 'i' added to the end of the breadcrumb on the confirmation page
when deleteing a MARC framework subfield.

Test plan:
1. Go to Administration > MARC bibliographic frameworks.
2. View the subfields for a tag for a framework, for example:
   . BKS framework > Actions > MARC structure
   . Search for tag 245
   . Select Actions > View subfields.
3. Select any subfield to delete and select the Delete button.
4. Note that there is an 'i' at the end of the breadcrumb, for example:
   . ... > Confirm deletion of subfield bi
5. Apply the patch.
6. Refresh the page and note that that 'i' is removed.
7. Sign off.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Tomás Cohen Arazi 2022-10-14 12:32:41 UTC
Vim users.... haha
Comment 9 Tomás Cohen Arazi 2022-10-14 12:33:47 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!