Closed (fixed)
Project:
MaxLength
Version:
7.x-3.x-dev
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2011 at 08:10 UTC
Updated:
28 Mar 2014 at 12:37 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dawehnerHere is the current progress on the issue.
Comment #2
Coupon Code Swap commentedI tried adding the patch and it does add some settings but isn't working. How's the progress so far? Will this allow to attach Maxlength only to the summary and not the body?
Comment #3
schnitzel commentedThis is for the unsupported 2.x version, you should upgrade to the 3.x
dup of: #1306562: Word Count for summary field in D7
Comment #4
dixon_Actually, I will put the other post as duplicate since this issue actually contains a patch that otherwise will get lost.
I will hopefully have reasons to test and finalize the patch in #1 soon.
Comment #5
nagy.balint commentedI had to rewrite the _maxlength_children function almost completely, but i managed to solve this issue.
Might need to work on the UI, but basically when its a text with summary, now you can specify a maxlength, and a label. And it works independently from the main text's maxlength and label. So it is possible to have maxlength on summary only.
Comment #6
Tunox commentedTested the patch provided in #5 and it works fine on a local sandbox.
Is there any hope to see this integrated in the 3.x stable release?
Edit: I just noticed that for non-english sites the summary countdown message:
Content limited to @limit characters, remaining: <strong>@remaining</strong>is not being translated. Even an added custom message is not being taken into consideration which suggests this isn't an translation issue. Maybe the newly added default values for the summary field in this patch aren't being rendered.
Comment #7
dlg0 commentedJust applied the patch in #5
and it works great.
Thanks a lot!
Comment #8
havran commentedPatch from #5 works great. Maybe we need add checkboxes Force text truncate and Truncate html for summary field too?
Comment #9
vincenzo commentedAny chance this will ever be consider for merging?
I can amend it if it's required to.
Comment #10
jOksanen commentedAfter 4 succesfull tests + me, im marking as tested by community.
Comment #11
kylahuff commentedJust letting you know that I have also tested and worked a treat.
Thanks
Comment #12
aron novak#5 is committed.