Link field for images is too short to provide links for every image
The field should be an textarea like it is for "alt" and "title" text.
Long and anoying discussion on the german newsgroup
The problem is very old and Gerhard Obermayr told about that about 2 years ago.
(issue imported from #M16462)
#1 Updated by Kay Strobach about 9 years ago
image_link varchar(255) DEFAULT '' NOT NULL,
image_link text DEFAULT '' NOT NULL,
change typo3/sysext/cms/tbl_tt_content.php: around 822
'type' => 'input',
'size' => '50',
'type' => 'text',
'cols' => '30',
'rows' => '3',
additionally we need to change the seperation to breaks instead of comma (or both?) in the template (css_styled_content i assume)
#2 Updated by Jo Hasenau about 9 years ago
Actually a first step has been made already, since the current TCA says:
'max' => '51200'
which is the maximum number of images (200) multiplied by a maximum value of 256 per URL
Original DIFF can be found here: #23766
Since this was my patch, I just implemented a resized DB field as well, but: The patch was just about the TCA, so the DB field size had to be removed before committing it.
IMHO we could do a follow up to this patch during the beta phase, since the current size of the DB field doesn't match the TCA.
#8 Updated by Jo Hasenau about 9 years ago
On the other hand, the URL definition http://www.ietf.org/rfc/rfc1738.txt says:
Thus, only alphanumerics, the special characters "$-_.+!*'(),", and reserved characters used for their reserved purposes may be used unencoded within a URL.
Which means: A comma is a legal part of a URL thus it must NOT be used as a separator.
This is what #18656 is about, so this one is related as well.
So after all it seems we do need a conversion script to fix all these bugs in one go.
#14 Updated by Jo Hasenau about 9 years ago
So do we agree on the following fixes to be necessary:
Change the DB field type to "text", because anything else won't fit 200 times the maximum size of a legal URL
Change the separation of links from CSV to linebreaks, because a comma can be a legal character used in a non encoded URL
Change the TCE form element to textarea, because this allows the necessary line breaks
#23 Updated by Jo Hasenau about 9 years ago
Attached another variant of the patch that will separate "old" field values by comma and new ones by line break
As soon as there is at least one linebreak available in the value, it must be a "new" one - therefor I changed the JS function to always include a linebreak at the end of the value
#25 Updated by Jo Hasenau about 9 years ago
Attached v2 of the alternative version that fixes an issue with the tabs in the link browser wizard not working when there are already 2 or more links in the textarea.
This will apply to the other version as well, so I will provide a v2 for that as well as soon as we got a decision which way to go.