pryley / site-reviews

plugin/Shortcodes/SiteReviewsPopup.php

Indentation +80 added lines, -80 removed lines

@@ -4,84 +4,84 @@
 
 class SiteReviewsPopup extends TinymcePopupGenerator
 {
-    /**
-     * @return array
-     */
-    public function fields()
-    {
-        return [[
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
-            'minWidth' => 320,
-            'type' => 'container',
-        ], [
-            'label' => esc_html__('Title', 'site-reviews'),
-            'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Display', 'site-reviews'),
-            'maxLength' => 5,
-            'name' => 'display',
-            'size' => 3,
-            'text' => '10',
-            'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Rating', 'site-reviews'),
-            'name' => 'rating',
-            'options' => [
-                '5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
-                '4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
-                '3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
-                '2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
-                '1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
-                '0' => esc_html(__('Unrated', 'site-reviews')),
-            ],
-            'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
-            'type' => 'listbox',
-        ], [
-            'label' => esc_html__('Pagination', 'site-reviews'),
-            'name' => 'pagination',
-            'options' => [
-                'true' => esc_html__('Enable', 'site-reviews'),
-                'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
-                'false' => esc_html__('Disable', 'site-reviews'),
-            ],
-            'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
-            'type' => 'listbox',
-        ],
-        $this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
-        [
-            'label' => esc_html__('Assigned To', 'site-reviews'),
-            'name' => 'assigned_to',
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Schema', 'site-reviews'),
-            'name' => 'schema',
-            'options' => [
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
-            ],
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
-            'type' => 'listbox',
-        ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
-            'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'columns' => 2,
-            'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
-            'layout' => 'grid',
-            'spacing' => 5,
-            'type' => 'container',
-        ], [
-            'hidden' => true,
-            'name' => 'id',
-            'type' => 'textbox',
-        ], ];
-    }
+	/**
+	 * @return array
+	 */
+	public function fields()
+	{
+		return [[
+			'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
+			'minWidth' => 320,
+			'type' => 'container',
+		], [
+			'label' => esc_html__('Title', 'site-reviews'),
+			'name' => 'title',
+			'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Display', 'site-reviews'),
+			'maxLength' => 5,
+			'name' => 'display',
+			'size' => 3,
+			'text' => '10',
+			'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Rating', 'site-reviews'),
+			'name' => 'rating',
+			'options' => [
+				'5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
+				'4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
+				'3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
+				'2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
+				'1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
+				'0' => esc_html(__('Unrated', 'site-reviews')),
+			],
+			'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
+			'type' => 'listbox',
+		], [
+			'label' => esc_html__('Pagination', 'site-reviews'),
+			'name' => 'pagination',
+			'options' => [
+				'true' => esc_html__('Enable', 'site-reviews'),
+				'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
+				'false' => esc_html__('Disable', 'site-reviews'),
+			],
+			'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
+			'type' => 'listbox',
+		],
+		$this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
+		$this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
+		[
+			'label' => esc_html__('Assigned To', 'site-reviews'),
+			'name' => 'assigned_to',
+			'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Schema', 'site-reviews'),
+			'name' => 'schema',
+			'options' => [
+				'true' => esc_html__('Enable rich snippets', 'site-reviews'),
+				'false' => esc_html__('Disable rich snippets', 'site-reviews'),
+			],
+			'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
+			'type' => 'listbox',
+		], [
+			'label' => esc_html__('Classes', 'site-reviews'),
+			'name' => 'class',
+			'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'columns' => 2,
+			'items' => $this->getHideOptions(),
+			'label' => esc_html__('Hide', 'site-reviews'),
+			'layout' => 'grid',
+			'spacing' => 5,
+			'type' => 'container',
+		], [
+			'hidden' => true,
+			'name' => 'id',
+			'type' => 'textbox',
+		], ];
+	}
 }

Spacing +29 added lines, -29 removed lines

