pryley / site-reviews

plugin/Handlers/RegisterTinymcePopups.php

Indentation +18 added lines, -18 removed lines

@@ -7,22 +7,22 @@
 
 class RegisterTinymcePopups
 {
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        foreach ($command->popups as $slug => $label) {
-            $buttonClass = glsr(Helper::class)->buildClassName($slug.'-popup', 'Shortcodes');
-            if (!class_exists($buttonClass)) {
-                glsr_log()->error(sprintf('Class missing (%s)', $buttonClass));
-                continue;
-            }
-            $shortcode = glsr($buttonClass)->register($slug, [
-                'label' => $label,
-                'title' => $label,
-            ]);
-            glsr()->mceShortcodes[$slug] = $shortcode->properties;
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		foreach ($command->popups as $slug => $label) {
+			$buttonClass = glsr(Helper::class)->buildClassName($slug.'-popup', 'Shortcodes');
+			if (!class_exists($buttonClass)) {
+				glsr_log()->error(sprintf('Class missing (%s)', $buttonClass));
+				continue;
+			}
+			$shortcode = glsr($buttonClass)->register($slug, [
+				'label' => $label,
+				'title' => $label,
+			]);
+			glsr()->mceShortcodes[$slug] = $shortcode->properties;
+		}
+	}
 }

plugin/Handlers/RegisterTaxonomy.php

Indentation +8 added lines, -8 removed lines

@@ -7,12 +7,12 @@
 
 class RegisterTaxonomy
 {
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        register_taxonomy(Application::TAXONOMY, Application::POST_TYPE, $command->args);
-        register_taxonomy_for_object_type(Application::TAXONOMY, Application::POST_TYPE);
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		register_taxonomy(Application::TAXONOMY, Application::POST_TYPE, $command->args);
+		register_taxonomy_for_object_type(Application::TAXONOMY, Application::POST_TYPE);
+	}
 }

plugin/Handlers/EnqueueAdminAssets.php

Indentation +146 added lines, -146 removed lines

@@ -10,158 +10,158 @@
 
 class EnqueueAdminAssets
 {
-    /**
-     * @var array
-     */
-    protected $pointers;
+	/**
+	 * @var array
+	 */
+	protected $pointers;
 
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        $this->generatePointers($command->pointers);
-        $this->enqueueAssets();
-        $this->localizeAssets();
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		$this->generatePointers($command->pointers);
+		$this->enqueueAssets();
+		$this->localizeAssets();
+	}
 
