Created attachment 10725 [details] screenshot When creating a new authority (example: PERS_NAME) and only entering 100$a, on saving the record an alert will show the missing mandatory fields. In the alert the descriptions of the messages are cut off and the tab number is missing (see screenshot).
This is still valid in current master. The tab numbers are not included in the error message and longer descriptions are cut off.
Created attachment 37405 [details] [review] Bug 8399: Display tab number on the mandatory field alert (authority) A wrong variable scope was used here, the number value is contained in the BIG_LOOP (note for later: change this variable name...). Test plan: 1/ Define 1+ mandatory fields for an authority type 2/ Create an authority and don't fill the mandatory fields 3/ Try to save: the popup alert should display the number of the tab.
*** Bug 8830 has been marked as a duplicate of this bug. ***
This patch only seems to solve one of the two issues in the bug. The tab number are there but longer descriptions are still being cut off. Please correct me if i have missed something or misunderstood.
(In reply to Andreas Hedström Mace from comment #4) > This patch only seems to solve one of the two issues in the bug. The tab > number are there but longer descriptions are still being cut off. Please detail, I don't understand the problem here.
Created attachment 37795 [details] Screenshot
Added a screenshot of the issue. The description of the fields are still being cut off, like "fixed length control field" turning up as "fixed length control f". Hope this helps, otherwise ask again! =)
Created attachment 37800 [details] [review] Bug 8399: No need to truncate the marc field text to 22 chars The text for the marc field are truncated to 22 chars when displaying an error on mandatory fields. This limitation is useless and this patch removes it. Test plan: Create a mandatory subfield and set a long text (more than 22 chars) edit or add an authority, click on save. You should get the error displayed with the full text Note that this patch modifies the same code for biblio. Test plan is the same for biblio.
(In reply to Andreas Hedström Mace from comment #7) > Added a screenshot of the issue. The description of the fields are still > being cut off, like "fixed length control field" turning up as "fixed length > control f". Hope this helps, otherwise ask again! =) Perfect thanks! The last patch fixes that. Same on the biblio side.
Created attachment 37804 [details] [review] Bug 8399: No need to truncate the marc field text to 22 chars The text for the marc field are truncated to 22 chars when displaying an error on mandatory fields. This limitation is useless and this patch removes it. Test plan: Create a mandatory subfield and set a long text (more than 22 chars) edit or add an authority, click on save. You should get the error displayed with the full text Note that this patch modifies the same code for biblio. Test plan is the same for biblio. Signed-off-by: Andreas Hedström Mace <andreas.hedstrom.mace@sub.su.se>
Created attachment 37995 [details] [review] [PASSED QA] Bug 8399: No need to truncate the marc field text to 22 chars The text for the marc field are truncated to 22 chars when displaying an error on mandatory fields. This limitation is useless and this patch removes it. Test plan: Create a mandatory subfield and set a long text (more than 22 chars) edit or add an authority, click on save. You should get the error displayed with the full text Note that this patch modifies the same code for biblio. Test plan is the same for biblio. Signed-off-by: Andreas Hedström Mace <andreas.hedstrom.mace@sub.su.se> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 37996 [details] [review] [PASSED QA] Bug 8399: Display tab number on the mandatory field alert (authority) A wrong variable scope was used here, the number value is contained in the BIG_LOOP (note for later: change this variable name...). Test plan: 1/ Define 1+ mandatory fields for an authority type 2/ Create an authority and don't fill the mandatory fields 3/ Try to save: the popup alert should display the number of the tab. Signed-off-by: Andreas Hedström Mace <andreas.hedstrom.mace@sub.su.se> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Patches pushed to master. Thanks Jonathan!