8000 [PropertyAccess] Allow custom methods on property accesses by lrlopez · Pull Request #18016 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[PropertyAccess] Allow custom methods on property accesses #18016

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 18 commits into from
Closed

[PropertyAccess] Allow custom methods on property accesses #18016

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
af3a824
[PropertyAccess] Allow custom methods on property accesses
lrlopez Mar 4, 2016
5b82237
Added support for YAML and XML mapping
lrlopez Mar 6, 2016
00a26eb
Renamed Attribute to Property on most classes and methods
lrlopez Mar 16, 2016
e4aaac9
Annotation renamed to @Property
lrlopez Mar 17, 2016
e46df98
Added method annotation for virtual properties
lrlopez Mar 21, 2016
5f206e6
Renamed "attribute" to "property" in XSD and YAML
lrlopez Mar 21, 2016
94e6157
Fixed the metadata caching mess
lrlopez Mar 21, 2016
09707ad
Replaced metadata caching with standard PSR-6 cache pool
lrlopez Mar 22, 2016
d02d93a
Added support to metadata factories in PropertyAccesorBuilder
lrlopez Mar 25, 2016
748b894
Fix outdated type hint in LazyLoadingMetadataFactory
lrlopez Mar 29, 2016
688cbda
Fixed typo in variable name
lrlopez Apr 1, 2016
003efdb
Added 'Property' prefix to method annotations
lrlopez Apr 2, 2016
dc5cacb
Renamed 'propertiesMetadata' to 'propertyMetadataCollection' in 'Clas…
lrlopez Apr 2, 2016
1a35d45
Minor rebase fixes
lrlopez Jul 3, 2016
1e29523
Addressed @xabbuh and @fabpot comments
lrlopez Jul 3, 2016
7c6a79c
Make fabbot.io happy :)
lrlopez Jul 3, 2016
d0bd777
Fixed class name clash in FrameworkExtension
lrlopez Jul 3, 2016
6b7eeff
Revert unintended change in composer.json
lrlopez Jul 4, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed typo in variable name
  • Loading branch information
@lrlopez
lrlopez committed Jul 3, 2016
commit 688cbdab73bfa73928b33b7c6eea5a26e39e78f1
4 changes: 2 additions & 2 deletions src/Symfony/Component/PropertyAccess/PropertyAccessor.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -750,15 +750,15 @@ private function getWriteAccessInfo($class, $property, $value)
$hasProperty = $reflClass->hasProperty($property);
$access[self::ACCESS_HAS_PROPERTY] = $hasProperty;

$transversable = is_array($value) || $value instanceof \Traversable;
$traversable = is_array($value) || $value instanceof \Traversable;
$done = false;

if ($this->classMetadataFactory) {
$metadata = $this->classMetadataFactory->getMetadataFor($class)->getPropertiesMetadata();
$metadata = isset($metadata[$property]) ? $metadata[$property] : null;

if ($metadata) {
if ($transversable && $metadata->getAdder() && $metadata->getRemover()) {
if ($traversable && $metadata->getAdder() && $metadata->getRemover()) {
$access[self::ACCESS_TYPE] = self::ACCESS_TYPE_ADDER_AND_REMOVER;
$access[self::ACCESS_ADDER] = $metadata->getAdder();
$access[self::ACCESS_REMOVER] = $metadata->getRemover();
Expand Down
0