Bug #82786: Invalid "recordOverview" in form YAML config - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #82786

closed

Invalid "recordOverview" in form YAML config

Added by Mathias Brodala about 7 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Form Framework

Target version:

next-patchlevel

Start date:

2017-10-17

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

no-brainer

Is Regression:

Sprint Focus:

Description

The form YAML config uses recordOverview in various locations which must be recordsOverview instead

Actions

Copy link

Updated by Gerrit Code Review about 7 years ago

Status changed from New 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/54435

Actions

Copy link

Updated by Björn Jacob about 7 years ago

Category changed from FormEngine aka TCEforms to Form Framework

Actions

Copy link

Updated by Mathias Brodala about 7 years ago

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

Applied in changeset e3cf49c9599f6932569d130d86c4df94948a696d.

Actions

Copy link

Updated by Ioulia Kondratovitch over 6 years ago

Target version changed from next-patchlevel to 8.7.13
TYPO3 Version changed from 9 to 8

Mathias Brodala wrote:

Applied in changeset e3cf49c9599f6932569d130d86c4df94948a696d.

The patch/bugfix seems to be included only in TYPO3 >= 9.0.
In TYPO3 8.7.12 we still have an old "recordOverview" which is wrong.
Patch for 8.7.12 needed.

Actions

Copy link

Updated by Mathias Brodala over 6 years ago

Target version changed from 8.7.13 to next-patchlevel

~~You are right, the backport for this one is still missing.~~

Apparently we don't need the backport, see the last comment in https://review.typo3.org/54435

Actions

Copy link

Updated by Mathias Brodala over 6 years ago

TYPO3 Version changed from 8 to 9

Actions

Copy link

Updated by Ioulia Kondratovitch over 6 years ago

Mathias Brodala wrote:

Apparently we don't need the backport, see the last comment in https://review.typo3.org/54435

Hi! I have seen the comment. But in reality we still have the line

                        recordOverview:
                          disabled: 1

in line 70,71 of

.../typo3/sysext/form/Configuration/Yaml/FormEngineSetup.yaml

in TYPO3 8.7.12. Which has no effect due to typo.
If you have a look at the code, you will find it.
I really think this should be fixed.

Actions

Copy link

Updated by Gerrit Code Review over 6 years ago

Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57818

Actions

Copy link