Use constants for version constraints to make languages more maintainable

Signed-off-by: Matt Friedman <maf675@gmail.com>

Author: iMattPro

ext.php

@@ -15,17 +15,28 @@
  */
 class ext extends \phpbb\extension\base
 {
+	/**
+	 * Require phpBB 3.3.12 due to new template and core events.
+	 */
+	public const PHPBB_MIN_VERSION = '3.3.12';
+
+	/**
+	 * Should not be installed in phpBB 4 because it already has push notifications.
+	 */
+	public const PHPBB_MAX_VERSION = '4.0.0-dev';
+
+	/**
+	 * Require PHP 7.3 due to 3rd party libraries included.
+	 */
+	public const PHP_MIN_VERSION = '7.3';
+
 	/**
 	 * @var array An array of installation error messages
 	 */
 	protected $errors = [];
 
 	/**
 	 * {@inheritdoc}
-	 *
-	 * Requires phpBB 3.3.12 due to new template and core events.
-	 * Should not be installed in phpBB 4.0.0-a1 because it already has push notifications.
-	 * Requires PHP 7.3 due to 3rd party libraries included.
 	 */
 	public function is_enableable()
 	{
@@ -42,12 +53,12 @@ public function is_enableable()
 	 */
 	protected function check_phpbb_version()
 	{
-		if (phpbb_version_compare(PHPBB_VERSION, '3.3.12', '<'))
+		if (phpbb_version_compare(PHPBB_VERSION, self::PHPBB_MIN_VERSION, '<'))
 		{
 			$this->errors[] = 'PHPBB_VERSION_MIN_ERROR';
 		}
 
-		if (phpbb_version_compare(PHPBB_VERSION, '4.0.0-dev', '>='))
+		if (phpbb_version_compare(PHPBB_VERSION, self::PHPBB_MAX_VERSION, '>='))
 		{
 			$this->errors[] = 'PHPBB_VERSION_MAX_ERROR';
 		}

language/en/install.php

@@ -39,7 +39,7 @@
 
 $lang = array_merge($lang, [
 	'PHPBB_VERSION_MAX_ERROR'	=> 'This extension can not be installed on this board. This is a phpBB 4 board, which already contains the features in this extension.',
-	'PHPBB_VERSION_MIN_ERROR'	=> 'phpBB 3.3.12 or newer is required.',
-	'PHP_VERSION_ERROR'			=> 'PHP 7.3 or newer is required.',
+	'PHPBB_VERSION_MIN_ERROR'	=> 'phpBB ' . \phpbb\webpushnotifications\ext::PHPBB_MIN_VERSION . ' or newer is required.',
+	'PHP_VERSION_ERROR'			=> 'PHP ' . \phpbb\webpushnotifications\ext::PHP_MIN_VERSION . ' or newer is required.',
 	'PHP_EXT_MISSING'			=> 'The “%s” extension for PHP must be installed on this server.',
 ]);

language/ru/install.php

@@ -39,6 +39,6 @@
 
 $lang = array_merge($lang, [
 	'PHPBB_VERSION_MAX_ERROR'	=> 'Данное расширение несовместимо с установленной версией конференции. Вы используете версию phpBB 4, которая уже содержит соответствующие функции.',
-	'PHPBB_VERSION_MIN_ERROR'	=> 'Данное расширение несовместимо с установленной версией конференции. Необходима версия phpBB 3.3.12 или выше.',
-	'PHP_VERSION_ERROR'	=> 'Данное расширение несовместимо с установленной на сервере версией PHP. Необходима версия PHP 7.3 или выше.',
+	'PHPBB_VERSION_MIN_ERROR'	=> 'Данное расширение несовместимо с установленной версией конференции. Необходима версия phpBB ' . \phpbb\webpushnotifications\ext::PHPBB_MIN_VERSION . ' или выше.',
+	'PHP_VERSION_ERROR'			=> 'Данное расширение несовместимо с установленной на сервере версией PHP. Необходима версия PHP ' . \phpbb\webpushnotifications\ext::PHP_MIN_VERSION . ' или выше.',
 ]);