Project

General

Profile

Actions

Bug #69183

closed

Story #69712: Further FormEngine development

Respect TCA option 'columnsOverrides' for server side field evaluations

Added by Sascha Egerer over 8 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
DataHandler aka TCEmain
Start date:
2015-08-19
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Remote Sprint

Description

With the TCA value columnsOverrides field configurations can be overwritten. This works for the Formengine stuff.
The field evaluation with the eval option can also be overwritten. But only the client side evaluation respects this setting as this is renderd trough the form enginge.
The server side evaluations are not respected as the `columnsOverrides` is not merged there.

This works:

$GLOBALS['TCA']['tt_content']['types']['text']['columnsOverrides'] = array(
    'header_link' => array(
        'config' => array(
            'eval' => 'trim,required',
        ),
    )
);

This does not work (evaluating the the value is an email address which is done on the server side):

$GLOBALS['TCA']['tt_content']['types']['text']['columnsOverrides'] = array(
    'header_link' => array(
        'config' => array(
            'eval' => 'trim,required,email',
        ),
    )
);


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Feature #76671: columnsOverrides does not affect newly added inline elementsClosed2016-06-17

Actions
Related to TYPO3 Core - Bug #87364: Slug generation wizard ignores columnsOverrides of TCAClosed2019-01-08

Actions
Actions #1

Updated by Christian Kuhn over 8 years ago

-1.

Please do not implement this now: the columnsOverrides is not handled in dataHandler systematically and it is also documented that columnsOverrides only works for FormEngine related stuff.

Please keep this situation until DataHandler was refactored.

Actions #2

Updated by Gerrit Code Review over 8 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 http://review.typo3.org/42762

Actions #3

Updated by Sascha Egerer over 8 years ago

  • Status changed from Under Review to New

Too late ;-)

Actions #4

Updated by Sascha Egerer over 8 years ago

@Christian
Would you give that -1 to the patch? I had it already done and just pushed it. So if you don't want it then please add you comments to the patch. Thanks.

Actions #5

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42762

Actions #6

Updated by Sascha Egerer over 8 years ago

Same problem at several other places (as I've expected it). So everything that is done on server side currently does not respect the setting.
Doing something like

$GLOBALS['TCA']['tt_content']['types']['foo_base_contact_banner']['columnsOverrides'] = array(
    'records' => array(
        'config' => array(
            'allowed' => 'my_custom_table',
        ),
    )
);

works in the Form but the record is never saved as it is evaluated on the server and then thrown away.

Actions #7

Updated by Sascha Egerer over 8 years ago

  • Status changed from Under Review to On Hold
  • Target version changed from 7.4 (Backend) to 7 LTS

Waiting for DataHandler refactoring

Actions #8

Updated by Mathias Schreiber over 8 years ago

  • Target version changed from 7 LTS to 8 LTS

unlikely to be completely refactored until Nov 10th ;-)

Actions #9

Updated by Sascha Egerer over 7 years ago

  • Parent task set to #69712
Actions #10

Updated by Benni Mack almost 7 years ago

  • Target version changed from 8 LTS to Candidate for patchlevel
Actions #11

Updated by Benni Mack over 6 years ago

  • Status changed from On Hold to Needs Feedback
  • Assignee changed from Sascha Egerer to Christian Kuhn

Hey Lolli,

could you state the opinion or the idea to solve (or not to solve) this issue?

Actions #12

Updated by Christian Kuhn over 6 years ago

  • Assignee deleted (Christian Kuhn)
Actions #13

Updated by Alexander Opitz almost 6 years ago

  • Category changed from FormEngine aka TCEforms to DataHandler aka TCEmain
  • Status changed from Needs Feedback to On Hold
  • Target version changed from Candidate for patchlevel to Candidate for Major Version
Actions #14

Updated by Daniel Goerz about 4 years ago

  • Related to Bug #87364: Slug generation wizard ignores columnsOverrides of TCA added
Actions #15

Updated by Daniel Goerz about 4 years ago

  • Status changed from On Hold to Resolved

This has been resolved with #87364

Actions #16

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF