From 6958628f0ad7cf41bcef1d0817bc5f119d8e0d37 Mon Sep 17 00:00:00 2001
From: Matt Friedman <maf675@gmail.com>
Date: Thu, 9 Jul 2020 09:37:46 -0700
Subject: [PATCH 1/4] Get QI version from composer.json now

---
 includes/class_qi.php     | 27 +++++++++++++++++++++++----
 includes/qi_constants.php |  1 -
 includes/qi_functions.php |  2 +-
 index.php                 |  2 +-
 modules/qi_create.php     |  4 ++--
 5 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/includes/class_qi.php b/includes/class_qi.php
index 8512605e..afc18ca1 100644
--- a/includes/class_qi.php
+++ b/includes/class_qi.php
@@ -50,9 +50,9 @@ public static function page_header($page_title = '')
 			'S_USER_LANG'			=> $user->lang['USER_LANG'],
 
 			'TRANSLATION_INFO'	=> $user->lang['TRANSLATION_INFO'],
-			'QI_VERSION'		=> QI_VERSION,
+			'QI_VERSION'		=> self::current_version(),
 
-			'VERSION_CHECK_TITLE'	=> !empty($update) ? sprintf($user->lang['VERSION_CHECK_TITLE'], $update['current'], QI_VERSION) : '',
+			'VERSION_CHECK_TITLE'	=> !empty($update) ? sprintf($user->lang['VERSION_CHECK_TITLE'], $update['current'], self::current_version()) : '',
 			'VERSION_CHECK_CURRENT'	=> !empty($update) ? $update['current'] : '',
 			'U_VERSION_CHECK_URL'	=> !empty($update) ? $update['download'] : '',
 		));
@@ -345,7 +345,7 @@ public static function msg_handler($errno, $msg_text, $errfile, $errline)
 					'MSG_EXPLAIN'          => '',
 					'SETTINGS_FORM'        => '',
 					'RETURN_LINKS'         => $l_return_index,
-					'QI_VERSION'           => QI_VERSION,
+					'QI_VERSION'           => self::current_version(),
 					'L_QUICKINSTALL'       => $user->lang['QUICKINSTALL'],
 					'L_PHPBB_QI_TEXT'      => $user->lang['PHPBB_QI_TEXT'],
 					'L_FOR_PHPBB_VERSIONS' => $user->lang['FOR_PHPBB_VERSIONS'],
@@ -405,9 +405,28 @@ public static function get_update()
 		$version_helper = new qi_version_helper();
 
 		return $version_helper
-			->set_current_version(QI_VERSION)
+			->set_current_version(self::current_version())
 			->force_stability('stable')
 			->set_file_location('www.phpbb.com', '/customise/db/official_tool/phpbb3_quickinstall', 'version_check')
 			->get_update();
 	}
+
+	/**
+	 * Get the current version of QuickInstall from composer.json
+	 *
+	 * @return string
+	 */
+	public static function current_version()
+	{
+		global $quickinstall_path;
+
+		static $composerJson = null;
+
+		if ($composerJson === null) {
+			$composerJson = file_get_contents("{$quickinstall_path}composer.json");
+			$composerJson = json_decode($composerJson, true);
+		}
+
+		return $composerJson["version"];
+	}
 }
diff --git a/includes/qi_constants.php b/includes/qi_constants.php
index 131f83f2..76ea13d3 100644
--- a/includes/qi_constants.php
+++ b/includes/qi_constants.php
@@ -7,7 +7,6 @@
 *
 */
 
-define('QI_VERSION', '1.4.0');
 //define('QI_DEBUG', true); // un-comment this disable template caching
 
 // Cookies set by QI
