[ticket/17191] Use di and avoid circular ref

PHPBB3-17191

Author: Crizz0

feed/controller/feed.php

@@ -395,7 +395,7 @@ protected function send_feed_do(feed_interface $feed)
 			'FEED_TITLE'			=> $this->config['sitename'],
 			'FEED_SUBTITLE'			=> $this->config['site_desc'],
 			'FEED_UPDATED'			=> $this->feed_helper->format_date($feed_updated_time),
-			'FEED_LANG'				=> $this->lang_helper->get_lang_key_value('user_lang'),
+			'FEED_LANG'				=> $this->lang_helper->get_lang_key_value('user_lang', $user->data['user_lang']),
 			'FEED_AUTHOR'			=> $this->config['sitename'],
 
 			// Feed entries

install/controller/helper.php

@@ -253,19 +253,16 @@ public function handle_navigation($iohandler = null)
 	 */
 	protected function page_header($page_title, $selected_language = false)
 	{
-		global $phpbb_container;
+		global $user;
 
 		// Path to templates
 		$paths = array($this->phpbb_root_path . 'install/update/new/adm/', $this->phpbb_admin_path);
 		$paths = array_filter($paths, 'is_dir');
 		$path = array_shift($paths);
 		$path = substr($path, strlen($this->phpbb_root_path));
-		// Get the language helper
-		/* @var $language_helper \phpbb\language\language_file_helper */
-		$language_file_helper = $phpbb_container->get('language.helper.language_file');
 
 		// Grab the users lang direction and store it for later use
-		$direction = $language_file_helper->get_lang_key_value('direction');
+		$direction = $this->lang_helper->get_lang_key_value('direction', $user->data['user_lang']);
 
 		$this->template->assign_vars(array(
 			'L_CHANGE'				=> $this->language->lang('CHANGE'),
@@ -285,7 +282,7 @@ protected function page_header($page_title, $selected_language = false)
 			'S_CONTENT_ENCODING' 	=> 'UTF-8',
 			'S_LANG_SELECT'			=> $selected_language,
 
-			'S_USER_LANG'			=> $language_file_helper->get_lang_key_value('user_lang'),
+			'S_USER_LANG'			=> $this->lang_helper->get_lang_key_value('user_lang', $user->data['user_lang']),
 		));
 
 		$this->render_navigation();

language/language.php

@@ -13,6 +13,7 @@
 
 namespace phpbb\language;
 
+use phpbb\config\config;
 use phpbb\language\exception\invalid_plural_rule_exception;
 
 /**
@@ -70,15 +71,29 @@ class language
 	 */
 	protected $loader;
 
+	/**
+	 * @var language_file_helper
+	 */
+	protected $lang_helper;
+
+	/**
+	 * @var \phpbb\config\config
+	 */
+	protected $config;
+
 	/**
 	 * Constructor
 	 *
 	 * @param \phpbb\language\language_file_loader	$loader			Language file loader
+	 * @param language_file_helper 					$lang_helper
 	 * @param array|null							$common_modules	Array of common language modules to load (optional)
+	 * @param config $config The config
 	 */
-	public function __construct(language_file_loader $loader, $common_modules = null)
+	public function __construct(language_file_loader $loader, language_file_helper $lang_helper, config $config, $common_modules = null)
 	{
 		$this->loader = $loader;
+		$this->lang_helper = $lang_helper;
+		$this->config = $config;
 
 		// Set up default information
 		$this->user_language		= null;
@@ -403,13 +418,10 @@ protected function load_common_language_files()
 	 */
 	public function get_plural_form($number, $force_rule = false)
 	{
-		global $phpbb_container;
-		// Get the language helper
-		/* @var $language_helper \phpbb\language\language_file_helper */
-		$language_file_helper = $phpbb_container->get('language.helper.language_file');
+		global $user;
 
 		// Grab the users lang plural rule
-		$plural_rule_content = $language_file_helper->get_lang_key_value('plural_rule');
+		$plural_rule_content = $this->lang_helper->get_lang_key_value('plural_rule', $user->data['user_lang']);
 
 		$number			= (int) $number;
 		$plural_rule	= ($force_rule !== false) ? $force_rule : ((isset($plural_rule_content)) ? $plural_rule_content : 1);

language/language_file_helper.php

@@ -15,8 +15,8 @@
 
 use DomainException;
 use phpbb\json\sanitizer as json_sanitizer;
-use phpbb\user;
 use Symfony\Component\Finder\Finder;
+use phpbb\config\config;
 
 /**
  * Helper class for language file related functions
@@ -28,19 +28,22 @@ class language_file_helper
 	 */
 	protected $phpbb_root_path;
 
-	/** @var user */
-	protected $user;
+	/**
+	 * @var \phpbb\config\config
+	 */
+	protected $config;
 
 	/**
 	 * Constructor
 	 *
-	 * @param string $phpbb_root_path Path to phpBB's root
-	 * @param user $user			  User Object
+	 * @param string $phpbb_root_path 		Path to phpBB's root
+	 * @param config $config 				The config
+	 *
 	 */
-	public function __construct(string $phpbb_root_path, \phpbb\user $user)
+	public function __construct(string $phpbb_root_path, config $config)
 	{
 		$this->phpbb_root_path = $phpbb_root_path;
-		$this->user = $user;
+		$this->config = $config;
 	}
 
 	/**
@@ -79,16 +82,18 @@ public function get_available_languages() : array
 	 * @return string
 	 *
 	 */
-	public function get_lang_key_value($lang_key) : string
+	public function get_lang_key_value($lang_key, $user_lang_db) : string
 	{
 		$available_languages = $this->get_available_languages();
 
 		foreach ($available_languages as $key => $value)
 		{
 			$available_languages[$value['iso']] = $value;
+			unset($available_languages[$key]);
 		}
-		$user_lang = $this->user->data['user_lang'] ?? 'en';
-		return $available_languages[$user_lang][$lang_key];
+		$board_default_lang = $this->config['default_lang'];
+		$user_lang_db = $user_lang_db ?? $board_default_lang;
+		return $available_languages[$user_lang_db][$lang_key];
 	}
 
 	/**