Bug #88418

CType table broken with empty space

Added by Franz Kugelmann 3 months ago. Updated 16 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
Content Rendering
Target version:
-
Start date:
2019-05-22
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
CodingNight
Complexity:
Is Regression:
Sprint Focus:

Description

How to replicate:
Create a content element table with following structure and leave all other settings with default values

1a | 1b | 1c
2a | 2b | 
3a | 3b | 3c

Add an empty space at the end of line 2. (Could easily happen, ie. if you remove the content from that cell, but leave the empty space)

Result:
Table is broken, because the empty space is handled as start of new table cell, but this cell is never closed, thus merging the whole line 3 into that cell.

Possible fix, add following TypoScript:

tt_content.table {
    dataProcessing {
        10 {
            fieldEnclosure.if.value = 0
            fieldEnclosure.if.equals.field = table_enclosure
            fieldEnclosure.if.negate = 1
        }
    }
}

Applies also for 8 LTS

Associated revisions

Revision 87837b5a (diff)
Added by Michael Blunck 16 days ago

[BUGFIX] Prevent tt_content.table being broken if last field is empty

This only applies to table elements where no text enclosure
is configured.

Resolves: #88418
Releases: master, 9.5
Change-Id: I8ba7cefe0da37fd7987858c1302dac9f3ba2ff10
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61423
Tested-by: TYPO3com <>
Tested-by: Marco Bresch <>
Tested-by: Susanne Moog <>
Reviewed-by: Marco Bresch <>
Reviewed-by: Susanne Moog <>

Revision cebfc475 (diff)
Added by Michael Blunck 16 days ago

[BUGFIX] Prevent tt_content.table being broken if last field is empty

This only applies to table elements where no text enclosure
is configured.

Resolves: #88418
Releases: master, 9.5
Change-Id: I8ba7cefe0da37fd7987858c1302dac9f3ba2ff10
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61366
Tested-by: TYPO3com <>
Tested-by: Susanne Moog <>
Reviewed-by: Susanne Moog <>

History

#1 Updated by Georg Ringer about 1 month ago

  • Status changed from New to Accepted

#2 Updated by Riccardo De Contardi 17 days ago

  • Tags set to CodingNight

#3 Updated by Gerrit Code Review 17 days ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61423

#4 Updated by Gerrit Code Review 16 days ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61366

#5 Updated by Michael Blunck 16 days ago

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

Also available in: Atom PDF