Rename constants

Author: iMattPro

ext.php

@@ -15,19 +15,13 @@
 
 class ext extends \phpbb\extension\base
 {
-	/** @var string The default value for PHP to use in skeletons */
-	const DEFAULT_PHP = '7.1.3';
-	/** @var string The default value for minimum phpBB to use in skeletons */
-	const DEFAULT_PHPBB_MIN = '3.3.0';
-	/** @var string The default value for maximum phpBB to use in skeletons */
-	const DEFAULT_PHPBB_MAX = '4.0.0@dev';
+	public const DEFAULT_SKELETON_PHP = '7.1.3';
+	public const DEFAULT_SKELETON_PHPBB_MIN = '3.3.0';
+	public const DEFAULT_SKELETON_PHPBB_MAX = '4.0.0@dev';
 
-	/** @var string The minimum version of phpBB this skeleton extension supports */
-	const REQUIRE_PHPBB_MIN = '3.3.0';
-	/** @var string The maximum version of phpBB this skeleton extension supports */
-	const REQUIRE_PHPBB_MAX = '4.0.0-dev';
-	/** @var string The minimum version of PHP this skeleton extension supports */
-	const REQUIRE_PHP = 70100;
+	public const MIN_PHPBB_ALLOWED = '3.3.0';
+	public const MAX_PHPBB_ALLOWED = '4.0.0-dev';
+	public const MIN_PHP_ALLOWED = 70100;
 
 	/**
 	 * @var array An array of installation error messages
@@ -57,11 +51,11 @@ public function is_enableable()
 	 */
 	protected function phpbb_requirement($phpBB_version = PHPBB_VERSION)
 	{
-		if (phpbb_version_compare($phpBB_version, self::REQUIRE_PHPBB_MIN, '<'))
+		if (phpbb_version_compare($phpBB_version, self::MIN_PHPBB_ALLOWED, '<'))
 		{
 			$this->errors[] = 'PHPBB_VERSION_MIN_ERROR';
 		}
-		else if (phpbb_version_compare($phpBB_version, self::REQUIRE_PHPBB_MAX, '>='))
+		else if (phpbb_version_compare($phpBB_version, self::MAX_PHPBB_ALLOWED, '>='))
 		{
 			$this->errors[] = 'PHPBB_VERSION_MAX_ERROR';
 		}
@@ -75,7 +69,7 @@ protected function phpbb_requirement($phpBB_version = PHPBB_VERSION)
 	 */
 	protected function php_requirement($php_version = PHP_VERSION_ID)
 	{
-		if ($php_version < self::REQUIRE_PHP)
+		if ($php_version < self::MIN_PHP_ALLOWED)
 		{
 			$this->errors[] = 'PHP_VERSION_ERROR';
 		}

helper/packager.php

@@ -74,9 +74,9 @@ public function get_composer_dialog_values()
 				'extension_homepage'     => null,
 			],
 			'requirements' => [
-				'php_version'       => '>=' . ext::DEFAULT_PHP,
-				'phpbb_version_min' => '>=' . ext::DEFAULT_PHPBB_MIN,
-				'phpbb_version_max' => '<' . ext::DEFAULT_PHPBB_MAX,
+				'php_version'       => '>=' . ext::DEFAULT_SKELETON_PHP,
+				'phpbb_version_min' => '>=' . ext::DEFAULT_SKELETON_PHPBB_MIN,
+				'phpbb_version_max' => '<' . ext::DEFAULT_SKELETON_PHPBB_MAX,
 			],
 		];
 	}

language/en/common.php

@@ -70,13 +70,13 @@
 
 	'SKELETON_QUESTION_PHP_VERSION'					=> 'Please enter the PHP requirement of the extension',
 	'SKELETON_QUESTION_PHP_VERSION_UI'				=> 'PHP requirement of the extension',
-	'SKELETON_QUESTION_PHP_VERSION_EXPLAIN'			=> 'default: >=' . \phpbb\skeleton\ext::DEFAULT_PHP,
+	'SKELETON_QUESTION_PHP_VERSION_EXPLAIN'			=> 'default: >=' . \phpbb\skeleton\ext::DEFAULT_SKELETON_PHP,
 	'SKELETON_QUESTION_PHPBB_VERSION_MIN'			=> 'Please enter the minimum phpBB requirement of the extension',
 	'SKELETON_QUESTION_PHPBB_VERSION_MIN_UI'		=> 'Minimum phpBB requirement of the extension',
-	'SKELETON_QUESTION_PHPBB_VERSION_MIN_EXPLAIN'	=> 'default: >=' . \phpbb\skeleton\ext::DEFAULT_PHPBB_MIN,
+	'SKELETON_QUESTION_PHPBB_VERSION_MIN_EXPLAIN'	=> 'default: >=' . \phpbb\skeleton\ext::DEFAULT_SKELETON_PHPBB_MIN,
 	'SKELETON_QUESTION_PHPBB_VERSION_MAX'			=> 'Please enter the maximum phpBB requirement of the extension',
 	'SKELETON_QUESTION_PHPBB_VERSION_MAX_UI'		=> 'Maximum phpBB requirement of the extension',
-	'SKELETON_QUESTION_PHPBB_VERSION_MAX_EXPLAIN'	=> 'default: <' . \phpbb\skeleton\ext::DEFAULT_PHPBB_MAX,
+	'SKELETON_QUESTION_PHPBB_VERSION_MAX_EXPLAIN'	=> 'default: <' . \phpbb\skeleton\ext::DEFAULT_SKELETON_PHPBB_MAX,
 
 	'SKELETON_QUESTION_COMPONENT_PHPLISTENER'		=> 'Create sample PHP event listeners',
 	'SKELETON_QUESTION_COMPONENT_PHPLISTENER_UI'	=> 'PHP event listeners',

tests/controller/main_test.php

@@ -146,7 +146,7 @@ public function test_handle($status_code, $page_content)
 				['vendor_name', '', true, \phpbb\request\request_interface::REQUEST, 'foo'],
 				['author_name', [''], true, \phpbb\request\request_interface::REQUEST, ['bar']],
 				['extension_version', '1.0.0-dev', true, \phpbb\request\request_interface::REQUEST, '1.0.0-dev'],
-				['php_version', '>=' . ext::DEFAULT_PHP, false, \phpbb\request\request_interface::REQUEST, '>=' . ext::DEFAULT_PHP],
+				['php_version', '>=' . ext::DEFAULT_SKELETON_PHP, false, \phpbb\request\request_interface::REQUEST, '>=' . ext::DEFAULT_SKELETON_PHP],
 				['component_phplistener', false, false, \phpbb\request\request_interface::REQUEST, true],
 			]);
 
@@ -229,7 +229,7 @@ public function test_handle($status_code, $page_content)
 					'NAME'         => 'php_version',
 					'DESC'         => 'SKELETON_QUESTION_PHP_VERSION_UI',
 					'DESC_EXPLAIN' => 'SKELETON_QUESTION_PHP_VERSION_EXPLAIN',
-					'VALUE'        => '>=' . ext::DEFAULT_PHP,
+					'VALUE'        => '>=' . ext::DEFAULT_SKELETON_PHP,
 				]],
 				['requirement', [
 					'NAME'         => 'phpbb_version_min',