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

Appearance settings

fix invalid call to PhpFileLoader::load() in a test #32288

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

Merged
merged 1 commit into from
Jul 2, 2019

Conversation

Tobion
Copy link
Contributor

@Tobion Tobion commented Jun 29, 2019

Q A
Branch? 4.3
Bug fix? yno
New feature? yno
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

This was calling \Symfony\Component\DependencyInjection\Loader\PhpFileLoader::load($resource, string $type = null) with the container object as $type which I found in #32286

@nicolas-grekas
Copy link
Member

For 3.4, isn't it?

@Tobion
Copy link
Contributor Author

Tobion commented Jun 30, 2019

The sodium_encoder.php does not exist in 3.4 but merge.php does exist. So would need two PRs then.

@Tobion Tobion force-pushed the fix-invalid-load-call-in-a-test branch from 12ed032 to 8930335 Compare June 30, 2019 23:10
@Tobion
Copy link
Contributor Author

Tobion commented Jun 30, 2019

For 3.4 see #32297

@Tobion Tobion modified the milestones: 3.4, 4.3 Jun 30, 2019
Tobion added a commit that referenced this pull request Jul 2, 2019
…ion)

This PR was merged into the 3.4 branch.

Discussion
----------

fix invalid call to PhpFileLoader::load() in a test

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets |
| License       | MIT
| Doc PR        |

#32288 for 3.4

Commits
-------

a03b5d8 fix invalid call to PhpFileLoader::load() in a test
@Tobion Tobion merged commit 8930335 into symfony:4.3 Jul 2, 2019
Tobion added a commit that referenced this pull request Jul 2, 2019
…ion)

This PR was merged into the 4.3 branch.

Discussion
----------

fix invalid call to PhpFileLoader::load() in a test

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yno
| New feature?  | yno <!-- please update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets |
| License       | MIT
| Doc PR        |

This was calling `\Symfony\Component\DependencyInjection\Loader\PhpFileLoader::load($resource, string $type = null)` with the container object as $type which I found in #32286

Commits
-------

8930335 fix invalid call to PhpFileLoader::load() in a test
@Tobion Tobion deleted the fix-invalid-load-call-in-a-test branch July 2, 2019 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.