Bug #37292

PropertyMappingConfiguration::mapUnknownProperties is not passed down to Subconfiguration

Added by Marc Neuhaus about 3 years ago. Updated over 1 year ago.

Status:Under Review Start date:2012-05-18
Priority:Should have Due date:
Assigned To:Sebastian Kurfuerst % Done:

0%

Category:Property
Target version:-
PHP Version: Complexity:
Has patch:Yes Affected Flow version:Git 1.1

Description

if i set allowAllProperties on my PropertyMappingConfiguration it still can fail because the setting doesn't get passed down in line 152 from getConfigurationFor.

I've attached a patch to pass down this setting.

PropertyMappingConfiguration.diff Magnifier (599 Bytes) Marc Neuhaus, 2012-05-23 14:57


Related issues

related to TYPO3.Flow - Feature #37436: PropertyMapper should allow settings like allowCreationFo... Resolved 2012-05-23
blocks Foo.ContentManagement - Major Feature #37293: Refactor Admin to work inside Phoenix as Foo.ContentManag... New 2012-05-18

History

#1 Updated by Marc Neuhaus about 3 years ago

forgot to add the Patch :)

#2 Updated by Bastian Waidelich about 3 years ago

  • Category set to Property
  • Assigned To set to Sebastian Kurfuerst
  • Target version set to 1.1 beta 2

Sebastian, I assign this to you as you are more into that topic ;)
I had similar issues: especially when working with the Form framework it's really tedious to get the PropertyMappingConfiguration right.. There should be some easy way to disable the pessimistic checks (as the form is secured by HMAC anyways) and/or to allow creation/modification & property changes for nested elements

#3 Updated by Bastian Waidelich about 3 years ago

  • Target version changed from 1.1 beta 2 to 1.1

#4 Updated by Karsten Dambekalns about 3 years ago

  • Target version changed from 1.1 to 1.1 RC1

#5 Updated by Karsten Dambekalns about 3 years ago

  • Target version deleted (1.1 RC1)
  • Affected Flow version changed from Git master to Git 1.1

#6 Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

Patch set 5 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/17204

Also available in: Atom PDF