Closed (fixed)
Project:
Link
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
27 Jan 2012 at 13:45 UTC
Updated:
17 Jul 2017 at 12:49 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
rvilarAttaching a patch that solves this feature request.
Comment #2
randyhook commentedThe patch works great for me.
I was a little confused with the patch's wording beneath the Static text field:
This title will always be used if "Static Title" is selected above or like a default title if "Optional title" is selected above.And wonder if it should be changed to:
This title will 1) always be used if "Static Title" is selected above or 2) used if "Optional title" is selected above and no title is entered when creating content.Comment #3
jcfiala commentedI'm willing to apply this ticket, but it no longer applies cleanly to the link module. Any chance you can do a quick fixup for me?
Comment #4
num37 commentedI came across this issue when looking for exactly this behavior. I've re-rolled the patch, applied randyhook's suggestion in #2, and added some text of my own to the title field's description as seen on a node add/edit form. The new description text shows up if the title is set to optional and a static/default title has been defined, like so:
The link title is limited to @maxlength characters maximum. If blank, "%default" will be used by default.This is my first patch contribution, so fingers crossed...
Comment #5
num37 commentedComment #8
num37 commentedAs far as I can tell, none of the failed tests had anything to do with the patch I uploaded, so I think the patch itself is still worth reviewing. If I've done something wrong, let me know!
Comment #9
renatog commentedHi people. Thank you very much.
I applied the new patch and works good for me.
@num37, your patch is ok. It was only necessary to re-roll your patch. (In attachment)
Thank you very much for contribution.
Good Work and Good Week.
Regards.
Comment #10
renatog commentedComment #12
renatog commentedFixed.
Commited in dev branch.
Thanks people.
Regards.