Bug #49847

PSR-0 compatible namespace with underscores is not loaded

Added by Sören Rohweder about 2 years ago. Updated about 1 year ago.

Status:Resolved Start date:2013-07-10
Priority:Should have Due date:
Assigned To:Sören Rohweder % Done:

100%

Category:-
Target version:-
PHP Version: Complexity:easy
Has patch:No Affected Flow version:Flow 2.0.0 beta 2

Description

When using Faker (fzaninotto/faker) the ClassLoader fails to load classes with underscores in the namespace like Faker\Provider\en_US\Person.

PSR-0 States that only underscores in the classname should be replaced by a \ while leaving underscores in the namespace untouched.

Associated revisions

Revision ced8e0be
Added by Sören Rohweder almost 2 years ago

[BUGFIX] Treat namespaces with underscores correctly

The actual classloading does not handle the presence of underscores in namespaces
correctly.

Change-Id: I589f687adf2cf71c5c53985def6d084a76188f0b
Fixes: #49847
Releases: master, 2.0

Revision eca1d1eb
Added by Sören Rohweder almost 2 years ago

[BUGFIX] Treat namespaces with underscores correctly

The actual classloading does not handle the presence of underscores in namespaces
correctly.

Change-Id: I589f687adf2cf71c5c53985def6d084a76188f0b
Fixes: #49847
Releases: master, 2.0

History

#1 Updated by Sören Rohweder about 2 years ago

Patch is ready, waiting for the CLA to be accepted.

#2 Updated by Gerrit Code Review about 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22175

#3 Updated by Gerrit Code Review about 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22175

#4 Updated by Gerrit Code Review about 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22175

#5 Updated by Gerrit Code Review about 2 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22175

#6 Updated by Gerrit Code Review almost 2 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22175

#7 Updated by Gerrit Code Review almost 2 years ago

Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/23896

#8 Updated by Christian Jul Jensen over 1 year ago

This patch has been merged in master, I guess we can close the ticekt?

#9 Updated by Sören Rohweder over 1 year ago

I think so too

#10 Updated by Sören Rohweder about 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF