diff options
Diffstat (limited to 'phpBB')
| -rwxr-xr-x | phpBB/bin/phpbbcli.php | 10 | ||||
| -rw-r--r-- | phpBB/common.php | 14 | ||||
| -rw-r--r-- | phpBB/download/file.php | 6 | ||||
| -rw-r--r-- | phpBB/install/database_update.php | 12 | ||||
| -rw-r--r-- | phpBB/install/index.php | 18 | ||||
| -rw-r--r-- | phpBB/install/install_convert.php | 4 | ||||
| -rw-r--r-- | phpBB/install/install_install.php | 4 | ||||
| -rw-r--r-- | phpBB/install/install_update.php | 12 | ||||
| -rw-r--r-- | phpBB/phpbb/di/container_builder.php | 504 | 
9 files changed, 290 insertions, 294 deletions
diff --git a/phpBB/bin/phpbbcli.php b/phpBB/bin/phpbbcli.php index fc78e5d04a..d72ca760c8 100755 --- a/phpBB/bin/phpbbcli.php +++ b/phpBB/bin/phpbbcli.php @@ -43,21 +43,21 @@ require($phpbb_root_path . 'includes/functions.' . $phpEx);  require($phpbb_root_path . 'includes/functions_admin.' . $phpEx);  require($phpbb_root_path . 'includes/utf/utf_tools.' . $phpEx); -$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -$phpbb_container_builder->set_dump_container(false); +$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +$phpbb_container = $phpbb_container_builder->with_config($phpbb_config_php_file); + +$phpbb_container_builder->without_cache();  $input = new ArgvInput();  if ($input->hasParameterOption(array('--safe-mode')))  { -	$phpbb_container_builder->set_use_extensions(false); -	$phpbb_container_builder->set_dump_container(false); +	$phpbb_container_builder->without_extensions();  }  else  {  	$phpbb_class_loader_ext = new \phpbb\class_loader('\\', "{$phpbb_root_path}ext/", $phpEx);  	$phpbb_class_loader_ext->register(); -	phpbb_load_extensions_autoloaders($phpbb_root_path);  }  $phpbb_container = $phpbb_container_builder->get_container(); diff --git a/phpBB/common.php b/phpBB/common.php index 633a37f25b..70fb34331c 100644 --- a/phpBB/common.php +++ b/phpBB/common.php @@ -96,21 +96,21 @@ set_error_handler(defined('PHPBB_MSG_HANDLER') ? PHPBB_MSG_HANDLER : 'msg_handle  $phpbb_class_loader_ext = new \phpbb\class_loader('\\', "{$phpbb_root_path}ext/", $phpEx);  $phpbb_class_loader_ext->register(); -phpbb_load_extensions_autoloaders($phpbb_root_path); +\Symfony\Component\Debug\Debug::enable(E_ALL);  // Set up container -try -{ -	$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -	$phpbb_container = $phpbb_container_builder->get_container(); -} +/*try +{*/ +	$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +	$phpbb_container = $phpbb_container_builder->with_config($phpbb_config_php_file)->get_container(); +/*}  catch (InvalidArgumentException $e)  {  	trigger_error(  		'The requested environment ' . PHPBB_ENVIRONMENT . ' is not available.',  		E_USER_ERROR  	); -} +}*/  $phpbb_class_loader->set_cache($phpbb_container->get('cache.driver'));  $phpbb_class_loader_ext->set_cache($phpbb_container->get('cache.driver')); diff --git a/phpBB/download/file.php b/phpBB/download/file.php index 235dd3c95c..9d54b824f8 100644 --- a/phpBB/download/file.php +++ b/phpBB/download/file.php @@ -61,11 +61,9 @@ if (isset($_GET['avatar']))  	$phpbb_class_loader_ext = new \phpbb\class_loader('\\', "{$phpbb_root_path}ext/", $phpEx);  	$phpbb_class_loader_ext->register(); -	phpbb_load_extensions_autoloaders($phpbb_root_path); -  	// Set up container -	$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -	$phpbb_container = $phpbb_container_builder->get_container(); +	$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +	$phpbb_container = $phpbb_container_builder->with_config($phpbb_config_php_file)->get_container();  	$phpbb_class_loader->set_cache($phpbb_container->get('cache.driver'));  	$phpbb_class_loader_ext->set_cache($phpbb_container->get('cache.driver')); diff --git a/phpBB/install/database_update.php b/phpBB/install/database_update.php index 4746bb11f2..40d07e7990 100644 --- a/phpBB/install/database_update.php +++ b/phpBB/install/database_update.php @@ -80,11 +80,13 @@ require($phpbb_root_path . 'includes/utf/utf_tools.' . $phpEx);  set_error_handler(defined('PHPBB_MSG_HANDLER') ? PHPBB_MSG_HANDLER : 'msg_handler');  // Set up container (must be done here because extensions table may not exist) -$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -$phpbb_container_builder->set_use_extensions(false); -$phpbb_container_builder->set_use_kernel_pass(false); -$phpbb_container_builder->set_dump_container(false); -$phpbb_container = $phpbb_container_builder->get_container(); +$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +$phpbb_container = $phpbb_container_builder +	->with_config($phpbb_config_php_file) +	->without_extensions() +	->without_cache() +	->get_container() +;  // set up caching  /* @var $cache \phpbb\cache\service */ diff --git a/phpBB/install/index.php b/phpBB/install/index.php index 056c827669..19cf27940b 100644 --- a/phpBB/install/index.php +++ b/phpBB/install/index.php @@ -116,18 +116,20 @@ $phpbb_class_loader_ext->register();  // Set up container  $phpbb_config_php_file = new \phpbb\config_php_file($phpbb_root_path, $phpEx); -$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -$phpbb_container_builder->set_use_extensions(false); -$phpbb_container_builder->set_dump_container(false); -$phpbb_container_builder->set_use_custom_pass(false); -$phpbb_container_builder->set_inject_config(false); -$phpbb_container_builder->set_compile_container(false); +$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +$phpbb_container = $phpbb_container_builder +	->with_config($phpbb_config_php_file) +	->without_extensions() +	->without_cache() +	->without_compiled_container() +	->get_container() +;  $other_config_path = $phpbb_root_path . 'install/update/new/config/';  $config_path = file_exists($other_config_path . 'services.yml') ? $other_config_path : $phpbb_root_path . 'config/'; -$phpbb_container_builder->set_config_path($config_path); +$phpbb_container_builder->with_config_path($config_path); -$phpbb_container_builder->set_custom_parameters(array( +$phpbb_container_builder->with_custom_parameters(array(  	'core.root_path'			=> $phpbb_root_path,  	'core.adm_relative_path'	=> $phpbb_adm_relative_path,  	'core.php_ext'				=> $phpEx, diff --git a/phpBB/install/install_convert.php b/phpBB/install/install_convert.php index 408c14e981..dd9835113b 100644 --- a/phpBB/install/install_convert.php +++ b/phpBB/install/install_convert.php @@ -131,8 +131,8 @@ class install_convert extends module  		// Enable super globals to prevent issues with the new \phpbb\request\request object  		$request->enable_super_globals();  		// Create a normal container now -		$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -		$phpbb_container = $phpbb_container_builder->get_container(); +		$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +		$phpbb_container = $phpbb_container_builder->with_config($phpbb_config_php_file)->get_container();  		// Create cache  		/* @var $cache \phpbb\cache\service */ diff --git a/phpBB/install/install_install.php b/phpBB/install/install_install.php index be6fa40566..e8890a3d08 100644 --- a/phpBB/install/install_install.php +++ b/phpBB/install/install_install.php @@ -110,8 +110,8 @@ class install_install extends module  				$request->enable_super_globals();  				// Create a normal container now -				$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -				$phpbb_container = $phpbb_container_builder->get_container(); +				$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +				$phpbb_container = $phpbb_container_builder->with_config($phpbb_config_php_file)->get_container();  				// Sets the global variables  				/* @var $cache \phpbb\cache\service */ diff --git a/phpBB/install/install_update.php b/phpBB/install/install_update.php index 288e728fce..d9ba2e7fe9 100644 --- a/phpBB/install/install_update.php +++ b/phpBB/install/install_update.php @@ -76,12 +76,16 @@ class install_update extends module  		$request->enable_super_globals();  		// Create a normal container now -		$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_config_php_file, $phpbb_root_path, $phpEx); -		$phpbb_container_builder->set_dump_container(false); -		$phpbb_container_builder->set_use_extensions(false); +		$phpbb_container_builder = new \phpbb\di\container_builder($phpbb_root_path, $phpEx); +		$phpbb_container = $phpbb_container_builder +			->with_config($phpbb_config_php_file) +			->without_cache() +			->without_extensions() +		; +  		if (file_exists($phpbb_root_path . 'install/update/new/config'))  		{ -			$phpbb_container_builder->set_config_path($phpbb_root_path . 'install/update/new/config'); +			$phpbb_container_builder->with_config_path($phpbb_root_path . 'install/update/new/config');  		}  		$phpbb_container = $phpbb_container_builder->get_container(); diff --git a/phpBB/phpbb/di/container_builder.php b/phpBB/phpbb/di/container_builder.php index 99576f9020..33eb4e59a8 100644 --- a/phpBB/phpbb/di/container_builder.php +++ b/phpBB/phpbb/di/container_builder.php @@ -13,6 +13,7 @@  namespace phpbb\di; +use phpbb\filesystem\filesystem;  use Symfony\Component\Config\ConfigCache;  use Symfony\Component\Config\FileLocator;  use Symfony\Component\DependencyInjection\ContainerBuilder; @@ -25,6 +26,11 @@ use Symfony\Component\HttpKernel\DependencyInjection\MergeExtensionConfiguration  class container_builder  {  	/** +	 * @var string The environment to use. +	 */ +	protected $environment; + +	/**  	 * @var string phpBB Root Path  	 */  	protected $phpbb_root_path; @@ -35,89 +41,58 @@ class container_builder  	protected $php_ext;  	/** -	* The container under construction -	* -	* @var ContainerBuilder -	*/ +	 * The container under construction +	 * +	 * @var ContainerBuilder +	 */  	protected $container;  	/** -	* @var \phpbb\db\driver\driver_interface -	*/ -	protected $dbal_connection = null; - -	/** -	* @var array the installed extensions -	*/ -	protected $installed_exts = null; - -	/** -	* Indicates whether the php config file should be injected into the container (default to true). -	* -	* @var bool -	*/ -	protected $inject_config = true; - -	/** -	* Indicates whether extensions should be used (default to true). -	* -	* @var bool -	*/ +	 * Indicates whether extensions should be used (default to true). +	 * +	 * @var bool +	 */  	protected $use_extensions = true;  	/** -	* Defines a custom path to find the configuration of the container (default to $this->phpbb_root_path . 'config') -	* -	* @var string -	*/ +	 * Defines a custom path to find the configuration of the container (default to $this->phpbb_root_path . 'config') +	 * +	 * @var string +	 */  	protected $config_path = null;  	/** -	* Indicates whether the phpBB compile pass should be used (default to true). -	* -	* @var bool -	*/ -	protected $use_custom_pass = true; - -	/** -	* Indicates whether the kernel compile pass should be used (default to true). -	* -	* @var bool -	*/ -	protected $use_kernel_pass = true; - -	/** -	* Indicates whether the container should be dumped to the filesystem (default to true). -	* -	* If DEBUG_CONTAINER is set this option is ignored and a new container is build. -	* -	* @var bool -	*/ -	protected $dump_container = true; +	 * Indicates whether the container should be dumped to the filesystem (default to true). +	 * +	 * If DEBUG_CONTAINER is set this option is ignored and a new container is build. +	 * +	 * @var bool +	 */ +	protected $use_cache = true;  	/** -	* Indicates if the container should be compiled automatically (default to true). -	* -	* @var bool -	*/ +	 * Indicates if the container should be compiled automatically (default to true). +	 * +	 * @var bool +	 */  	protected $compile_container = true;  	/** -	* Custom parameters to inject into the container. -	* -	* Default to true: -	* 	array( -	* 		'core.root_path', $this->phpbb_root_path, -	* 		'core.php_ext', $this->php_ext, -	* ); -	* -	* @var array -	*/ +	 * Custom parameters to inject into the container. +	 * +	 * Default to: +	 * 	array( +	 * 		'core.root_path', $this->phpbb_root_path, +	 * 		'core.php_ext', $this->php_ext, +	 * ); +	 * +	 * @var array +	 */  	protected $custom_parameters = null;  	/** -	* @var \phpbb\config_php_file -	*/ +	 * @var \phpbb\config_php_file +	 */  	protected $config_php_file;  	/** @@ -126,295 +101,311 @@ class container_builder  	protected $cache_dir;  	/** -	* Constructor -	* -	* @param \phpbb\config_php_file $config_php_file -	* @param string $phpbb_root_path Path to the phpbb includes directory. -	* @param string $php_ext php file extension -	*/ -	function __construct(\phpbb\config_php_file $config_php_file, $phpbb_root_path, $php_ext) +	 * @var array +	 */ +	private $container_extensions; + +	/** +	 * Constructor +	 * +	 * @param string $phpbb_root_path Path to the phpbb includes directory. +	 * @param string $php_ext php file extension +	 */ +	function __construct($phpbb_root_path, $php_ext)  	{ -		$this->config_php_file = $config_php_file;  		$this->phpbb_root_path = $phpbb_root_path;  		$this->php_ext = $php_ext;  	}  	/** -	* Build and return a new Container respecting the current configuration -	* -	* @return \phpbb_cache_container|ContainerBuilder -	*/ +	 * Build and return a new Container respecting the current configuration +	 * +	 * @return \phpbb_cache_container|ContainerBuilder +	 */  	public function get_container()  	{  		$container_filename = $this->get_container_filename();  		$config_cache = new ConfigCache($container_filename, defined('DEBUG')); -		if ($this->dump_container && $config_cache->isFresh()) +		if ($this->use_cache && $config_cache->isFresh())  		{  			require($config_cache->getPath());  			$this->container = new \phpbb_cache_container(); + +			return $this->container;  		} -		else -		{ -			$container_extensions = array(new \phpbb\di\extension\core($this->get_config_path())); -			if ($this->use_extensions) -			{ -				$installed_exts = $this->get_installed_extensions(); -				foreach ($installed_exts as $ext_name => $path) -				{ -					$extension_class = '\\' . str_replace('/', '\\', $ext_name) . '\\di\\extension'; +		$this->container_extensions = array(new extension\core($this->get_config_path())); -					if (!class_exists($extension_class)) -					{ -						$extension_class = '\phpbb\extension\di\extension_base'; -					} +		if ($this->use_extensions) +		{ +			$this->load_extensions(); +		} -					$container_extensions[] = new $extension_class($ext_name, $path); -				} -			} +		// Inject the config +		$this->container_extensions[] = new extension\config($this->config_php_file); -			if ($this->inject_config) -			{ -				$container_extensions[] = new \phpbb\di\extension\config($this->config_php_file); -			} +		$this->container = $this->create_container($this->container_extensions); -			$this->container = $this->create_container($container_extensions); +		// Easy collections through tags +		$this->container->addCompilerPass(new pass\collection_pass()); -			if ($this->use_custom_pass) -			{ -				// Symfony Kernel Listeners -				$this->container->addCompilerPass(new \phpbb\di\pass\collection_pass()); -				$this->container->addCompilerPass(new RegisterListenersPass('dispatcher', 'event.listener_listener', 'event.listener')); +		// Event listeners "phpBB style" +		$this->container->addCompilerPass(new RegisterListenersPass('dispatcher', 'event.listener_listener', 'event.listener')); -				if ($this->use_kernel_pass) -				{ -					$this->container->addCompilerPass(new RegisterListenersPass('dispatcher')); -				} -			} +		// Event listeners "Symfony style" +		$this->container->addCompilerPass(new RegisterListenersPass('dispatcher')); -			$filesystem = new \phpbb\filesystem\filesystem(); -			$loader = new YamlFileLoader($this->container, new FileLocator($filesystem->realpath($this->get_config_path()))); -			$loader->load($this->container->getParameter('core.environment') . '/config.yml'); +		$filesystem = new filesystem(); +		$loader = new YamlFileLoader($this->container, new FileLocator($filesystem->realpath($this->get_config_path()))); +		$loader->load($this->container->getParameter('core.environment') . '/config.yml'); -			$this->inject_custom_parameters(); +		$this->inject_custom_parameters(); -			if ($this->compile_container) -			{ -				$this->container->compile(); -			} +		if ($this->compile_container) +		{ +			$this->container->compile(); -			if ($this->dump_container) +			if ($this->use_cache)  			{  				$this->dump_container($config_cache);  			}  		} -		$this->container->set('config.php', $this->config_php_file); - -		if ($this->compile_container) -		{ -			$this->inject_dbal(); -		} -  		return $this->container;  	}  	/** -	* Set if the extensions should be used. -	* -	* @param bool $use_extensions -	*/ -	public function set_use_extensions($use_extensions) +	 * Enable the extensions. +	 * +	 * @param string $environment The environment to use +	 * @return $this +	 */ +	public function with_environment($environment)  	{ -		$this->use_extensions = $use_extensions; -	} +		$this->environment = $environment; -	/** -	* Set if the phpBB compile pass have to be used. -	* -	* @param bool $use_custom_pass -	*/ -	public function set_use_custom_pass($use_custom_pass) -	{ -		$this->use_custom_pass = $use_custom_pass; +		return $this;  	}  	/** -	* Set if the kernel compile pass have to be used. -	* -	* @param bool $use_kernel_pass -	*/ -	public function set_use_kernel_pass($use_kernel_pass) +	 * Enable the extensions. +	 * +	 * @return $this +	 */ +	public function with_extensions()  	{ -		$this->use_kernel_pass = $use_kernel_pass; +		$this->use_extensions = true; + +		return $this;  	}  	/** -	* Set if the php config file should be injecting into the container. -	* -	* @param bool $inject_config -	*/ -	public function set_inject_config($inject_config) +	 * Disable the extensions. +	 * +	 * @return $this +	 */ +	public function without_extensions()  	{ -		$this->inject_config = $inject_config; +		$this->use_extensions = false; + +		return $this;  	}  	/** -	* Set if a dump container should be used. -	* -	* If DEBUG_CONTAINER is set this option is ignored and a new container is build. -	* -	* @var bool $dump_container -	*/ -	public function set_dump_container($dump_container) +	 * Enable the caching of the container. +	 * +	 * If DEBUG_CONTAINER is set this option is ignored and a new container is build. +	 * +	 * @return $this +	 */ +	public function with_cache()  	{ -		$this->dump_container = $dump_container; +		$this->use_cache = true; + +		return $this;  	}  	/** -	* Set if the container should be compiled automatically (default to true). -	* -	* @var bool $dump_container -	*/ -	public function set_compile_container($compile_container) +	 * Disable the caching of the container. +	 * +	 * @return $this +	 */ +	public function without_cache()  	{ -		$this->compile_container = $compile_container; +		$this->use_cache = false; + +		return $this;  	}  	/** -	* Set a custom path to find the configuration of the container -	* -	* @param string $config_path -	*/ -	public function set_config_path($config_path) +	 * Set the cache directory. +	 * +	 * @param string $cache_dir The cache directory. +	 * @return $this +	 */ +	public function with_cache_dir($cache_dir)  	{ -		$this->config_path = $config_path; +		$this->cache_dir = $cache_dir; + +		return $this;  	}  	/** -	 * Returns the path to the container configuration (default: root_path/config) +	 * Enable the compilation of the container.  	 * -	 * @return string +	 * @return $this  	 */ -	protected function get_config_path() +	public function with_compiled_container()  	{ -		return $this->config_path ?: $this->phpbb_root_path . 'config'; +		$this->compile_container = true; + +		return $this;  	}  	/** -	* Set custom parameters to inject into the container. -	* -	* @param array $custom_parameters -	*/ -	public function set_custom_parameters($custom_parameters) +	 * Disable the compilation of the container. +	 * +	 * @return $this +	 */ +	public function without_compiled_container()  	{ -		$this->custom_parameters = $custom_parameters; +		$this->compile_container = false; + +		return $this;  	}  	/** -	 * Set the path to the cache directory. +	 * Set a custom path to find the configuration of the container.  	 * -	 * @param string $cache_dir Path to the cache directory +	 * @param string $config_path +	 * @return $this  	 */ -	public function set_cache_dir($cache_dir) +	public function with_config_path($config_path)  	{ -		$this->cache_dir = $cache_dir; +		$this->config_path = $config_path; + +		return $this;  	}  	/** -	 * Returns the path to the cache directory (default: root_path/cache/environment). +	 * Set custom parameters to inject into the container.  	 * -	 * @return string Path to the cache directory. +	 * @param array $custom_parameters +	 * @return $this  	 */ -	protected function get_cache_dir() +	public function with_custom_parameters($custom_parameters)  	{ -		return $this->cache_dir ?: $this->phpbb_root_path . 'cache/' . $this->get_environment() . '/'; +		$this->custom_parameters = $custom_parameters; + +		return $this;  	}  	/** -	* Dump the container to the disk. -	* -	* @param ConfigCache $cache The config cache -	*/ -	protected function dump_container($cache) +	 * Set custom parameters to inject into the container. +	 * +	 * @param \phpbb\config_php_file $config_php_file +	 * @return $this +	 */ +	public function with_config(\phpbb\config_php_file $config_php_file)  	{ -		$dumper = new PhpDumper($this->container); -		$cached_container_dump = $dumper->dump(array( -			'class'         => 'phpbb_cache_container', -			'base_class'    => 'Symfony\\Component\\DependencyInjection\\ContainerBuilder', -		)); +		$this->config_php_file = $config_php_file; -		$cache->write($cached_container_dump, $this->container->getResources()); +		return $this;  	}  	/** -	* Inject the connection into the container if one was opened. -	*/ -	protected function inject_dbal() +	 * Returns the path to the container configuration (default: root_path/config) +	 * +	 * @return string +	 */ +	protected function get_config_path()  	{ -		if ($this->dbal_connection !== null) -		{ -			$this->container->get('dbal.conn')->set_driver($this->dbal_connection); -		} +		return $this->config_path ?: $this->phpbb_root_path . 'config';  	}  	/** -	* Get DB connection. -	* -	* @return \phpbb\db\driver\driver_interface -	*/ -	protected function get_dbal_connection() +	 * Returns the path to the cache directory (default: root_path/cache/environment). +	 * +	 * @return string Path to the cache directory. +	 */ +	protected function get_cache_dir()  	{ -		if ($this->dbal_connection === null) -		{ -			$dbal_driver_class = $this->config_php_file->convert_30_dbms_to_31($this->config_php_file->get('dbms')); -			$this->dbal_connection = new $dbal_driver_class(); -			$this->dbal_connection->sql_connect( -				$this->config_php_file->get('dbhost'), -				$this->config_php_file->get('dbuser'), -				$this->config_php_file->get('dbpasswd'), -				$this->config_php_file->get('dbname'), -				$this->config_php_file->get('dbport'), -				defined('PHPBB_DB_NEW_LINK') && PHPBB_DB_NEW_LINK -			); -		} - -		return $this->dbal_connection; +		return $this->cache_dir ?: $this->phpbb_root_path . 'cache/' . $this->get_environment() . '/';  	}  	/** -	* Get enabled extensions. -	* -	* @return array enabled extensions -	*/ -	protected function get_installed_extensions() +	 * Load the enabled extensions. +	 */ +	protected function load_extensions()  	{ -		$db = $this->get_dbal_connection(); -		$extension_table = $this->config_php_file->get('table_prefix') . 'ext'; +		if ($this->config_php_file !== null) +		{ +			// Build an intermediate container to load the ext list from the database +			$container_builder = new container_builder($this->phpbb_root_path, $this->php_ext); +			$ext_container = $container_builder +				->without_cache() +				->without_extensions() +				->with_config($this->config_php_file) +				->with_environment('production') +				->without_compiled_container() +				->get_container() +			; + +			$ext_container->register('cache.driver', '\\phpbb\\cache\\driver\\null'); +			$ext_container->compile(); + +			$extensions = $ext_container->get('ext.manager')->all_enabled(); + +			// Load each extension found +			foreach ($extensions as $ext_name => $path) +			{ +				$extension_class = '\\' . str_replace('/', '\\', $ext_name) . '\\di\\extension'; -		$sql = 'SELECT * -			FROM ' . $extension_table . ' -			WHERE ext_active = 1'; +				if (!class_exists($extension_class)) +				{ +					$extension_class = '\\phpbb\\extension\\di\\extension_base'; +				} -		$result = $db->sql_query($sql); -		$rows = $db->sql_fetchrowset($result); -		$db->sql_freeresult($result); +				$this->container_extensions[] = new $extension_class($ext_name, $path); -		$exts = array(); -		foreach ($rows as $row) +				// Load extension autoloader +				$filename = $path . 'vendor/autoload.php'; +				if (file_exists($filename)) +				{ +					require $filename; +				} +			} +		} +		else  		{ -			$exts[$row['ext_name']] = $this->phpbb_root_path . 'ext/' . $row['ext_name'] . '/'; +			// To load the extensions we need the database credentials. +			// Automatically disable the extensions if we don't have them. +			$this->use_extensions = false;  		} +	} -		return $exts; +	/** +	 * Dump the container to the disk. +	 * +	 * @param ConfigCache $cache The config cache +	 */ +	protected function dump_container($cache) +	{ +		$dumper = new PhpDumper($this->container); +		$cached_container_dump = $dumper->dump(array( +			'class'         => 'phpbb_cache_container', +			'base_class'    => 'Symfony\\Component\\DependencyInjection\\ContainerBuilder', +		)); + +		$cache->write($cached_container_dump, $this->container->getResources());  	}  	/** -	* Create the ContainerBuilder object -	* -	* @param array $extensions Array of Container extension objects -	* @return ContainerBuilder object -	*/ +	 * Create the ContainerBuilder object +	 * +	 * @param array $extensions Array of Container extension objects +	 * @return ContainerBuilder object +	 */  	protected function create_container(array $extensions)  	{  		$container = new ContainerBuilder(new ParameterBag($this->get_core_parameters())); @@ -425,7 +416,6 @@ class container_builder  		{  			$container->registerExtension($extension);  			$extensions_alias[] = $extension->getAlias(); -			//$container->loadFromExtension($extension->getAlias());  		}  		$container->getCompilerPassConfig()->setMergePass(new MergeExtensionConfigurationPass($extensions_alias)); @@ -487,10 +477,10 @@ class container_builder  	}  	/** -	* Get the filename under which the dumped container will be stored. -	* -	* @return string Path for dumped container -	*/ +	 * Get the filename under which the dumped container will be stored. +	 * +	 * @return string Path for dumped container +	 */  	protected function get_container_filename()  	{  		$filename = str_replace(array('/', '.'), array('slash', 'dot'), $this->phpbb_root_path); @@ -504,6 +494,6 @@ class container_builder  	 */  	protected function get_environment()  	{ -		return PHPBB_ENVIRONMENT; +		return $this->environment ?: PHPBB_ENVIRONMENT;  	}  }  | 
