[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com Version(s)|24.05.00,23.11.02 |24.05.00,23.11.02,23.05.08 released in|| Status|Pushed to stable|Pushed to oldstable --- Comment #32 from Lucas Gass --- Backported to 23.05.x for upcoming 23.05.08 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Fridolin Somers changed: What|Removed |Added Status|Pushed to master|Pushed to stable CC||fridolin.som...@biblibre.co ||m Version(s)|24.05.00|24.05.00,23.11.02 released in|| --- Comment #31 from Fridolin Somers --- Pushed to 23.11.x for 23.11.02 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer changed: What|Removed |Added Keywords|additional_work_needed | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #30 from Katrin Fischer --- Pushed for 24.05! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer changed: What|Removed |Added Version(s)||24.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #29 from Katrin Fischer --- Forget the last part of the previous comment, only keep: I added a follow-up to use our standard validation with our own error messages, see https://wiki.koha-community.org/wiki/Coding_Guidelines#JS10:_Form_Validation. You can see the difference in how the error messages display. A proper restart_all fixes most issues... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer changed: What|Removed |Added Keywords||additional_work_needed --- Comment #28 from Katrin Fischer --- I added a follow-up to use our standard validation with our own error messages, see https://wiki.koha-community.org/wiki/Coding_Guidelines#JS10:_Form_Validation. You can see the difference in how the error messages display. But we still have a big issue here on saving: C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Incorrect integer value: '' for column `koha_kohadev`.`subscription_numberpatterns`.`add1` at row 1 at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 70 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 This appears for all 3 columns and all numeric values in the prediction pattern. NONE of these should be required or only be required if you have at least one input field in the column (more complicated, let's go with not exploding for now. Please follow-up here or on a separate bug. Additionally we explode when display order is left empty. This should be possible as well (compare to adding from the subscription form, where the field is not included). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #27 from Nick Clemens --- Created attachment 160250 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160250=edit Bug 28012: (QA follow-up) Add numeric imput mode to display order field Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #26 from Nick Clemens --- Created attachment 160249 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160249=edit Bug 28012: Creating numbering pattern: mark mandatory fields required If any of these fields were missing, notably the Numbering formula then saving the new numbering pattern silently failed, it didn't give any error but it didn't save any new pattern either. Note that there is a plan to make description field optional, so even if it currently is mandatory on the DB level we leave it optional here (and due to people using production DB's in non-strict mode which allows this to be optional). To test: 1) After applying this patch make sure all the 3 fields must be filled in /cgi-bin/koha/serials/subscription-numberpatterns.pl?op=new page and it cannot be saved without those. Signed-off-by: David Nind Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Nick Clemens changed: What|Removed |Added Attachment #141947|0 |1 is obsolete|| Attachment #141948|0 |1 is obsolete|| --- Comment #25 from Nick Clemens --- Created attachment 160248 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160248=edit Bug 28012: Use definedness test instead of bool when checking whether to null fields Saving a new numbering pattern didn't work without having to fill all the fields, even those that are not mandatory. To test: 1) Go to /cgi-bin/koha/serials/subscription-numberpatterns.pl and try creating a new pattern, notice that only Name and Description are mandatory after applying this patch 2) Make sure editing existing numbering patterns still works Signed-off-by: David Nind Signed-off-by: Nick Clemens -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Nick Clemens changed: What|Removed |Added Status|Failed QA |Passed QA Depends on||31297 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31297 [Bug 31297] Cannot add new subscription patterns from edit subscription page -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Jonathan Druart changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl --- Comment #24 from Jonathan Druart --- *** Bug 33678 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #23 from Joonas Kylmälä --- (In reply to Kyle M Hall from comment #22) > I cannot leave description empty: > C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: > DBD::mysql::st execute failed: Column 'description' cannot be null at > /kohadevbox/koha/serials/subscription-numberpatterns.pl line 73 > at /usr/share/perl5/DBIx/Class/Exception.pm line 77 See the explanation in the patch "Creating numbering pattern: mark mandatory fields required": [...] Note that there is a plan to make description field optional, so even if it currently is mandatory on the DB level we leave it optional here (and due to people using production DB's in non-strict mode which allows this to be optional). [...] Also, I believe this was discussed with Katrin in this bug report as well. > > "Display order" should be set to only allow integer inputs. It will be a separate patch I believe, would it be possible for someone to add a new patch here or submit it in a separate report? I don't have koha development environment set up anymore unfortunately. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Kyle M Hall changed: What|Removed |Added CC||k...@bywatersolutions.com Status|Signed Off |Failed QA --- Comment #22 from Kyle M Hall --- I cannot leave description empty: C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Column 'description' cannot be null at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 73 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 "Display order" should be set to only allow integer inputs. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Joonas Kylmälä changed: What|Removed |Added Status|Failed QA |Signed Off --- Comment #21 from Joonas Kylmälä --- (In reply to Katrin Fischer from comment #20) > As noted in comments before this one, I don't think that making the > description mandatory is the right solution here. I addressed this after the last time you commented on this, please see the patch again, it is not mandatory. Setting the status back to Signed off, let me know if there is something I'm not understanding. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #20 from Katrin Fischer --- As noted in comments before this one, I don't think that making the description mandatory is the right solution here. These features work currently for all libraries as we don't set strict on production systems. So making them required, will be a regression. I confirmed that without the strict, it's possible to save numbering patterns in master with only name and numbering pattern set. We should instead fix it on database level, perhaps by making the field nullable. Note: bug 31297 was supposed to take care of this, but I believe Joonas is no longer around and it's currently in FQA. So we cannot expect a fix there right now. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer changed: What|Removed |Added Summary|Error on Saving new |Error on saving new |numbering pattern |numbering pattern -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 David Nind changed: What|Removed |Added Text to go in the||This fixes the serials new release notes||numbering pattern input ||form so that the name and ||numbering formula fields ||are marked as required. ||Before this, there was no ||indication that these ||fields were required and ||error trace messages were ||displayed if these were not ||completed - saving a new ||pattern or editing an ||existing pattern would also ||silently fail. || ||NOTE: ||Making the description ||field optional will be ||fixed in bug 31297. Until ||this is done, a value needs ||to be entered into this ||field - even though it ||doesn't indicate that it is ||required. CC||da...@davidnind.com --- Comment #19 from David Nind --- Testing notes (using koha-testing-docker) - I hope I got this right! Replicate the issue: 1. Go to Serials > Manage numbering patterns. 2. Add a new numbering pattern, but leave the description field blank (copy an existing numbering pattern such as Number) - note that no fields are marked as required: ==> C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Incorrect integer value: '' for column `koha_kohadev`.`subscription_numberpatterns`.`add2` at row 1 at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 70 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 3. Repeat step 2 but leave the display order field blank: ==> C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Incorrect integer value: '' for column `koha_kohadev`.`subscription_numberpatterns`.`displayorder` at row 1 at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 70 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 4. Repeat step 2, leaving name or numbering formula fields blank: ==> adding the new numbering pattern silently fails - you are return to the list of numbering patterns and the your new numbering pattern is not saved. 5. Fill in all the fields (name, description, numbering formula, and display order) and save: ==> C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Incorrect integer value: '' for column `koha_kohadev`.`subscription_numberpatterns`.`add2` at row 1 at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 70 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 6. Edit an existing pattern with the various combinations from steps 2-5 (for example, the existing number pattern) - no fields are marked as required, you get these results: . Name: C4::Serials::Numberpattern::ModSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Column 'label' cannot be null at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 95 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 . Numbering formula: C4::Serials::Numberpattern::ModSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Column 'numberingmethod' cannot be null at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 95 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 . Description: C4::Serials::Numberpattern::ModSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Column 'description' cannot be null at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 95 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 . Display order: can make blank or renumber This fixes the new numbering pattern input form so that the name and numbering formula fields are marked as required. Before this, there was no indication that these fields were required and error trace messages were
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 David Nind changed: What|Removed |Added Attachment #140322|0 |1 is obsolete|| --- Comment #18 from David Nind --- Created attachment 141948 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141948=edit Bug 28012: Creating numbering pattern: mark mandatory fields required If any of these fields were missing, notably the Numbering formula then saving the new numbering pattern silently failed, it didn't give any error but it didn't save any new pattern either. Note that there is a plan to make description field optional, so even if it currently is mandatory on the DB level we leave it optional here (and due to people using production DB's in non-strict mode which allows this to be optional). To test: 1) After applying this patch make sure all the 3 fields must be filled in /cgi-bin/koha/serials/subscription-numberpatterns.pl?op=new page and it cannot be saved without those. Signed-off-by: David Nind -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 David Nind changed: What|Removed |Added Attachment #140196|0 |1 is obsolete|| --- Comment #17 from David Nind --- Created attachment 141947 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=141947=edit Bug 28012: Use definedness test instead of bool when checking whether to null fields Saving a new numbering pattern didn't work without having to fill all the fields, even those that are not mandatory. To test: 1) Go to /cgi-bin/koha/serials/subscription-numberpatterns.pl and try creating a new pattern, notice that only Name and Description are mandatory after applying this patch 2) Make sure editing existing numbering patterns still works Signed-off-by: David Nind -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 David Nind changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #16 from Katrin Fischer --- Just a note: Patch doesn't apply means you could't test because the patches couldn't be applied to the installation. If you find an error in testing, the status is "Failed QA" (not limited to QA team) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Joonas Kylmälä changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff --- Comment #15 from Joonas Kylmälä --- (In reply to Emmanuel Bétemps from comment #13) > I still can edit existing numbering patterns but if a delete the > description, an error occurs when I (try to) save : It's expected, as explained in the second patch's description. This bug is left to be fixed in bug 31297. You also quoted in your message the test plan for the first patch btw, in the first patch's test plan I meant with mandatory that it is mandatory to fill those to prevent the error. The second plan then concerns the GUI changes, and it has its own test plan. > I can't create a new one : > > C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: > DBD::mysql::st execute failed: Incorrect integer value: '' for column > `koha_test`.`subscription_numberpatterns`.`add2` at row 1 at > /kohadevbox/koha/serials/subscription-numberpatterns.pl line 70 > at /usr/share/perl5/DBIx/Class/Exception.pm line 77 Did you remember to run restart_all after applying patches to kohadevbox? This is the exact error the first patch attached solves. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Emmanuel Bétemps changed: What|Removed |Added Status|Failed QA |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #14 from Emmanuel Bétemps --- Created attachment 140337 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=140337=edit screenshot Modify a pattern -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Emmanuel Bétemps changed: What|Removed |Added Status|Needs Signoff |Failed QA CC||e.bete...@gmail.com --- Comment #13 from Emmanuel Bétemps --- (In reply to Joonas Kylmälä from comment #4) > To test: > 1) Go to /cgi-bin/koha/serials/subscription-numberpatterns.pl > and try creating a new pattern, notice that only Name and > Description are mandatory after applying this patch Hi, after applying the patch, when I want to create a new pattern : - Name is labeled as mandatory - Description is not labeled as mandatory - Numbering formula is labeled as mandatory (see attachment 'Screenshot - Modify pattern.png') I still can edit existing numbering patterns but if a delete the description, an error occurs when I (try to) save : C4::Serials::Numberpattern::ModSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Column 'description' cannot be null at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 95 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 I can't create a new one : C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Incorrect integer value: '' for column `koha_test`.`subscription_numberpatterns`.`add2` at row 1 at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 70 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Joonas Kylmälä changed: What|Removed |Added Attachment #140197|0 |1 is obsolete|| --- Comment #12 from Joonas Kylmälä --- Created attachment 140322 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=140322=edit Bug 28012: Creating numbering pattern: mark mandatory fields required If any of these fields were missing, notably the Numbering formula then saving the new numbering pattern silently failed, it didn't give any error but it didn't save any new pattern either. Note that there is a plan to make description field optional, so even if it currently is mandatory on the DB level we leave it optional here (and due to people using production DB's in non-strict mode which allows this to be optional). To test: 1) After applying this patch make sure all the 3 fields must be filled in /cgi-bin/koha/serials/subscription-numberpatterns.pl?op=new page and it cannot be saved without those. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Joonas Kylmälä changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #11 from Joonas Kylmälä --- (In reply to Katrin Fischer from comment #8) > I just confirmed that you can save a numbering pattern in a production 20.11 > (package installation) without adding a description. Possibly due to DBMS > configuration (non-strict?). I missed this. So maybe remove it directly from here if in non-strict mode you can avoid the DB error. Then after that no need to adjust patch depending on bug 31297. Will attach updated patch soon. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #10 from Joonas Kylmälä --- (In reply to Joonas Kylmälä from comment #9) > I was thinking of just focusing here on the UI side since description is > mandatory on the DB level. Is it okay for you if we focus on the whether > description is mandatory on bug 3129? If so, and bug 3129 gets merged then > this patch might have to be adjusted but until it is then I think there's no > problem with this patch, if you agree can you change the status back to need > sign-off? I meant to type bug 31297. Typo. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #9 from Joonas Kylmälä --- (In reply to Katrin Fischer from comment #8) > As we have name - which is not a code in this case - we should not make > description mandatory. > > I just confirmed that you can save a numbering pattern in a production 20.11 > (package installation) without adding a description. Possibly due to DBMS > configuration (non-strict?). > > So there should be a lot of patterns out there without description already, > that would then throw an error on editing. > > Same fix as for bug 31297? I was thinking of just focusing here on the UI side since description is mandatory on the DB level. Is it okay for you if we focus on the whether description is mandatory on bug 3129? If so, and bug 3129 gets merged then this patch might have to be adjusted but until it is then I think there's no problem with this patch, if you agree can you change the status back to need sign-off? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #8 from Katrin Fischer --- As we have name - which is not a code in this case - we should not make description mandatory. I just confirmed that you can save a numbering pattern in a production 20.11 (package installation) without adding a description. Possibly due to DBMS configuration (non-strict?). So there should be a lot of patterns out there without description already, that would then throw an error on editing. Same fix as for bug 31297? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #7 from Katrin Fischer --- (In reply to Joonas Kylmälä from comment #6) > Added a couple patches that make adding a new numbering pattern usable, I > don't understand how it can have been, apparently already since bug 7688 / > year 2013, this broken and nobody has fixed this. > > Further improvement can be done, like I don't think description should be > mandatory but will leave this like this to be easily reviewable patch so we > can make this feature usable ASAP. > > Ready for sign-off. If this relates to MySQL 5.7 database settings might mitigate the issue (non-strict?) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Joonas Kylmälä changed: What|Removed |Added CC||joonas.kylm...@iki.fi Severity|minor |normal Assignee|koha-b...@lists.koha-commun |joonas.kylm...@iki.fi |ity.org | --- Comment #6 from Joonas Kylmälä --- Added a couple patches that make adding a new numbering pattern usable, I don't understand how it can have been, apparently already since bug 7688 / year 2013, this broken and nobody has fixed this. Further improvement can be done, like I don't think description should be mandatory but will leave this like this to be easily reviewable patch so we can make this feature usable ASAP. Ready for sign-off. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #5 from Joonas Kylmälä --- Created attachment 140197 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=140197=edit Bug 28012: Creating numbering pattern: mark mandatory fields required If any of these fields were missing, notably the Numbering formula then saving the new numbering pattern silently failed, it didn't give any error but it didn't save any new pattern either. To test: 1) After applying this patch make sure all the 3 fields must be filled in /cgi-bin/koha/serials/subscription-numberpatterns.pl?op=new page and it cannot be saved without those. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #4 from Joonas Kylmälä --- Created attachment 140196 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=140196=edit Bug 28012: Use definedness test instead of bool when checking whether to null fields Saving a new numbering pattern didn't work without having to fill all the fields, even those that are not mandatory. To test: 1) Go to /cgi-bin/koha/serials/subscription-numberpatterns.pl and try creating a new pattern, notice that only Name and Description are mandatory after applying this patch 2) Make sure editing existing numbering patterns still works -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Joonas Kylmälä changed: What|Removed |Added Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Nick Clemens changed: What|Removed |Added CC||n...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Jonathan Druart changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=31297 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #3 from Katrin Fischer --- I think it would be nice if the field could remain empty. Changing the display numbering can be tedious and often just alphabetic or chronological ordering might be preferred. How is it done when a numbering pattern is added from the subscription form? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 --- Comment #2 from Owen Leonard --- I get an different error, "C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Field 'description' doesn't have a default value" -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Jonathan Druart changed: What|Removed |Added Blocks||17258 CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #1 from Jonathan Druart --- Globally this form is missing data validation. We could add a default value for displayorder (1?) at DBMS level but then we get another one: Incorrect integer value: '' for column 'add1' and so on... Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17258 [Bug 17258] [OMNIBUS] MySQL 5.7 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Katrin Fischer 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. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28012] Error on Saving new numbering pattern
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012 Christian Stelzenmüller changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=27397 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
