[Bug 18536] New: Generating CSV using profile in serials late issues doesn 't work as described
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Bug ID: 18536 Summary: Generating CSV using profile in serials late issues doesn't work as described Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Serials Assignee: koha-bugs@lists.koha-community.org Reporter: josef.moravec@gmail.com QA Contact: testopia@bugs.koha-community.org CC: colin.campbell@ptfs-europe.com Description on editing csv profiles says: "You can also use your own headers (instead of the ones from Koha) by prefixing the field name with an header, followed by the equal sign." So the header should be optional, but in fact it's mandatory. Also the regular expression to cut table name from beginning of db column is not right. Patch follows -- 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=18536 Josef Moravec <josef.moravec@gmail.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=18536 --- Comment #1 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63058 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63058&action=edit Bug 18536: Generating CSV using profile in serials late issues doesn't work as described Description on editing csv profiles says: "You can also use your own headers (instead of the ones from Koha) by prefixing the field name with an header, followed by the equal sign." So the header should be optional, but in fact it's mandatory. Also the regular expression to cut table name from beginning of db column is not right. Test plan: 0. Don't apply the patch 1. Make two CSV profiles for exporting late issues a) SUPPLIER=aqbooksellers.name|TITLE=subscription.title|ISSUENUMBER=serial.serialseq|LATE SINCE=serial.planneddate b) aqbooksellers.name|TITLE=subscription.title|ISSUE NUMBER=serial.serialseq|LATE SINCE=serial.planneddate 2. Export late issues, profile a) works as expected, profile b) doesn't 3. Apply the patch 4. Both profiles should work -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63058|0 |1 is obsolete| | --- Comment #2 from Josef Moravec <josef.moravec@gmail.com> --- Created attachment 63342 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63342&action=edit Bug 18536: Generating CSV using profile in serials late issues doesn't work as described Description on editing csv profiles says: "You can also use your own headers (instead of the ones from Koha) by prefixing the field name with an header, followed by the equal sign." So the header should be optional, but in fact it's mandatory. Also the regular expression to cut table name from beginning of db column is not right. Test plan: 0. Don't apply the patch 1. Make two CSV profiles for exporting late issues a) SUPPLIER=aqbooksellers.name|TITLE=subscription.title|ISSUENUMBER=serial.serialseq|LATE SINCE=serial.planneddate b) aqbooksellers.name|TITLE=subscription.title|ISSUE NUMBER=serial.serialseq|LATE SINCE=serial.planneddate 2. Export late issues, profile a) works as expected, profile b) doesn't 3. Apply the patch 4. Both profiles should work -- 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=18536 Josef Moravec <josef.moravec@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |minor -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63342|0 |1 is obsolete| | --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 63373 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63373&action=edit [SIGNED OFF] Bug 18536: Generating CSV using profile in serials late issues doesn't work as described Description on editing csv profiles says: "You can also use your own headers (instead of the ones from Koha) by prefixing the field name with an header, followed by the equal sign." So the header should be optional, but in fact it's mandatory. Also the regular expression to cut table name from beginning of db column is not right. Test plan: 0. Don't apply the patch 1. Make two CSV profiles for exporting late issues a) SUPPLIER=aqbooksellers.name|TITLE=subscription.title|ISSUENUMBER=serial.serialseq|LATE SINCE=serial.planneddate b) aqbooksellers.name|TITLE=subscription.title|ISSUE NUMBER=serial.serialseq|LATE SINCE=serial.planneddate 2. Export late issues, profile a) works as expected, profile b) doesn't 3. Apply the patch 4. Both profiles should work Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |josef.moravec@gmail.com |ity.org | QA Contact|testopia@bugs.koha-communit |m.de.rooy@rijksmuseum.nl |y.org | CC| |m.de.rooy@rijksmuseum.nl -- 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=18536 --- Comment #4 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 63574 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63574&action=edit Bug 18536: Generating CSV using profile in serials late issues doesn't work as described Description on editing csv profiles says: "You can also use your own headers (instead of the ones from Koha) by prefixing the field name with an header, followed by the equal sign." So the header should be optional, but in fact it's mandatory. Also the regular expression to cut table name from beginning of db column is not right. Test plan: 0. Don't apply the patch 1. Make two CSV profiles for exporting late issues a) SUPPLIER=aqbooksellers.name|TITLE=subscription.title|ISSUENUMBER=serial.serialseq|LATE SINCE=serial.planneddate b) aqbooksellers.name|TITLE=subscription.title|ISSUE NUMBER=serial.serialseq|LATE SINCE=serial.planneddate 2. Export late issues, profile a) works as expected, profile b) doesn't 3. Apply the patch 4. Both profiles should work Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 --- Comment #5 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 63575 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=63575&action=edit Bug 18536: [QA Follow-up] Tiny regex simplification Simplify regex for removing table name. No need to escape a dot between the square brackets. No need to specify a number of 1 between parentheses. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #63373|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #6 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 17.05, thanks Josef! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Status|Pushed to Master |Pushed to Stable --- Comment #7 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- These patches have been pushed to 16.11.x and will be in 16.11.08. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice@biblibre.com --- Comment #8 from Julian Maurice <julian.maurice@biblibre.com> --- Pushed to 3.22.x for 3.22.21 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18536 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtj@kohaaloha.com --- Comment #9 from Mason James <mtj@kohaaloha.com> --- Pushed to 16.05.x, for 16.05.13 release -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org