Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should setStaticPropertyValue trigger autoloading? #265

Closed
asgrim opened this issue Apr 3, 2017 · 4 comments
Closed

Should setStaticPropertyValue trigger autoloading? #265

asgrim opened this issue Apr 3, 2017 · 4 comments
Assignees
Labels
Milestone

Comments

@asgrim
Copy link
Member

asgrim commented Apr 3, 2017

#258 (comment)

Currently, we throw a ClassDoesNotExist exception if the class has not yet been loaded yet. Discuss whether we should just trigger autoloading, and also how the hijacked autoloader might work with that.

@Ocramius
Copy link
Member

Ocramius commented Sep 8, 2017

As of

$declaringClassName::${$propertyName} = $value;
( #343 ) it does

@kukulich
Copy link
Collaborator

kukulich commented Sep 8, 2017

No, the check is still there:

$this->assertClassExist($declaringClassName);

@Ocramius
Copy link
Member

Ocramius commented Sep 8, 2017

Gotcha, totally missed it @_@

@Ocramius
Copy link
Member

Closing here: the question itself is answered

@Ocramius Ocramius self-assigned this Nov 13, 2021
@Ocramius Ocramius added this to the 5.0.0 milestone Nov 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants