Some catalogers would find it handy if the cn_browser plugin included the item type description in the table of cn_browser results.
Created attachment 148941 [details] [review] Bug 33365: Add item type column to cn_browser results
Created attachment 148942 [details] [review] Bug 33365: Add item type column to cn_browser results To test: 1. Go to MARC bibliographic framework, pick a framework and go to 952, subfield "o". Turn the cn_browser plugin on. 2. Pick or create an item in that framework, edit that item. 3. TO the right of the 952$o notice the "...". Click that start the call number browser. 4. Notice there is no column for item type. 5. Apply patch, restart_all 6. Notice there is now a column for item type.
Created attachment 149139 [details] [review] Bug 33365: Add item type column to cn_browser results To test: 1. Go to MARC bibliographic framework, pick a framework and go to 952, subfield "o". Turn the cn_browser plugin on. 2. Pick or create an item in that framework, edit that item. 3. TO the right of the 952$o notice the "...". Click that start the call number browser. 4. Notice there is no column for item type. 5. Apply patch, restart_all 6. Notice there is now a column for item type. Signed-off-by: David Nind <david@davidnind.com>
Created attachment 150079 [details] [review] Bug 33365: Add item type column to cn_browser results To test: 1. Go to MARC bibliographic framework, pick a framework and go to 952, subfield "o". Turn the cn_browser plugin on. 2. Pick or create an item in that framework, edit that item. 3. TO the right of the 952$o notice the "...". Click that start the call number browser. 4. Notice there is no column for item type. 5. Apply patch, restart_all 6. Notice there is now a column for item type. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 150080 [details] [review] Bug 33365: (QA follow-up) Add check on item-level_itypes With biblio level itemtypes you won't have any on the item type level, so I feel like it will make sense to hide the column here like we do in other spots. To test: * pref at specific item = itemtype column shows * pref at biblio level = no itemtype column Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Should we have some check on item-level_itypes here? Just in case... I added a follow-up :)
Pushed to master for 23.05. Nice work everyone, thanks!
Enhancement. Not pushing to 22.11.x