Bug #80394
closedckeditor: js error when changing a link
0%
Description
Hi,
I was trying out the new ckeditor and found out that I get an js error when trying to change a link.
SCRIPT5007: Unable to get property 'length' of undefined or null reference
it happens in typo3link.js (71,9)
var additionalAttributes = editor.config.typo3link.additionalAttributes;
for (i = additionalAttributes.length; --i >= 0;) {
the object editor.config.typo3link has no additionalAttributes.
then it results in the above mentioned error:
var additionalAttributes = editor.config.typo3link.additionalAttributes;
for (i = additionalAttributes.length; --i >= 0;) {
when trying in the above coding to access additionalAttributes.length. Maybe there should be a check if the additionalAttributes is not null.
The result is thaton IE I can not change the link, have to tremove and create it new.
I am using latest typo3 version 8.6.1. if you wish I could retest a bugfix.
Best regards
Eckard.
Updated by Benni Mack over 7 years ago
- Status changed from New to Accepted
- Assignee set to Benni Mack
- Priority changed from Should have to Must have
- Target version set to 8 LTS
Updated by Frank Nägler over 7 years ago
- Category changed from Backend JavaScript to RTE (rtehtmlarea + ckeditor)
Updated by Frank Nägler over 7 years ago
- Status changed from Accepted to Needs Feedback
Hi Eckard,
can you please re-check the error with current master?
If the error still exists, please provide some more information about the browser. I need the version number to try to re-produce the bug.
Updated by Eckard Gehrke over 7 years ago
Hi Frank,
I checked with the latest version 87dev and it does not occur any more.
new coding var additionalAttributes = getAdditionalAttributes(editor);
returns now an empty table.
You can close the ticket. Seems that I can't change the status of the ticket.
Best regards
Eckard.
Updated by Riccardo De Contardi over 7 years ago
- Status changed from Needs Feedback to Closed
- Assignee deleted (
Benni Mack)
Hello Eckard, thank you for testing. I'm closing this issue.