Bug #34634

Inserting Tables is broken in 4.5.12 and 4.6.5

Added by Jan Loderhose over 10 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Must have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
Start date:
2012-03-07
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

After upgrading TYPO3 to 4.5.12 and 4.6.5 inserting tables via RTE fails.

Latest FF says:

Error: HTMLArea.util.Color.colorToHex(element && element.style.borderColor ? element.style.borderColor : this.properties.borderColor &&
this.properties.borderColor.defaultValue ? this.properties.borderColor.defaultValue : "") is null

Source file: http://xxxxxx/typo3temp/compressor/htmlarea_cumulative_26d8390a9806dabb7c6c_compressed-
6e184a46173deb6d9f67984f1417affe.js.gzip
Line: 657

where as latest Chrome mumbles

Uncaught TypeError: Cannot call method 'substr' of null

The window to define the table parameters does not appear.


Related issues

Related to TYPO3 Core - Bug #54188: table propertiesClosedMathias Schreiber2013-12-04

Actions
#1

Updated by Christian Hennecke over 10 years ago

This also affects the properties dialogs of table, row, column, and cell. Same error message.

The error is related to the border settings. It does not occur if

disableBordersFieldsetInTableOperations = 1
#2

Updated by Stanislas Rolland over 10 years ago

  • Status changed from New to Accepted
  • Assignee set to Stanislas Rolland
  • Target version set to 4.5.13
#3

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/9465

#4

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/9466

#5

Updated by Stanislas Rolland over 10 years ago

Please test these changes for 4.5 and 4.6.

#6

Updated by Jan Loderhose over 10 years ago

Reviewed on gerrit. +1 for both patches.

#7

Updated by Stanislas Rolland over 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#8

Updated by Riccardo De Contardi almost 5 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF