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

[VarExporter] Improve partial-initialization API for ghost objects#48312

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
fabpot merged 1 commit intosymfony:6.2fromnicolas-grekas:lazy-partial
Nov 25, 2022

Conversation

@nicolas-grekas
Copy link
Member

@nicolas-grekasnicolas-grekas commentedNov 24, 2022
edited
Loading

QA
Branch?6.2
Bug fix?yes
New feature?no
Deprecations?no
Tickets-
LicenseMIT
Doc PR-

Discussed with@alcaeus at SymfonyCon, this improves support for partial-initialization of lazy ghost objects.

  • isLazyObjectInitialized(bool $partial = false): bool allows checking if an object is partially-initialized.
  • a partial initializer set to the special"\0" key can be used to initialize all/many properties at once.

Targeting 6.2 because this would be a BC break in 6.3, also because this improves a new unreleased feature.

Copy link
Contributor

@alcaeusalcaeus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I've tested this against a small proof of concept I've been working on, and the object initialisation works as expected. Thanks@nicolas-grekas!

@fabpot
Copy link
Member

Thank you@nicolas-grekas.

@fabpotfabpot merged commit894adcb intosymfony:6.2Nov 25, 2022
@nicolas-grekasnicolas-grekas deleted the lazy-partial branchNovember 25, 2022 08:15
@fabpotfabpot mentioned this pull requestNov 25, 2022
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@stofstofstof left review comments

+1 more reviewer

@alcaeusalcaeusalcaeus approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

6.2

Development

Successfully merging this pull request may close these issues.

5 participants

@nicolas-grekas@fabpot@alcaeus@stof@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp