Bug 27397

Summary: Serials: The description input field when defining numbering patterns is too short
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: SerialsAssignee: Catherine <catherinem2003>
Status: CLOSED FIXED QA Contact: Martin Renvoize <martin.renvoize>
Severity: trivial    
Priority: P5 - low CC: amitddng135, andrewfh, catherinem2003, christian.stelzenmueller, fridolin.somers, jonathan.druart, martin.renvoize, victor
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28012
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00,20.11.05,20.05.11
Attachments: Bug 25054: (QA follow-up) Add space after comma for multiple aliases
Bug 27397: The description input field when defining numbering patterns is too short
Bug 27397: The description input field when defining numbering patterns is too short
Bug 27397: The description input field when defining numbering patterns is too short
Bug 27397: The description input field when defining numbering patterns is too short

Description Katrin Fischer 2021-01-11 21:34:33 UTC
When adding or editing a numbering pattern in the serials module the description input field is quite short which makes adding and editing notes unnecessarily hard.

To test:
- go to the serials module
- click on manage numbering patterns in the left hand navigation
- add or edit a pattern
- check the description field is as short as the name field, but can take much more text
Comment 1 Catherine 2021-01-18 23:27:50 UTC Comment hidden (obsolete)
Comment 2 Catherine 2021-01-19 02:33:25 UTC
Created attachment 115289 [details] [review]
Bug 27397: The description input field when defining numbering patterns is too short

Test Plan:
1. go to the serials module
2. click on manage numbering patterns in the left hand navigation
3. add or edit a pattern
4. check the description field is as short as the name field, but can
take much more text
5. add patch
6. repeat steps 1-3
7. check the description field is larger than the name field (it should)

Sponsored by Catalyst IT
Comment 3 Jonathan Druart 2021-01-19 09:00:53 UTC
Hi Catherine,

I don't think the following change is needed:

-                  <input type="text" id="label" name="label" value="[% label | html %]" />
+                  <input type="text" id="label" name="label" value="[% label | html %]" >
Comment 4 Catherine 2021-01-19 21:07:55 UTC
Created attachment 115361 [details] [review]
Bug 27397: The description input field when defining numbering patterns is too short

Test Plan:
1. go to the serials module
2. click on manage numbering patterns in the left hand navigation
3. add or edit a pattern
4. check the description field is as short as the name field, but can
take much more text
5. add patch
6. repeat steps 1-3
7. check the description field is larger than the name field (it should)

Sponsored by Catalyst IT
Comment 5 Christian Stelzenmüller 2021-03-18 07:57:00 UTC
I don't know wheter this has something to do with this patch. I got an error after klicking on the Save button for saving a new numbering pattern at testing this patch. 
The description field is indeed longer than before.

First lines of error trace:

C4::Serials::Numberpattern::AddSubscriptionNumberpattern(): DBI Exception: DBD::mysql::st execute failed: Incorrect integer value: '' for column `koha_christian8`.`subscription_numberpatterns`.`displayorder` at row 1 [for Statement "INSERT INTO subscription_numberpatterns(label,description,numberingmethod,displayorder,label1,label2,label3,add1,add2,add3,every1,every2,every3,setto1,setto2,setto3,whenmorethan1,whenmorethan2,whenmorethan3,numbering1,numbering2,numbering3) VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)" with ParamValues: 0="blablub", 1="blablubdescription", 2="{X}, H. {Y} {Z} ", 3="", 4="Jahr", 5="Jahrgang", 6="Heft", 7="1", 8="1", 9="1", 10="12", 11="6", 12="1", 13="1", 14="1", 15="1", 16="9999", 17="9999", 18="6", 19="", 20="", 21=""] at /kohadevbox/koha/serials/subscription-numberpatterns.pl line 62
 at /usr/share/perl5/DBIx/Class/Exception.pm line 77
Comment 6 Christian Stelzenmüller 2021-03-18 08:16:59 UTC
Addition: if i'm emptying the Numbering formula field, i won't get an error after saving, but the New numbering pattern won't be saved (on the surface?)
Comment 7 Jonathan Druart 2021-03-18 09:36:27 UTC
The order appears when you create a new numbering patterns and the "display order" is empty. It's a bug that is not directly related to this patch and bug report. Can you open a new one please?
Comment 8 Jonathan Druart 2021-03-18 09:36:51 UTC
(In reply to Jonathan Druart from comment #7)
> The order appears when you create a new numbering patterns and the "display
> order" is empty. It's a bug that is not directly related to this patch and
> bug report. Can you open a new one please?

s/order/error
Comment 9 Amit Gupta 2021-03-22 11:36:43 UTC
Created attachment 118590 [details] [review]
Bug 27397: The description input field when defining  numbering patterns is too short

Test Plan:
1. go to the serials module
2. click on manage numbering patterns in the left hand navigation
3. add or edit a pattern
4. check the description field is as short as the name field, but can
take much more text
5. add patch
6. repeat steps 1-3
7. check the description field is larger than the name field (it should)
Comment 10 Martin Renvoize 2021-03-25 14:04:46 UTC
Created attachment 118809 [details] [review]
Bug 27397: The description input field when defining numbering patterns is too short

Test Plan:
1. go to the serials module
2. click on manage numbering patterns in the left hand navigation
3. add or edit a pattern
4. check the description field is as short as the name field, but can
take much more text
5. add patch
6. repeat steps 1-3
7. check the description field is larger than the name field (it should)

Sponsored by Catalyst IT

Works OK!

Signed-off-by: Amit Gupta <amitddng135@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 11 Martin Renvoize 2021-03-25 14:05:03 UTC
Trivial display fix, Passing QA
Comment 12 Jonathan Druart 2021-04-01 16:53:57 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 13 Fridolin Somers 2021-04-15 10:34:42 UTC
Pushed to 20.11.x for 20.11.05
Comment 14 Andrew Fuerste-Henry 2021-04-24 16:33:55 UTC
Pushed to 20.05.x for 20.05.11
Comment 15 Victor Grousset/tuxayo 2021-04-24 17:16:52 UTC
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.