@@ -10,71 +10,71 @@
     public function fields()
     {
         return [[
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
+            'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
             'minWidth' => 320,
             'type' => 'container',
         ], [
-            'label' => esc_html__('Title', 'site-reviews'),
+            'label' => esc_html__( 'Title', 'site-reviews' ),
             'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+            'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Display', 'site-reviews'),
+            'label' => esc_html__( 'Display', 'site-reviews' ),
             'maxLength' => 5,
             'name' => 'display',
             'size' => 3,
             'text' => '10',
-            'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
+            'tooltip' => __( 'How many reviews would you like to display (default: 10)?', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Rating', 'site-reviews'),
+            'label' => esc_html__( 'Rating', 'site-reviews' ),
             'name' => 'rating',
             'options' => [
-                '5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
-                '4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
-                '3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
-                '2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
-                '1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
-                '0' => esc_html(__('Unrated', 'site-reviews')),
+                '5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 ) ),
+                '4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 ) ),
+                '3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 ) ),
+                '2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ) ),
+                '1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ) ),
+                '0' => esc_html( __( 'Unrated', 'site-reviews' ) ),
             ],
-            'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
+            'tooltip' => __( 'What is the minimum rating to display (default: 1 star)?', 'site-reviews' ),
             'type' => 'listbox',
         ], [
-            'label' => esc_html__('Pagination', 'site-reviews'),
+            'label' => esc_html__( 'Pagination', 'site-reviews' ),
             'name' => 'pagination',
             'options' => [
-                'true' => esc_html__('Enable', 'site-reviews'),
-                'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
-                'false' => esc_html__('Disable', 'site-reviews'),
+                'true' => esc_html__( 'Enable', 'site-reviews' ),
+                'ajax' => esc_html__( 'Enable (using ajax)', 'site-reviews' ),
+                'false' => esc_html__( 'Disable', 'site-reviews' ),
             ],
-            'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
+            'tooltip' => __( 'When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews' ),
             'type' => 'listbox',
         ],
-        $this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
+        $this->getTypes( __( 'Which type of review would you like to display?', 'site-reviews' ) ),
+        $this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' ) ),
         [
-            'label' => esc_html__('Assigned To', 'site-reviews'),
+            'label' => esc_html__( 'Assigned To', 'site-reviews' ),
             'name' => 'assigned_to',
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
+            'tooltip' => __( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Schema', 'site-reviews'),
+            'label' => esc_html__( 'Schema', 'site-reviews' ),
             'name' => 'schema',
             'options' => [
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
+                'true' => esc_html__( 'Enable rich snippets', 'site-reviews' ),
+                'false' => esc_html__( 'Disable rich snippets', 'site-reviews' ),
             ],
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
+            'tooltip' => __( 'Rich snippets are disabled by default.', 'site-reviews' ),
             'type' => 'listbox',
         ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
+            'label' => esc_html__( 'Classes', 'site-reviews' ),
             'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+            'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
             'columns' => 2,
             'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
+            'label' => esc_html__( 'Hide', 'site-reviews' ),
             'layout' => 'grid',
             'spacing' => 5,
             'type' => 'container',

plugin/Controllers/WelcomeController.php

Indentation +88 added lines, -88 removed lines

@@ -8,97 +8,97 @@
 
 class WelcomeController extends Controller
 {
-    /**
-     * @return array
-     * @filter plugin_action_links_site-reviews/site-reviews.php
-     */
-    public function filterActionLinks(array $links)
-    {
-        $links['welcome'] = glsr(Builder::class)->a(__('About', 'site-reviews'), [
-            'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'),
-        ]);
-        return $links;
-    }
+	/**
+	 * @return array
+	 * @filter plugin_action_links_site-reviews/site-reviews.php
+	 */
+	public function filterActionLinks(array $links)
+	{
+		$links['welcome'] = glsr(Builder::class)->a(__('About', 'site-reviews'), [
+			'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'),
+		]);
+		return $links;
+	}
 
-    /**
-     * @return string
-     * @filter admin_title
-     */
-    public function filterAdminTitle($title)
-    {
-        return Application::POST_TYPE.'_page_welcome' == glsr_current_screen()->id
-            ? sprintf(__('Welcome to %s — WordPress', 'site-reviews'), glsr()->name)
-            : $title;
-    }
+	/**
+	 * @return string
+	 * @filter admin_title
+	 */
+	public function filterAdminTitle($title)
+	{
+		return Application::POST_TYPE.'_page_welcome' == glsr_current_screen()->id
+			? sprintf(__('Welcome to %s — WordPress', 'site-reviews'), glsr()->name)
+			: $title;
+	}
 
-    /**
-     * @param string $text
-     * @return string
-     * @filter admin_footer_text
-     */
-    public function filterFooterText($text)
-    {
-        if (Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id) {
-            return $text;
-        }
-        $url = 'https://wordpress.org/support/view/plugin-reviews/site-reviews?filter=5#new-post';
-        return wp_kses_post(sprintf(
-            __('Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews'),
-            '<strong>'.glsr()->name.'</strong> <a href="'.$url.'" target="_blank">&#9733;&#9733;&#9733;&#9733;&#9733;</a>',
-            '<a href="'.$url.'" target="_blank">wordpress.org</a>'
-        ));
-    }
+	/**
+	 * @param string $text
+	 * @return string
+	 * @filter admin_footer_text
+	 */
+	public function filterFooterText($text)
+	{
+		if (Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id) {
+			return $text;
+		}
+		$url = 'https://wordpress.org/support/view/plugin-reviews/site-reviews?filter=5#new-post';
+		return wp_kses_post(sprintf(
+			__('Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews'),
+			'<strong>'.glsr()->name.'</strong> <a href="'.$url.'" target="_blank">&#9733;&#9733;&#9733;&#9733;&#9733;</a>',
+			'<a href="'.$url.'" target="_blank">wordpress.org</a>'
+		));
+	}
 
-    /**
-     * @param string $plugin
-     * @param bool $isNetworkActivation
-     * @return void
-     * @action activated_plugin
-     */
-    public function redirectOnActivation($plugin, $isNetworkActivation)
-    {
-        if (!$isNetworkActivation
-            && 'cli' !== php_sapi_name()
-            && $plugin === plugin_basename(glsr()->file)) {
-            wp_safe_redirect(admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'));
-            exit;
-        }
-    }
+	/**
+	 * @param string $plugin
+	 * @param bool $isNetworkActivation
+	 * @return void
+	 * @action activated_plugin
+	 */
+	public function redirectOnActivation($plugin, $isNetworkActivation)
+	{
+		if (!$isNetworkActivation
+			&& 'cli' !== php_sapi_name()
+			&& $plugin === plugin_basename(glsr()->file)) {
+			wp_safe_redirect(admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'));
+			exit;
+		}
+	}
 
-    /**
-     * @return void
-     * @action admin_menu
-     */
-    public function registerPage()
-    {
-        add_submenu_page('edit.php?post_type='.Application::POST_TYPE,
-            sprintf(__('Welcome to %s', 'site-reviews'), glsr()->name),
-            glsr()->name,
-            glsr()->getPermission('welcome'),
-            'welcome',
-            [$this, 'renderPage']
-        );
-        remove_submenu_page('edit.php?post_type='.Application::POST_TYPE, 'welcome');
-    }
+	/**
+	 * @return void
+	 * @action admin_menu
+	 */
+	public function registerPage()
+	{
+		add_submenu_page('edit.php?post_type='.Application::POST_TYPE,
+			sprintf(__('Welcome to %s', 'site-reviews'), glsr()->name),
+			glsr()->name,
+			glsr()->getPermission('welcome'),
+			'welcome',
+			[$this, 'renderPage']
+		);
+		remove_submenu_page('edit.php?post_type='.Application::POST_TYPE, 'welcome');
+	}
 
-    /**
-     * @return void
-     * @see $this->registerPage()
-     * @callback add_submenu_page
-     */
-    public function renderPage()
-    {
-        $tabs = apply_filters('site-reviews/addon/welcome/tabs', [
-            'getting-started' => __('Getting Started', 'site-reviews'),
-            'whatsnew' => __('What\'s New', 'site-reviews'),
-            'upgrade-guide' => __('Upgrade Guide', 'site-reviews'),
-            'support' => __('Support', 'site-reviews'),
-        ]);
-        glsr()->render('pages/welcome/index', [
-            'data' => ['context' => []],
-            'http_referer' => (string) wp_get_referer(),
-            'tabs' => $tabs,
-            'template' => glsr(Template::class),
-        ]);
-    }
+	/**
+	 * @return void
+	 * @see $this->registerPage()
+	 * @callback add_submenu_page
+	 */
+	public function renderPage()
+	{
+		$tabs = apply_filters('site-reviews/addon/welcome/tabs', [
+			'getting-started' => __('Getting Started', 'site-reviews'),
+			'whatsnew' => __('What\'s New', 'site-reviews'),
+			'upgrade-guide' => __('Upgrade Guide', 'site-reviews'),
+			'support' => __('Support', 'site-reviews'),
+		]);
+		glsr()->render('pages/welcome/index', [
+			'data' => ['context' => []],
+			'http_referer' => (string) wp_get_referer(),
+			'tabs' => $tabs,
+			'template' => glsr(Template::class),
+		]);
+	}
 }

Spacing +29 added lines, -29 removed lines

@@ -12,11 +12,11 @@ 
      * @return array
      * @filter plugin_action_links_site-reviews/site-reviews.php
      */
-    public function filterActionLinks(array $links)
+    public function filterActionLinks( array $links )
     {
-        $links['welcome'] = glsr(Builder::class)->a(__('About', 'site-reviews'), [
-            'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'),
-        ]);
+        $links['welcome'] = glsr( Builder::class )->a( __( 'About', 'site-reviews' ), [
+            'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=welcome' ),
+        ] );
         return $links;
     }
 
@@ -24,10 +24,10 @@ 
      * @return string
      * @filter admin_title
      */
-    public function filterAdminTitle($title)
+    public function filterAdminTitle( $title )
     {
         return Application::POST_TYPE.'_page_welcome' == glsr_current_screen()->id
-            ? sprintf(__('Welcome to %s — WordPress', 'site-reviews'), glsr()->name)
+            ? sprintf( __( 'Welcome to %s — WordPress', 'site-reviews' ), glsr()->name )
             : $title;
     }
 
@@ -36,17 +36,17 @@ 
      * @return string
      * @filter admin_footer_text
      */
-    public function filterFooterText($text)
+    public function filterFooterText( $text )
     {
-        if (Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id) {
+        if( Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id ) {
             return $text;
         }
         $url = 'https://wordpress.org/support/view/plugin-reviews/site-reviews?filter=5#new-post';
-        return wp_kses_post(sprintf(
-            __('Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews'),
+        return wp_kses_post( sprintf(
+            __( 'Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews' ),
             '<strong>'.glsr()->name.'</strong> <a href="'.$url.'" target="_blank">&#9733;&#9733;&#9733;&#9733;&#9733;</a>',
             '<a href="'.$url.'" target="_blank">wordpress.org</a>'
-        ));
+        ) );
     }
 
     /**
@@ -55,12 +55,12 @@ 
      * @return void
      * @action activated_plugin
      */
-    public function redirectOnActivation($plugin, $isNetworkActivation)
+    public function redirectOnActivation( $plugin, $isNetworkActivation )
     {
-        if (!$isNetworkActivation
+        if( !$isNetworkActivation
             && 'cli' !== php_sapi_name()
-            && $plugin === plugin_basename(glsr()->file)) {
-            wp_safe_redirect(admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'));
+            && $plugin === plugin_basename( glsr()->file ) ) {
+            wp_safe_redirect( admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=welcome' ) );
             exit;
         }
     }
@@ -71,14 +71,14 @@ 
      */
     public function registerPage()
     {
-        add_submenu_page('edit.php?post_type='.Application::POST_TYPE,
-            sprintf(__('Welcome to %s', 'site-reviews'), glsr()->name),
+        add_submenu_page( 'edit.php?post_type='.Application::POST_TYPE,
+            sprintf( __( 'Welcome to %s', 'site-reviews' ), glsr()->name ),
             glsr()->name,
-            glsr()->getPermission('welcome'),
+            glsr()->getPermission( 'welcome' ),
             'welcome',
             [$this, 'renderPage']
         );
-        remove_submenu_page('edit.php?post_type='.Application::POST_TYPE, 'welcome');
+        remove_submenu_page( 'edit.php?post_type='.Application::POST_TYPE, 'welcome' );
     }
 
     /**
@@ -88,17 +88,17 @@ 
      */
     public function renderPage()
     {
-        $tabs = apply_filters('site-reviews/addon/welcome/tabs', [
-            'getting-started' => __('Getting Started', 'site-reviews'),
-            'whatsnew' => __('What\'s New', 'site-reviews'),
-            'upgrade-guide' => __('Upgrade Guide', 'site-reviews'),
-            'support' => __('Support', 'site-reviews'),
-        ]);
-        glsr()->render('pages/welcome/index', [
+        $tabs = apply_filters( 'site-reviews/addon/welcome/tabs', [
+            'getting-started' => __( 'Getting Started', 'site-reviews' ),
+            'whatsnew' => __( 'What\'s New', 'site-reviews' ),
+            'upgrade-guide' => __( 'Upgrade Guide', 'site-reviews' ),
+            'support' => __( 'Support', 'site-reviews' ),
+        ] );
+        glsr()->render( 'pages/welcome/index', [
             'data' => ['context' => []],
-            'http_referer' => (string) wp_get_referer(),
+            'http_referer' => (string)wp_get_referer(),
             'tabs' => $tabs,
-            'template' => glsr(Template::class),
-        ]);
+            'template' => glsr( Template::class ),
+        ] );
     }
 }

plugin/Defaults/SiteReviewsDefaults.php

Indentation +35 added lines, -35 removed lines

@@ -6,41 +6,41 @@
 
 class SiteReviewsDefaults extends Defaults
 {
-    /**
-     * @var array
-     */
-    protected $guarded = [
-        'fallback',
-        'title',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $guarded = [
+		'fallback',
+		'title',
+	];
 
-    /**
-     * @var array
-     */
-    protected $mapped = [
-        'count' => 'display', // @deprecated since v4.1.0
-        'per_page' => 'display',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $mapped = [
+		'count' => 'display', // @deprecated since v4.1.0
+		'per_page' => 'display',
+	];
 
-    /**
-     * @return array
-     */
-    protected function defaults()
-    {
-        return [
-            'assigned_to' => '',
-            'category' => '',
-            'class' => '',
-            'display' => 5,
-            'fallback' => '',
-            'hide' => [],
-            'id' => '',
-            'offset' => '',
-            'pagination' => false,
-            'rating' => 0,
-            'schema' => false,
-            'title' => '',
-            'type' => 'local',
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function defaults()
+	{
+		return [
+			'assigned_to' => '',
+			'category' => '',
+			'class' => '',
+			'display' => 5,
+			'fallback' => '',
+			'hide' => [],
+			'id' => '',
+			'offset' => '',
+			'pagination' => false,
+			'rating' => 0,
+			'schema' => false,
+			'title' => '',
+			'type' => 'local',
+		];
+	}
 }

plugin/Defaults/ReviewsDefaults.php

Indentation +26 added lines, -26 removed lines

@@ -6,31 +6,31 @@
 
 class ReviewsDefaults extends Defaults
 {
-    /**
-     * @var array
-     */
-    protected $mapped = [
-        'count' => 'per_page', // @deprecated since v4.1.0
-        'display' => 'per_page',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $mapped = [
+		'count' => 'per_page', // @deprecated since v4.1.0
+		'display' => 'per_page',
+	];
 
-    /**
-     * @return array
-     */
-    protected function defaults()
-    {
-        return [
-            'assigned_to' => '',
-            'category' => '',
-            'offset' => '',
-            'order' => 'DESC',
-            'orderby' => 'date',
-            'pagination' => false,
-            'per_page' => 10,
-            'post__in' => [],
-            'post__not_in' => [],
-            'rating' => '',
-            'type' => '',
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function defaults()
+	{
+		return [
+			'assigned_to' => '',
+			'category' => '',
+			'offset' => '',
+			'order' => 'DESC',
+			'orderby' => 'date',
+			'pagination' => false,
+			'per_page' => 10,
+			'post__in' => [],
+			'post__not_in' => [],
+			'rating' => '',
+			'type' => '',
+		];
+	}
 }

plugin/Defaults/DefaultsAbstract.php

Indentation +133 added lines, -133 removed lines

@@ -8,137 +8,137 @@
 
 abstract class DefaultsAbstract
 {
-    /**
-     * @var array
-     */
-    protected $callable = [
-        'defaults', 'filter', 'merge', 'restrict', 'unguarded',
-    ];
-
-    /**
-     * @var array
-     */
-    protected $guarded = [];
-
-    /**
-     * @var array
-     */
-    protected $mapped = [];
-
-    /**
-     * @param string $name
-     * @return void|array
-     */
-    public function __call($name, array $args = [])
-    {
-        if (!method_exists($this, $name) || !in_array($name, $this->callable)) {
-            return;
-        }
-        $args[0] = $this->mapKeys(Arr::get($args, 0, []));
-        $defaults = call_user_func_array([$this, $name], $args);
-        $hookName = (new ReflectionClass($this))->getShortName();
-        $hookName = str_replace('Defaults', '', $hookName);
-        $hookName = Str::dashCase($hookName);
-        return apply_filters('site-reviews/defaults/'.$hookName, $defaults, $name);
-    }
-
-    /**
-     * @return array
-     */
-    abstract protected function defaults();
-
-    /**
-     * @return array
-     */
-    protected function filter(array $values = [])
-    {
-        return $this->normalize($this->merge(array_filter($values)), $values);
-    }
-
-    /**
-     * @return string
-     */
-    protected function filteredJson(array $values = [])
-    {
-        $defaults = $this->flattenArray(
-            array_diff_key($this->defaults(), array_flip($this->guarded))
-        );
-        $values = $this->flattenArray(
-            shortcode_atts($defaults, $values)
-        );
-        $filtered = array_filter(array_diff_assoc($values, $defaults), function ($value) {
-            return !$this->isEmpty($value);
-        });
-        return json_encode($filtered, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
-    }
-
-    /**
-     * @return array
-     */
-    protected function flattenArray(array $values)
-    {
-        array_walk($values, function (&$value) {
-            if (!is_array($value)) {
-                return;
-            }
-            $value = implode(',', $value);
-        });
-        return $values;
-    }
-
-    /**
-     * @param mixed $var
-     * @return bool
-     */
-    protected function isEmpty($var)
-    {
-        return !is_numeric($var) && !is_bool($var) && empty($var);
-    }
-
-    /**
-     * @return array
-     */
-    protected function mapKeys(array $args)
-    {
-        foreach ($this->mapped as $old => $new) {
-            if (array_key_exists($old, $args)) {
-                $args[$new] = $args[$old];
-                unset($args[$old]);
-            }
-        }
-        return $args;
-    }
-
-    /**
-     * @return array
-     */
-    protected function merge(array $values = [])
-    {
-        return $this->normalize(wp_parse_args($values, $this->defaults()), $values);
-    }
-
-    /**
-     * @return array
-     */
-    protected function normalize(array $values, array $originalValues)
-    {
-        $values['json'] = $this->filteredJson($originalValues);
-        return $values;
-    }
-
-    /**
-     * @return array
-     */
-    protected function restrict(array $values = [])
-    {
-        return $this->normalize(shortcode_atts($this->defaults(), $values), $values);
-    }
-
-    /**
-     * @return array
-     */
-    protected function unguarded()
-    {
-        return array_diff_key($this->defaults(), array_flip($this->guarded));
-    }
+	/**
+	 * @var array
+	 */
+	protected $callable = [
+		'defaults', 'filter', 'merge', 'restrict', 'unguarded',
+	];
+
+	/**
+	 * @var array
+	 */
+	protected $guarded = [];
+
+	/**
+	 * @var array
+	 */
+	protected $mapped = [];
+
+	/**
+	 * @param string $name
+	 * @return void|array
+	 */
+	public function __call($name, array $args = [])
+	{
+		if (!method_exists($this, $name) || !in_array($name, $this->callable)) {
+			return;
+		}
+		$args[0] = $this->mapKeys(Arr::get($args, 0, []));
+		$defaults = call_user_func_array([$this, $name], $args);
+		$hookName = (new ReflectionClass($this))->getShortName();
+		$hookName = str_replace('Defaults', '', $hookName);
+		$hookName = Str::dashCase($hookName);
+		return apply_filters('site-reviews/defaults/'.$hookName, $defaults, $name);
+	}
+
+	/**
+	 * @return array
+	 */
+	abstract protected function defaults();
+
+	/**
+	 * @return array
+	 */
+	protected function filter(array $values = [])
+	{
+		return $this->normalize($this->merge(array_filter($values)), $values);
+	}
+
+	/**
+	 * @return string
+	 */
+	protected function filteredJson(array $values = [])
+	{
+		$defaults = $this->flattenArray(
+			array_diff_key($this->defaults(), array_flip($this->guarded))
+		);
+		$values = $this->flattenArray(
+			shortcode_atts($defaults, $values)
+		);
+		$filtered = array_filter(array_diff_assoc($values, $defaults), function ($value) {
+			return !$this->isEmpty($value);
+		});
+		return json_encode($filtered, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function flattenArray(array $values)
+	{
+		array_walk($values, function (&$value) {
+			if (!is_array($value)) {
+				return;
+			}
+			$value = implode(',', $value);
+		});
+		return $values;
+	}
+
+	/**
+	 * @param mixed $var
+	 * @return bool
+	 */
+	protected function isEmpty($var)
+	{
+		return !is_numeric($var) && !is_bool($var) && empty($var);
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function mapKeys(array $args)
+	{
+		foreach ($this->mapped as $old => $new) {
+			if (array_key_exists($old, $args)) {
+				$args[$new] = $args[$old];
+				unset($args[$old]);
+			}
+		}
+		return $args;
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function merge(array $values = [])
+	{
+		return $this->normalize(wp_parse_args($values, $this->defaults()), $values);
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function normalize(array $values, array $originalValues)
+	{
+		$values['json'] = $this->filteredJson($originalValues);
+		return $values;
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function restrict(array $values = [])
+	{
+		return $this->normalize(shortcode_atts($this->defaults(), $values), $values);
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function unguarded()
+	{
+		return array_diff_key($this->defaults(), array_flip($this->guarded));
+	}
 }

Spacing +32 added lines, -32 removed lines

@@ -29,17 +29,17 @@ 
      * @param string $name
      * @return void|array
      */
-    public function __call($name, array $args = [])
+    public function __call( $name, array $args = [] )
     {
-        if (!method_exists($this, $name) || !in_array($name, $this->callable)) {
+        if( !method_exists( $this, $name ) || !in_array( $name, $this->callable ) ) {
             return;
         }
-        $args[0] = $this->mapKeys(Arr::get($args, 0, []));
-        $defaults = call_user_func_array([$this, $name], $args);
-        $hookName = (new ReflectionClass($this))->getShortName();
-        $hookName = str_replace('Defaults', '', $hookName);
-        $hookName = Str::dashCase($hookName);
-        return apply_filters('site-reviews/defaults/'.$hookName, $defaults, $name);
+        $args[0] = $this->mapKeys( Arr::get( $args, 0, [] ) );
+        $defaults = call_user_func_array( [$this, $name], $args );
+        $hookName = (new ReflectionClass( $this ))->getShortName();
+        $hookName = str_replace( 'Defaults', '', $hookName );
+        $hookName = Str::dashCase( $hookName );
+        return apply_filters( 'site-reviews/defaults/'.$hookName, $defaults, $name );
     }
 
     /**
@@ -50,38 +50,38 @@ 
     /**
      * @return array
      */
-    protected function filter(array $values = [])
+    protected function filter( array $values = [] )
     {
-        return $this->normalize($this->merge(array_filter($values)), $values);
+        return $this->normalize( $this->merge( array_filter( $values ) ), $values );
     }
 
     /**
      * @return string
      */
-    protected function filteredJson(array $values = [])
+    protected function filteredJson( array $values = [] )
     {
         $defaults = $this->flattenArray(
-            array_diff_key($this->defaults(), array_flip($this->guarded))
+            array_diff_key( $this->defaults(), array_flip( $this->guarded ) )
         );
         $values = $this->flattenArray(
-            shortcode_atts($defaults, $values)
+            shortcode_atts( $defaults, $values )
         );
-        $filtered = array_filter(array_diff_assoc($values, $defaults), function ($value) {
-            return !$this->isEmpty($value);
+        $filtered = array_filter( array_diff_assoc( $values, $defaults ), function( $value ) {
+            return !$this->isEmpty( $value );
         });
-        return json_encode($filtered, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+        return json_encode( $filtered, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE );
     }
 
     /**
      * @return array
      */
-    protected function flattenArray(array $values)
+    protected function flattenArray( array $values )
     {
-        array_walk($values, function (&$value) {
-            if (!is_array($value)) {
+        array_walk( $values, function( &$value ) {
+            if( !is_array( $value ) ) {
                 return;
             }
-            $value = implode(',', $value);
+            $value = implode( ',', $value );
         });
         return $values;
     }
@@ -90,18 +90,18 @@ 
      * @param mixed $var
      * @return bool
      */
-    protected function isEmpty($var)
+    protected function isEmpty( $var )
     {
-        return !is_numeric($var) && !is_bool($var) && empty($var);
+        return !is_numeric( $var ) && !is_bool( $var ) && empty($var);
     }
 
     /**
      * @return array
      */
-    protected function mapKeys(array $args)
+    protected function mapKeys( array $args )
     {
-        foreach ($this->mapped as $old => $new) {
-            if (array_key_exists($old, $args)) {
+        foreach( $this->mapped as $old => $new ) {
+            if( array_key_exists( $old, $args ) ) {
                 $args[$new] = $args[$old];
                 unset($args[$old]);
             }
@@ -112,26 +112,26 @@ 
     /**
      * @return array
      */
-    protected function merge(array $values = [])
+    protected function merge( array $values = [] )
     {
-        return $this->normalize(wp_parse_args($values, $this->defaults()), $values);
+        return $this->normalize( wp_parse_args( $values, $this->defaults() ), $values );
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $values, array $originalValues)
+    protected function normalize( array $values, array $originalValues )
     {
-        $values['json'] = $this->filteredJson($originalValues);
+        $values['json'] = $this->filteredJson( $originalValues );
         return $values;
     }
 
     /**
      * @return array
      */
-    protected function restrict(array $values = [])
+    protected function restrict( array $values = [] )
     {
-        return $this->normalize(shortcode_atts($this->defaults(), $values), $values);
+        return $this->normalize( shortcode_atts( $this->defaults(), $values ), $values );
     }
 
     /**
@@ -139,6 +139,6 @@ 
      */
     protected function unguarded()
     {
-        return array_diff_key($this->defaults(), array_flip($this->guarded));
+        return array_diff_key( $this->defaults(), array_flip( $this->guarded ) );
     }
 }

plugin/Controllers/EditorController/Labels.php

Indentation +98 added lines, -98 removed lines

@@ -7,106 +7,106 @@
 
 class Labels
 {
-    /**
-     * @param string $translation
-     * @param string $test
-     * @return string
-     */
-    public function filterPostStatusLabels($translation, $text)
-    {
-        $replacements = $this->getStatusLabels();
-        return array_key_exists($text, $replacements)
-            ? $replacements[$text]
-            : $translation;
-    }
+	/**
+	 * @param string $translation
+	 * @param string $test
+	 * @return string
+	 */
+	public function filterPostStatusLabels($translation, $text)
+	{
+		$replacements = $this->getStatusLabels();
+		return array_key_exists($text, $replacements)
+			? $replacements[$text]
+			: $translation;
+	}
 
-    /**
-     * @return array
-     */
-    public function filterUpdateMessages(array $messages)
-    {
-        $post = get_post();
-        if (!($post instanceof WP_Post)) {
-            return;
-        }
-        $strings = $this->getReviewLabels();
-        $restored = filter_input(INPUT_GET, 'revision');
-        if ($revisionTitle = wp_post_revision_title(intval($restored), false)) {
-            $restored = sprintf($strings['restored'], $revisionTitle);
-        }
-        $scheduled_date = date_i18n('M j, Y @ H:i', strtotime($post->post_date));
-        $messages[Application::POST_TYPE] = [
-             1 => $strings['updated'],
-             4 => $strings['updated'],
-             5 => $restored,
-             6 => $strings['published'],
-             7 => $strings['saved'],
-             8 => $strings['submitted'],
-             9 => sprintf($strings['scheduled'], '<strong>'.$scheduled_date.'</strong>'),
-            10 => $strings['draft_updated'],
-            50 => $strings['approved'],
-            51 => $strings['unapproved'],
-            52 => $strings['reverted'],
-        ];
-        return $messages;
-    }
+	/**
+	 * @return array
+	 */
+	public function filterUpdateMessages(array $messages)
+	{
+		$post = get_post();
+		if (!($post instanceof WP_Post)) {
+			return;
+		}
+		$strings = $this->getReviewLabels();
+		$restored = filter_input(INPUT_GET, 'revision');
+		if ($revisionTitle = wp_post_revision_title(intval($restored), false)) {
+			$restored = sprintf($strings['restored'], $revisionTitle);
+		}
+		$scheduled_date = date_i18n('M j, Y @ H:i', strtotime($post->post_date));
+		$messages[Application::POST_TYPE] = [
+			 1 => $strings['updated'],
+			 4 => $strings['updated'],
+			 5 => $restored,
+			 6 => $strings['published'],
+			 7 => $strings['saved'],
+			 8 => $strings['submitted'],
+			 9 => sprintf($strings['scheduled'], '<strong>'.$scheduled_date.'</strong>'),
+			10 => $strings['draft_updated'],
+			50 => $strings['approved'],
+			51 => $strings['unapproved'],
+			52 => $strings['reverted'],
+		];
+		return $messages;
+	}
 
-    /**
-     * @return void
-     */
-    public function translatePostStatusLabels()
-    {
-        global $wp_scripts;
-        $strings = [
-            'savePending' => __('Save as Unapproved', 'site-reviews'),
-            'published' => __('Approved', 'site-reviews'),
-        ];
-        if (isset($wp_scripts->registered['post']->extra['data'])) {
-            $l10n = &$wp_scripts->registered['post']->extra['data'];
-            foreach ($strings as $search => $replace) {
-                $l10n = preg_replace('/("'.$search.'":")([^"]+)/', '$1'.$replace, $l10n);
-            }
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function translatePostStatusLabels()
+	{
+		global $wp_scripts;
+		$strings = [
+			'savePending' => __('Save as Unapproved', 'site-reviews'),
+			'published' => __('Approved', 'site-reviews'),
+		];
+		if (isset($wp_scripts->registered['post']->extra['data'])) {
+			$l10n = &$wp_scripts->registered['post']->extra['data'];
+			foreach ($strings as $search => $replace) {
+				$l10n = preg_replace('/("'.$search.'":")([^"]+)/', '$1'.$replace, $l10n);
+			}
+		}
+	}
 
-    /**
-     * @return array
-     */
-    protected function getReviewLabels()
-    {
-        return [
-            'approved' => __('Review has been approved and published.', 'site-reviews'),
-            'draft_updated' => __('Review draft updated.', 'site-reviews'),
-            'preview' => __('Preview review', 'site-reviews'),
-            'published' => __('Review approved and published.', 'site-reviews'),
-            'restored' => __('Review restored to revision from %s.', 'site-reviews'),
-            'reverted' => __('Review has been reverted to its original submission state (title, content, and submission date).', 'site-reviews'),
-            'saved' => __('Review saved.', 'site-reviews'),
-            'scheduled' => __('Review scheduled for: %s.', 'site-reviews'),
-            'submitted' => __('Review submitted.', 'site-reviews'),
-            'unapproved' => __('Review has been unapproved and is now pending.', 'site-reviews'),
-            'updated' => __('Review updated.', 'site-reviews'),
-            'view' => __('View review', 'site-reviews'),
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function getReviewLabels()
+	{
+		return [
+			'approved' => __('Review has been approved and published.', 'site-reviews'),
+			'draft_updated' => __('Review draft updated.', 'site-reviews'),
+			'preview' => __('Preview review', 'site-reviews'),
+			'published' => __('Review approved and published.', 'site-reviews'),
+			'restored' => __('Review restored to revision from %s.', 'site-reviews'),
+			'reverted' => __('Review has been reverted to its original submission state (title, content, and submission date).', 'site-reviews'),
+			'saved' => __('Review saved.', 'site-reviews'),
+			'scheduled' => __('Review scheduled for: %s.', 'site-reviews'),
+			'submitted' => __('Review submitted.', 'site-reviews'),
+			'unapproved' => __('Review has been unapproved and is now pending.', 'site-reviews'),
+			'updated' => __('Review updated.', 'site-reviews'),
+			'view' => __('View review', 'site-reviews'),
+		];
+	}
 
-    /**
-     * Store the labels to avoid unnecessary loops.
-     * @return array
-     */
-    protected function getStatusLabels()
-    {
-        static $labels;
-        if (empty($labels)) {
-            $labels = [
-                'Pending' => __('Unapproved', 'site-reviews'),
-                'Pending Review' => __('Unapproved', 'site-reviews'),
-                'Privately Published' => __('Privately Approved', 'site-reviews'),
-                'Publish' => __('Approve', 'site-reviews'),
-                'Published' => __('Approved', 'site-reviews'),
-                'Save as Pending' => __('Save as Unapproved', 'site-reviews'),
-            ];
-        }
-        return $labels;
-    }
+	/**
+	 * Store the labels to avoid unnecessary loops.
+	 * @return array
+	 */
+	protected function getStatusLabels()
+	{
+		static $labels;
+		if (empty($labels)) {
+			$labels = [
+				'Pending' => __('Unapproved', 'site-reviews'),
+				'Pending Review' => __('Unapproved', 'site-reviews'),
+				'Privately Published' => __('Privately Approved', 'site-reviews'),
+				'Publish' => __('Approve', 'site-reviews'),
+				'Published' => __('Approved', 'site-reviews'),
+				'Save as Pending' => __('Save as Unapproved', 'site-reviews'),
+			];
+		}
+		return $labels;
+	}
 }

Spacing +33 added lines, -33 removed lines

@@ -12,10 +12,10 @@ 
      * @param string $test
      * @return string
      */
-    public function filterPostStatusLabels($translation, $text)
+    public function filterPostStatusLabels( $translation, $text )
     {
         $replacements = $this->getStatusLabels();
-        return array_key_exists($text, $replacements)
+        return array_key_exists( $text, $replacements )
             ? $replacements[$text]
             : $translation;
     }
@@ -23,18 +23,18 @@ 
     /**
      * @return array
      */
-    public function filterUpdateMessages(array $messages)
+    public function filterUpdateMessages( array $messages )
     {
         $post = get_post();
-        if (!($post instanceof WP_Post)) {
+        if( !($post instanceof WP_Post) ) {
             return;
         }
         $strings = $this->getReviewLabels();
-        $restored = filter_input(INPUT_GET, 'revision');
-        if ($revisionTitle = wp_post_revision_title(intval($restored), false)) {
-            $restored = sprintf($strings['restored'], $revisionTitle);
+        $restored = filter_input( INPUT_GET, 'revision' );
+        if( $revisionTitle = wp_post_revision_title( intval( $restored ), false ) ) {
+            $restored = sprintf( $strings['restored'], $revisionTitle );
         }
-        $scheduled_date = date_i18n('M j, Y @ H:i', strtotime($post->post_date));
+        $scheduled_date = date_i18n( 'M j, Y @ H:i', strtotime( $post->post_date ) );
         $messages[Application::POST_TYPE] = [
              1 => $strings['updated'],
              4 => $strings['updated'],
@@ -42,7 +42,7 @@ 
              6 => $strings['published'],
              7 => $strings['saved'],
              8 => $strings['submitted'],
-             9 => sprintf($strings['scheduled'], '<strong>'.$scheduled_date.'</strong>'),
+             9 => sprintf( $strings['scheduled'], '<strong>'.$scheduled_date.'</strong>' ),
             10 => $strings['draft_updated'],
             50 => $strings['approved'],
             51 => $strings['unapproved'],
@@ -58,13 +58,13 @@ 
     {
         global $wp_scripts;
         $strings = [
-            'savePending' => __('Save as Unapproved', 'site-reviews'),
-            'published' => __('Approved', 'site-reviews'),
+            'savePending' => __( 'Save as Unapproved', 'site-reviews' ),
+            'published' => __( 'Approved', 'site-reviews' ),
         ];
-        if (isset($wp_scripts->registered['post']->extra['data'])) {
+        if( isset($wp_scripts->registered['post']->extra['data']) ) {
             $l10n = &$wp_scripts->registered['post']->extra['data'];
-            foreach ($strings as $search => $replace) {
-                $l10n = preg_replace('/("'.$search.'":")([^"]+)/', '$1'.$replace, $l10n);
+            foreach( $strings as $search => $replace ) {
+                $l10n = preg_replace( '/("'.$search.'":")([^"]+)/', '$1'.$replace, $l10n );
             }
         }
     }
@@ -75,18 +75,18 @@ 
     protected function getReviewLabels()
     {
         return [
-            'approved' => __('Review has been approved and published.', 'site-reviews'),
-            'draft_updated' => __('Review draft updated.', 'site-reviews'),
-            'preview' => __('Preview review', 'site-reviews'),
-            'published' => __('Review approved and published.', 'site-reviews'),
-            'restored' => __('Review restored to revision from %s.', 'site-reviews'),
-            'reverted' => __('Review has been reverted to its original submission state (title, content, and submission date).', 'site-reviews'),
-            'saved' => __('Review saved.', 'site-reviews'),
-            'scheduled' => __('Review scheduled for: %s.', 'site-reviews'),
-            'submitted' => __('Review submitted.', 'site-reviews'),
-            'unapproved' => __('Review has been unapproved and is now pending.', 'site-reviews'),
-            'updated' => __('Review updated.', 'site-reviews'),
-            'view' => __('View review', 'site-reviews'),
+            'approved' => __( 'Review has been approved and published.', 'site-reviews' ),
+            'draft_updated' => __( 'Review draft updated.', 'site-reviews' ),
+            'preview' => __( 'Preview review', 'site-reviews' ),
+            'published' => __( 'Review approved and published.', 'site-reviews' ),
+            'restored' => __( 'Review restored to revision from %s.', 'site-reviews' ),
+            'reverted' => __( 'Review has been reverted to its original submission state (title, content, and submission date).', 'site-reviews' ),
+            'saved' => __( 'Review saved.', 'site-reviews' ),
+            'scheduled' => __( 'Review scheduled for: %s.', 'site-reviews' ),
+            'submitted' => __( 'Review submitted.', 'site-reviews' ),
+            'unapproved' => __( 'Review has been unapproved and is now pending.', 'site-reviews' ),
+            'updated' => __( 'Review updated.', 'site-reviews' ),
+            'view' => __( 'View review', 'site-reviews' ),
         ];
     }
 
@@ -97,14 +97,14 @@ 
     protected function getStatusLabels()
     {
         static $labels;
-        if (empty($labels)) {
+        if( empty($labels) ) {
             $labels = [
-                'Pending' => __('Unapproved', 'site-reviews'),
-                'Pending Review' => __('Unapproved', 'site-reviews'),
-                'Privately Published' => __('Privately Approved', 'site-reviews'),
-                'Publish' => __('Approve', 'site-reviews'),
-                'Published' => __('Approved', 'site-reviews'),
-                'Save as Pending' => __('Save as Unapproved', 'site-reviews'),
+                'Pending' => __( 'Unapproved', 'site-reviews' ),
+                'Pending Review' => __( 'Unapproved', 'site-reviews' ),
+                'Privately Published' => __( 'Privately Approved', 'site-reviews' ),
+                'Publish' => __( 'Approve', 'site-reviews' ),
+                'Published' => __( 'Approved', 'site-reviews' ),
+                'Save as Pending' => __( 'Save as Unapproved', 'site-reviews' ),
             ];
         }
         return $labels;

plugin/Modules/Translation.php

Indentation +250 added lines, -250 removed lines

@@ -11,272 +11,272 @@
 
 class Translation
 {
-    const SEARCH_THRESHOLD = 3;
+	const SEARCH_THRESHOLD = 3;
 
-    /**
-     * @var array
-     */
-    protected $entries;
+	/**
+	 * @var array
+	 */
+	protected $entries;
 
-    /**
-     * @var array
-     */
-    protected $results;
+	/**
+	 * @var array
+	 */
+	protected $results;
 
-    /**
-     * Returns all saved custom translations with translation context.
-     * @return array
-     */
-    public function all()
-    {
-        $translations = $this->translations();
-        $entries = $this->filter($translations, $this->entries())->results();
-        array_walk($translations, function (&$entry) use ($entries) {
-            $entry['desc'] = array_key_exists($entry['id'], $entries)
-                ? $this->getEntryString($entries[$entry['id']], 'msgctxt')
-                : '';
-        });
-        return $translations;
-    }
+	/**
+	 * Returns all saved custom translations with translation context.
+	 * @return array
+	 */
+	public function all()
+	{
+		$translations = $this->translations();
+		$entries = $this->filter($translations, $this->entries())->results();
+		array_walk($translations, function (&$entry) use ($entries) {
+			$entry['desc'] = array_key_exists($entry['id'], $entries)
+				? $this->getEntryString($entries[$entry['id']], 'msgctxt')
+				: '';
+		});
+		return $translations;
+	}
 
-    /**
-     * @return array
-     */
-    public function entries()
-    {
-        if (!isset($this->entries)) {
-            $potFile = glsr()->path(glsr()->languages.'/'.Application::ID.'.pot');
-            $entries = $this->extractEntriesFromPotFile($potFile);
-            $entries = apply_filters('site-reviews/translation/entries', $entries);
-            $this->entries = $entries;
-        }
-        return $this->entries;
-    }
+	/**
+	 * @return array
+	 */
+	public function entries()
+	{
+		if (!isset($this->entries)) {
+			$potFile = glsr()->path(glsr()->languages.'/'.Application::ID.'.pot');
+			$entries = $this->extractEntriesFromPotFile($potFile);
+			$entries = apply_filters('site-reviews/translation/entries', $entries);
+			$this->entries = $entries;
+		}
+		return $this->entries;
+	}
 
-    /**
-     * @param array|null $entriesToExclude
-     * @param array|null $entries
-     * @return static
-     */
-    public function exclude($entriesToExclude = null, $entries = null)
-    {
-        return $this->filter($entriesToExclude, $entries, false);
-    }
+	/**
+	 * @param array|null $entriesToExclude
+	 * @param array|null $entries
+	 * @return static
+	 */
+	public function exclude($entriesToExclude = null, $entries = null)
+	{
+		return $this->filter($entriesToExclude, $entries, false);
+	}
 
-    /**
-     * @param string $potFile
-     * @return array
-     */
-    public function extractEntriesFromPotFile($potFile, array $entries = [])
-    {
-        try {
-            $potEntries = $this->normalize(Parser::parseFile($potFile)->getEntries());
-            foreach ($potEntries as $key => $entry) {
-                $entries[html_entity_decode($key, ENT_COMPAT, 'UTF-8')] = $entry;
-            }
-        } catch (Exception $e) {
-            glsr_log()->error($e->getMessage());
-        }
-        return $entries;
-    }
+	/**
+	 * @param string $potFile
+	 * @return array
+	 */
+	public function extractEntriesFromPotFile($potFile, array $entries = [])
+	{
+		try {
+			$potEntries = $this->normalize(Parser::parseFile($potFile)->getEntries());
+			foreach ($potEntries as $key => $entry) {
+				$entries[html_entity_decode($key, ENT_COMPAT, 'UTF-8')] = $entry;
+			}
+		} catch (Exception $e) {
+			glsr_log()->error($e->getMessage());
+		}
+		return $entries;
+	}
 
-    /**
-     * @param array|null $filterWith
-     * @param array|null $entries
-     * @param bool $intersect
-     * @return static
-     */
-    public function filter($filterWith = null, $entries = null, $intersect = true)
-    {
-        if (!is_array($entries)) {
-            $entries = $this->results;
-        }
-        if (!is_array($filterWith)) {
-            $filterWith = $this->translations();
-        }
-        $keys = array_flip(glsr_array_column($filterWith, 'id'));
-        $this->results = $intersect
-            ? array_intersect_key($entries, $keys)
-            : array_diff_key($entries, $keys);
-        return $this;
-    }
+	/**
+	 * @param array|null $filterWith
+	 * @param array|null $entries
+	 * @param bool $intersect
+	 * @return static
+	 */
+	public function filter($filterWith = null, $entries = null, $intersect = true)
+	{
+		if (!is_array($entries)) {
+			$entries = $this->results;
+		}
+		if (!is_array($filterWith)) {
+			$filterWith = $this->translations();
+		}
+		$keys = array_flip(glsr_array_column($filterWith, 'id'));
+		$this->results = $intersect
+			? array_intersect_key($entries, $keys)
+			: array_diff_key($entries, $keys);
+		return $this;
+	}
 
-    /**
-     * @param string $template
-     * @return string
-     */
-    public function render($template, array $entry)
-    {
-        $data = array_combine(
-            array_map(function ($key) { return 'data.'.$key; }, array_keys($entry)),
-            $entry
-        );
-        $data['data.class'] = $data['data.error'] = '';
-        if (false === array_search($entry['s1'], glsr_array_column($this->entries(), 'msgid'))) {
-            $data['data.class'] = 'is-invalid';
-            $data['data.error'] = __('This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews');
-        }
-        return glsr(Template::class)->build('partials/translations/'.$template, [
-            'context' => array_map('esc_html', $data),
-        ]);
-    }
+	/**
+	 * @param string $template
+	 * @return string
+	 */
+	public function render($template, array $entry)
+	{
+		$data = array_combine(
+			array_map(function ($key) { return 'data.'.$key; }, array_keys($entry)),
+			$entry
+		);
+		$data['data.class'] = $data['data.error'] = '';
+		if (false === array_search($entry['s1'], glsr_array_column($this->entries(), 'msgid'))) {
+			$data['data.class'] = 'is-invalid';
+			$data['data.error'] = __('This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews');
+		}
+		return glsr(Template::class)->build('partials/translations/'.$template, [
+			'context' => array_map('esc_html', $data),
+		]);
+	}
 
-    /**
-     * Returns a rendered string of all saved custom translations with translation context.
-     * @return string
-     */
-    public function renderAll()
-    {
-        $rendered = '';
-        foreach ($this->all() as $index => $entry) {
-            $entry['index'] = $index;
-            $entry['prefix'] = OptionManager::databaseKey();
-            $rendered.= $this->render($entry['type'], $entry);
-        }
-        return $rendered;
-    }
+	/**
+	 * Returns a rendered string of all saved custom translations with translation context.
+	 * @return string
+	 */
+	public function renderAll()
+	{
+		$rendered = '';
+		foreach ($this->all() as $index => $entry) {
+			$entry['index'] = $index;
+			$entry['prefix'] = OptionManager::databaseKey();
+			$rendered.= $this->render($entry['type'], $entry);
+		}
+		return $rendered;
+	}
 
-    /**
-     * @param bool $resetAfterRender
-     * @return string
-     */
-    public function renderResults($resetAfterRender = true)
-    {
-        $rendered = '';
-        foreach ($this->results as $id => $entry) {
-            $data = [
-                'desc' => $this->getEntryString($entry, 'msgctxt'),
-                'id' => $id,
-                'p1' => $this->getEntryString($entry, 'msgid_plural'),
-                's1' => $this->getEntryString($entry, 'msgid'),
-            ];
-            $text = !empty($data['p1'])
-                ? sprintf('%s | %s', $data['s1'], $data['p1'])
-                : $data['s1'];
-            $rendered.= $this->render('result', [
-                'entry' => json_encode($data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE),
-                'text' => wp_strip_all_tags($text),
-            ]);
-        }
-        if ($resetAfterRender) {
-            $this->reset();
-        }
-        return $rendered;
-    }
+	/**
+	 * @param bool $resetAfterRender
+	 * @return string
+	 */
+	public function renderResults($resetAfterRender = true)
+	{
+		$rendered = '';
+		foreach ($this->results as $id => $entry) {
+			$data = [
+				'desc' => $this->getEntryString($entry, 'msgctxt'),
+				'id' => $id,
+				'p1' => $this->getEntryString($entry, 'msgid_plural'),
+				's1' => $this->getEntryString($entry, 'msgid'),
+			];
+			$text = !empty($data['p1'])
+				? sprintf('%s | %s', $data['s1'], $data['p1'])
+				: $data['s1'];
+			$rendered.= $this->render('result', [
+				'entry' => json_encode($data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE),
+				'text' => wp_strip_all_tags($text),
+			]);
+		}
+		if ($resetAfterRender) {
+			$this->reset();
+		}
+		return $rendered;
+	}
 
-    /**
-     * @return void
-     */
-    public function reset()
-    {
-        $this->results = [];
-    }
+	/**
+	 * @return void
+	 */
+	public function reset()
+	{
+		$this->results = [];
+	}
 
-    /**
-     * @return array
-     */
-    public function results()
-    {
-        $results = $this->results;
-        $this->reset();
-        return $results;
-    }
+	/**
+	 * @return array
+	 */
+	public function results()
+	{
+		$results = $this->results;
+		$this->reset();
+		return $results;
+	}
 
-    /**
-     * @param string $needle
-     * @return static
-     */
-    public function search($needle = '')
-    {
-        $this->reset();
-        $needle = trim(strtolower($needle));
-        foreach ($this->entries() as $key => $entry) {
-            $single = strtolower($this->getEntryString($entry, 'msgid'));
-            $plural = strtolower($this->getEntryString($entry, 'msgid_plural'));
-            if (strlen($needle) < static::SEARCH_THRESHOLD) {
-                if (in_array($needle, [$single, $plural])) {
-                    $this->results[$key] = $entry;
-                }
-            } elseif (Str::contains(sprintf('%s %s', $single, $plural), $needle)) {
-                $this->results[$key] = $entry;
-            }
-        }
-        return $this;
-    }
+	/**
+	 * @param string $needle
+	 * @return static
+	 */
+	public function search($needle = '')
+	{
+		$this->reset();
+		$needle = trim(strtolower($needle));
+		foreach ($this->entries() as $key => $entry) {
+			$single = strtolower($this->getEntryString($entry, 'msgid'));
+			$plural = strtolower($this->getEntryString($entry, 'msgid_plural'));
+			if (strlen($needle) < static::SEARCH_THRESHOLD) {
+				if (in_array($needle, [$single, $plural])) {
+					$this->results[$key] = $entry;
+				}
+			} elseif (Str::contains(sprintf('%s %s', $single, $plural), $needle)) {
+				$this->results[$key] = $entry;
+			}
+		}
+		return $this;
+	}
 
-    /**
-     * Store the translations to avoid unnecessary loops.
-     * @return array
-     */
-    public function translations()
-    {
-        static $translations;
-        if (empty($translations)) {
-            $settings = glsr(OptionManager::class)->get('settings');
-            $translations = isset($settings['strings'])
-                ? $this->normalizeSettings((array) $settings['strings'])
-                : [];
-        }
-        return $translations;
-    }
+	/**
+	 * Store the translations to avoid unnecessary loops.
+	 * @return array
+	 */
+	public function translations()
+	{
+		static $translations;
+		if (empty($translations)) {
+			$settings = glsr(OptionManager::class)->get('settings');
+			$translations = isset($settings['strings'])
+				? $this->normalizeSettings((array) $settings['strings'])
+				: [];
+		}
+		return $translations;
+	}
 
-    /**
-     * @param string $key
-     * @return string
-     */
-    protected function getEntryString(array $entry, $key)
-    {
-        return isset($entry[$key])
-            ? implode('', (array) $entry[$key])
-            : '';
-    }
+	/**
+	 * @param string $key
+	 * @return string
+	 */
+	protected function getEntryString(array $entry, $key)
+	{
+		return isset($entry[$key])
+			? implode('', (array) $entry[$key])
+			: '';
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalize(array $entries)
-    {
-        $keys = [
-            'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]',
-        ];
-        array_walk($entries, function (&$entry) use ($keys) {
-            foreach ($keys as $key) {
-                try {
-                    $entry = $this->normalizeEntryString($entry, $key);
-                } catch (\TypeError $error) {
-                    glsr_log()->once('error', 'Translation/normalize', $error);
-                    glsr_log()->once('debug', 'Translation/normalize', $entry);
-                }
-            }
-        });
-        return $entries;
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalize(array $entries)
+	{
+		$keys = [
+			'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]',
+		];
+		array_walk($entries, function (&$entry) use ($keys) {
+			foreach ($keys as $key) {
+				try {
+					$entry = $this->normalizeEntryString($entry, $key);
+				} catch (\TypeError $error) {
+					glsr_log()->once('error', 'Translation/normalize', $error);
+					glsr_log()->once('debug', 'Translation/normalize', $entry);
+				}
+			}
+		});
+		return $entries;
+	}
 
-    /**
-     * @param string $key
-     * @return array
-     */
-    protected function normalizeEntryString(array $entry, $key)
-    {
-        if (isset($entry[$key])) {
-            $entry[$key] = $this->getEntryString($entry, $key);
-        }
-        return $entry;
-    }
+	/**
+	 * @param string $key
+	 * @return array
+	 */
+	protected function normalizeEntryString(array $entry, $key)
+	{
+		if (isset($entry[$key])) {
+			$entry[$key] = $this->getEntryString($entry, $key);
+		}
+		return $entry;
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalizeSettings(array $strings)
-    {
-        $defaultString = array_fill_keys(['id', 's1', 's2', 'p1', 'p2'], '');
-        $strings = array_filter($strings, 'is_array');
-        foreach ($strings as &$string) {
-            $string['type'] = isset($string['p1']) ? 'plural' : 'single';
-            $string = wp_parse_args($string, $defaultString);
-        }
-        return array_filter($strings, function ($string) {
-            return !empty($string['id']);
-        });
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalizeSettings(array $strings)
+	{
+		$defaultString = array_fill_keys(['id', 's1', 's2', 'p1', 'p2'], '');
+		$strings = array_filter($strings, 'is_array');
+		foreach ($strings as &$string) {
+			$string['type'] = isset($string['p1']) ? 'plural' : 'single';
+			$string = wp_parse_args($string, $defaultString);
+		}
+		return array_filter($strings, function ($string) {
+			return !empty($string['id']);
+		});
+	}
 }

Spacing +71 added lines, -71 removed lines

@@ -30,10 +30,10 @@ 
     public function all()
     {
         $translations = $this->translations();
-        $entries = $this->filter($translations, $this->entries())->results();
-        array_walk($translations, function (&$entry) use ($entries) {
-            $entry['desc'] = array_key_exists($entry['id'], $entries)
-                ? $this->getEntryString($entries[$entry['id']], 'msgctxt')
+        $entries = $this->filter( $translations, $this->entries() )->results();
+        array_walk( $translations, function( &$entry ) use ($entries) {
+            $entry['desc'] = array_key_exists( $entry['id'], $entries )
+                ? $this->getEntryString( $entries[$entry['id']], 'msgctxt' )
                 : '';
         });
         return $translations;
@@ -44,10 +44,10 @@ 
      */
     public function entries()
     {
-        if (!isset($this->entries)) {
-            $potFile = glsr()->path(glsr()->languages.'/'.Application::ID.'.pot');
-            $entries = $this->extractEntriesFromPotFile($potFile);
-            $entries = apply_filters('site-reviews/translation/entries', $entries);
+        if( !isset($this->entries) ) {
+            $potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' );
+            $entries = $this->extractEntriesFromPotFile( $potFile );
+            $entries = apply_filters( 'site-reviews/translation/entries', $entries );
             $this->entries = $entries;
         }
         return $this->entries;
@@ -58,24 +58,24 @@ 
      * @param array|null $entries
      * @return static
      */
-    public function exclude($entriesToExclude = null, $entries = null)
+    public function exclude( $entriesToExclude = null, $entries = null )
     {
-        return $this->filter($entriesToExclude, $entries, false);
+        return $this->filter( $entriesToExclude, $entries, false );
     }
 
     /**
      * @param string $potFile
      * @return array
      */
-    public function extractEntriesFromPotFile($potFile, array $entries = [])
+    public function extractEntriesFromPotFile( $potFile, array $entries = [] )
     {
         try {
-            $potEntries = $this->normalize(Parser::parseFile($potFile)->getEntries());
-            foreach ($potEntries as $key => $entry) {
-                $entries[html_entity_decode($key, ENT_COMPAT, 'UTF-8')] = $entry;
+            $potEntries = $this->normalize( Parser::parseFile( $potFile )->getEntries() );
+            foreach( $potEntries as $key => $entry ) {
+                $entries[html_entity_decode( $key, ENT_COMPAT, 'UTF-8' )] = $entry;
             }
-        } catch (Exception $e) {
-            glsr_log()->error($e->getMessage());
+        } catch( Exception $e ) {
+            glsr_log()->error( $e->getMessage() );
         }
         return $entries;
     }
@@ -86,18 +86,18 @@ 
      * @param bool $intersect
      * @return static
      */
-    public function filter($filterWith = null, $entries = null, $intersect = true)
+    public function filter( $filterWith = null, $entries = null, $intersect = true )
     {
-        if (!is_array($entries)) {
+        if( !is_array( $entries ) ) {
             $entries = $this->results;
         }
-        if (!is_array($filterWith)) {
+        if( !is_array( $filterWith ) ) {
             $filterWith = $this->translations();
         }
-        $keys = array_flip(glsr_array_column($filterWith, 'id'));
+        $keys = array_flip( glsr_array_column( $filterWith, 'id' ) );
         $this->results = $intersect
-            ? array_intersect_key($entries, $keys)
-            : array_diff_key($entries, $keys);
+            ? array_intersect_key( $entries, $keys )
+            : array_diff_key( $entries, $keys );
         return $this;
     }
 
@@ -105,20 +105,20 @@ 
      * @param string $template
      * @return string
      */
-    public function render($template, array $entry)
+    public function render( $template, array $entry )
     {
         $data = array_combine(
-            array_map(function ($key) { return 'data.'.$key; }, array_keys($entry)),
+            array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry ) ),
             $entry
         );
         $data['data.class'] = $data['data.error'] = '';
-        if (false === array_search($entry['s1'], glsr_array_column($this->entries(), 'msgid'))) {
+        if( false === array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' ) ) ) {
             $data['data.class'] = 'is-invalid';
-            $data['data.error'] = __('This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews');
+            $data['data.error'] = __( 'This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews' );
         }
-        return glsr(Template::class)->build('partials/translations/'.$template, [
-            'context' => array_map('esc_html', $data),
-        ]);
+        return glsr( Template::class )->build( 'partials/translations/'.$template, [
+            'context' => array_map( 'esc_html', $data ),
+        ] );
     }
 
     /**
@@ -128,10 +128,10 @@ 
     public function renderAll()
     {
         $rendered = '';
-        foreach ($this->all() as $index => $entry) {
+        foreach( $this->all() as $index => $entry ) {
             $entry['index'] = $index;
             $entry['prefix'] = OptionManager::databaseKey();
-            $rendered.= $this->render($entry['type'], $entry);
+            $rendered .= $this->render( $entry['type'], $entry );
         }
         return $rendered;
     }
@@ -140,25 +140,25 @@ 
      * @param bool $resetAfterRender
      * @return string
      */
-    public function renderResults($resetAfterRender = true)
+    public function renderResults( $resetAfterRender = true )
     {
         $rendered = '';
-        foreach ($this->results as $id => $entry) {
+        foreach( $this->results as $id => $entry ) {
             $data = [
-                'desc' => $this->getEntryString($entry, 'msgctxt'),
+                'desc' => $this->getEntryString( $entry, 'msgctxt' ),
                 'id' => $id,
-                'p1' => $this->getEntryString($entry, 'msgid_plural'),
-                's1' => $this->getEntryString($entry, 'msgid'),
+                'p1' => $this->getEntryString( $entry, 'msgid_plural' ),
+                's1' => $this->getEntryString( $entry, 'msgid' ),
             ];
             $text = !empty($data['p1'])
-                ? sprintf('%s | %s', $data['s1'], $data['p1'])
+                ? sprintf( '%s | %s', $data['s1'], $data['p1'] )
                 : $data['s1'];
-            $rendered.= $this->render('result', [
-                'entry' => json_encode($data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE),
-                'text' => wp_strip_all_tags($text),
-            ]);
+            $rendered .= $this->render( 'result', [
+                'entry' => json_encode( $data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ),
+                'text' => wp_strip_all_tags( $text ),
+            ] );
         }
-        if ($resetAfterRender) {
+        if( $resetAfterRender ) {
             $this->reset();
         }
         return $rendered;
@@ -186,18 +186,18 @@ 
      * @param string $needle
      * @return static
      */
-    public function search($needle = '')
+    public function search( $needle = '' )
     {
         $this->reset();
-        $needle = trim(strtolower($needle));
-        foreach ($this->entries() as $key => $entry) {
-            $single = strtolower($this->getEntryString($entry, 'msgid'));
-            $plural = strtolower($this->getEntryString($entry, 'msgid_plural'));
-            if (strlen($needle) < static::SEARCH_THRESHOLD) {
-                if (in_array($needle, [$single, $plural])) {
+        $needle = trim( strtolower( $needle ) );
+        foreach( $this->entries() as $key => $entry ) {
+            $single = strtolower( $this->getEntryString( $entry, 'msgid' ) );
+            $plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ) );
+            if( strlen( $needle ) < static::SEARCH_THRESHOLD ) {
+                if( in_array( $needle, [$single, $plural] ) ) {
                     $this->results[$key] = $entry;
                 }
-            } elseif (Str::contains(sprintf('%s %s', $single, $plural), $needle)) {
+            } elseif( Str::contains( sprintf( '%s %s', $single, $plural ), $needle ) ) {
                 $this->results[$key] = $entry;
             }
         }
@@ -211,10 +211,10 @@ 
     public function translations()
     {
         static $translations;
-        if (empty($translations)) {
-            $settings = glsr(OptionManager::class)->get('settings');
+        if( empty($translations) ) {
+            $settings = glsr( OptionManager::class )->get( 'settings' );
             $translations = isset($settings['strings'])
-                ? $this->normalizeSettings((array) $settings['strings'])
+                ? $this->normalizeSettings( (array)$settings['strings'] )
                 : [];
         }
         return $translations;
@@ -224,28 +224,28 @@ 
      * @param string $key
      * @return string
      */
-    protected function getEntryString(array $entry, $key)
+    protected function getEntryString( array $entry, $key )
     {
         return isset($entry[$key])
-            ? implode('', (array) $entry[$key])
+            ? implode( '', (array)$entry[$key] )
             : '';
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $entries)
+    protected function normalize( array $entries )
     {
         $keys = [
             'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]',
         ];
-        array_walk($entries, function (&$entry) use ($keys) {
-            foreach ($keys as $key) {
+        array_walk( $entries, function( &$entry ) use ($keys) {
+            foreach( $keys as $key ) {
                 try {
-                    $entry = $this->normalizeEntryString($entry, $key);
-                } catch (\TypeError $error) {
-                    glsr_log()->once('error', 'Translation/normalize', $error);
-                    glsr_log()->once('debug', 'Translation/normalize', $entry);
+                    $entry = $this->normalizeEntryString( $entry, $key );
+                } catch( \TypeError $error ) {
+                    glsr_log()->once( 'error', 'Translation/normalize', $error );
+                    glsr_log()->once( 'debug', 'Translation/normalize', $entry );
                 }
             }
         });
@@ -256,10 +256,10 @@ 
      * @param string $key
      * @return array
      */
-    protected function normalizeEntryString(array $entry, $key)
+    protected function normalizeEntryString( array $entry, $key )
     {
-        if (isset($entry[$key])) {
-            $entry[$key] = $this->getEntryString($entry, $key);
+        if( isset($entry[$key]) ) {
+            $entry[$key] = $this->getEntryString( $entry, $key );
         }
         return $entry;
     }
@@ -267,15 +267,15 @@ 
     /**
      * @return array
      */
-    protected function normalizeSettings(array $strings)
+    protected function normalizeSettings( array $strings )
     {
-        $defaultString = array_fill_keys(['id', 's1', 's2', 'p1', 'p2'], '');
-        $strings = array_filter($strings, 'is_array');
-        foreach ($strings as &$string) {
+        $defaultString = array_fill_keys( ['id', 's1', 's2', 'p1', 'p2'], '' );
+        $strings = array_filter( $strings, 'is_array' );
+        foreach( $strings as &$string ) {
             $string['type'] = isset($string['p1']) ? 'plural' : 'single';
-            $string = wp_parse_args($string, $defaultString);
+            $string = wp_parse_args( $string, $defaultString );
         }
-        return array_filter($strings, function ($string) {
+        return array_filter( $strings, function( $string ) {
             return !empty($string['id']);
         });
     }

Braces +6 added lines, -3 removed lines

@@ -74,7 +74,8 @@ 
             foreach ($potEntries as $key => $entry) {
                 $entries[html_entity_decode($key, ENT_COMPAT, 'UTF-8')] = $entry;
             }
-        } catch (Exception $e) {
+        }
+        catch (Exception $e) {
             glsr_log()->error($e->getMessage());
         }
         return $entries;
@@ -197,7 +198,8 @@ 
                 if (in_array($needle, [$single, $plural])) {
                     $this->results[$key] = $entry;
                 }
-            } elseif (Str::contains(sprintf('%s %s', $single, $plural), $needle)) {
+            }
+            elseif (Str::contains(sprintf('%s %s', $single, $plural), $needle)) {
                 $this->results[$key] = $entry;
             }
         }
@@ -243,7 +245,8 @@ 
             foreach ($keys as $key) {
                 try {
                     $entry = $this->normalizeEntryString($entry, $key);
-                } catch (\TypeError $error) {
+                }
+                catch (\TypeError $error) {
                     glsr_log()->once('error', 'Translation/normalize', $error);
                     glsr_log()->once('debug', 'Translation/normalize', $entry);
                 }

plugin/Modules/Translator.php

Indentation +88 added lines, -88 removed lines

@@ -6,97 +6,97 @@
 
 class Translator
 {
-    /**
-     * @param string $original
-     * @param string $domain
-     * @return string
-     */
-    public function translate($original, $domain, array $args)
-    {
-        $domains = apply_filters('site-reviews/translator/domains', [Application::ID]);
-        if (!in_array($domain, $domains)) {
-            return $original;
-        }
-        $args = $this->normalizeTranslationArgs($args);
-        $strings = $this->getTranslationStrings($args['single'], $args['plural']);
-        if (empty($strings)) {
-            return $original;
-        }
-        $string = current($strings);
-        return 'plural' == $string['type']
-            ? $this->translatePlural($domain, $string, $args)
-            : $this->translateSingle($domain, $string, $args);
-    }
+	/**
+	 * @param string $original
+	 * @param string $domain
+	 * @return string
+	 */
+	public function translate($original, $domain, array $args)
+	{
+		$domains = apply_filters('site-reviews/translator/domains', [Application::ID]);
+		if (!in_array($domain, $domains)) {
+			return $original;
+		}
+		$args = $this->normalizeTranslationArgs($args);
+		$strings = $this->getTranslationStrings($args['single'], $args['plural']);
+		if (empty($strings)) {
+			return $original;
+		}
+		$string = current($strings);
+		return 'plural' == $string['type']
+			? $this->translatePlural($domain, $string, $args)
+			: $this->translateSingle($domain, $string, $args);
+	}
 
-    /**
-     * Used when search/replacing a default text-domain translation
-     * @return string
-     */
-    public function getTranslation(array $args)
-    {
-        $args = $this->normalizeTranslationArgs($args);
-        return get_translations_for_domain(Application::ID)->translate_plural($args['single'], $args['plural'], $args['number']);
-    }
+	/**
+	 * Used when search/replacing a default text-domain translation
+	 * @return string
+	 */
+	public function getTranslation(array $args)
+	{
+		$args = $this->normalizeTranslationArgs($args);
+		return get_translations_for_domain(Application::ID)->translate_plural($args['single'], $args['plural'], $args['number']);
+	}
 
-    /**
-     * @param string $single
-     * @param string $plural
-     * @return array
-     */
-    protected function getTranslationStrings($single, $plural)
-    {
-        return array_filter(glsr(Translation::class)->translations(), function ($string) use ($single, $plural) {
-            return $string['s1'] == html_entity_decode($single, ENT_COMPAT, 'UTF-8')
-                && $string['p1'] == html_entity_decode($plural, ENT_COMPAT, 'UTF-8');
-        });
-    }
+	/**
+	 * @param string $single
+	 * @param string $plural
+	 * @return array
+	 */
+	protected function getTranslationStrings($single, $plural)
+	{
+		return array_filter(glsr(Translation::class)->translations(), function ($string) use ($single, $plural) {
+			return $string['s1'] == html_entity_decode($single, ENT_COMPAT, 'UTF-8')
+				&& $string['p1'] == html_entity_decode($plural, ENT_COMPAT, 'UTF-8');
+		});
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalizeTranslationArgs(array $args)
-    {
-        $defaults = [
-            'context' => '',
-            'number' => 1,
-            'plural' => '',
-            'single' => '',
-        ];
-        return shortcode_atts($defaults, $args);
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalizeTranslationArgs(array $args)
+	{
+		$defaults = [
+			'context' => '',
+			'number' => 1,
+			'plural' => '',
+			'single' => '',
+		];
+		return shortcode_atts($defaults, $args);
+	}
 
-    /**
-     * @param string $domain
-     * @return string
-     */
-    protected function translatePlural($domain, array $string, array $args)
-    {
-        if (!empty($string['s2'])) {
-            $args['single'] = $string['s2'];
-        }
-        if (!empty($string['p2'])) {
-            $args['plural'] = $string['p2'];
-        }
-        return get_translations_for_domain($domain)->translate_plural(
-            $args['single'],
-            $args['plural'],
-            $args['number'],
-            $args['context']
-        );
-    }
+	/**
+	 * @param string $domain
+	 * @return string
+	 */
+	protected function translatePlural($domain, array $string, array $args)
+	{
+		if (!empty($string['s2'])) {
+			$args['single'] = $string['s2'];
+		}
+		if (!empty($string['p2'])) {
+			$args['plural'] = $string['p2'];
+		}
+		return get_translations_for_domain($domain)->translate_plural(
+			$args['single'],
+			$args['plural'],
+			$args['number'],
+			$args['context']
+		);
+	}
 
-    /**
-     * @param string $domain
-     * @return string
-     */
-    protected function translateSingle($domain, array $string, array $args)
-    {
-        if (!empty($string['s2'])) {
-            $args['single'] = $string['s2'];
-        }
-        return get_translations_for_domain($domain)->translate(
-            $args['single'],
-            $args['context']
-        );
-    }
+	/**
+	 * @param string $domain
+	 * @return string
+	 */
+	protected function translateSingle($domain, array $string, array $args)
+	{
+		if (!empty($string['s2'])) {
+			$args['single'] = $string['s2'];
+		}
+		return get_translations_for_domain($domain)->translate(
+			$args['single'],
+			$args['context']
+		);
+	}
 }

Spacing +25 added lines, -25 removed lines

@@ -11,31 +11,31 @@ 
      * @param string $domain
      * @return string
      */
-    public function translate($original, $domain, array $args)
+    public function translate( $original, $domain, array $args )
     {
-        $domains = apply_filters('site-reviews/translator/domains', [Application::ID]);
-        if (!in_array($domain, $domains)) {
+        $domains = apply_filters( 'site-reviews/translator/domains', [Application::ID] );
+        if( !in_array( $domain, $domains ) ) {
             return $original;
         }
-        $args = $this->normalizeTranslationArgs($args);
-        $strings = $this->getTranslationStrings($args['single'], $args['plural']);
-        if (empty($strings)) {
+        $args = $this->normalizeTranslationArgs( $args );
+        $strings = $this->getTranslationStrings( $args['single'], $args['plural'] );
+        if( empty($strings) ) {
             return $original;
         }
-        $string = current($strings);
+        $string = current( $strings );
         return 'plural' == $string['type']
-            ? $this->translatePlural($domain, $string, $args)
-            : $this->translateSingle($domain, $string, $args);
+            ? $this->translatePlural( $domain, $string, $args )
+            : $this->translateSingle( $domain, $string, $args );
     }
 
     /**
      * Used when search/replacing a default text-domain translation
      * @return string
      */
-    public function getTranslation(array $args)
+    public function getTranslation( array $args )
     {
-        $args = $this->normalizeTranslationArgs($args);
-        return get_translations_for_domain(Application::ID)->translate_plural($args['single'], $args['plural'], $args['number']);
+        $args = $this->normalizeTranslationArgs( $args );
+        return get_translations_for_domain( Application::ID )->translate_plural( $args['single'], $args['plural'], $args['number'] );
     }
 
     /**
@@ -43,18 +43,18 @@ 
      * @param string $plural
      * @return array
      */
-    protected function getTranslationStrings($single, $plural)
+    protected function getTranslationStrings( $single, $plural )
     {
-        return array_filter(glsr(Translation::class)->translations(), function ($string) use ($single, $plural) {
-            return $string['s1'] == html_entity_decode($single, ENT_COMPAT, 'UTF-8')
-                && $string['p1'] == html_entity_decode($plural, ENT_COMPAT, 'UTF-8');
+        return array_filter( glsr( Translation::class )->translations(), function( $string ) use ($single, $plural) {
+            return $string['s1'] == html_entity_decode( $single, ENT_COMPAT, 'UTF-8' )
+                && $string['p1'] == html_entity_decode( $plural, ENT_COMPAT, 'UTF-8' );
         });
     }
 
     /**
      * @return array
      */
-    protected function normalizeTranslationArgs(array $args)
+    protected function normalizeTranslationArgs( array $args )
     {
         $defaults = [
             'context' => '',
@@ -62,22 +62,22 @@ 
             'plural' => '',
             'single' => '',
         ];
-        return shortcode_atts($defaults, $args);
+        return shortcode_atts( $defaults, $args );
     }
 
     /**
      * @param string $domain
      * @return string
      */
-    protected function translatePlural($domain, array $string, array $args)
+    protected function translatePlural( $domain, array $string, array $args )
     {
-        if (!empty($string['s2'])) {
+        if( !empty($string['s2']) ) {
             $args['single'] = $string['s2'];
         }
-        if (!empty($string['p2'])) {
+        if( !empty($string['p2']) ) {
             $args['plural'] = $string['p2'];
         }
-        return get_translations_for_domain($domain)->translate_plural(
+        return get_translations_for_domain( $domain )->translate_plural(
             $args['single'],
             $args['plural'],
             $args['number'],
@@ -89,12 +89,12 @@ 
      * @param string $domain
      * @return string
      */
-    protected function translateSingle($domain, array $string, array $args)
+    protected function translateSingle( $domain, array $string, array $args )
     {
-        if (!empty($string['s2'])) {
+        if( !empty($string['s2']) ) {
             $args['single'] = $string['s2'];
         }
-        return get_translations_for_domain($domain)->translate(
+        return get_translations_for_domain( $domain )->translate(
             $args['single'],
             $args['context']
         );

plugin/Controllers/TranslationController.php

Indentation +224 added lines, -224 removed lines

@@ -10,228 +10,228 @@
 
 class TranslationController
 {
-    /**
-     * @var Translator
-     */
-    public $translator;
-
-    public function __construct(Translator $translator)
-    {
-        $this->translator = $translator;
-    }
-
-    /**
-     * @param array $messages
-     * @return array
-     * @filter bulk_post_updated_messages
-     */
-    public function filterBulkUpdateMessages($messages, array $counts)
-    {
-        $messages = Arr::consolidateArray($messages);
-        $messages[Application::POST_TYPE] = [
-            'updated' => _n('%s review updated.', '%s reviews updated.', $counts['updated'], 'site-reviews'),
-            'locked' => _n('%s review not updated, somebody is editing it.', '%s reviews not updated, somebody is editing them.', $counts['locked'], 'site-reviews'),
-            'deleted' => _n('%s review permanently deleted.', '%s reviews permanently deleted.', $counts['deleted'], 'site-reviews'),
-            'trashed' => _n('%s review moved to the Trash.', '%s reviews moved to the Trash.', $counts['trashed'], 'site-reviews'),
-            'untrashed' => _n('%s review restored from the Trash.', '%s reviews restored from the Trash.', $counts['untrashed'], 'site-reviews'),
-        ];
-        return $messages;
-    }
-
-    /**
-     * @param string $translation
-     * @param string $text
-     * @param string $domain
-     * @return string
-     * @filter gettext
-     */
-    public function filterGettext($translation, $text, $domain)
-    {
-        return apply_filters('site-reviews/gettext/'.$domain, $translation, $text);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $text
-     * @return string
-     * @filter site-reviews/gettext/site-reviews
-     */
-    public function filterGettextSiteReviews($translation, $text)
-    {
-        return $this->translator->translate($translation, Application::ID, [
-            'single' => $text,
-        ]);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $text
-     * @param string $context
-     * @param string $domain
-     * @return string
-     * @filter gettext_with_context
-     */
-    public function filterGettextWithContext($translation, $text, $context, $domain)
-    {
-        return apply_filters('site-reviews/gettext_with_context/'.$domain, $translation, $text, $context);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $text
-     * @param string $context
-     * @return string
-     * @filter site-reviews/gettext_with_context/site-reviews
-     */
-    public function filterGettextWithContextSiteReviews($translation, $text, $context)
-    {
-        return $this->translator->translate($translation, Application::ID, [
-            'context' => $context,
-            'single' => $text,
-        ]);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $single
-     * @param string $plural
-     * @param int $number
-     * @param string $domain
-     * @return string
-     * @filter ngettext
-     */
-    public function filterNgettext($translation, $single, $plural, $number, $domain)
-    {
-        return apply_filters('site-reviews/ngettext/'.$domain, $translation, $single, $plural, $number);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $single
-     * @param string $plural
-     * @param int $number
-     * @return string
-     * @filter site-reviews/ngettext/site-reviews
-     */
-    public function filterNgettextSiteReviews($translation, $single, $plural, $number)
-    {
-        return $this->translator->translate($translation, Application::ID, [
-            'number' => $number,
-            'plural' => $plural,
-            'single' => $single,
-        ]);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $single
-     * @param string $plural
-     * @param int $number
-     * @param string $context
-     * @param string $domain
-     * @return string
-     * @filter ngettext_with_context
-     */
-    public function filterNgettextWithContext($translation, $single, $plural, $number, $context, $domain)
-    {
-        return apply_filters('site-reviews/ngettext_with_context/'.$domain, $translation, $single, $plural, $number, $context);
-    }
-
-    /**
-     * @param string $translation
-     * @param string $single
-     * @param string $plural
-     * @param int $number
-     * @param string $context
-     * @return string
-     * @filter site-reviews/ngettext_with_context/site-reviews
-     */
-    public function filterNgettextWithContextSiteReviews($translation, $single, $plural, $number, $context)
-    {
-        return $this->translator->translate($translation, Application::ID, [
-            'context' => $context,
-            'number' => $number,
-            'plural' => $plural,
-            'single' => $single,
-        ]);
-    }
-
-    /**
-     * @param array $postStates
-     * @param \WP_Post $post
-     * @return array
-     * @filter display_post_states
-     */
-    public function filterPostStates($postStates, $post)
-    {
-        $postStates = Arr::consolidateArray($postStates);
-        if (Application::POST_TYPE == Arr::get($post, 'post_type') && array_key_exists('pending', $postStates)) {
-            $postStates['pending'] = __('Unapproved', 'site-reviews');
-        }
-        return $postStates;
-    }
-
-    /**
-     * @param string $translation
-     * @param string $text
-     * @return string
-     * @filter site-reviews/gettext/default
-     * @filter site-reviews/gettext_with_context/default
-     */
-    public function filterPostStatusLabels($translation, $text)
-    {
-        return $this->canModifyTranslation()
-            ? glsr(Labels::class)->filterPostStatusLabels($translation, $text)
-            : $translation;
-    }
-
-    /**
-     * @param string $translation
-     * @param string $single
-     * @param string $plural
-     * @param int $number
-     * @return string
-     * @filter site-reviews/ngettext/default
-     */
-    public function filterPostStatusText($translation, $single, $plural, $number)
-    {
-        if ($this->canModifyTranslation()) {
-            $strings = [
-                'Published' => __('Approved', 'site-reviews'),
-                'Pending' => __('Unapproved', 'site-reviews'),
-            ];
-            foreach ($strings as $search => $replace) {
-                if (!Str::contains($single, $search)) {
-                    continue;
-                }
-                return $this->translator->getTranslation([
-                    'number' => $number,
-                    'plural' => str_replace($search, $replace, $plural),
-                    'single' => str_replace($search, $replace, $single),
-                ]);
-            }
-        }
-        return $translation;
-    }
-
-    /**
-     * @return void
-     * @action admin_enqueue_scripts
-     */
-    public function translatePostStatusLabels()
-    {
-        if ($this->canModifyTranslation()) {
-            glsr(Labels::class)->translatePostStatusLabels();
-        }
-    }
-
-    /**
-     * @return bool
-     */
-    protected function canModifyTranslation()
-    {
-        $screen = glsr_current_screen();
-        return Application::POST_TYPE == $screen->post_type 
-            && in_array($screen->base, ['edit', 'post']);
-    }
+	/**
+	 * @var Translator
+	 */
+	public $translator;
+
+	public function __construct(Translator $translator)
+	{
+		$this->translator = $translator;
+	}
+
+	/**
+	 * @param array $messages
+	 * @return array
+	 * @filter bulk_post_updated_messages
+	 */
+	public function filterBulkUpdateMessages($messages, array $counts)
+	{
+		$messages = Arr::consolidateArray($messages);
+		$messages[Application::POST_TYPE] = [
+			'updated' => _n('%s review updated.', '%s reviews updated.', $counts['updated'], 'site-reviews'),
+			'locked' => _n('%s review not updated, somebody is editing it.', '%s reviews not updated, somebody is editing them.', $counts['locked'], 'site-reviews'),
+			'deleted' => _n('%s review permanently deleted.', '%s reviews permanently deleted.', $counts['deleted'], 'site-reviews'),
+			'trashed' => _n('%s review moved to the Trash.', '%s reviews moved to the Trash.', $counts['trashed'], 'site-reviews'),
+			'untrashed' => _n('%s review restored from the Trash.', '%s reviews restored from the Trash.', $counts['untrashed'], 'site-reviews'),
+		];
+		return $messages;
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $text
+	 * @param string $domain
+	 * @return string
+	 * @filter gettext
+	 */
+	public function filterGettext($translation, $text, $domain)
+	{
+		return apply_filters('site-reviews/gettext/'.$domain, $translation, $text);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $text
+	 * @return string
+	 * @filter site-reviews/gettext/site-reviews
+	 */
+	public function filterGettextSiteReviews($translation, $text)
+	{
+		return $this->translator->translate($translation, Application::ID, [
+			'single' => $text,
+		]);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $text
+	 * @param string $context
+	 * @param string $domain
+	 * @return string
+	 * @filter gettext_with_context
+	 */
+	public function filterGettextWithContext($translation, $text, $context, $domain)
+	{
+		return apply_filters('site-reviews/gettext_with_context/'.$domain, $translation, $text, $context);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $text
+	 * @param string $context
+	 * @return string
+	 * @filter site-reviews/gettext_with_context/site-reviews
+	 */
+	public function filterGettextWithContextSiteReviews($translation, $text, $context)
+	{
+		return $this->translator->translate($translation, Application::ID, [
+			'context' => $context,
+			'single' => $text,
+		]);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $single
+	 * @param string $plural
+	 * @param int $number
+	 * @param string $domain
+	 * @return string
+	 * @filter ngettext
+	 */
+	public function filterNgettext($translation, $single, $plural, $number, $domain)
+	{
+		return apply_filters('site-reviews/ngettext/'.$domain, $translation, $single, $plural, $number);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $single
+	 * @param string $plural
+	 * @param int $number
+	 * @return string
+	 * @filter site-reviews/ngettext/site-reviews
+	 */
+	public function filterNgettextSiteReviews($translation, $single, $plural, $number)
+	{
+		return $this->translator->translate($translation, Application::ID, [
+			'number' => $number,
+			'plural' => $plural,
+			'single' => $single,
+		]);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $single
+	 * @param string $plural
+	 * @param int $number
+	 * @param string $context
+	 * @param string $domain
+	 * @return string
+	 * @filter ngettext_with_context
+	 */
+	public function filterNgettextWithContext($translation, $single, $plural, $number, $context, $domain)
+	{
+		return apply_filters('site-reviews/ngettext_with_context/'.$domain, $translation, $single, $plural, $number, $context);
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $single
+	 * @param string $plural
+	 * @param int $number
+	 * @param string $context
+	 * @return string
+	 * @filter site-reviews/ngettext_with_context/site-reviews
+	 */
+	public function filterNgettextWithContextSiteReviews($translation, $single, $plural, $number, $context)
+	{
+		return $this->translator->translate($translation, Application::ID, [
+			'context' => $context,
+			'number' => $number,
+			'plural' => $plural,
+			'single' => $single,
+		]);
+	}
+
+	/**
+	 * @param array $postStates
+	 * @param \WP_Post $post
+	 * @return array
+	 * @filter display_post_states
+	 */
+	public function filterPostStates($postStates, $post)
+	{
+		$postStates = Arr::consolidateArray($postStates);
+		if (Application::POST_TYPE == Arr::get($post, 'post_type') && array_key_exists('pending', $postStates)) {
+			$postStates['pending'] = __('Unapproved', 'site-reviews');
+		}
+		return $postStates;
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $text
+	 * @return string
+	 * @filter site-reviews/gettext/default
+	 * @filter site-reviews/gettext_with_context/default
+	 */
+	public function filterPostStatusLabels($translation, $text)
+	{
+		return $this->canModifyTranslation()
+			? glsr(Labels::class)->filterPostStatusLabels($translation, $text)
+			: $translation;
+	}
+
+	/**
+	 * @param string $translation
+	 * @param string $single
+	 * @param string $plural
+	 * @param int $number
+	 * @return string
+	 * @filter site-reviews/ngettext/default
+	 */
+	public function filterPostStatusText($translation, $single, $plural, $number)
+	{
+		if ($this->canModifyTranslation()) {
+			$strings = [
+				'Published' => __('Approved', 'site-reviews'),
+				'Pending' => __('Unapproved', 'site-reviews'),
+			];
+			foreach ($strings as $search => $replace) {
+				if (!Str::contains($single, $search)) {
+					continue;
+				}
+				return $this->translator->getTranslation([
+					'number' => $number,
+					'plural' => str_replace($search, $replace, $plural),
+					'single' => str_replace($search, $replace, $single),
+				]);
+			}
+		}
+		return $translation;
+	}
+
+	/**
+	 * @return void
+	 * @action admin_enqueue_scripts
+	 */
+	public function translatePostStatusLabels()
+	{
+		if ($this->canModifyTranslation()) {
+			glsr(Labels::class)->translatePostStatusLabels();
+		}
+	}
+
+	/**
+	 * @return bool
+	 */
+	protected function canModifyTranslation()
+	{
+		$screen = glsr_current_screen();
+		return Application::POST_TYPE == $screen->post_type 
+			&& in_array($screen->base, ['edit', 'post']);
+	}
 }

Spacing +47 added lines, -47 removed lines

@@ -15,7 +15,7 @@ 
      */
     public $translator;
 
-    public function __construct(Translator $translator)
+    public function __construct( Translator $translator )
     {
         $this->translator = $translator;
     }
@@ -25,15 +25,15 @@ 
      * @return array
      * @filter bulk_post_updated_messages
      */
-    public function filterBulkUpdateMessages($messages, array $counts)
+    public function filterBulkUpdateMessages( $messages, array $counts )
     {
-        $messages = Arr::consolidateArray($messages);
+        $messages = Arr::consolidateArray( $messages );
         $messages[Application::POST_TYPE] = [
-            'updated' => _n('%s review updated.', '%s reviews updated.', $counts['updated'], 'site-reviews'),
-            'locked' => _n('%s review not updated, somebody is editing it.', '%s reviews not updated, somebody is editing them.', $counts['locked'], 'site-reviews'),
-            'deleted' => _n('%s review permanently deleted.', '%s reviews permanently deleted.', $counts['deleted'], 'site-reviews'),
-            'trashed' => _n('%s review moved to the Trash.', '%s reviews moved to the Trash.', $counts['trashed'], 'site-reviews'),
-            'untrashed' => _n('%s review restored from the Trash.', '%s reviews restored from the Trash.', $counts['untrashed'], 'site-reviews'),
+            'updated' => _n( '%s review updated.', '%s reviews updated.', $counts['updated'], 'site-reviews' ),
+            'locked' => _n( '%s review not updated, somebody is editing it.', '%s reviews not updated, somebody is editing them.', $counts['locked'], 'site-reviews' ),
+            'deleted' => _n( '%s review permanently deleted.', '%s reviews permanently deleted.', $counts['deleted'], 'site-reviews' ),
+            'trashed' => _n( '%s review moved to the Trash.', '%s reviews moved to the Trash.', $counts['trashed'], 'site-reviews' ),
+            'untrashed' => _n( '%s review restored from the Trash.', '%s reviews restored from the Trash.', $counts['untrashed'], 'site-reviews' ),
         ];
         return $messages;
     }
@@ -45,9 +45,9 @@ 
      * @return string
      * @filter gettext
      */
-    public function filterGettext($translation, $text, $domain)
+    public function filterGettext( $translation, $text, $domain )
     {
-        return apply_filters('site-reviews/gettext/'.$domain, $translation, $text);
+        return apply_filters( 'site-reviews/gettext/'.$domain, $translation, $text );
     }
 
     /**
@@ -56,11 +56,11 @@ 
      * @return string
      * @filter site-reviews/gettext/site-reviews
      */
-    public function filterGettextSiteReviews($translation, $text)
+    public function filterGettextSiteReviews( $translation, $text )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'single' => $text,
-        ]);
+        ] );
     }
 
     /**
@@ -71,9 +71,9 @@ 
      * @return string
      * @filter gettext_with_context
      */
-    public function filterGettextWithContext($translation, $text, $context, $domain)
+    public function filterGettextWithContext( $translation, $text, $context, $domain )
     {
-        return apply_filters('site-reviews/gettext_with_context/'.$domain, $translation, $text, $context);
+        return apply_filters( 'site-reviews/gettext_with_context/'.$domain, $translation, $text, $context );
     }
 
     /**
@@ -83,12 +83,12 @@ 
      * @return string
      * @filter site-reviews/gettext_with_context/site-reviews
      */
-    public function filterGettextWithContextSiteReviews($translation, $text, $context)
+    public function filterGettextWithContextSiteReviews( $translation, $text, $context )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'context' => $context,
             'single' => $text,
-        ]);
+        ] );
     }
 
     /**
@@ -100,9 +100,9 @@ 
      * @return string
      * @filter ngettext
      */
-    public function filterNgettext($translation, $single, $plural, $number, $domain)
+    public function filterNgettext( $translation, $single, $plural, $number, $domain )
     {
-        return apply_filters('site-reviews/ngettext/'.$domain, $translation, $single, $plural, $number);
+        return apply_filters( 'site-reviews/ngettext/'.$domain, $translation, $single, $plural, $number );
     }
 
     /**
@@ -113,13 +113,13 @@ 
      * @return string
      * @filter site-reviews/ngettext/site-reviews
      */
-    public function filterNgettextSiteReviews($translation, $single, $plural, $number)
+    public function filterNgettextSiteReviews( $translation, $single, $plural, $number )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'number' => $number,
             'plural' => $plural,
             'single' => $single,
-        ]);
+        ] );
     }
 
     /**
@@ -132,9 +132,9 @@ 
      * @return string
      * @filter ngettext_with_context
      */
-    public function filterNgettextWithContext($translation, $single, $plural, $number, $context, $domain)
+    public function filterNgettextWithContext( $translation, $single, $plural, $number, $context, $domain )
     {
-        return apply_filters('site-reviews/ngettext_with_context/'.$domain, $translation, $single, $plural, $number, $context);
+        return apply_filters( 'site-reviews/ngettext_with_context/'.$domain, $translation, $single, $plural, $number, $context );
     }
 
     /**
@@ -146,14 +146,14 @@ 
      * @return string
      * @filter site-reviews/ngettext_with_context/site-reviews
      */
-    public function filterNgettextWithContextSiteReviews($translation, $single, $plural, $number, $context)
+    public function filterNgettextWithContextSiteReviews( $translation, $single, $plural, $number, $context )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'context' => $context,
             'number' => $number,
             'plural' => $plural,
             'single' => $single,
-        ]);
+        ] );
     }
 
     /**
@@ -162,11 +162,11 @@ 
      * @return array
      * @filter display_post_states
      */
-    public function filterPostStates($postStates, $post)
+    public function filterPostStates( $postStates, $post )
     {
-        $postStates = Arr::consolidateArray($postStates);
-        if (Application::POST_TYPE == Arr::get($post, 'post_type') && array_key_exists('pending', $postStates)) {
-            $postStates['pending'] = __('Unapproved', 'site-reviews');
+        $postStates = Arr::consolidateArray( $postStates );
+        if( Application::POST_TYPE == Arr::get( $post, 'post_type' ) && array_key_exists( 'pending', $postStates ) ) {
+            $postStates['pending'] = __( 'Unapproved', 'site-reviews' );
         }
         return $postStates;
     }
@@ -178,10 +178,10 @@ 
      * @filter site-reviews/gettext/default
      * @filter site-reviews/gettext_with_context/default
      */
-    public function filterPostStatusLabels($translation, $text)
+    public function filterPostStatusLabels( $translation, $text )
     {
         return $this->canModifyTranslation()
-            ? glsr(Labels::class)->filterPostStatusLabels($translation, $text)
+            ? glsr( Labels::class )->filterPostStatusLabels( $translation, $text )
             : $translation;
     }
 
@@ -193,22 +193,22 @@ 
      * @return string
      * @filter site-reviews/ngettext/default
      */
-    public function filterPostStatusText($translation, $single, $plural, $number)
+    public function filterPostStatusText( $translation, $single, $plural, $number )
     {
-        if ($this->canModifyTranslation()) {
+        if( $this->canModifyTranslation() ) {
             $strings = [
-                'Published' => __('Approved', 'site-reviews'),
-                'Pending' => __('Unapproved', 'site-reviews'),
+                'Published' => __( 'Approved', 'site-reviews' ),
+                'Pending' => __( 'Unapproved', 'site-reviews' ),
             ];
-            foreach ($strings as $search => $replace) {
-                if (!Str::contains($single, $search)) {
+            foreach( $strings as $search => $replace ) {
+                if( !Str::contains( $single, $search ) ) {
                     continue;
                 }
-                return $this->translator->getTranslation([
+                return $this->translator->getTranslation( [
                     'number' => $number,
-                    'plural' => str_replace($search, $replace, $plural),
-                    'single' => str_replace($search, $replace, $single),
-                ]);
+                    'plural' => str_replace( $search, $replace, $plural ),
+                    'single' => str_replace( $search, $replace, $single ),
+                ] );
             }
         }
         return $translation;
@@ -220,8 +220,8 @@ 
      */
     public function translatePostStatusLabels()
     {
-        if ($this->canModifyTranslation()) {
-            glsr(Labels::class)->translatePostStatusLabels();
+        if( $this->canModifyTranslation() ) {
+            glsr( Labels::class )->translatePostStatusLabels();
         }
     }
 
@@ -232,6 +232,6 @@ 
     {
         $screen = glsr_current_screen();
         return Application::POST_TYPE == $screen->post_type 
-            && in_array($screen->base, ['edit', 'post']);
+            && in_array( $screen->base, ['edit', 'post'] );
     }
 }