I need this module because I need to see the number of characters in the field, but I don't need to limit the length. I can specify maxlength as 99999 or something, but I would like to be able to just display the counter in the desired field. Thank you!

CommentFileSizeAuthor
#5 interdiff-2329441-2-5.txt1.05 KBJeroenT
#5 unlimited_field_just_a-2329441-5.patch1.79 KBJeroenT
#2 unlimited_field_just_a-2329441-2.patch1.79 KBJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: Unlimited field. Just a count! » Unlimited field, just a character count
Component: Code » User interface
Related issues: +#1918738: Character count on fields via Maxlength integration (D7)

That would be useful for integration with e.g. Metatag (#1918738: Character count on fields via Maxlength integration (D7)).

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Status: Active » Needs review
FileSize
unlimited_field_just_a-2329441-2.patch1.79 KB

I created a simple initial patch for this. I added a new element to the form 'maxlength_js_limit'.

How to test? Add maxlength_js and maxlength_js_limit to your form element :

$form['foo'] = array(
'#type' => 'textfield',
'#maxlength_js' => TRUE,
'#maxlength_js_limit' => 150,
);

I add the 'maxlength_js_limit' as a data-attribute on the form element and in javascript, I check if this data-maxlength-js-limit attribute exists. When it exists, I set the enforce option to false.

What do you think? Any suggestions?

seanB’s picture

seanB
Primary language Dutch
Location Netherlands
CreditAttribution: seanB commented
Status: Needs review » Reviewed & tested by the community

Seems to work fine! +1

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Needs work

One minuscule thing - the "if" statements should have a space after them for both files to match the Drupal coding standards.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Needs work » Needs review
FileSize
unlimited_field_just_a-2329441-5.patch1.79 KB
interdiff-2329441-2-5.txt1.05 KB

Added the code styling updates.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Needs review » Reviewed & tested by the community
weri’s picture

weri CreditAttribution: weri at Previon Plus AG commented
Related issues: +#2329441: Unlimited field, just a character count

This is related to #1841028: Add field setting "Do not restrict maxlength on writing"

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

@weri: The link you provided was to this issue, did you mean to link to a different one?

weri’s picture

weri CreditAttribution: weri at Previon Plus AG commented
Related issues: -#2329441: Unlimited field, just a character count +#1841028: Add field setting "Do not restrict maxlength on writing"

Thank you @DamienMcKenna for the hint!! I linked now the correct issue.

cedewey’s picture

cedewey
he/him
Primary language English
Location Denver, CO
CreditAttribution: cedewey at DevCollaborative, LLC commented
Status: Reviewed & tested by the community » Closed (won't fix)

We are putting a feature freeze on the Drupal 7 version of the module, so I'm marking this Closed, won't fix. Thank you everyone for working on this issue. If you do want to maintain the Drupal 7 version, do reach out. We'd be happy to bring you on board as a maintainer.

Also, this is a feature that exists in the Drupal 8/9 version so I do encourage people to upgrade to Drupal 8/9 and enjoy this enhancement along with the many other benefits of the latest version of Drupal.