? Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
25 Jul 2015

Issue

As explained in #7539 the version note will be truncated without notice if it is longer than 255 characters. That is because the database field is no bigger. See https://github.com/joomla/joomla-cms/blob/staging/installation/sql/mysql/joomla.sql#L1724

Solution

Add the HTML maxlength attribute so the user isn't able to enter more than 255 characters into the version note field

Testing

  • Add a long string (more than 255 characters) into the version note field in any of the forms supporting it. Eg article, banner, contact. Check after saving and see that the string was truncated. You can use this string:
abcdefghijklmnopqrstuvwxyz1abcdefghijklmnopqrstuvwxyz2abcdefghijklmnopqrstuvwxyz3abcdefghijklmnopqrstuvwxyz4abcdefghijklmnopqrstuvwxyz5abcdefghijklmnopqrstuvwxyz6abcdefghijklmnopqrstuvwxyz7abcdefghijklmnopqrstuvwxyz8abcdefghijklmnopqrstuvwxyz9abcdefghijklmnopqrstuvwxyz10
  • Apply patch and try to enter the string again. See that it is truncated before saving already. You will not be able to paste the full string.
avatar Bakual Bakual - open - 25 Jul 2015
avatar Bakual Bakual - change - 25 Jul 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Jul 2015
Labels Added: ?
avatar brianteeman
brianteeman - comment - 25 Jul 2015

I guess the same should be applied to the Note field in Menus and Modules?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7541.

avatar Bakual
Bakual - comment - 25 Jul 2015

True. I was just looking for the version_note ones. But those have the same limit in the database, so adding it there would make sense.
Same also for categories and tags.

For the module it was currently already limited to 100 characters. I tracked it back to 2010 when the note was added. No clue why it was set to 100 when the database allows for 255. I changed it to 255 now.

avatar RickR2H
RickR2H - comment - 27 Jul 2015

Test successful. The note field text gets truncated when pasted.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7541.

avatar zero-24 zero-24 - alter_testresult - 31 Jul 2015 - RickR2H: Tested successfully
avatar zero-24 zero-24 - change - 31 Jul 2015
Category Administration UI/UX
avatar zero-24 zero-24 - change - 31 Jul 2015
Easy No Yes
avatar zero-24 zero-24 - change - 7 Aug 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 7 Aug 2015

Works as expected now :smile: Thanks. --> RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7541.

avatar joomla-cms-bot joomla-cms-bot - change - 7 Aug 2015
Labels Added: ?
avatar zero-24 zero-24 - test_item - 7 Aug 2015 - Tested successfully
avatar Kubik-Rubik Kubik-Rubik - change - 10 Aug 2015
Milestone Added:
avatar Kubik-Rubik Kubik-Rubik - reference | 5e782b3 - 10 Aug 15
avatar Kubik-Rubik Kubik-Rubik - merge - 10 Aug 2015
avatar Kubik-Rubik Kubik-Rubik - close - 10 Aug 2015
avatar Kubik-Rubik Kubik-Rubik - change - 10 Aug 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-08-10 18:35:21
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - close - 10 Aug 2015
avatar joomla-cms-bot joomla-cms-bot - close - 10 Aug 2015
avatar Kubik-Rubik
Kubik-Rubik - comment - 10 Aug 2015

Thank you @Bakual! Merged.

avatar joomla-cms-bot joomla-cms-bot - change - 10 Aug 2015
Labels Removed: ?
avatar Bakual Bakual - head_ref_deleted - 10 Aug 2015

Add a Comment

Login with GitHub to post a comment