Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Serializer] Refactoring and object_to_populate support.#13252

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.

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

Already on GitHub?Sign in to your account

Merged

Conversation

@dunglas
Copy link
Member

QA
Bug fix?no
New feature?yes
BC breaks?no
Deprecations?no
Tests pass?yes
LicenseMIT
Doc PRn/a

This PR is mainly a refactoring. It move up the common code ofPorpertyNormalizer andGetSetMethodNormalizer intoAbstractNormalizer.

It also adds a news feature: the ability to use an existing object instance for denormalization:

$dummy =newGetSetDummy();$dummy->setFoo('foo');$obj =$this->normalizer->denormalize(array('bar' =>'bar'),'GetSetDummy',null,array('object_to_populate' =>$dummy));// $obj and $dummy are references to the same object

@dunglas
Copy link
MemberAuthor

The Travis failure is strange but doesn't seem related with this PR. Any hint@nicolas-grekas?

@fabpot
Copy link
Member

👍

@fabpot
Copy link
Member

Thank you@dunglas.

@fabpotfabpot merged commitb5990be intosymfony:2.7Jan 7, 2015
fabpot added a commit that referenced this pull requestJan 7, 2015
…t. (dunglas)This PR was merged into the 2.7 branch.Discussion----------[Serializer] Refactoring and object_to_populate support.| Q             | A| ------------- | ---| Bug fix?      | no| New feature?  | yes| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| License       | MIT| Doc PR        | n/aThis PR is mainly a refactoring. It move up the common code of `PorpertyNormalizer` and `GetSetMethodNormalizer` into `AbstractNormalizer`.It also adds a news feature: the ability to use an existing object instance for denormalization:```php$dummy = new GetSetDummy();$dummy->setFoo('foo');$obj = $this->normalizer->denormalize(    array('bar' => 'bar'),    'GetSetDummy',     null,     array('object_to_populate' => $dummy));// $obj and $dummy are references to the same object```Commits-------b5990be [Serializer] Refactoring and object_to_populate support.
dunglas added a commit that referenced this pull requestMar 6, 2015
This PR was merged into the 2.7 branch.Discussion----------[Serializer] ObjectNormalizer| Q             | A| ------------- | ---| Bug fix?      | no| New feature?  | yes| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| License       | MIT| Doc PR       | not yet`PropertyAccessNormalizer` is a new normalizer leveraging the PropertyAccess Component. It is able to handle classes containing both public properties and properties only accessibles trough getters / setters / issers / hassers...As it extends `AbstractNormalizer`, it supports circular reference handling, name converters and existing object population.What do you think about making this new normalizer the default one as it's the most convenient to use and the most consistent with the behavior of other components.#13120,#13252 and#13255 need to be merged to make this PR working.Commits-------0050bbb [Serializer] Introduce ObjectNormalizer
@dunglasdunglas deleted the serializer_abstract_refactoring branchDecember 5, 2015 09:01
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dunglas@fabpot@StanAngeloff

[8]ページ先頭

©2009-2025 Movatter.jp