Project

General

Profile

Actions
Copy link

Bug #80394

closed

ckeditor: js error when changing a link

Added by Eckard Gehrke about 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
8 LTS
Start date:
2017-03-22
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

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.

Actions
Copy link
 #1

Updated by Benni Mack about 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
Actions
Copy link
 #2

Updated by Frank Nägler about 7 years ago

  • Category changed from Backend JavaScript to RTE (rtehtmlarea + ckeditor)
Actions
Copy link
 #3

Updated by Frank Nägler about 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.

Actions
Copy link
 #4

Updated by Eckard Gehrke about 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.

Actions
Copy link
 #5

Updated by Riccardo De Contardi about 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.

Actions
Copy link

Also available in: Atom PDF