Fix role exists install issues

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

Author: iMattPro

composer.json

@@ -29,7 +29,7 @@
 		}
 	],
 	"require": {
-		"php": ">=5.4",
+		"php": ">=7.1.3",
 		"composer/installers": "~1.0"
 	},
 	"require-dev": {
@@ -38,7 +38,7 @@
 	"extra": {
 		"display-name": "Board Rules",
 		"soft-require": {
-			"phpbb/phpbb": ">=3.2.0"
+			"phpbb/phpbb": ">=3.3.2"
 		},
 		"version-check": {
 			"host": "www.phpbb.com",

ext.php

@@ -31,15 +31,14 @@ class ext extends \phpbb\extension\base
 	* The current phpBB version should meet or exceed
 	* the minimum version required by this extension:
 	*
-	* Requires phpBB 3.2.0 due to new faq controller route for bbcodes,
-	* the revised notifications system, font awesome and the text reparser.
+	* Requires phpBB 3.3.2 due to using role_exists check in permission migration.
 	*
 	* @return bool
 	* @access public
 	*/
 	public function is_enableable()
 	{
-		return phpbb_version_compare(PHPBB_VERSION, '3.2.0', '>=');
+		return phpbb_version_compare(PHPBB_VERSION, '3.3.2', '>=');
 	}
 
 	/**

migrations/v10x/m5_initial_permission.php

@@ -40,8 +40,14 @@ public function update_data()
 			array('permission.add', array('a_boardrules', true)),
 
 			// Set permissions
-			array('permission.permission_set', array('ROLE_ADMIN_FULL', 'a_boardrules')),
-			array('permission.permission_set', array('ROLE_ADMIN_STANDARD', 'a_boardrules')),
+			array('if', array(
+				array('permission.role_exists', array('ROLE_ADMIN_FULL')),
+				array('permission.permission_set', array('ROLE_ADMIN_FULL', 'a_boardrules')),
+			)),
+			array('if', array(
+				array('permission.role_exists', array('ROLE_ADMIN_STANDARD')),
+				array('permission.permission_set', array('ROLE_ADMIN_STANDARD', 'a_boardrules')),
+			)),
 		);
 	}
 }