I was trying to use beautytips_manager on some views classes and got some fatal errors because the form element didn't have a maxlength specified so when it tried to save my values they were too long for the field. The patch I'll post once I've got an issue number extends the length to 255 and sets a maxlength on the form element.

CommentFileSizeAuthor
#2 beautytips_1390802.patch1.83 KBdrewish
#1 beautytips_1390802.patch64 bytesdrewish
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

drewish CreditAttribution: drewish commented
FileSize
beautytips_1390802.patch64 bytes
drewish’s picture

drewish CreditAttribution: drewish commented
FileSize
beautytips_1390802.patch1.83 KB

whoops forgot --stdout.

kleinmp’s picture

kleinmp CreditAttribution: kleinmp commented

Thanks for the patch. Committed it here. Also fixed the issue in the 6.x-2.x-dev branch.

kleinmp’s picture

kleinmp CreditAttribution: kleinmp commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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