Bug #37292

PropertyMappingConfiguration::mapUnknownProperties is not passed down to Subconfiguration

Added by Marc Neuhaus about 9 years ago. Updated over 7 years ago.

Status:
Under Review
Priority:
Should have
Category:
Property
Target version:
-
Start date:
2012-05-18
Due date:
% Done:

0%

Estimated time:
PHP Version:
Has patch:
Yes
Complexity:

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.


Files

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

Related issues

Related to TYPO3.Flow - Feature #37436: PropertyMapper should allow settings like allowCreationForSubProperty('products.[n].tierprices.[n]')ResolvedChristopher Hlubek2012-05-23

Actions
Blocks Foo.ContentManagement - Major Feature #37293: Refactor Admin to work inside Phoenix as Foo.ContentManagementNew2012-05-18

Actions
#2

Updated by Bastian Waidelich about 9 years ago

  • Category set to Property
  • Assignee 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 9 years ago

  • Target version changed from 1.1 beta 2 to 1.1
#4

Updated by Karsten Dambekalns about 9 years ago

  • Target version changed from 1.1 to 1.1 RC1
#5

Updated by Karsten Dambekalns about 9 years ago

  • Target version deleted (1.1 RC1)
#6

Updated by Gerrit Code Review over 7 years 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