Permalink
Browse files

BUG Class autoloading in i18nTextCollector

The collector didn't find classes marked as entity providers
unless they were already loaded as a sideeffect of the preceding
execution path. Since we want to introspect the whole codebase,
autoloading is mandatory here.
  • Loading branch information...
1 parent d0a9811 commit 1db830771d9fb234a406e8534b0acd4392264077 @chillu chillu committed Aug 6, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 i18n/i18nTextCollector.php
View
2 i18n/i18nTextCollector.php
@@ -334,7 +334,7 @@ function collectFromEntityProviders($filePath) {
// Not all classes can be instanciated without mandatory arguments,
// so entity collection doesn't work for all SilverStripe classes currently
// Requires PHP 5.1+
- if(class_exists($class, false) && in_array('i18nEntityProvider', class_implements($class))) {
+ if(class_exists($class) && in_array('i18nEntityProvider', class_implements($class))) {
$reflectionClass = new ReflectionClass($class);
if($reflectionClass->isAbstract()) continue;

0 comments on commit 1db8307

Please sign in to comment.