Bug #69183

Story #69712: Further FormEngine development

Respect TCA option 'columnsOverrides' for server side field evaluations

Added by Sascha Egerer over 4 years ago. Updated over 1 year ago.

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

0%

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

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

History

#1 Updated by Christian Kuhn over 4 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.

#2 Updated by Gerrit Code Review over 4 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

#3 Updated by Sascha Egerer over 4 years ago

  • Status changed from Under Review to New

Too late ;-)

#4 Updated by Sascha Egerer over 4 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.

#5 Updated by Gerrit Code Review over 4 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

#6 Updated by Sascha Egerer over 4 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.

#7 Updated by Sascha Egerer about 4 years ago

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

Waiting for DataHandler refactoring

#8 Updated by Mathias Schreiber about 4 years ago

  • Target version changed from 7 LTS to 8 LTS

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

#9 Updated by Sascha Egerer over 3 years ago

  • Parent task set to #69712

#10 Updated by Benni Mack over 2 years ago

  • Target version changed from 8 LTS to Candidate for patchlevel

#11 Updated by Benni Mack about 2 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?

#12 Updated by Christian Kuhn almost 2 years ago

  • Assignee deleted (Christian Kuhn)

#13 Updated by Alexander Opitz over 1 year 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

Also available in: Atom PDF