Summary

If no minimum provided code doesn't work. Because sometimes minumum value is in js is undefined. Added a check in code.

Steps to reproduce

  • Configure a text field to have a "Soft length limit" at any content type.
  • Don't set any value at "Soft length minimum"
  • Save
  • Go to the add node form, type some chars at the configured field.
  • The warning message is not displayed and we get a JS error.
CommentFileSizeAuthor
#3 interdiff-3119953-1-3.txt420 bytesmrinalini9
#3 soft_length_no_min-3119953-3.patch1.02 KBmrinalini9
soft_length_no_min.patch479 bytesarthur.baghdasar

Comments

arthur.baghdasar created an issue. See original summary.

ahebrank’s picture

ahebrank commented
Status: Active » Needs review

Thanks for the patch. I can replicate; without the patch if min-length is left blank you get a JS error and no counter.

mrinalini9’s picture

mrinalini9 commented
StatusFileSize
new soft_length_no_min-3119953-3.patch1.02 KB
new interdiff-3119953-1-3.txt420 bytes

Fixing test groups issue in the above patch, please review.

jlbellido’s picture

jlbellido
Primary language Spanish
commented
Issue summary: View changes
aaron.ferris’s picture

aaron.ferris commented
Status: Needs review » Reviewed & tested by the community

Can confirm this still occurs on the current 2.x branch, this patch looks and works well.

ahebrank’s picture

ahebrank commented
Version: 8.x-1.0 » 2.0.x-dev
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.