makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-08 04:18:52 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/13733] Allow tests the skip class validation

Browse source 
PHPBB3-13733
This commit is contained in:
David King 2015-05-23 21:13:30 -04:00
parent 4ecc13af83
commit 65316cffaf
2 changed files with 19 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

31
phpBB/phpbb/extension/base.php
View file

@ -121,9 +121,11 @@ class base implements \phpbb\extension\extension_interface
/** /**
* Get the list of migration files from this extension * Get the list of migration files from this extension
* *
* @var bool $validate_classes Whether or not to check that the migration
* class exists and extends the base migration class.
* @return array * @return array
*/ */
protected function get_migration_file_list() protected function get_migration_file_list($validate_classes = true)
{ {
if ($this->migrations !== false) if ($this->migrations !== false)
{ {
@ -137,23 +139,26 @@ class base implements \phpbb\extension\extension_interface
$migrations = $this->extension_finder->get_classes_from_files($migrations); $migrations = $this->extension_finder->get_classes_from_files($migrations);
foreach ($migrations as $key => $migration) if ($validate_classes)
{ {
// If the class exists and is a subclass of the foreach ($migrations as $key => $migration)
// \phpbb\db\migration\migration abstract class {
// we skip it. // If the class exists and is a subclass of the
// \phpbb\db\migration\migration abstract class
// we skip it.
// Otherwise, i.e. if it doesn't exist or it is
// not an extend the abstract class, we unset it
if (class_exists($migration)) {
$reflector = new \ReflectionClass($migration);
if ($reflector->isSubclassOf('\phpbb\db\migration\migration')) {
continue;
}
// Otherwise, i.e. if it doesn't exist or it is
// not an extend the abstract class, we unset it
if (class_exists($migration)) {
$reflector = new \ReflectionClass($migration);
if ($reflector->isSubclassOf('\phpbb\db\migration\migration')) {
continue;
} }
unset($migrations[$key]);
} }
unset($migrations[$key]);
} }
return $migrations; return $migrations;

2
tests/extension/extension_base_test.php
View file

@ -74,6 +74,6 @@ class phpbb_extension_extension_base_test extends phpbb_test_case
public function test_suffix_get_classes($extension_name, $expected) public function test_suffix_get_classes($extension_name, $expected)
{ {
$extension = $this->extension_manager->get_extension($extension_name); $extension = $this->extension_manager->get_extension($extension_name);
$this->assertEquals($expected, self::$reflection_method_get_migration_file_list->invoke($extension)); $this->assertEquals($expected, self::$reflection_method_get_migration_file_list->invoke($extension, false));
} }
} }