[Bug 33639] New: Adding item to item group from 'Add item' screen doesn't work
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Bug ID: 33639 Summary: Adding item to item group from 'Add item' screen doesn't work Change sponsored?: --- Product: Koha Version: 21.11 Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: Cataloging Assignee: koha-bugs@lists.koha-community.org Reporter: sbrown@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl When creating a new item in 22.11.02, the 'Add to item group' feature at the bottom of the 'Add item' screen doesn't successfully add that new item to a group. To replicate: -Create one or more item groups for a bib if needed -Create a new item, and at the bottom of the 'Add item' screen, select the item group to which you want this item added. Save. -On the Normal tab, see that the item has not been added to the item group you selected. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #1 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 150422 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150422&action=edit Bug 33639: Fix selectors for adding to item group To test: Apply patch 1 - Browse to details page for a record 2 - Create or ensure the record has item group(s) 3 - Click New->New item 4 - Note the bottom of the page has a form to attach to existing group 5 - Note when a group is selected the enumchron field is populated 6 - Confirm item is saved to group when saved Note: this does not fix the 'Create new item group' feature in the dropdown. I am having trouble seeing why that would work in the first in place! -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |lucas@bywatersolutions.com |ity.org | Version|21.11 |master CC| |lucas@bywatersolutions.com Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #2 from Lucas Gass <lucas@bywatersolutions.com> --- Filed Bug 33640 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |24857 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24857 [Bug 24857] Add ability to group items for records -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- This fixes the problem, but it creates another: * When enumchron was already filled out, selecting an item group now will overwrite the entered value and replace it. I feel like the initial intention here was to avoid this and I think that would be correct: if ( ! $('input.items-enumchron').val() ) { I think if we keep the check, but change it to checking for the name with . not -, it might actually work? if ( ! $('input[name="items.enumchron"]').val() ) { -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #4 from Lucas Gass <lucas@bywatersolutions.com> --- (In reply to Katrin Fischer from comment #3)
This fixes the problem, but it creates another:
* When enumchron was already filled out, selecting an item group now will overwrite the entered value and replace it.
I feel like the initial intention here was to avoid this and I think that would be correct:
if ( ! $('input.items-enumchron').val() ) {
I think if we keep the check, but change it to checking for the name with . not -, it might actually work?
if ( ! $('input[name="items.enumchron"]').val() ) {
That makes sense. Should we also warn the user that enumchron value is not changing? If I clicked the dropdown and suspected the value was going to change but it does not I might be confused. So maybe we need a pop-up? "You are about to to change this value from A to B. Would you like to continue?" -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Actually I was quite surprised about the forced connection to enumchron. But then I realized that I always grouped the items from the catalog detail page. If you do that, enumchron is not altered/updated neither when empty nor when set. So we have some inconsistency there. We do display the item group in the staff interface items table, but we don't display it in the OPAC. Enumchron shows in a lot of places, soo I understand the idea. At the same time it feels not quite right yet. The initial intention was to not overwrite existing data, I think this is a good default. I am not sure if a dialog could get annoying, ideally maybe a checkbox setting to turn it off, that is remembered? But we should try to limit the changes here to the necessary. I think maybe we should stick to 'don't overwrite if already set' and work on a more sophisticated solution separately (keeping backporting in mind) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #150422|0 |1 is obsolete| | --- Comment #6 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 151383 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151383&action=edit Bug 33639: Fix selectors for adding to item group To test: Apply patch 1 - Browse to details page for a record 2 - Create or ensure the record has item group(s) 3 - Click New->New item 4 - Note the bottom of the page has a form to attach to existing group 5 - Note when a group is selected the enumchron field is populated if it is blank 6 - Note that the enumchron field is NOT update is there is a an existing value 7 - Confirm item is saved to group when saved -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 ByWater Sandboxes <bws.sandboxes@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #151383|0 |1 is obsolete| | --- Comment #7 from ByWater Sandboxes <bws.sandboxes@gmail.com> --- Created attachment 151428 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=151428&action=edit Bug 33639: Fix selectors for adding to item group To test: Apply patch 1 - Browse to details page for a record 2 - Create or ensure the record has item group(s) 3 - Click New->New item 4 - Note the bottom of the page has a form to attach to existing group 5 - Note when a group is selected the enumchron field is populated if it is blank 6 - Note that the enumchron field is NOT update is there is a an existing value 7 - Confirm item is saved to group when saved Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Laura Escamilla <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |Laura.escamilla@bywatersolu | |tions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Sam <samalau@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |samalau@gmail.com Status|Signed Off |Failed QA --- Comment #8 from Sam <samalau@gmail.com> --- When adding a new item, a group is already preselected in the dropdown box, which makes the enumchron box empty by default, even though a group is selected. Had to deselect the group and then reselect it in order for the enumchron to be filled. Another issue was that when the new item was added, the selected group did not show up in the group column, instead, the group column was left empty. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Miranda Nero <mnero@oslri.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mnero@oslri.net Severity|trivial |normal --- Comment #9 from Miranda Nero <mnero@oslri.net> --- Adding a comment to revive this bug! I think this was opened during our inital Koha training at Ocean State Libraries. We are still very interested in being able to add an item to an existing item group when creating said item. We do not want to be able to create a new item group at this step. We do not use the enumchron field to store data, so affecting what is in that field is not an issue for us. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com, | |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #151428|0 |1 is obsolete| | --- Comment #10 from Lucas Gass <lucas@bywatersolutions.com> --- Created attachment 160418 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160418&action=edit Bug 33639: Make add item to item group work from additem.pl To test: 1, Create one or more item groups for a bib if needed 2. Create a new item, and at the bottom of the 'Add item' screen, select the item group to which you want this item added. Save. 3. On the Normal tab, see that the item has not been added to the item group you selected. 4. APPLY PATCH, restart_all 5. Try 1-3 again but this time the item should succesffuly be added to the item group. 6. Add a new item and to the same record and this time try the 'Create a new item group' from the dropdown. 7. Name the new item group. 8. Make sure it is correctly added to the item group. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #11 from Lucas Gass <lucas@bywatersolutions.com> --- *** Bug 33640 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #160418|0 |1 is obsolete| | --- Comment #12 from David Nind <david@davidnind.com> --- Created attachment 160441 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160441&action=edit Bug 33639: Make add item to item group work from additem.pl To test: 1, Create one or more item groups for a bib if needed 2. Create a new item, and at the bottom of the 'Add item' screen, select the item group to which you want this item added. Save. 3. On the Normal tab, see that the item has not been added to the item group you selected. 4. APPLY PATCH, restart_all 5. Try 1-3 again but this time the item should succesffuly be added to the item group. 6. Add a new item and to the same record and this time try the 'Create a new item group' from the dropdown. 7. Name the new item group. 8. Make sure it is correctly added to the item group. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com Text to go in the| |This fixes adding a new release notes| |item to an item group (when | |using the item groups | |feature - EnableItemGroups | |system preference). before | |this fix, even if you | |selected an item group, it | |was not added to it. --- Comment #13 from David Nind <david@davidnind.com> --- Testing notes (using KTD): 1. Enable the EnableItemGroups system preference. 2. enumchron issue noted in previous comments: after selecting New item, I added a value for serial enumeration, a call number, and a barcode. Under the "Add item group" it defaults to "Do not add to item group" - selecting a group does not change the value already entered in h - Serial enumeration / chronology. So I'm assuming that this is no longer an issue. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |35695 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35695 [Bug 35695] Remove useless item group code from cataloging_additem.js -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart+koha@gmail. | |com --- Comment #14 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- 446 add_item_to_item_group( $item->biblionumber, $item->biblioitemnumber, $item_group, $item_group_description ); Should it be $item->itemnumber instead of biblioitemnumber here as well? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #160441|0 |1 is obsolete| | --- Comment #15 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 160839 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160839&action=edit Bug 33639: Make add item to item group work from additem.pl To test: 1, Create one or more item groups for a bib if needed 2. Create a new item, and at the bottom of the 'Add item' screen, select the item group to which you want this item added. Save. 3. On the Normal tab, see that the item has not been added to the item group you selected. 4. APPLY PATCH, restart_all 5. Try 1-3 again but this time the item should succesffuly be added to the item group. 6. Add a new item and to the same record and this time try the 'Create a new item group' from the dropdown. 7. Name the new item group. 8. Make sure it is correctly added to the item group. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com Status|Signed Off |Passed QA --- Comment #16 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Trivial QA, this patch is fixing issues that came in at the last moment when the feature was renamed from 'volumes' to 'item groups'. All working as expected now, no QA warnings.. Passing QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #17 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- (In reply to Martin Renvoize from comment #16)
Trivial QA, this patch is fixing issues that came in at the last moment when the feature was renamed from 'volumes' to 'item groups'.
All working as expected now, no QA warnings.. Passing QA
What about comment 14? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |24.05.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #18 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #19 from Lucas Gass <lucas@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #14)
446 add_item_to_item_group( $item->biblionumber, $item->biblioitemnumber, $item_group, $item_group_description );
Should it be $item->itemnumber instead of biblioitemnumber here as well?
Even though this works as is, I think you're right. I will open a follow-up. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |35774 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35774 [Bug 35774] add_item_to_item_group additem.pl should be $item->itemnumber instead of biblioitemnumber -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #20 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- For consistency you're right, thanks for spotting it. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #21 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- (In reply to Martin Renvoize from comment #20)
For consistency you're right, thanks for spotting it.
Consistency only? Then why do we need this parameter if we can pass a wrong value? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |additional_work_needed --- Comment #22 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Let me know when the follow-up is ready. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #23 from Lucas Gass <lucas@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #21)
(In reply to Martin Renvoize from comment #20)
For consistency you're right, thanks for spotting it.
Consistency only? Then why do we need this parameter if we can pass a wrong value?
We do need to pass the itemnumber for item_group_items.item_id. Its a FK for the items table. Without it you'd get "Broken FK constraint". -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 --- Comment #24 from Lucas Gass <lucas@bywatersolutions.com> --- (In reply to Katrin Fischer from comment #22)
Let me know when the follow-up is ready.
35774 is ready for testing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m Version(s)|24.05.00 |24.05.00,23.11.02 released in| | Status|Pushed to master |Pushed to stable --- Comment #25 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.11.x for 23.11.02 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Pushed to oldstable Version(s)|24.05.00,23.11.02 |24.05.00,23.11.02,23.05.09 released in| | --- Comment #26 from Lucas Gass <lucas@bywatersolutions.com> --- Backported to 23.05.x for upcoming 23.05.09 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|additional_work_needed | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |36498 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36498 [Bug 36498] Allow ability to set display order when adding an item group from additem.tt -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|martin.renvoize@ptfs-europe | |.com | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33639 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to oldstable |RESOLVED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org