[Bug 14510] New: Allow column visibility in additem.pl to be customized
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Bug ID: 14510 Summary: Allow column visibility in additem.pl to be customized Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: jweaver@bywatersolutions.com Reporter: jweaver@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl It should be possible to customize column visibility in the items table in additem.pl through the columns configuration tool. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #1 from Jesse Weaver <jweaver@bywatersolutions.com> --- Created attachment 40977 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40977&action=edit Bug 14510: Allow column visibility in additem.pl to be customized This patch adds the table of items on additem.pl to the columns customizer. Test plan: 1. Open item editor on a record, and verify that all columns are visible. 2. Apply patch. 3. Reload editor, and verify that column visibility hasn't changed. 4. Open "Hide/show columns," and verify that you can add and remove columns. 5. Change the visibility and togglability of some columns in columns_settings.pl, and verify that these correctly apply to additem.pl. NOTE: The columns that are configurable are selected from the non-hidden columns that have mappings to MARC subfields in the default MARC21 framework (and can thus be displayed in the item editor). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nengard@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 J Schmidt <jschmidt@switchinc.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jschmidt@switchinc.org -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA CC| |veron@veron.ch --- Comment #2 from Marc Véron <veron@veron.ch> --- Sorry, QA tools found some forbidden tab chars: cataloguing/additem.pl forbidden pattern: tab char (line 807) koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc forbidden pattern: tab char (line 13) forbidden pattern: tab char (line 12) koha-tmpl/intranet-tmpl/prog/en/modules/admin/columns_settings.tt forbidden pattern: tab char (line 15) forbidden pattern: tab char (line 22) forbidden pattern: tab char (line 17) forbidden pattern: tab char (line 16) koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt forbidden pattern: tab char (line 41) forbidden pattern: tab char (line 40) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #40977|0 |1 is obsolete| | --- Comment #3 from Jesse Weaver <jweaver@bywatersolutions.com> --- Created attachment 41418 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41418&action=edit Bug 14510: Allow column visibility in additem.pl to be customized This patch adds the table of items on additem.pl to the columns customizer. Test plan: 1. Open item editor on a record, and verify that all columns are visible. 2. Apply patch. 3. Reload editor, and verify that column visibility hasn't changed. 4. Open "Hide/show columns," and verify that you can add and remove columns. 5. Change the visibility and togglability of some columns in columns_settings.pl, and verify that these correctly apply to additem.pl. NOTE: The columns that are configurable are selected from the non-hidden columns that have mappings to MARC subfields in the default MARC21 framework (and can thus be displayed in the item editor). Signed-off-by: Jenny Schmidt <jschmidt@switchinc.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #4 from Jesse Weaver <jweaver@bywatersolutions.com> --- Created attachment 41419 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41419&action=edit Bug 14510: (QA followup) remove extraneous whitespace -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #5 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Jesse, This works quite well! How did you choose the order of the fields on the column settings configuration page? What about the stocknumber and statisticvalue fields, for instance? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #6 from Jesse Weaver <jweaver@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #5)
Jesse, This works quite well! How did you choose the order of the fields on the column settings configuration page? What about the stocknumber and statisticvalue fields, for instance?
The order and selection of the columns in question is based on which columns are mapped to subfields in the MARC21 framework. These are the only columns that can appear in the table. Are there any such fields that are mapped in UNIMARC but not MARC21? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |ASSIGNED --- Comment #7 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Joubu: pianohacker: for instance stocknumber is not mapped by default, but can be mapped pianohacker: hrm, decent point. pianohacker: Joubu: based on my patch, and your work with the datatables stuff, even if I throw in all those columns it will only show the ones that are actually visible in the "show/hide columns" menu, and in the correct order, right? Joubu: I think so, yes matts est désormais connu sous le nom de matts_away Joubu: that's the point, I think it's better to show all columns pianohacker: agreed nengard est désormais connu sous le nom de nengard_webinar Joubu: and display them either the order on admin/koha2marclinks.pl either the DB order edveal-afk est désormais connu sous le nom de edveal Joubu: DB order ie. kohastructure.sql pianohacker: yeah, to be consistent with something pianohacker: Joubu: all right, have that on my todo list. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #8 from Jesse Weaver <jweaver@bywatersolutions.com> --- Created attachment 41800 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41800&action=edit Bug 14510: (QA followup) Add more columns, resort Per a conversation with Jonathan Druart, add all columns that could be reasonably mapped, and order them the same as in kohastructure.sql. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Jesse, This does not work (even without the last patch). To reproduce: hide the first column by default (barcode), save and refresh the add item page. The first column is hidden (in my case 'damaged'). You will have to change the way the feature works. The way it works currently is based on the order of the columns, not the names. Marked as Failed QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #10 from Jesse Weaver <jweaver@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #9)
Jesse,
This does not work (even without the last patch). To reproduce: hide the first column by default (barcode), save and refresh the add item page. The first column is hidden (in my case 'damaged').
You will have to change the way the feature works. The way it works currently is based on the order of the columns, not the names.
Marked as Failed QA.
Was this on a UNIMARC system? If so, is there any chance you could post a sandbox link that I could look at? I thought I'd made it possible for it to work based on name. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #11 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Yes it was a UNIMARC install. But same on a MARC21 fresh install. Check "cn_source is hidden by default". Refresh cataloguing/additem.pl?biblionumber=109 The third column is hidden (damaged status). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #12 from Jesse Weaver <jweaver@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #11)
Yes it was a UNIMARC install.
But same on a MARC21 fresh install.
Check "cn_source is hidden by default". Refresh cataloguing/additem.pl?biblionumber=109
The third column is hidden (damaged status).
This is very bizarre; I can't duplicate this issue. Could I ask you to post a link to a sandbox where this issue occurs, so I can poke around? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #13 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- On sandbox 13 (reminder: the DB is erased every day). http://pro.test13.biblibre.com/cgi-bin/koha/admin/columns_settings.pl : barcode, dateaccessioned and homebranch are hidden by default. http://pro.test13.biblibre.com/cgi-bin/koha/cataloguing/additem.pl?biblionum... "Propriétaire", "Dépositaire" and "Code barres" are hidden Propriétaire: homebranch Dépositaire: holdingbranch Code barres: barcode -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #14 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Ok, I found why it does not work: dateaccessioned is not mapped. Same with my previous example, barcode is not mapped on my local install. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #15 from Jesse Weaver <jweaver@bywatersolutions.com> --- Created attachment 42353 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42353&action=edit Bug 14510: (QA followup) Fix bug with unmapped columns Fall back more smoothly when items subfields are not mapped to a kohafield. Note, however, that this development does not allow applying default settings to subfields that aren't mapped. A note has been added to the columns settings page, as there is no known workaround for this. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #16 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42379 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42379&action=edit Bug 14510: Redirect to the items' fields mapped -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #17 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Jesse, Great, it works perfectly, for the items editor table... Not the others :) Try the late orders table for instance, the ft is broken on it. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 --- Comment #18 from Jesse Weaver <jweaver@bywatersolutions.com> --- Created attachment 42624 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42624&action=edit Bug 14510: (QA followup) fix columns settings in other pages -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jesse Weaver <jweaver@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #41418|0 |1 is obsolete| | --- Comment #19 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42628 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42628&action=edit Bug 14510: Allow column visibility in additem.pl to be customized This patch adds the table of items on additem.pl to the columns customizer. Test plan: 1. Open item editor on a record, and verify that all columns are visible. 2. Apply patch. 3. Reload editor, and verify that column visibility hasn't changed. 4. Open "Hide/show columns," and verify that you can add and remove columns. 5. Change the visibility and togglability of some columns in columns_settings.pl, and verify that these correctly apply to additem.pl. NOTE: The columns that are configurable are selected from the non-hidden columns that have mappings to MARC subfields in the default MARC21 framework (and can thus be displayed in the item editor). Signed-off-by: Jenny Schmidt <jschmidt@switchinc.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #41419|0 |1 is obsolete| | --- Comment #20 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42629 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42629&action=edit Bug 14510: (QA followup) remove extraneous whitespace Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #41800|0 |1 is obsolete| | --- Comment #21 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42630 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42630&action=edit Bug 14510: (QA followup) Add more columns, resort Per a conversation with Jonathan Druart, add all columns that could be reasonably mapped, and order them the same as in kohastructure.sql. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42353|0 |1 is obsolete| | --- Comment #22 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42631 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42631&action=edit Bug 14510: (QA followup) Fix bug with unmapped columns Fall back more smoothly when items subfields are not mapped to a kohafield. Note, however, that this development does not allow applying default settings to subfields that aren't mapped. A note has been added to the columns settings page, as there is no known workaround for this. Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42379|0 |1 is obsolete| | --- Comment #23 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42632 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42632&action=edit Bug 14510: Redirect to the items' fields mapped Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #42624|0 |1 is obsolete| | --- Comment #24 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 42633 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42633&action=edit Bug 14510: (QA followup) fix columns settings in other pages Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Nicole C. Engard <nengard@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nicole@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14510 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tomascohen@gmail.com Status|Passed QA |Pushed to Master --- Comment #25 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Patches pushed to master. Thanks Jesse! -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org