diff --git a/includes/qi_functions.php b/includes/qi_functions.php
index d87e3069..25fac9ba 100644
--- a/includes/qi_functions.php
+++ b/includes/qi_functions.php
@@ -270,7 +270,7 @@ function gen_error_msg($msg_text, $msg_title = 'General Error', $msg_explain = '
 		'MSG_EXPLAIN'          => $msg_explain,
 		'SETTINGS_FORM'        => $settings_form,
 		'RETURN_LINKS'         => $l_return_index,
-		'QI_VERSION'           => QI_VERSION,
+		'QI_VERSION'           => qi::current_version(),
 		'L_QUICKINSTALL'       => $l_quickinstall,
 		'L_PHPBB_QI_TEXT'      => $l_phpbb_qi_text,
 		'L_FOR_PHPBB_VERSIONS' => $l_for_phpbb_versions,
diff --git a/index.php b/index.php
index c45ad7f5..dfbbfbb3 100644
--- a/index.php
+++ b/index.php
@@ -249,7 +249,7 @@
 else if (!empty($settings->update_text)) // PROFILE_UPDATED
 {
 	$update_title	= sprintf($user->lang['PROFILE_UPDATED'], $settings->profile);
-	$update_explain	= sprintf($user->lang['UPDATED_EXPLAIN'], QI_VERSION);
+	$update_explain	= sprintf($user->lang['UPDATED_EXPLAIN'], qi::current_version());
 
 	$update_msg = '<ul>';
 	foreach ($settings->update_text as $update)
diff --git a/modules/qi_create.php b/modules/qi_create.php
index 362b5c93..66837c59 100644
--- a/modules/qi_create.php
+++ b/modules/qi_create.php
@@ -831,11 +831,11 @@ public function __construct()
 		$user->ip = &$user_ip;
 		if (defined('PHPBB_31'))
 		{
-			add_log('admin', sprintf($user->lang['LOG_INSTALL_INSTALLED_QI'], QI_VERSION));
+			add_log('admin', sprintf($user->lang['LOG_INSTALL_INSTALLED_QI'], qi::current_version()));
 		}
 		else
 		{
-			add_log('admin', 'LOG_INSTALL_INSTALLED_QI', QI_VERSION);
+			add_log('admin', 'LOG_INSTALL_INSTALLED_QI', qi::current_version());
 		}
 
 		// purge cache

From 963ec2a95d3fbfd8762d2edd69de3ed5f24ab970 Mon Sep 17 00:00:00 2001
From: Matt Friedman <maf675@gmail.com>
Date: Thu, 9 Jul 2020 09:38:10 -0700
Subject: [PATCH 2/4] =?UTF-8?q?Move=20QI=20cookie=20constant=20into=20sett?=
 =?UTF-8?q?ings=20class=20where=20it=E2=80=99s=20used?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 includes/class_qi_settings.php | 13 +++++++++----
 includes/qi_constants.php      |  3 ---
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/includes/class_qi_settings.php b/includes/class_qi_settings.php
index 88c54884..f0a60766 100644
--- a/includes/class_qi_settings.php
+++ b/includes/class_qi_settings.php
@@ -21,6 +21,11 @@
  */
 class settings
 {
+	/**
+	 * Cookie with the latest used profile name as payload.
+	 */
+	const QI_PROFILE_COOKIE = 'qi_profile';
+
 	/**
 	 * Array with configuration settings.
 	 * @private
@@ -84,11 +89,11 @@ function __construct($profile = '', $mode = '')
 			$this->profile = $profile;
 			$this->set_profile_cookie($profile);
 		}
-		else if (!empty($_COOKIE[QI_PROFILE_COOKIE]) && is_readable("{$quickinstall_path}settings/{$_COOKIE[QI_PROFILE_COOKIE]}.cfg"))
+		else if (!empty($_COOKIE[self::QI_PROFILE_COOKIE]) && is_readable("{$quickinstall_path}settings/{$_COOKIE[self::QI_PROFILE_COOKIE]}.cfg"))
 		{
 			// Get the previously used profile.
-			$config = file("{$quickinstall_path}settings/{$_COOKIE[QI_PROFILE_COOKIE]}.cfg", FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
-			$this->profile = $_COOKIE[QI_PROFILE_COOKIE];
+			$config = file("{$quickinstall_path}settings/{$_COOKIE[self::QI_PROFILE_COOKIE]}.cfg", FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
+			$this->profile = $_COOKIE[self::QI_PROFILE_COOKIE];
 		}
 		else
 		{
@@ -827,7 +832,7 @@ function set_profile_cookie($profile = 'default')
 	{
 		// A Julian year == 365.25 days * 86,400 seconds
 		$expire_time = time() + 31557600;
-		setcookie(QI_PROFILE_COOKIE, $profile, $expire_time);
+		setcookie(self::QI_PROFILE_COOKIE, $profile, $expire_time);
 	}
 
 	/**
diff --git a/includes/qi_constants.php b/includes/qi_constants.php
index 76ea13d3..ef911975 100644
--- a/includes/qi_constants.php
+++ b/includes/qi_constants.php
@@ -8,6 +8,3 @@
 */
 
 //define('QI_DEBUG', true); // un-comment this disable template caching
-
-// Cookies set by QI
-define('QI_PROFILE_COOKIE', 'qi_profile');	// Cookie with the latest used profile name as payload.

From 0e7e0659199019dc20fee7129efe407cf53c4c71 Mon Sep 17 00:00:00 2001
From: Matt Friedman <maf675@gmail.com>
Date: Thu, 9 Jul 2020 09:41:48 -0700
Subject: [PATCH 3/4] Update constants file with notice to no longer use it

---
 includes/qi_constants.php | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/includes/qi_constants.php b/includes/qi_constants.php
index ef911975..c63a805d 100644
--- a/includes/qi_constants.php
+++ b/includes/qi_constants.php
@@ -7,4 +7,8 @@
 *
 */
 
+// IMPORTANT NOTE:
+// This file is for development and debugging purposes only. Use class constants instead of
+// defining global constants here. This file is ignored by git.
+
 //define('QI_DEBUG', true); // un-comment this disable template caching

From ccab2b6550915bec39b9976f63a7791a4ee2628b Mon Sep 17 00:00:00 2001
From: Matt Friedman <maf675@gmail.com>
Date: Thu, 9 Jul 2020 09:42:59 -0700
Subject: [PATCH 4/4] Ignore old qi constants file

---
 .gitignore | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitignore b/.gitignore
index aa1a8011..7b4ec838 100644
--- a/.gitignore
+++ b/.gitignore
@@ -9,3 +9,4 @@ build/*
 *~
 .idea
 */node_modules
+includes/qi_constants.php