To Support URL Tagging for long URL's:
Line 728 in curlypage.admin.inc
Currently:
// URL to open on curlypage click.
$form['details']['linking_settings']['link'] = array(
'#type' => 'textfield',
'#title' => t('URL'),
'#description' => t('URL to go when user click on the ad.'),
'#required' => TRUE,
'#default_value' => $curlypage->link,
'#size' => 40,
);
Needs to be changed to:
// URL to open on curlypage click.
$form['details']['linking_settings']['link'] = array(
'#type' => 'textfield',
'#title' => t('URL'),
'#description' => t('URL to go when user click on the ad.'),
'#required' => TRUE,
'#default_value' => $curlypage->link,
'#size' => 100,
'#maxlength' => 255,
);This now allows up to 255 characters entered into the field and now allows you to see the first 100 characters without scrolling.
Previously the field would not accept long tagged URL's in the Admin.
Comments
Comment #1
manfer commentedAs it seems the default 128 value for maxlength is not enough for the link parameter in some cases then it is not problem to raise it to 255. The database already accepts that length so there are no more changes to do.
About the view size of the textfield in my opinion 100 is a little high value so I have raised it from 40 to 80.
Done and pushed. New version 6.x-3.7 released that includes this change.
Comment #2
manfer commentedThis change needs to be ported to 7.x-1.x version too.
Comment #3.0
(not verified) commentedWhy I suggested the change.