[Bug 15501] New: Planned Irregularities are deleted when modifying subscription
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Bug ID: 15501 Summary: Planned Irregularities are deleted when modifying subscription Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Serials Assignee: koha-bugs@lists.koha-community.org Reporter: koha@univ-lyon3.fr QA Contact: testopia@bugs.koha-community.org CC: colin.campbell@ptfs-europe.com In an existing subscription where non published issues have been planned through the checkboxes of the prediction patern test, if you modify later a parameter (adding a note for example) without launching the prediction test (you think it's unuseful because you didn't modify the planning itself), there's is no alert,theese data are deleted. Olivier Crouzet -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |koha@univ-lyon3.fr --- Comment #1 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Created attachment 46341 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46341&action=edit Add warning when irregularities are set in a subscription pattern -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Koha Team Lyon 3 <koha@univ-lyon3.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #46341|0 |1 is obsolete| | --- Comment #2 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Created attachment 46385 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46385&action=edit Bug 15501 - Planned Irregularities are deleted when modifying subscription Test plan : Before applying patch 1) Go to serials Home page 2) Choose an existing subscription (from a daily, weekly or monthly serial) and edit 3) Go to second page and launch the prediction pattern test 4) Check some checkboxes 5) Save. 6) Edit again and add a note on first page 7) Go to next page and Save : you can see there had been no alert. 8) Edit again, go to next page and launch prediction pattern test : You see that the checkboxes you had selected are not any more checked. Now apply the patch and redo all steps 2 to 6. 7) when you click on Save button, an alert warn you that you must launch the prediciton test before. Do it and save. 8) When you edit again and launch again the prediction test, you can see that the checkboxes selected are still checked. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Is it still ready to be signed off? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Koha Team Lyon 3 <koha@univ-lyon3.fr> 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.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Liz Rea <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Liz Rea <liz@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #46385|0 |1 is obsolete| | --- Comment #4 from Liz Rea <liz@catalyst.net.nz> --- Created attachment 46411 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46411&action=edit Bug 15501 - Planned Irregularities are deleted when modifying subscription Add a warning to avoid saving a subscription whitout testing prediction pattern when there are planned no published issues Signed-off-by: Liz Rea <liz@catalyst.net.nz> Ran through test plan, looks all good. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #46411|0 |1 is obsolete| | --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 46422 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46422&action=edit Bug 15501 - Planned Irregularities are deleted when modifying subscription Add a warning to avoid saving a subscription whitout testing prediction pattern when there are planned no published issues Signed-off-by: Liz Rea <liz@catalyst.net.nz> Ran through test plan, looks all good. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 David Cook <dcook@prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Brendan Gallagher <brendan@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |brendan@bywatersolutions.co | |m --- Comment #6 from Brendan Gallagher <brendan@bywatersolutions.com> --- Pushed to master - Thanks -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |julian.maurice@biblibre.com --- Comment #7 from Julian Maurice <julian.maurice@biblibre.com> --- Patch pushed to 3.22.x, will be in 3.22.2 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Frédéric Demians <frederic@tamil.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |frederic@tamil.fr --- Comment #8 from Frédéric Demians <frederic@tamil.fr> --- Isn't this line problematic? [% IF ( irregularity ) %]patternneedtobetested=1[% END %] Shouldn't it be (with ;): [% IF ( irregularity ) %]patternneedtobetested=1;[% END %] -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Frédéric Demians from comment #8)
Isn't this line problematic?
[% IF ( irregularity ) %]patternneedtobetested=1[% END %]
Shouldn't it be (with ;):
[% IF ( irregularity ) %]patternneedtobetested=1;[% END %]
I have just tested: no JS error with irregularities. But it's obviously an error, must be fixed. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 --- Comment #10 from Koha Team Lyon 3 <koha@univ-lyon3.fr> --- Arghhh! you are right, I will post a follow-up Olivier Crouzet -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15501 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=15657 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org