-    /**
-     * @return void
-     */
-    public function enqueueAssets()
-    {
-        wp_enqueue_style(
-            Application::ID,
-            glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
-            [],
-            glsr()->version
-        );
-        if (!$this->isCurrentScreen()) {
-            return;
-        }
-        wp_enqueue_script(
-            Application::ID,
-            glsr()->url('assets/scripts/'.Application::ID.'-admin.js'),
-            $this->getDependencies(),
-            glsr()->version,
-            true
-        );
-        if (!empty($this->pointers)) {
-            wp_enqueue_style('wp-pointer');
-            wp_enqueue_script('wp-pointer');
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function enqueueAssets()
+	{
+		wp_enqueue_style(
+			Application::ID,
+			glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
+			[],
+			glsr()->version
+		);
+		if (!$this->isCurrentScreen()) {
+			return;
+		}
+		wp_enqueue_script(
+			Application::ID,
+			glsr()->url('assets/scripts/'.Application::ID.'-admin.js'),
+			$this->getDependencies(),
+			glsr()->version,
+			true
+		);
+		if (!empty($this->pointers)) {
+			wp_enqueue_style('wp-pointer');
+			wp_enqueue_script('wp-pointer');
+		}
+	}
 
-    /**
-     * @return void
-     */
-    public function localizeAssets()
-    {
-        $variables = [
-            'action' => Application::PREFIX.'action',
-            'addons' => [],
-            'ajaxurl' => admin_url('admin-ajax.php'),
-            'hideoptions' => [
-                'site_reviews' => glsr(SiteReviewsShortcode::class)->getHideOptions(),
-                'site_reviews_form' => glsr(SiteReviewsFormShortcode::class)->getHideOptions(),
-                'site_reviews_summary' => glsr(SiteReviewsSummaryShortcode::class)->getHideOptions(),
-            ],
-            'nameprefix' => Application::ID,
-            'nonce' => [
-                'change-status' => wp_create_nonce('change-status'),
-                'clear-console' => wp_create_nonce('clear-console'),
-                'count-reviews' => wp_create_nonce('count-reviews'),
-                'fetch-console' => wp_create_nonce('fetch-console'),
-                'mce-shortcode' => wp_create_nonce('mce-shortcode'),
-                'sync-reviews' => wp_create_nonce('sync-reviews'),
-                'toggle-pinned' => wp_create_nonce('toggle-pinned'),
-            ],
-            'pointers' => $this->pointers,
-            'shortcodes' => [],
-            'tinymce' => [
-                'glsr_shortcode' => glsr()->url('assets/scripts/mce-plugin.js'),
-            ],
-        ];
-        if (user_can_richedit()) {
-            $variables['shortcodes'] = $this->localizeShortcodes();
-        }
-        $variables = apply_filters('site-reviews/enqueue/admin/localize', $variables);
-        wp_localize_script(Application::ID, 'GLSR', $variables);
-    }
+	/**
+	 * @return void
+	 */
+	public function localizeAssets()
+	{
+		$variables = [
+			'action' => Application::PREFIX.'action',
+			'addons' => [],
+			'ajaxurl' => admin_url('admin-ajax.php'),
+			'hideoptions' => [
+				'site_reviews' => glsr(SiteReviewsShortcode::class)->getHideOptions(),
+				'site_reviews_form' => glsr(SiteReviewsFormShortcode::class)->getHideOptions(),
+				'site_reviews_summary' => glsr(SiteReviewsSummaryShortcode::class)->getHideOptions(),
+			],
+			'nameprefix' => Application::ID,
+			'nonce' => [
+				'change-status' => wp_create_nonce('change-status'),
+				'clear-console' => wp_create_nonce('clear-console'),
+				'count-reviews' => wp_create_nonce('count-reviews'),
+				'fetch-console' => wp_create_nonce('fetch-console'),
+				'mce-shortcode' => wp_create_nonce('mce-shortcode'),
+				'sync-reviews' => wp_create_nonce('sync-reviews'),
+				'toggle-pinned' => wp_create_nonce('toggle-pinned'),
+			],
+			'pointers' => $this->pointers,
+			'shortcodes' => [],
+			'tinymce' => [
+				'glsr_shortcode' => glsr()->url('assets/scripts/mce-plugin.js'),
+			],
+		];
+		if (user_can_richedit()) {
+			$variables['shortcodes'] = $this->localizeShortcodes();
+		}
+		$variables = apply_filters('site-reviews/enqueue/admin/localize', $variables);
+		wp_localize_script(Application::ID, 'GLSR', $variables);
+	}
 
-    /**
-     * @return array
-     */
-    protected function getDependencies()
-    {
-        $dependencies = apply_filters('site-reviews/enqueue/admin/dependencies', []);
-        $dependencies = array_merge($dependencies, [
-            'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
-        ]);
-        return $dependencies;
-    }
+	/**
+	 * @return array
+	 */
+	protected function getDependencies()
+	{
+		$dependencies = apply_filters('site-reviews/enqueue/admin/dependencies', []);
+		$dependencies = array_merge($dependencies, [
+			'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
+		]);
+		return $dependencies;
+	}
 
-    /**
-     * @return array
-     */
-    protected function generatePointer(array $pointer)
-    {
-        return [
-            'id' => $pointer['id'],
-            'options' => [
-                'content' => '<h3>'.$pointer['title'].'</h3><p>'.$pointer['content'].'</p>',
-                'position' => $pointer['position'],
-            ],
-            'screen' => $pointer['screen'],
-            'target' => $pointer['target'],
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function generatePointer(array $pointer)
+	{
+		return [
+			'id' => $pointer['id'],
+			'options' => [
+				'content' => '<h3>'.$pointer['title'].'</h3><p>'.$pointer['content'].'</p>',
+				'position' => $pointer['position'],
+			],
+			'screen' => $pointer['screen'],
+			'target' => $pointer['target'],
+		];
+	}
 
-    /**
-     * @return void
-     */
-    protected function generatePointers(array $pointers)
-    {
-        $dismissedPointers = get_user_meta(get_current_user_id(), 'dismissed_wp_pointers', true);
-        $dismissedPointers = explode(',', (string) $dismissedPointers);
-        $generatedPointers = [];
-        foreach ($pointers as $pointer) {
-            if ($pointer['screen'] != glsr_current_screen()->id) {
-                continue;
-            }
-            if (in_array($pointer['id'], $dismissedPointers)) {
-                continue;
-            }
-            $generatedPointers[] = $this->generatePointer($pointer);
-        }
-        $this->pointers = $generatedPointers;
-    }
+	/**
+	 * @return void
+	 */
+	protected function generatePointers(array $pointers)
+	{
+		$dismissedPointers = get_user_meta(get_current_user_id(), 'dismissed_wp_pointers', true);
+		$dismissedPointers = explode(',', (string) $dismissedPointers);
+		$generatedPointers = [];
+		foreach ($pointers as $pointer) {
+			if ($pointer['screen'] != glsr_current_screen()->id) {
+				continue;
+			}
+			if (in_array($pointer['id'], $dismissedPointers)) {
+				continue;
+			}
+			$generatedPointers[] = $this->generatePointer($pointer);
+		}
+		$this->pointers = $generatedPointers;
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isCurrentScreen()
-    {
-        $screen = glsr_current_screen();
-        return $screen && (Application::POST_TYPE == $screen->post_type
-            || 'post' == $screen->base
-            || 'dashboard' == $screen->id
-            || 'widgets' == $screen->id
-        );
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isCurrentScreen()
+	{
+		$screen = glsr_current_screen();
+		return $screen && (Application::POST_TYPE == $screen->post_type
+			|| 'post' == $screen->base
+			|| 'dashboard' == $screen->id
+			|| 'widgets' == $screen->id
+		);
+	}
 
-    /**
-     * @return array
-     */
-    protected function localizeShortcodes()
-    {
-        $variables = [];
-        foreach (glsr()->mceShortcodes as $tag => $args) {
-            if (empty($args['required'])) {
-                continue;
-            }
-            $variables[$tag] = $args['required'];
-        }
-        return $variables;
-    }
+	/**
+	 * @return array
+	 */
+	protected function localizeShortcodes()
+	{
+		$variables = [];
+		foreach (glsr()->mceShortcodes as $tag => $args) {
+			if (empty($args['required'])) {
+				continue;
+			}
+			$variables[$tag] = $args['required'];
+		}
+		return $variables;
+	}
 }

plugin/Handlers/RegisterPostType.php

Indentation +12 added lines, -12 removed lines

@@ -6,16 +6,16 @@
 
 class RegisterPostType
 {
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        if (!in_array($command->postType, get_post_types(['_builtin' => true]))) {
-            register_post_type($command->postType, $command->args);
-            glsr()->postTypeColumns = wp_parse_args(glsr()->postTypeColumns, [
-                $command->postType => $command->columns,
-            ]);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		if (!in_array($command->postType, get_post_types(['_builtin' => true]))) {
+			register_post_type($command->postType, $command->args);
+			glsr()->postTypeColumns = wp_parse_args(glsr()->postTypeColumns, [
+				$command->postType => $command->columns,
+			]);
+		}
+	}
 }

plugin/Handlers/RegisterShortcodes.php

Indentation +14 added lines, -14 removed lines

@@ -7,18 +7,18 @@
 
 class RegisterShortcodes
 {
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        foreach ($command->shortcodes as $shortcode) {
-            $shortcodeClass = glsr(Helper::class)->buildClassName($shortcode.'-shortcode', 'Shortcodes');
-            if (!class_exists($shortcodeClass)) {
-                glsr_log()->error(sprintf('Class missing (%s)', $shortcodeClass));
-                continue;
-            }
-            add_shortcode($shortcode, [glsr($shortcodeClass), 'buildShortcode']);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		foreach ($command->shortcodes as $shortcode) {
+			$shortcodeClass = glsr(Helper::class)->buildClassName($shortcode.'-shortcode', 'Shortcodes');
+			if (!class_exists($shortcodeClass)) {
+				glsr_log()->error(sprintf('Class missing (%s)', $shortcodeClass));
+				continue;
+			}
+			add_shortcode($shortcode, [glsr($shortcodeClass), 'buildShortcode']);
+		}
+	}
 }

plugin/Shortcodes/Shortcode.php

Indentation +218 added lines, -218 removed lines

@@ -10,222 +10,222 @@
 
 abstract class Shortcode implements ShortcodeContract
 {
-    /**
-     * @var string
-     */
-    protected $partialName;
-
-    /**
-     * @var string
-     */
-    protected $shortcodeName;
-
-    /**
-     * @param string|array $atts
-     * @param string $type
-     * @return string
-     */
-    public function build($atts, array $args = [], $type = 'shortcode')
-    {
-        $this->partialName = $this->getShortcodePartialName();
-        $this->shortcodeName = $this->getShortcodeName();
-        $args = $this->normalizeArgs($args, $type);
-        $atts = $this->normalizeAtts($atts, $type);
-        $partial = glsr(Partial::class)->build($this->partialName, $atts);
-        $title = !empty($atts['title'])
-            ? $args['before_title'].$atts['title'].$args['after_title']
-            : '';
-        $debug = sprintf('<glsr-%1$s hidden data-atts=\'%2$s\'></glsr-%1$s>', $type, $atts['json']);
-        return $args['before_widget'].$title.$partial.$debug.$args['after_widget'];
-    }
-
-    /**
-     * @param string|array $atts
-     * @return string
-     */
-    public function buildShortcode($atts = [])
-    {
-        return $this->build($atts);
-    }
-
-    /**
-     * @return array
-     */
-    public function getDefaults($atts)
-    {
-        return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
-    }
-
-    /**
-     * @return array
-     */
-    public function getHideOptions()
-    {
-        $options = $this->hideOptions();
-        return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodeClassName($replace = '', $search = 'Shortcode')
-    {
-        return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodeDefaultsClassName()
-    {
-        return glsr(Helper::class)->buildClassName(
-            $this->getShortcodeClassName('Defaults'),
-            'Defaults'
-        );
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodeName()
-    {
-        return glsr(Helper::class)->snakeCase($this->getShortcodeClassName());
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodePartialName()
-    {
-        return glsr(Helper::class)->dashCase($this->getShortcodeClassName());
-    }
-
-    /**
-     * @param array|string $args
-     * @param string $type
-     * @return array
-     */
-    public function normalizeArgs($args, $type = 'shortcode')
-    {
-        $args = wp_parse_args($args, [
-            'before_widget' => '<div class="glsr-'.$type.' '.$type.'-'.$this->partialName.'">',
-            'after_widget' => '</div>',
-            'before_title' => '<h3 class="glsr-'.$type.'-title">',
-            'after_title' => '</h3>',
-        ]);
-        return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
-    }
-
-    /**
-     * @param array|string $atts
-     * @param string $type
-     * @return array
-     */
-    public function normalizeAtts($atts, $type = 'shortcode')
-    {
-        $atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
-        $atts = $this->getDefaults($atts);
-        array_walk($atts, function (&$value, $key) {
-            $methodName = glsr(Helper::class)->buildMethodName($key, 'normalize');
-            if (!method_exists($this, $methodName)) {
-                return;
-            }
-            $value = $this->$methodName($value);
-        });
-        return $atts;
-    }
-
-    /**
-     * @return array
-     */
-    abstract protected function hideOptions();
-
-    /**
-     * @param string $postId
-     * @return int|string
-     */
-    protected function normalizeAssignedTo($postId)
-    {
-        if ('parent_id' == $postId) {
-            $postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
-        } elseif ('post_id' == $postId) {
-            $postId = intval(get_the_ID());
-        }
-        return $postId;
-    }
-
-    /**
-     * @param string $postId
-     * @return int|string
-     */
-    protected function normalizeAssignTo($postId)
-    {
-        return $this->normalizeAssignedTo($postId);
-    }
-
-    /**
-     * @param string|array $hide
-     * @return array
-     */
-    protected function normalizeHide($hide)
-    {
-        if (is_string($hide)) {
-            $hide = explode(',', $hide);
-        }
-        $hideKeys = array_keys($this->getHideOptions());
-        return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
-            return in_array($value, $hideKeys);
-        });
-    }
-
-    /**
-     * @param string $id
-     * @return string
-     */
-    protected function normalizeId($id)
-    {
-        return sanitize_title($id);
-    }
-
-    /**
-     * @param string $labels
-     * @return array
-     */
-    protected function normalizeLabels($labels)
-    {
-        $defaults = [
-            __('Excellent', 'site-reviews'),
-            __('Very good', 'site-reviews'),
-            __('Average', 'site-reviews'),
-            __('Poor', 'site-reviews'),
-            __('Terrible', 'site-reviews'),
-        ];
-        $defaults = array_pad($defaults, glsr()->constant('MAX_RATING', Rating::class), '');
-        $labels = array_map('trim', explode(',', $labels));
-        foreach ($defaults as $i => $label) {
-            if (empty($labels[$i])) {
-                continue;
-            }
-            $defaults[$i] = $labels[$i];
-        }
-        return array_combine(range(glsr()->constant('MAX_RATING', Rating::class), 1), $defaults);
-    }
-
-    /**
-     * @param string $schema
-     * @return bool
-     */
-    protected function normalizeSchema($schema)
-    {
-        return wp_validate_boolean($schema);
-    }
-
-    /**
-     * @param string $text
-     * @return string
-     */
-    protected function normalizeText($text)
-    {
-        return trim($text);
-    }
+	/**
+	 * @var string
+	 */
+	protected $partialName;
+
+	/**
+	 * @var string
+	 */
+	protected $shortcodeName;
+
+	/**
+	 * @param string|array $atts
+	 * @param string $type
+	 * @return string
+	 */
+	public function build($atts, array $args = [], $type = 'shortcode')
+	{
+		$this->partialName = $this->getShortcodePartialName();
+		$this->shortcodeName = $this->getShortcodeName();
+		$args = $this->normalizeArgs($args, $type);
+		$atts = $this->normalizeAtts($atts, $type);
+		$partial = glsr(Partial::class)->build($this->partialName, $atts);
+		$title = !empty($atts['title'])
+			? $args['before_title'].$atts['title'].$args['after_title']
+			: '';
+		$debug = sprintf('<glsr-%1$s hidden data-atts=\'%2$s\'></glsr-%1$s>', $type, $atts['json']);
+		return $args['before_widget'].$title.$partial.$debug.$args['after_widget'];
+	}
+
+	/**
+	 * @param string|array $atts
+	 * @return string
+	 */
+	public function buildShortcode($atts = [])
+	{
+		return $this->build($atts);
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getDefaults($atts)
+	{
+		return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getHideOptions()
+	{
+		$options = $this->hideOptions();
+		return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodeClassName($replace = '', $search = 'Shortcode')
+	{
+		return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodeDefaultsClassName()
+	{
+		return glsr(Helper::class)->buildClassName(
+			$this->getShortcodeClassName('Defaults'),
+			'Defaults'
+		);
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodeName()
+	{
+		return glsr(Helper::class)->snakeCase($this->getShortcodeClassName());
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodePartialName()
+	{
+		return glsr(Helper::class)->dashCase($this->getShortcodeClassName());
+	}
+
+	/**
+	 * @param array|string $args
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeArgs($args, $type = 'shortcode')
+	{
+		$args = wp_parse_args($args, [
+			'before_widget' => '<div class="glsr-'.$type.' '.$type.'-'.$this->partialName.'">',
+			'after_widget' => '</div>',
+			'before_title' => '<h3 class="glsr-'.$type.'-title">',
+			'after_title' => '</h3>',
+		]);
+		return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
+	}
+
+	/**
+	 * @param array|string $atts
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeAtts($atts, $type = 'shortcode')
+	{
+		$atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
+		$atts = $this->getDefaults($atts);
+		array_walk($atts, function (&$value, $key) {
+			$methodName = glsr(Helper::class)->buildMethodName($key, 'normalize');
+			if (!method_exists($this, $methodName)) {
+				return;
+			}
+			$value = $this->$methodName($value);
+		});
+		return $atts;
+	}
+
+	/**
+	 * @return array
+	 */
+	abstract protected function hideOptions();
+
+	/**
+	 * @param string $postId
+	 * @return int|string
+	 */
+	protected function normalizeAssignedTo($postId)
+	{
+		if ('parent_id' == $postId) {
+			$postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
+		} elseif ('post_id' == $postId) {
+			$postId = intval(get_the_ID());
+		}
+		return $postId;
+	}
+
+	/**
+	 * @param string $postId
+	 * @return int|string
+	 */
+	protected function normalizeAssignTo($postId)
+	{
+		return $this->normalizeAssignedTo($postId);
+	}
+
+	/**
+	 * @param string|array $hide
+	 * @return array
+	 */
+	protected function normalizeHide($hide)
+	{
+		if (is_string($hide)) {
+			$hide = explode(',', $hide);
+		}
+		$hideKeys = array_keys($this->getHideOptions());
+		return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
+			return in_array($value, $hideKeys);
+		});
+	}
+
+	/**
+	 * @param string $id
+	 * @return string
+	 */
+	protected function normalizeId($id)
+	{
+		return sanitize_title($id);
+	}
+
+	/**
+	 * @param string $labels
+	 * @return array
+	 */
+	protected function normalizeLabels($labels)
+	{
+		$defaults = [
+			__('Excellent', 'site-reviews'),
+			__('Very good', 'site-reviews'),
+			__('Average', 'site-reviews'),
+			__('Poor', 'site-reviews'),
+			__('Terrible', 'site-reviews'),
+		];
+		$defaults = array_pad($defaults, glsr()->constant('MAX_RATING', Rating::class), '');
+		$labels = array_map('trim', explode(',', $labels));
+		foreach ($defaults as $i => $label) {
+			if (empty($labels[$i])) {
+				continue;
+			}
+			$defaults[$i] = $labels[$i];
+		}
+		return array_combine(range(glsr()->constant('MAX_RATING', Rating::class), 1), $defaults);
+	}
+
+	/**
+	 * @param string $schema
+	 * @return bool
+	 */
+	protected function normalizeSchema($schema)
+	{
+		return wp_validate_boolean($schema);
+	}
+
+	/**
+	 * @param string $text
+	 * @return string
+	 */
+	protected function normalizeText($text)
+	{
+		return trim($text);
+	}
 }

plugin/Shortcodes/SiteReviewsFormPopup.php

Indentation +46 added lines, -46 removed lines

@@ -4,50 +4,50 @@
 
 class SiteReviewsFormPopup extends TinymcePopupGenerator
 {
-    /**
-     * @return array
-     */
-    public function fields()
-    {
-        return [[
-            'type' => 'container',
-            'html' => '<p class="strong">'.esc_html__('All settings are optional.', 'site-reviews').'</p>',
-        ], [
-            'label' => esc_html__('Title', 'site-reviews'),
-            'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Description', 'site-reviews'),
-            'minHeight' => 60,
-            'minWidth' => 240,
-            'multiline' => true,
-            'name' => 'description',
-            'tooltip' => __('Enter a custom shortcode description.', 'site-reviews'),
-            'type' => 'textbox',
-        ],
-        $this->getCategories(__('Automatically assign a category to reviews submitted with this shortcode.', 'site-reviews')),
-        [
-            'label' => esc_html__('Assign To', 'site-reviews'),
-            'name' => 'assign_to',
-            'tooltip' => __('Assign submitted reviews to a custom page/post ID. You can also enter "post_id" to assign reviews to the ID of the current page.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            '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 [[
+			'type' => 'container',
+			'html' => '<p class="strong">'.esc_html__('All settings are optional.', 'site-reviews').'</p>',
+		], [
+			'label' => esc_html__('Title', 'site-reviews'),
+			'name' => 'title',
+			'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Description', 'site-reviews'),
+			'minHeight' => 60,
+			'minWidth' => 240,
+			'multiline' => true,
+			'name' => 'description',
+			'tooltip' => __('Enter a custom shortcode description.', 'site-reviews'),
+			'type' => 'textbox',
+		],
+		$this->getCategories(__('Automatically assign a category to reviews submitted with this shortcode.', 'site-reviews')),
+		[
+			'label' => esc_html__('Assign To', 'site-reviews'),
+			'name' => 'assign_to',
+			'tooltip' => __('Assign submitted reviews to a custom page/post ID. You can also enter "post_id" to assign reviews to the ID of the current page.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'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',
+		], ];
+	}
 }

plugin/Shortcodes/SiteReviewsSummaryPopup.php

Indentation +45 added lines, -45 removed lines

@@ -4,49 +4,49 @@
 
 class SiteReviewsSummaryPopup extends SiteReviewsPopup
 {
-    /**
-     * @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',
-        ],
-        $this->getTypes(__('Which type of review would you like to use?', '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',
-        ], ];
-    }
+	/**
+	 * @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',
+		],
+		$this->getTypes(__('Which type of review would you like to use?', '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',
+		], ];
+	}
 }

plugin/Shortcodes/SiteReviewsFormShortcode.php

Indentation +24 added lines, -24 removed lines

@@ -4,29 +4,29 @@
 
 class SiteReviewsFormShortcode extends Shortcode
 {
-    protected function hideOptions()
-    {
-        return [
-            'rating' => __('Hide the rating field', 'site-reviews'),
-            'title' => __('Hide the title field', 'site-reviews'),
-            'content' => __('Hide the review field', 'site-reviews'),
-            'name' => __('Hide the name field', 'site-reviews'),
-            'email' => __('Hide the email field', 'site-reviews'),
-            'terms' => __('Hide the terms field', 'site-reviews'),
-        ];
-    }
+	protected function hideOptions()
+	{
+		return [
+			'rating' => __('Hide the rating field', 'site-reviews'),
+			'title' => __('Hide the title field', 'site-reviews'),
+			'content' => __('Hide the review field', 'site-reviews'),
+			'name' => __('Hide the name field', 'site-reviews'),
+			'email' => __('Hide the email field', 'site-reviews'),
+			'terms' => __('Hide the terms field', 'site-reviews'),
+		];
+	}
 
-    /**
-     * @param array|string $atts
-     * @param string $type
-     * @return array
-     */
-    public function normalizeAtts($atts, $type = 'shortcode')
-    {
-        $atts = parent::normalizeAtts($atts, $type);
-        if (empty($atts['id'])) {
-            $atts['id'] = substr(md5(serialize($atts)), 0, 8);
-        }
-        return $atts;
-    }
+	/**
+	 * @param array|string $atts
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeAtts($atts, $type = 'shortcode')
+	{
+		$atts = parent::normalizeAtts($atts, $type);
+		if (empty($atts['id'])) {
+			$atts['id'] = substr(md5(serialize($atts)), 0, 8);
+		}
+		return $atts;
+	}
 }