2531
93%

Ran 09 Jan 2017 12:44PM UTC

Jobs 2

Files 175

Run time 19s

Badge

Embed ▾

pending completion

Build # 2531

Build Type

push

travis-ci

Committed by

NamelessCoder

Commit Message

[BUGFIX] Fix FlexForm empty nodes removal (#1273)

This commit changes the way Flux cleans a FlexForm. One of the things done by
this cleaning function was to delete all the empty `sheet` elements, because it
makes TYPO3 throw an exception in some cases.

But there was a problem in the way this feature was handled: looping on the
elements and removing elements at the same time could cause nodes to be skipped.
For instance, if three `sheet` nodes were empty in a row, the second one was
skipped because the first one was removed and the looping pointer was not
correctly updated.

The implemented solution to this (big) issue is to first loop on all `sheet`
nodes, and store the one which _should_ be deleted first, _then_ loop again on
this list and remove the nodes one by one.

Run Details

6082 of 6747 relevant lines covered (90.14%)

25.61 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	2531.1 (TYPO3_VERSION=">=7.6.13, <7.6.99")	09 Jan 2017 12:42PM UTC	0	90.14	Travis Job 2531.1
2	2531.2 (TYPO3_VERSION=">=7.6.13, <7.6.99")	09 Jan 2017 12:42PM UTC	0	90.14	Travis Job 2531.2

Source Files on build 2531

Detailed source file information is not available for this build.

FluidTYPO3 / flux / 2531
93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2531

FluidTYPO3 / flux / 2531 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 2531

FluidTYPO3 / flux / 2531
93%

README BADGES
x