pryley / site-reviews

plugin/Modules/Multilingual/Wpml.php

Indentation +73 added lines, -73 removed lines

@@ -7,83 +7,83 @@
 
 class Wpml implements Contract
 {
-    public $pluginName = 'WPML';
-    public $supportedVersion = '3.3.5';
+	public $pluginName = 'WPML';
+	public $supportedVersion = '3.3.5';
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPost($postId)
-    {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
-            return;
-        }
-        if ($this->isEnabled()) {
-            $postId = apply_filters('wpml_object_id', $postId, 'any', true);
-        }
-        return get_post(intval($postId));
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPost($postId)
+	{
+		$postId = trim($postId);
+		if (!is_numeric($postId)) {
+			return;
+		}
+		if ($this->isEnabled()) {
+			$postId = apply_filters('wpml_object_id', $postId, 'any', true);
+		}
+		return get_post(intval($postId));
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPostIds(array $postIds)
-    {
-        if (!$this->isEnabled()) {
-            return $postIds;
-        }
-        $newPostIds = [];
-        foreach ($this->cleanIds($postIds) as $postId) {
-            $postType = get_post_type($postId);
-            if (!$postType) {
-                continue;
-            }
-            $elementType = 'post_'.$postType;
-            $trid = apply_filters('wpml_element_trid', null, $postId, $elementType);
-            $translations = apply_filters('wpml_get_element_translations', null, $trid, $elementType);
-            if (!is_array($translations)) {
-                $translations = [];
-            }
-            $newPostIds = array_merge(
-                $newPostIds,
-                array_column($translations, 'element_id')
-            );
-        }
-        return $this->cleanIds($newPostIds);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPostIds(array $postIds)
+	{
+		if (!$this->isEnabled()) {
+			return $postIds;
+		}
+		$newPostIds = [];
+		foreach ($this->cleanIds($postIds) as $postId) {
+			$postType = get_post_type($postId);
+			if (!$postType) {
+				continue;
+			}
+			$elementType = 'post_'.$postType;
+			$trid = apply_filters('wpml_element_trid', null, $postId, $elementType);
+			$translations = apply_filters('wpml_get_element_translations', null, $trid, $elementType);
+			if (!is_array($translations)) {
+				$translations = [];
+			}
+			$newPostIds = array_merge(
+				$newPostIds,
+				array_column($translations, 'element_id')
+			);
+		}
+		return $this->cleanIds($newPostIds);
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isActive()
-    {
-        return defined('ICL_SITEPRESS_VERSION');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isActive()
+	{
+		return defined('ICL_SITEPRESS_VERSION');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isEnabled()
-    {
-        return $this->isActive()
-            && 'wpml' == glsr(OptionManager::class)->get('settings.general.multilingual');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isEnabled()
+	{
+		return $this->isActive()
+			&& 'wpml' == glsr(OptionManager::class)->get('settings.general.multilingual');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isSupported()
-    {
-        return $this->isActive()
-            && version_compare(ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isSupported()
+	{
+		return $this->isActive()
+			&& version_compare(ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=');
+	}
 
-    /**
-     * @return array
-     */
-    protected function cleanIds(array $postIds)
-    {
-        return array_filter(array_unique($postIds));
-    }
+	/**
+	 * @return array
+	 */
+	protected function cleanIds(array $postIds)
+	{
+		return array_filter(array_unique($postIds));
+	}
 }

Spacing +21 added lines, -21 removed lines

@@ -13,44 +13,44 @@ 
     /**
      * {@inheritdoc}
      */
-    public function getPost($postId)
+    public function getPost( $postId )
     {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
+        $postId = trim( $postId );
+        if( !is_numeric( $postId ) ) {
             return;
         }
-        if ($this->isEnabled()) {
-            $postId = apply_filters('wpml_object_id', $postId, 'any', true);
+        if( $this->isEnabled() ) {
+            $postId = apply_filters( 'wpml_object_id', $postId, 'any', true );
         }
-        return get_post(intval($postId));
+        return get_post( intval( $postId ) );
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getPostIds(array $postIds)
+    public function getPostIds( array $postIds )
     {
-        if (!$this->isEnabled()) {
+        if( !$this->isEnabled() ) {
             return $postIds;
         }
         $newPostIds = [];
-        foreach ($this->cleanIds($postIds) as $postId) {
-            $postType = get_post_type($postId);
-            if (!$postType) {
+        foreach( $this->cleanIds( $postIds ) as $postId ) {
+            $postType = get_post_type( $postId );
+            if( !$postType ) {
                 continue;
             }
             $elementType = 'post_'.$postType;
-            $trid = apply_filters('wpml_element_trid', null, $postId, $elementType);
-            $translations = apply_filters('wpml_get_element_translations', null, $trid, $elementType);
-            if (!is_array($translations)) {
+            $trid = apply_filters( 'wpml_element_trid', null, $postId, $elementType );
+            $translations = apply_filters( 'wpml_get_element_translations', null, $trid, $elementType );
+            if( !is_array( $translations ) ) {
                 $translations = [];
             }
             $newPostIds = array_merge(
                 $newPostIds,
-                array_column($translations, 'element_id')
+                array_column( $translations, 'element_id' )
             );
         }
-        return $this->cleanIds($newPostIds);
+        return $this->cleanIds( $newPostIds );
     }
 
     /**
@@ -58,7 +58,7 @@ 
      */
     public function isActive()
     {
-        return defined('ICL_SITEPRESS_VERSION');
+        return defined( 'ICL_SITEPRESS_VERSION' );
     }
 
     /**
@@ -67,7 +67,7 @@ 
     public function isEnabled()
     {
         return $this->isActive()
-            && 'wpml' == glsr(OptionManager::class)->get('settings.general.multilingual');
+            && 'wpml' == glsr( OptionManager::class )->get( 'settings.general.multilingual' );
     }
 
     /**
@@ -76,14 +76,14 @@ 
     public function isSupported()
     {
         return $this->isActive()
-            && version_compare(ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=');
+            && version_compare( ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=' );
     }
 
     /**
      * @return array
      */
-    protected function cleanIds(array $postIds)
+    protected function cleanIds( array $postIds )
     {
-        return array_filter(array_unique($postIds));
+        return array_filter( array_unique( $postIds ) );
     }
 }

plugin/Modules/Multilingual/Polylang.php

Indentation +69 added lines, -69 removed lines

@@ -7,79 +7,79 @@
 
 class Polylang implements Contract
 {
-    public $pluginName = 'Polylang';
-    public $supportedVersion = '2.3';
+	public $pluginName = 'Polylang';
+	public $supportedVersion = '2.3';
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPost($postId)
-    {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
-            return;
-        }
-        if ($this->isEnabled()) {
-            $polylangPostId = pll_get_post($postId, pll_get_post_language(get_the_ID()));
-        }
-        if (!empty($polylangPostId)) {
-            $postId = $polylangPostId;
-        }
-        return get_post(intval($postId));
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPost($postId)
+	{
+		$postId = trim($postId);
+		if (!is_numeric($postId)) {
+			return;
+		}
+		if ($this->isEnabled()) {
+			$polylangPostId = pll_get_post($postId, pll_get_post_language(get_the_ID()));
+		}
+		if (!empty($polylangPostId)) {
+			$postId = $polylangPostId;
+		}
+		return get_post(intval($postId));
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPostIds(array $postIds)
-    {
-        if (!$this->isEnabled()) {
-            return $postIds;
-        }
-        $newPostIds = [];
-        foreach ($this->cleanIds($postIds) as $postId) {
-            $newPostIds = array_merge(
-                $newPostIds,
-                array_values(pll_get_post_translations($postId))
-            );
-        }
-        return $this->cleanIds($newPostIds);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPostIds(array $postIds)
+	{
+		if (!$this->isEnabled()) {
+			return $postIds;
+		}
+		$newPostIds = [];
+		foreach ($this->cleanIds($postIds) as $postId) {
+			$newPostIds = array_merge(
+				$newPostIds,
+				array_values(pll_get_post_translations($postId))
+			);
+		}
+		return $this->cleanIds($newPostIds);
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isActive()
-    {
-        return function_exists('PLL')
-            && function_exists('pll_get_post')
-            && function_exists('pll_get_post_language')
-            && function_exists('pll_get_post_translations');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isActive()
+	{
+		return function_exists('PLL')
+			&& function_exists('pll_get_post')
+			&& function_exists('pll_get_post_language')
+			&& function_exists('pll_get_post_translations');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isEnabled()
-    {
-        return $this->isActive()
-            && 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isEnabled()
+	{
+		return $this->isActive()
+			&& 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isSupported()
-    {
-        return defined('POLYLANG_VERSION')
-            && version_compare(POLYLANG_VERSION, $this->supportedVersion, '>=');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isSupported()
+	{
+		return defined('POLYLANG_VERSION')
+			&& version_compare(POLYLANG_VERSION, $this->supportedVersion, '>=');
+	}
 
-    /**
-     * @return array
-     */
-    protected function cleanIds(array $postIds)
-    {
-        return array_filter(array_unique($postIds));
-    }
+	/**
+	 * @return array
+	 */
+	protected function cleanIds(array $postIds)
+	{
+		return array_filter(array_unique($postIds));
+	}
 }

Spacing +21 added lines, -21 removed lines

@@ -13,37 +13,37 @@ 
     /**
      * {@inheritdoc}
      */
-    public function getPost($postId)
+    public function getPost( $postId )
     {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
+        $postId = trim( $postId );
+        if( !is_numeric( $postId ) ) {
             return;
         }
-        if ($this->isEnabled()) {
-            $polylangPostId = pll_get_post($postId, pll_get_post_language(get_the_ID()));
+        if( $this->isEnabled() ) {
+            $polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ) );
         }
-        if (!empty($polylangPostId)) {
+        if( !empty($polylangPostId) ) {
             $postId = $polylangPostId;
         }
-        return get_post(intval($postId));
+        return get_post( intval( $postId ) );
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getPostIds(array $postIds)
+    public function getPostIds( array $postIds )
     {
-        if (!$this->isEnabled()) {
+        if( !$this->isEnabled() ) {
             return $postIds;
         }
         $newPostIds = [];
-        foreach ($this->cleanIds($postIds) as $postId) {
+        foreach( $this->cleanIds( $postIds ) as $postId ) {
             $newPostIds = array_merge(
                 $newPostIds,
-                array_values(pll_get_post_translations($postId))
+                array_values( pll_get_post_translations( $postId ) )
             );
         }
-        return $this->cleanIds($newPostIds);
+        return $this->cleanIds( $newPostIds );
     }
 
     /**
@@ -51,10 +51,10 @@ 
      */
     public function isActive()
     {
-        return function_exists('PLL')
-            && function_exists('pll_get_post')
-            && function_exists('pll_get_post_language')
-            && function_exists('pll_get_post_translations');
+        return function_exists( 'PLL' )
+            && function_exists( 'pll_get_post' )
+            && function_exists( 'pll_get_post_language' )
+            && function_exists( 'pll_get_post_translations' );
     }
 
     /**
@@ -63,7 +63,7 @@ 
     public function isEnabled()
     {
         return $this->isActive()
-            && 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
+            && 'polylang' == glsr( OptionManager::class )->get( 'settings.general.multilingual' );
     }
 
     /**
@@ -71,15 +71,15 @@ 
      */
     public function isSupported()
     {
-        return defined('POLYLANG_VERSION')
-            && version_compare(POLYLANG_VERSION, $this->supportedVersion, '>=');
+        return defined( 'POLYLANG_VERSION' )
+            && version_compare( POLYLANG_VERSION, $this->supportedVersion, '>=' );
     }
 
     /**
      * @return array
      */
-    protected function cleanIds(array $postIds)
+    protected function cleanIds( array $postIds )
     {
-        return array_filter(array_unique($postIds));
+        return array_filter( array_unique( $postIds ) );
     }
 }

plugin/Modules/Multilingual.php

Indentation +76 added lines, -76 removed lines

@@ -7,87 +7,87 @@
 
 class Multilingual implements Contract
 {
-    protected $integration;
+	protected $integration;
 
-    /**
-     * @param string $integration
-     * @return false|\GeminiLabs\SiteReviews\Modules\Multilingual\Polylang|\GeminiLabs\SiteReviews\Modules\Multilingual\Wpml
-     */
-    public function getIntegration($integration = '')
-    {
-        if (empty($integration)) {
-            $integration = ucfirst(glsr(OptionManager::class)->get('settings.general.multilingual'));
-        }
-        $integrationClass = 'GeminiLabs\SiteReviews\Modules\Multilingual\\'.$integration;
-        if (class_exists($integrationClass)) {
-            return glsr($integrationClass);
-        }
-        return false;
-    }
+	/**
+	 * @param string $integration
+	 * @return false|\GeminiLabs\SiteReviews\Modules\Multilingual\Polylang|\GeminiLabs\SiteReviews\Modules\Multilingual\Wpml
+	 */
+	public function getIntegration($integration = '')
+	{
+		if (empty($integration)) {
+			$integration = ucfirst(glsr(OptionManager::class)->get('settings.general.multilingual'));
+		}
+		$integrationClass = 'GeminiLabs\SiteReviews\Modules\Multilingual\\'.$integration;
+		if (class_exists($integrationClass)) {
+			return glsr($integrationClass);
+		}
+		return false;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPost($postId)
-    {
-        return $this->isIntegrated()
-            ? $this->integration->getPostIds($postId)
-            : $postId;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPost($postId)
+	{
+		return $this->isIntegrated()
+			? $this->integration->getPostIds($postId)
+			: $postId;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPostIds(array $postIds)
-    {
-        return $this->isIntegrated()
-            ? $this->integration->getPostIds($postIds)
-            : $postIds;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPostIds(array $postIds)
+	{
+		return $this->isIntegrated()
+			? $this->integration->getPostIds($postIds)
+			: $postIds;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isActive()
-    {
-        return $this->isIntegrated()
-            ? $this->integration->isActive()
-            : false;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isActive()
+	{
+		return $this->isIntegrated()
+			? $this->integration->isActive()
+			: false;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isEnabled()
-    {
-        return $this->isIntegrated()
-            ? $this->integration->isEnabled()
-            : false;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isEnabled()
+	{
+		return $this->isIntegrated()
+			? $this->integration->isEnabled()
+			: false;
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isSupported()
-    {
-        return $this->isIntegrated()
-            ? $this->integration->isSupported()
-            : false;
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isSupported()
+	{
+		return $this->isIntegrated()
+			? $this->integration->isSupported()
+			: false;
+	}
 
-    /**
-     * return bool
-     */
-    protected function isIntegrated()
-    {
-        if (!empty($this->integration)) {
-            return true;
-        }
-        if ($integration = $this->getIntegration()) {
-            $this->integration = $integration;
-            return true;
-        }
-        glsr_log()->error($integrationClass.' does not exist');
-        return false;
-    }
+	/**
+	 * return bool
+	 */
+	protected function isIntegrated()
+	{
+		if (!empty($this->integration)) {
+			return true;
+		}
+		if ($integration = $this->getIntegration()) {
+			$this->integration = $integration;
+			return true;
+		}
+		glsr_log()->error($integrationClass.' does not exist');
+		return false;
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -13,14 +13,14 @@ 
      * @param string $integration
      * @return false|\GeminiLabs\SiteReviews\Modules\Multilingual\Polylang|\GeminiLabs\SiteReviews\Modules\Multilingual\Wpml
      */
-    public function getIntegration($integration = '')
+    public function getIntegration( $integration = '' )
     {
-        if (empty($integration)) {
-            $integration = ucfirst(glsr(OptionManager::class)->get('settings.general.multilingual'));
+        if( empty($integration) ) {
+            $integration = ucfirst( glsr( OptionManager::class )->get( 'settings.general.multilingual' ) );
         }
         $integrationClass = 'GeminiLabs\SiteReviews\Modules\Multilingual\\'.$integration;
-        if (class_exists($integrationClass)) {
-            return glsr($integrationClass);
+        if( class_exists( $integrationClass ) ) {
+            return glsr( $integrationClass );
         }
         return false;
     }
@@ -28,20 +28,20 @@ 
     /**
      * {@inheritdoc}
      */
-    public function getPost($postId)
+    public function getPost( $postId )
     {
         return $this->isIntegrated()
-            ? $this->integration->getPostIds($postId)
+            ? $this->integration->getPostIds( $postId )
             : $postId;
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getPostIds(array $postIds)
+    public function getPostIds( array $postIds )
     {
         return $this->isIntegrated()
-            ? $this->integration->getPostIds($postIds)
+            ? $this->integration->getPostIds( $postIds )
             : $postIds;
     }
 
@@ -80,14 +80,14 @@ 
      */
     protected function isIntegrated()
     {
-        if (!empty($this->integration)) {
+        if( !empty($this->integration) ) {
             return true;
         }
-        if ($integration = $this->getIntegration()) {
+        if( $integration = $this->getIntegration() ) {
             $this->integration = $integration;
             return true;
         }
-        glsr_log()->error($integrationClass.' does not exist');
+        glsr_log()->error( $integrationClass.' does not exist' );
         return false;
     }
 }

plugin/Controllers/SettingsController.php

Indentation +112 added lines, -112 removed lines

@@ -9,121 +9,121 @@
 
 class SettingsController extends Controller
 {
-    /**
-     * @param mixed $input
-     * @return array
-     * @callback register_setting
-     */
-    public function callbackRegisterSettings($input)
-    {
-        $settings = Arr::consolidateArray($input);
-        if (1 === count($settings) && array_key_exists('settings', $settings)) {
-            $options = array_replace_recursive(glsr(OptionManager::class)->all(), $input);
-            $options = $this->sanitizeGeneral($input, $options);
-            $options = $this->sanitizeSubmissions($input, $options);
-            $options = $this->sanitizeTranslations($input, $options);
-            $options = apply_filters('site-reviews/settings/callback', $options, $settings);
-            if (filter_input(INPUT_POST, 'option_page') == Application::ID.'-settings') {
-                glsr(Notice::class)->addSuccess(__('Settings updated.', 'site-reviews'));
-            }
-            return $options;
-        }
-        return $input;
-    }
+	/**
+	 * @param mixed $input
+	 * @return array
+	 * @callback register_setting
+	 */
+	public function callbackRegisterSettings($input)
+	{
+		$settings = Arr::consolidateArray($input);
+		if (1 === count($settings) && array_key_exists('settings', $settings)) {
+			$options = array_replace_recursive(glsr(OptionManager::class)->all(), $input);
+			$options = $this->sanitizeGeneral($input, $options);
+			$options = $this->sanitizeSubmissions($input, $options);
+			$options = $this->sanitizeTranslations($input, $options);
+			$options = apply_filters('site-reviews/settings/callback', $options, $settings);
+			if (filter_input(INPUT_POST, 'option_page') == Application::ID.'-settings') {
+				glsr(Notice::class)->addSuccess(__('Settings updated.', 'site-reviews'));
+			}
+			return $options;
+		}
+		return $input;
+	}
 
-    /**
-     * @return void
-     * @action admin_init
-     */
-    public function registerSettings()
-    {
-        register_setting(Application::ID.'-settings', OptionManager::databaseKey(), [
-            'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-        ]);
-    }
+	/**
+	 * @return void
+	 * @action admin_init
+	 */
+	public function registerSettings()
+	{
+		register_setting(Application::ID.'-settings', OptionManager::databaseKey(), [
+			'sanitize_callback' => [$this, 'callbackRegisterSettings'],
+		]);
+	}
 
-    /**
-     * @return array
-     */
-    protected function sanitizeGeneral(array $input, array $options)
-    {
-        $key = 'settings.general';
-        $inputForm = Arr::get($input, $key);
-        if (!$this->hasMultilingualIntegration(Arr::get($inputForm, 'multilingual'))) {
-            $options = Arr::set($options, $key.'.multilingual', '');
-        }
-        if ('' == trim(Arr::get($inputForm, 'notification_message'))) {
-            $defaultValue = Arr::get(glsr()->defaults, $key.'.notification_message');
-            $options = Arr::set($options, $key.'.notification_message', $defaultValue);
-        }
-        $defaultValue = Arr::get($inputForm, 'notifications', []);
-        $options = Arr::set($options, $key.'.notifications', $defaultValue);
-        return $options;
-    }
+	/**
+	 * @return array
+	 */
+	protected function sanitizeGeneral(array $input, array $options)
+	{
+		$key = 'settings.general';
+		$inputForm = Arr::get($input, $key);
+		if (!$this->hasMultilingualIntegration(Arr::get($inputForm, 'multilingual'))) {
+			$options = Arr::set($options, $key.'.multilingual', '');
+		}
+		if ('' == trim(Arr::get($inputForm, 'notification_message'))) {
+			$defaultValue = Arr::get(glsr()->defaults, $key.'.notification_message');
+			$options = Arr::set($options, $key.'.notification_message', $defaultValue);
+		}
+		$defaultValue = Arr::get($inputForm, 'notifications', []);
+		$options = Arr::set($options, $key.'.notifications', $defaultValue);
+		return $options;
+	}
 
-    /**
-     * @return array
-     */
-    protected function sanitizeSubmissions(array $input, array $options)
-    {
-        $key = 'settings.submissions';
-        $inputForm = Arr::get($input, $key);
-        $defaultValue = isset($inputForm['required'])
-            ? $inputForm['required']
-            : [];
-        $options = Arr::set($options, $key.'.required', $defaultValue);
-        return $options;
-    }
+	/**
+	 * @return array
+	 */
+	protected function sanitizeSubmissions(array $input, array $options)
+	{
+		$key = 'settings.submissions';
+		$inputForm = Arr::get($input, $key);
+		$defaultValue = isset($inputForm['required'])
+			? $inputForm['required']
+			: [];
+		$options = Arr::set($options, $key.'.required', $defaultValue);
+		return $options;
+	}
 
-    /**
-     * @return array
-     */
-    protected function sanitizeTranslations(array $input, array $options)
-    {
-        $key = 'settings.strings';
-        $inputForm = Arr::consolidateArray(Arr::get($input, $key));
-        if (!empty($inputForm)) {
-            $options = Arr::set($options, $key, array_values(array_filter($inputForm)));
-            $allowedTags = [
-                'a' => ['class' => [], 'href' => [], 'target' => []],
-                'span' => ['class' => []],
-            ];
-            array_walk($options['settings']['strings'], function (&$string) use ($allowedTags) {
-                if (isset($string['s2'])) {
-                    $string['s2'] = wp_kses($string['s2'], $allowedTags);
-                }
-                if (isset($string['p2'])) {
-                    $string['p2'] = wp_kses($string['p2'], $allowedTags);
-                }
-            });
-        }
-        return $options;
-    }
+	/**
+	 * @return array
+	 */
+	protected function sanitizeTranslations(array $input, array $options)
+	{
+		$key = 'settings.strings';
+		$inputForm = Arr::consolidateArray(Arr::get($input, $key));
+		if (!empty($inputForm)) {
+			$options = Arr::set($options, $key, array_values(array_filter($inputForm)));
+			$allowedTags = [
+				'a' => ['class' => [], 'href' => [], 'target' => []],
+				'span' => ['class' => []],
+			];
+			array_walk($options['settings']['strings'], function (&$string) use ($allowedTags) {
+				if (isset($string['s2'])) {
+					$string['s2'] = wp_kses($string['s2'], $allowedTags);
+				}
+				if (isset($string['p2'])) {
+					$string['p2'] = wp_kses($string['p2'], $allowedTags);
+				}
+			});
+		}
+		return $options;
+	}
 
-    /**
-     * @param string $integrationSlug
-     * @return bool
-     */
-    protected function hasMultilingualIntegration($integrationSlug)
-    {
-        $integration = glsr(Multilingual::class)->getIntegration($integrationSlug);
-        if (!$integration) {
-            return false;
-        }
-        if (!$integration->isActive()) {
-            glsr(Notice::class)->addError(sprintf(
-                __('Please install/activate the %s plugin to enable integration.', 'site-reviews'),
-                $integration->pluginName
-            ));
-            return false;
-        } elseif (!$integration->isSupported()) {
-            glsr(Notice::class)->addError(sprintf(
-                __('Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews'),
-                $integration->pluginName,
-                $integration->supportedVersion
-            ));
-            return false;
-        }
-        return true;
-    }
+	/**
+	 * @param string $integrationSlug
+	 * @return bool
+	 */
+	protected function hasMultilingualIntegration($integrationSlug)
+	{
+		$integration = glsr(Multilingual::class)->getIntegration($integrationSlug);
+		if (!$integration) {
+			return false;
+		}
+		if (!$integration->isActive()) {
+			glsr(Notice::class)->addError(sprintf(
+				__('Please install/activate the %s plugin to enable integration.', 'site-reviews'),
+				$integration->pluginName
+			));
+			return false;
+		} elseif (!$integration->isSupported()) {
+			glsr(Notice::class)->addError(sprintf(
+				__('Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews'),
+				$integration->pluginName,
+				$integration->supportedVersion
+			));
+			return false;
+		}
+		return true;
+	}
 }

Spacing +44 added lines, -44 removed lines

@@ -14,17 +14,17 @@ 
      * @return array
      * @callback register_setting
      */
-    public function callbackRegisterSettings($input)
+    public function callbackRegisterSettings( $input )
     {
-        $settings = Arr::consolidateArray($input);
-        if (1 === count($settings) && array_key_exists('settings', $settings)) {
-            $options = array_replace_recursive(glsr(OptionManager::class)->all(), $input);
-            $options = $this->sanitizeGeneral($input, $options);
-            $options = $this->sanitizeSubmissions($input, $options);
-            $options = $this->sanitizeTranslations($input, $options);
-            $options = apply_filters('site-reviews/settings/callback', $options, $settings);
-            if (filter_input(INPUT_POST, 'option_page') == Application::ID.'-settings') {
-                glsr(Notice::class)->addSuccess(__('Settings updated.', 'site-reviews'));
+        $settings = Arr::consolidateArray( $input );
+        if( 1 === count( $settings ) && array_key_exists( 'settings', $settings ) ) {
+            $options = array_replace_recursive( glsr( OptionManager::class )->all(), $input );
+            $options = $this->sanitizeGeneral( $input, $options );
+            $options = $this->sanitizeSubmissions( $input, $options );
+            $options = $this->sanitizeTranslations( $input, $options );
+            $options = apply_filters( 'site-reviews/settings/callback', $options, $settings );
+            if( filter_input( INPUT_POST, 'option_page' ) == Application::ID.'-settings' ) {
+                glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) );
             }
             return $options;
         }
@@ -37,63 +37,63 @@ 
      */
     public function registerSettings()
     {
-        register_setting(Application::ID.'-settings', OptionManager::databaseKey(), [
+        register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [
             'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-        ]);
+        ] );
     }
 
     /**
      * @return array
      */
-    protected function sanitizeGeneral(array $input, array $options)
+    protected function sanitizeGeneral( array $input, array $options )
     {
         $key = 'settings.general';
-        $inputForm = Arr::get($input, $key);
-        if (!$this->hasMultilingualIntegration(Arr::get($inputForm, 'multilingual'))) {
-            $options = Arr::set($options, $key.'.multilingual', '');
+        $inputForm = Arr::get( $input, $key );
+        if( !$this->hasMultilingualIntegration( Arr::get( $inputForm, 'multilingual' ) ) ) {
+            $options = Arr::set( $options, $key.'.multilingual', '' );
         }
-        if ('' == trim(Arr::get($inputForm, 'notification_message'))) {
-            $defaultValue = Arr::get(glsr()->defaults, $key.'.notification_message');
-            $options = Arr::set($options, $key.'.notification_message', $defaultValue);
+        if( '' == trim( Arr::get( $inputForm, 'notification_message' ) ) ) {
+            $defaultValue = Arr::get( glsr()->defaults, $key.'.notification_message' );
+            $options = Arr::set( $options, $key.'.notification_message', $defaultValue );
         }
-        $defaultValue = Arr::get($inputForm, 'notifications', []);
-        $options = Arr::set($options, $key.'.notifications', $defaultValue);
+        $defaultValue = Arr::get( $inputForm, 'notifications', [] );
+        $options = Arr::set( $options, $key.'.notifications', $defaultValue );
         return $options;
     }
 
     /**
      * @return array
      */
-    protected function sanitizeSubmissions(array $input, array $options)
+    protected function sanitizeSubmissions( array $input, array $options )
     {
         $key = 'settings.submissions';
-        $inputForm = Arr::get($input, $key);
+        $inputForm = Arr::get( $input, $key );
         $defaultValue = isset($inputForm['required'])
             ? $inputForm['required']
             : [];
-        $options = Arr::set($options, $key.'.required', $defaultValue);
+        $options = Arr::set( $options, $key.'.required', $defaultValue );
         return $options;
     }
 
     /**
      * @return array
      */
-    protected function sanitizeTranslations(array $input, array $options)
+    protected function sanitizeTranslations( array $input, array $options )
     {
         $key = 'settings.strings';
-        $inputForm = Arr::consolidateArray(Arr::get($input, $key));
-        if (!empty($inputForm)) {
-            $options = Arr::set($options, $key, array_values(array_filter($inputForm)));
+        $inputForm = Arr::consolidateArray( Arr::get( $input, $key ) );
+        if( !empty($inputForm) ) {
+            $options = Arr::set( $options, $key, array_values( array_filter( $inputForm ) ) );
             $allowedTags = [
                 'a' => ['class' => [], 'href' => [], 'target' => []],
                 'span' => ['class' => []],
             ];
-            array_walk($options['settings']['strings'], function (&$string) use ($allowedTags) {
-                if (isset($string['s2'])) {
-                    $string['s2'] = wp_kses($string['s2'], $allowedTags);
+            array_walk( $options['settings']['strings'], function( &$string ) use ($allowedTags) {
+                if( isset($string['s2']) ) {
+                    $string['s2'] = wp_kses( $string['s2'], $allowedTags );
                 }
-                if (isset($string['p2'])) {
-                    $string['p2'] = wp_kses($string['p2'], $allowedTags);
+                if( isset($string['p2']) ) {
+                    $string['p2'] = wp_kses( $string['p2'], $allowedTags );
                 }
             });
         }
@@ -104,24 +104,24 @@ 
      * @param string $integrationSlug
      * @return bool
      */
-    protected function hasMultilingualIntegration($integrationSlug)
+    protected function hasMultilingualIntegration( $integrationSlug )
     {
-        $integration = glsr(Multilingual::class)->getIntegration($integrationSlug);
-        if (!$integration) {
+        $integration = glsr( Multilingual::class )->getIntegration( $integrationSlug );
+        if( !$integration ) {
             return false;
         }
-        if (!$integration->isActive()) {
-            glsr(Notice::class)->addError(sprintf(
-                __('Please install/activate the %s plugin to enable integration.', 'site-reviews'),
+        if( !$integration->isActive() ) {
+            glsr( Notice::class )->addError( sprintf(
+                __( 'Please install/activate the %s plugin to enable integration.', 'site-reviews' ),
                 $integration->pluginName
-            ));
+            ) );
             return false;
-        } elseif (!$integration->isSupported()) {
-            glsr(Notice::class)->addError(sprintf(
-                __('Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews'),
+        } elseif( !$integration->isSupported() ) {
+            glsr( Notice::class )->addError( sprintf(
+                __( 'Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews' ),
                 $integration->pluginName,
                 $integration->supportedVersion
-            ));
+            ) );
             return false;
         }
         return true;

Braces +2 added lines, -1 removed lines

@@ -116,7 +116,8 @@
                 $integration->pluginName
             ));
             return false;
-        } elseif (!$integration->isSupported()) {
+        }
+        elseif (!$integration->isSupported()) {
             glsr(Notice::class)->addError(sprintf(
                 __('Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews'),
                 $integration->pluginName,