Inspection of "fix WPML integration" - pryley/site-reviews - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 7f1e4d...e207fc )

by Paul

created 2019-12-06 01:20 UTC

Status

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

@@ -8,24 +8,24 @@
 block discarded – undo
 
 class TogglePinned
 {
-    /**
-     * @return bool
-     */
-    public function handle(Command $command)
-    {
-        if (!get_post($command->id)) {
-            return false;
-        }
-        if (is_null($command->pinned)) {
-            $meta = glsr(Database::class)->get($command->id, 'pinned');
-            $command->pinned = !wp_validate_boolean($meta);
-        } else {
-            $notice = $command->pinned
-                ? __('Review pinned.', 'site-reviews')
-                : __('Review unpinned.', 'site-reviews');
-            glsr(Notice::class)->addSuccess($notice);
-        }
-        glsr(Database::class)->update($command->id, 'pinned', $command->pinned);
-        return $command->pinned;
-    }
+	/**
+	 * @return bool
+	 */
+	public function handle(Command $command)
+	{
+		if (!get_post($command->id)) {
+			return false;
+		}
+		if (is_null($command->pinned)) {
+			$meta = glsr(Database::class)->get($command->id, 'pinned');
+			$command->pinned = !wp_validate_boolean($meta);
+		} else {
+			$notice = $command->pinned
+				? __('Review pinned.', 'site-reviews')
+				: __('Review unpinned.', 'site-reviews');
+			glsr(Notice::class)->addSuccess($notice);
+		}
+		glsr(Database::class)->update($command->id, 'pinned', $command->pinned);
+		return $command->pinned;
+	}
 }

Please login to merge, or discard this patch.

plugin/Modules/Akismet.php 1 patch

Indentation +75 added lines, -75 removed lines patch added patch discarded remove patch

@@ -7,82 +7,82 @@
 block discarded – undo
 
 class Akismet
 {
-    /**
-     * @return bool
-     */
-    public function isSpam(array $review)
-    {
-        if (!$this->isActive()) {
-            return false;
-        }
-        $submission = [
-            'blog' => glsr(OptionManager::class)->getWP('home'),
-            'blog_charset' => glsr(OptionManager::class)->getWP('blog_charset', 'UTF-8'),
-            'blog_lang' => get_locale(),
-            'comment_author' => $review['name'],
-            'comment_author_email' => $review['email'],
-            'comment_content' => $review['title']."\n\n".$review['content'],
-            'comment_type' => 'review',
-            'referrer' => filter_input(INPUT_SERVER, 'HTTP_REFERER'),
-            'user_agent' => filter_input(INPUT_SERVER, 'HTTP_USER_AGENT'),
-            'user_ip' => $review['ip_address'],
-            // 'user_role' => 'administrator',
-            // 'is_test' => 1,
-        ];
-        foreach ($_SERVER as $key => $value) {
-            if (is_array($value) || in_array($key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'])) {
-                continue;
-            }
-            $submission[$key] = $value;
-        }
-        return $this->check(apply_filters('site-reviews/akismet/submission', $submission, $review));
-    }
+	/**
+	 * @return bool
+	 */
+	public function isSpam(array $review)
+	{
+		if (!$this->isActive()) {
+			return false;
+		}
+		$submission = [
+			'blog' => glsr(OptionManager::class)->getWP('home'),
+			'blog_charset' => glsr(OptionManager::class)->getWP('blog_charset', 'UTF-8'),
+			'blog_lang' => get_locale(),
+			'comment_author' => $review['name'],
+			'comment_author_email' => $review['email'],
+			'comment_content' => $review['title']."\n\n".$review['content'],
+			'comment_type' => 'review',
+			'referrer' => filter_input(INPUT_SERVER, 'HTTP_REFERER'),
+			'user_agent' => filter_input(INPUT_SERVER, 'HTTP_USER_AGENT'),
+			'user_ip' => $review['ip_address'],
+			// 'user_role' => 'administrator',
+			// 'is_test' => 1,
+		];
+		foreach ($_SERVER as $key => $value) {
+			if (is_array($value) || in_array($key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'])) {
+				continue;
+			}
+			$submission[$key] = $value;
+		}
+		return $this->check(apply_filters('site-reviews/akismet/submission', $submission, $review));
+	}
 
-    /**
-     * @return bool
-     */
-    protected function check(array $submission)
-    {
-        $response = AkismetPlugin::http_post($this->buildQuery($submission), 'comment-check');
-        return apply_filters('site-reviews/akismet/is-spam',
-            'true' == $response[1],
-            $submission,
-            $response
-        );
-    }
+	/**
+	 * @return bool
+	 */
+	protected function check(array $submission)
+	{
+		$response = AkismetPlugin::http_post($this->buildQuery($submission), 'comment-check');
+		return apply_filters('site-reviews/akismet/is-spam',
+			'true' == $response[1],
+			$submission,
+			$response
+		);
+	}
 
-    /**
-     * @return string
-     */
-    protected function buildQuery(array $data)
-    {
-        $query = [];
-        foreach ($data as $key => $value) {
-            if (is_array($value) || is_object($value)) {
-                continue;
-            }
-            if (false === $value) {
-                $value = '0';
-            }
-            $value = trim($value);
-            if (!strlen($value)) {
-                continue;
-            }
-            $query[] = urlencode($key).'='.urlencode($value);
-        }
-        return implode('&', $query);
-    }
+	/**
+	 * @return string
+	 */
+	protected function buildQuery(array $data)
+	{
+		$query = [];
+		foreach ($data as $key => $value) {
+			if (is_array($value) || is_object($value)) {
+				continue;
+			}
+			if (false === $value) {
+				$value = '0';
+			}
+			$value = trim($value);
+			if (!strlen($value)) {
+				continue;
+			}
+			$query[] = urlencode($key).'='.urlencode($value);
+		}
+		return implode('&', $query);
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isActive()
-    {
-        $check = !glsr(OptionManager::class)->getBool('settings.submissions.akismet')
-            || !is_callable(['Akismet', 'get_api_key'])
-            || !is_callable(['Akismet', 'http_post'])
-            ? false
-            : (bool) AkismetPlugin::get_api_key();
-        return apply_filters('site-reviews/akismet/is-active', $check);
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isActive()
+	{
+		$check = !glsr(OptionManager::class)->getBool('settings.submissions.akismet')
+			|| !is_callable(['Akismet', 'get_api_key'])
+			|| !is_callable(['Akismet', 'http_post'])
+			? false
+			: (bool) AkismetPlugin::get_api_key();
+		return apply_filters('site-reviews/akismet/is-active', $check);
+	}
 }

Please login to merge, or discard this patch.

plugin/Modules/Blacklist.php 1 patch

Indentation +48 added lines, -48 removed lines patch added patch discarded remove patch

@@ -6,54 +6,54 @@
 block discarded – undo
 
 class Blacklist
 {
-    /**
-     * @return bool
-     */
-    public function isBlacklisted(array $review)
-    {
-        $target = implode("\n", array_filter([
-            $review['name'],
-            $review['content'],
-            $review['email'],
-            $review['ip_address'],
-            $review['title'],
-        ]));
-        return (bool) apply_filters('site-reviews/blacklist/is-blacklisted',
-            $this->check($target),
-            $review,
-            $target
-        );
-    }
+	/**
+	 * @return bool
+	 */
+	public function isBlacklisted(array $review)
+	{
+		$target = implode("\n", array_filter([
+			$review['name'],
+			$review['content'],
+			$review['email'],
+			$review['ip_address'],
+			$review['title'],
+		]));
+		return (bool) apply_filters('site-reviews/blacklist/is-blacklisted',
+			$this->check($target),
+			$review,
+			$target
+		);
+	}
 
-    /**
-     * @param string $target
-     * @return bool
-     */
-    protected function check($target)
-    {
-        $blacklist = $this->getBlacklist();
-        if (empty($blacklist)) {
-            return false;
-        }
-        $lines = explode("\n", $blacklist);
-        foreach ((array) $lines as $line) {
-            $line = trim($line);
-            if (empty($line) || 256 < strlen($line)) {
-                continue;
-            }
-            $pattern = sprintf('#%s#i', preg_quote($line, '#'));
-            if (preg_match($pattern, $target)) {
-                return true;
-            }
-        }
-        return false;
-    }
+	/**
+	 * @param string $target
+	 * @return bool
+	 */
+	protected function check($target)
+	{
+		$blacklist = $this->getBlacklist();
+		if (empty($blacklist)) {
+			return false;
+		}
+		$lines = explode("\n", $blacklist);
+		foreach ((array) $lines as $line) {
+			$line = trim($line);
+			if (empty($line) || 256 < strlen($line)) {
+				continue;
+			}
+			$pattern = sprintf('#%s#i', preg_quote($line, '#'));
+			if (preg_match($pattern, $target)) {
+				return true;
+			}
+		}
+		return false;
+	}
 
-    protected function getBlacklist()
-    {
-        $option = glsr(OptionManager::class)->get('settings.submissions.blacklist.integration');
-        return $option == 'comments'
-            ? trim(glsr(OptionManager::class)->getWP('blacklist_keys'))
-            : trim(glsr(OptionManager::class)->get('settings.submissions.blacklist.entries'));
-    }
+	protected function getBlacklist()
+	{
+		$option = glsr(OptionManager::class)->get('settings.submissions.blacklist.integration');
+		return $option == 'comments'
+			? trim(glsr(OptionManager::class)->getWP('blacklist_keys'))
+			: trim(glsr(OptionManager::class)->get('settings.submissions.blacklist.entries'));
+	}
 }

Please login to merge, or discard this patch.

plugin/Modules/Notification.php 1 patch

Indentation +148 added lines, -148 removed lines patch added patch discarded remove patch

@@ -8,164 +8,164 @@
 block discarded – undo
 
 class Notification
 {
-    /**
-     * @var bool
-     */
-    protected $email;
+	/**
+	 * @var bool
+	 */
+	protected $email;
 
-    /**
-     * @var Review
-     */
-    protected $review;
+	/**
+	 * @var Review
+	 */
+	protected $review;
 
-    /**
-     * @var bool
-     */
-    protected $slack;
+	/**
+	 * @var bool
+	 */
+	protected $slack;
 
-    /**
-     * @var array
-     */
-    protected $types;
+	/**
+	 * @var array
+	 */
+	protected $types;
 
-    public function __construct()
-    {
-        $types = glsr(OptionManager::class)->get('settings.general.notifications', []);
-        $this->email = count(array_intersect(['admin', 'custom'], $types)) > 0;
-        $this->slack = in_array('slack', $types);
-        $this->types = $types;
-    }
+	public function __construct()
+	{
+		$types = glsr(OptionManager::class)->get('settings.general.notifications', []);
+		$this->email = count(array_intersect(['admin', 'custom'], $types)) > 0;
+		$this->slack = in_array('slack', $types);
+		$this->types = $types;
+	}
 
-    /**
-     * @return void
-     */
-    public function send(Review $review)
-    {
-        if (empty($this->types)) {
-            return;
-        }
-        $this->review = $review;
-        $args = [
-            'link' => $this->getLink(),
-            'title' => $this->getTitle(),
-        ];
-        $this->sendToEmail($args);
-        $this->sendToSlack($args);
-    }
+	/**
+	 * @return void
+	 */
+	public function send(Review $review)
+	{
+		if (empty($this->types)) {
+			return;
+		}
+		$this->review = $review;
+		$args = [
+			'link' => $this->getLink(),
+			'title' => $this->getTitle(),
+		];
+		$this->sendToEmail($args);
+		$this->sendToSlack($args);
+	}
 
-    /**
-     * @return Email
-     */
-    protected function buildEmail(array $args)
-    {
-        return glsr(Email::class)->compose([
-            'to' => $this->getEmailAddresses(),
-            'subject' => $args['title'],
-            'template' => 'email-notification',
-            'template-tags' => [
-                'review_author' => $this->review->author ?: __('Anonymous', 'site-reviews'),
-                'review_content' => $this->review->content,
-                'review_email' => $this->review->email,
-                'review_ip' => $this->review->ip_address,
-                'review_link' => sprintf('<a href="%1$s">%1$s</a>', $args['link']),
-                'review_rating' => $this->review->rating,
-                'review_title' => $this->review->title,
-            ],
-        ]);
-    }
+	/**
+	 * @return Email
+	 */
+	protected function buildEmail(array $args)
+	{
+		return glsr(Email::class)->compose([
+			'to' => $this->getEmailAddresses(),
+			'subject' => $args['title'],
+			'template' => 'email-notification',
+			'template-tags' => [
+				'review_author' => $this->review->author ?: __('Anonymous', 'site-reviews'),
+				'review_content' => $this->review->content,
+				'review_email' => $this->review->email,
+				'review_ip' => $this->review->ip_address,
+				'review_link' => sprintf('<a href="%1$s">%1$s</a>', $args['link']),
+				'review_rating' => $this->review->rating,
+				'review_title' => $this->review->title,
+			],
+		]);
+	}
 
-    /**
-     * @return Slack
-     */
-    protected function buildSlackNotification(array $args)
-    {
-        return glsr(Slack::class)->compose($this->review, [
-            'button_url' => $args['link'],
-            'fallback' => $this->buildEmail($args)->read('plaintext'),
-            'pretext' => $args['title'],
-        ]);
-    }
+	/**
+	 * @return Slack
+	 */
+	protected function buildSlackNotification(array $args)
+	{
+		return glsr(Slack::class)->compose($this->review, [
+			'button_url' => $args['link'],
+			'fallback' => $this->buildEmail($args)->read('plaintext'),
+			'pretext' => $args['title'],
+		]);
+	}
 
-    /**
-     * @return array
-     */
-    protected function getEmailAddresses()
-    {
-        $emails = [];
-        if (in_array('admin', $this->types)) {
-            $emails[] = glsr(OptionManager::class)->getWP('admin_email');
-        }
-        if (in_array('author', $this->types)) {
-            $assignedPost = get_post(intval($this->review->assigned_to));
-            if ($assignedPost instanceof WP_Post) {
-                $this->email = true;
-                $emails[] = get_the_author_meta('user_email', intval($assignedPost->post_author));
-            }
-        }
-        if (in_array('custom', $this->types)) {
-            $customEmails = glsr(OptionManager::class)->get('settings.general.notification_email');
-            $customEmails = str_replace([' ', ',', ';'], ',', $customEmails);
-            $customEmails = explode(',', $customEmails);
-            $emails = array_merge($emails, $customEmails);
-        }
-        $emails = array_filter(array_keys(array_flip($emails)));
-        return apply_filters('site-reviews/notification/emails', $emails, $this->review);
-    }
+	/**
+	 * @return array
+	 */
+	protected function getEmailAddresses()
+	{
+		$emails = [];
+		if (in_array('admin', $this->types)) {
+			$emails[] = glsr(OptionManager::class)->getWP('admin_email');
+		}
+		if (in_array('author', $this->types)) {
+			$assignedPost = get_post(intval($this->review->assigned_to));
+			if ($assignedPost instanceof WP_Post) {
+				$this->email = true;
+				$emails[] = get_the_author_meta('user_email', intval($assignedPost->post_author));
+			}
+		}
+		if (in_array('custom', $this->types)) {
+			$customEmails = glsr(OptionManager::class)->get('settings.general.notification_email');
+			$customEmails = str_replace([' ', ',', ';'], ',', $customEmails);
+			$customEmails = explode(',', $customEmails);
+			$emails = array_merge($emails, $customEmails);
+		}
+		$emails = array_filter(array_keys(array_flip($emails)));
+		return apply_filters('site-reviews/notification/emails', $emails, $this->review);
+	}
 
-    /**
-     * @return string
-     */
-    protected function getLink()
-    {
-        return admin_url('post.php?post='.$this->review->ID.'&action=edit');
-    }
+	/**
+	 * @return string
+	 */
+	protected function getLink()
+	{
+		return admin_url('post.php?post='.$this->review->ID.'&action=edit');
+	}
 
-    /**
-     * @return string
-     */
-    protected function getTitle()
-    {
-        $assignedTitle = get_the_title(intval($this->review->assigned_to));
-        $title = _nx(
-            'New %s-star review',
-            'New %s-star review of: %s',
-            intval(empty($assignedTitle)),
-            'This string differs depending on whether or not the review has been assigned to a post.',
-            'site-reviews'
-        );
-        $title = sprintf('[%s] %s',
-            wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES),
-            sprintf($title, $this->review->rating, $assignedTitle)
-        );
-        return apply_filters('site-reviews/notification/title', $title, $this->review);
-    }
+	/**
+	 * @return string
+	 */
+	protected function getTitle()
+	{
+		$assignedTitle = get_the_title(intval($this->review->assigned_to));
+		$title = _nx(
+			'New %s-star review',
+			'New %s-star review of: %s',
+			intval(empty($assignedTitle)),
+			'This string differs depending on whether or not the review has been assigned to a post.',
+			'site-reviews'
+		);
+		$title = sprintf('[%s] %s',
+			wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES),
+			sprintf($title, $this->review->rating, $assignedTitle)
+		);
+		return apply_filters('site-reviews/notification/title', $title, $this->review);
+	}
 
-    /**
-     * @return void
-     */
-    protected function sendToEmail(array $args)
-    {
-        $email = $this->buildEmail($args);
-        if (empty($email->to)) {
-            glsr_log()->error('Email notification was not sent (missing email address)');
-            return;
-        }
-        $email->send();
-    }
+	/**
+	 * @return void
+	 */
+	protected function sendToEmail(array $args)
+	{
+		$email = $this->buildEmail($args);
+		if (empty($email->to)) {
+			glsr_log()->error('Email notification was not sent (missing email address)');
+			return;
+		}
+		$email->send();
+	}
 
-    /**
-     * @return void
-     */
-    protected function sendToSlack(array $args)
-    {
-        if (!$this->slack) {
-            return;
-        }
-        $notification = $this->buildSlackNotification($args);
-        $result = $notification->send();
-        if (is_wp_error($result)) {
-            $notification->review = null;
-            glsr_log()->error($result->get_error_message())->debug($notification);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	protected function sendToSlack(array $args)
+	{
+		if (!$this->slack) {
+			return;
+		}
+		$notification = $this->buildSlackNotification($args);
+		$result = $notification->send();
+		if (is_wp_error($result)) {
+			$notification->review = null;
+			glsr_log()->error($result->get_error_message())->debug($notification);
+		}
+	}
 }

Please login to merge, or discard this patch.

plugin/Controllers/EditorController/Customization.php 1 patch

Indentation +74 added lines, -74 removed lines patch added patch discarded remove patch

@@ -7,84 +7,84 @@
 block discarded – undo
 
 class Customization
 {
-    /**
-     * @return array
-     */
-    public function filterEditorSettings(array $settings)
-    {
-        if ($this->isReviewEditable()) {
-            $settings = [
-                'media_buttons' => false,
-                'quicktags' => false,
-                'textarea_rows' => 12,
-                'tinymce' => false,
-            ];
-        }
-        return $settings;
-    }
+	/**
+	 * @return array
+	 */
+	public function filterEditorSettings(array $settings)
+	{
+		if ($this->isReviewEditable()) {
+			$settings = [
+				'media_buttons' => false,
+				'quicktags' => false,
+				'textarea_rows' => 12,
+				'tinymce' => false,
+			];
+		}
+		return $settings;
+	}
 
-    /**
-     * @param string $html
-     * @return string
-     */
-    public function filterEditorTextarea($html)
-    {
-        if ($this->isReviewEditable()) {
-            $html = str_replace('<textarea', '<div id="ed_toolbar"></div><textarea', $html);
-        }
-        return $html;
-    }
+	/**
+	 * @param string $html
+	 * @return string
+	 */
+	public function filterEditorTextarea($html)
+	{
+		if ($this->isReviewEditable()) {
+			$html = str_replace('<textarea', '<div id="ed_toolbar"></div><textarea', $html);
+		}
+		return $html;
+	}
 
-    /**
-     * @return void
-     */
-    public function removeAutosave()
-    {
-        if (!$this->isReviewEditor() || $this->isReviewEditable()) {
-            return;
-        }
-        wp_deregister_script('autosave');
-    }
+	/**
+	 * @return void
+	 */
+	public function removeAutosave()
+	{
+		if (!$this->isReviewEditor() || $this->isReviewEditable()) {
+			return;
+		}
+		wp_deregister_script('autosave');
+	}
 
-    /**
-     * @return void
-     */
-    public function removeMetaBoxes()
-    {
-        remove_meta_box('slugdiv', Application::POST_TYPE, 'advanced');
-    }
+	/**
+	 * @return void
+	 */
+	public function removeMetaBoxes()
+	{
+		remove_meta_box('slugdiv', Application::POST_TYPE, 'advanced');
+	}
 
-    /**
-     * @return void
-     */
-    public function removePostTypeSupport()
-    {
-        if (!$this->isReviewEditor() || $this->isReviewEditable()) {
-            return;
-        }
-        remove_post_type_support(Application::POST_TYPE, 'title');
-        remove_post_type_support(Application::POST_TYPE, 'editor');
-    }
+	/**
+	 * @return void
+	 */
+	public function removePostTypeSupport()
+	{
+		if (!$this->isReviewEditor() || $this->isReviewEditable()) {
+			return;
+		}
+		remove_post_type_support(Application::POST_TYPE, 'title');
+		remove_post_type_support(Application::POST_TYPE, 'editor');
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isReviewEditable()
-    {
-        $postId = intval(filter_input(INPUT_GET, 'post'));
-        return $postId > 0
-            && 'local' == glsr(Database::class)->get($postId, 'review_type')
-            && $this->isReviewEditor();
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isReviewEditable()
+	{
+		$postId = intval(filter_input(INPUT_GET, 'post'));
+		return $postId > 0
+			&& 'local' == glsr(Database::class)->get($postId, 'review_type')
+			&& $this->isReviewEditor();
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isReviewEditor()
-    {
-        $screen = glsr_current_screen();
-        return 'post' == $screen->base
-            && Application::POST_TYPE == $screen->id
-            && Application::POST_TYPE == $screen->post_type;
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isReviewEditor()
+	{
+		$screen = glsr_current_screen();
+		return 'post' == $screen->base
+			&& Application::POST_TYPE == $screen->id
+			&& Application::POST_TYPE == $screen->post_type;
+	}
 }

Please login to merge, or discard this patch.

plugin/Contracts/MultilingualContract.php 1 patch

Indentation +21 added lines, -21 removed lines patch added patch discarded remove patch

@@ -4,29 +4,29 @@
 block discarded – undo
 
 interface MultilingualContract
 {
-    /**
-     * @param int|string $postId
-     * @return \WP_Post|void|null
-     */
-    public function getPost($postId);
+	/**
+	 * @param int|string $postId
+	 * @return \WP_Post|void|null
+	 */
+	public function getPost($postId);
 
-    /**
-     * @return array
-     */
-    public function getPostIds(array $postIds);
+	/**
+	 * @return array
+	 */
+	public function getPostIds(array $postIds);
 
-    /**
-     * @return bool
-     */
-    public function isActive();
+	/**
+	 * @return bool
+	 */
+	public function isActive();
 
-    /**
-     * @return bool
-     */
-    public function isEnabled();
+	/**
+	 * @return bool
+	 */
+	public function isEnabled();
 
-    /**
-     * @return bool
-     */
-    public function isSupported();
+	/**
+	 * @return bool
+	 */
+	public function isSupported();
 }

Please login to merge, or discard this patch.

plugin/Defaults/EmailDefaults.php 1 patch

Indentation +34 added lines, -34 removed lines patch added patch discarded remove patch

@@ -7,40 +7,40 @@
 block discarded – undo
 
 class EmailDefaults extends Defaults
 {
-    /**
-     * @return array
-     */
-    protected function defaults()
-    {
-        return [
-            'after' => '',
-            'attachments' => [],
-            'bcc' => '',
-            'before' => '',
-            'cc' => '',
-            'from' => $this->getFromName().' <'.$this->getFromEmail().'>',
-            'message' => '',
-            'reply-to' => '',
-            'subject' => '',
-            'template' => '',
-            'template-tags' => [],
-            'to' => '',
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function defaults()
+	{
+		return [
+			'after' => '',
+			'attachments' => [],
+			'bcc' => '',
+			'before' => '',
+			'cc' => '',
+			'from' => $this->getFromName().' <'.$this->getFromEmail().'>',
+			'message' => '',
+			'reply-to' => '',
+			'subject' => '',
+			'template' => '',
+			'template-tags' => [],
+			'to' => '',
+		];
+	}
 
-    /**
-     * @return string
-     */
-    protected function getFromEmail()
-    {
-        return glsr(OptionManager::class)->getWP('admin_email');
-    }
+	/**
+	 * @return string
+	 */
+	protected function getFromEmail()
+	{
+		return glsr(OptionManager::class)->getWP('admin_email');
+	}
 
-    /**
-     * @return string
-     */
-    protected function getFromName()
-    {
-        return wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES);
-    }
+	/**
+	 * @return string
+	 */
+	protected function getFromName()
+	{
+		return wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES);
+	}
 }

Please login to merge, or discard this patch.

site-reviews.php 1 patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -18,11 +18,11 @@
 block discarded – undo
 defined('WPINC') || die;
 
 if (!class_exists('GL_Plugin_Check_v3')) {
-    require_once __DIR__.'/activate.php';
+	require_once __DIR__.'/activate.php';
 }
 $check = new GL_Plugin_Check_v3(__FILE__);
 if (!$check->canProceed()) {
-    return;
+	return;
 }
 require_once __DIR__.'/autoload.php';
 require_once __DIR__.'/compatibility.php';

Please login to merge, or discard this patch.

plugin/Modules/Notice.php 1 patch

Indentation +101 added lines, -101 removed lines patch added patch discarded remove patch

@@ -8,112 +8,112 @@
 block discarded – undo
 
 class Notice
 {
-    /**
-     * @param string $type
-     * @param string|array|WP_Error $message
-     * @return void
-     */
-    public function add($type, $message, array $args = [])
-    {
-        if (empty(array_filter([$message, $type]))) {
-            return;
-        }
-        $args['message'] = $message;
-        $args['type'] = $type;
-        add_settings_error(Application::ID, '', json_encode($this->normalize($args)));
-    }
+	/**
+	 * @param string $type
+	 * @param string|array|WP_Error $message
+	 * @return void
+	 */
+	public function add($type, $message, array $args = [])
+	{
+		if (empty(array_filter([$message, $type]))) {
+			return;
+		}
+		$args['message'] = $message;
+		$args['type'] = $type;
+		add_settings_error(Application::ID, '', json_encode($this->normalize($args)));
+	}
 
-    /**
-     * @param string|array|WP_Error $message
-     * @return void
-     */
-    public function addError($message, array $args = [])
-    {
-        $this->add('error', $message, $args);
-    }
+	/**
+	 * @param string|array|WP_Error $message
+	 * @return void
+	 */
+	public function addError($message, array $args = [])
+	{
+		$this->add('error', $message, $args);
+	}
 
-    /**
-     * @param string|array|WP_Error $message
-     * @return void
-     */
-    public function addSuccess($message, array $args = [])
-    {
-        $this->add('success', $message, $args);
-    }
+	/**
+	 * @param string|array|WP_Error $message
+	 * @return void
+	 */
+	public function addSuccess($message, array $args = [])
+	{
+		$this->add('success', $message, $args);
+	}
 
-    /**
-     * @param string|array|WP_Error $message
-     * @return void
-     */
-    public function addWarning($message, array $args = [])
-    {
-        $this->add('warning', $message, $args);
-    }
+	/**
+	 * @param string|array|WP_Error $message
+	 * @return void
+	 */
+	public function addWarning($message, array $args = [])
+	{
+		$this->add('warning', $message, $args);
+	}
 
-    /**
-     * @return static
-     */
-    public function clear()
-    {
-        global $wp_settings_errors;
-        $wp_settings_errors = [];
-        delete_transient('settings_errors');
-        return $this;
-    }
+	/**
+	 * @return static
+	 */
+	public function clear()
+	{
+		global $wp_settings_errors;
+		$wp_settings_errors = [];
+		delete_transient('settings_errors');
+		return $this;
+	}
 
-    /**
-     * @return string
-     */
-    public function get()
-    {
-        $notices = array_map('unserialize',
-            array_unique(array_map('serialize', get_settings_errors(Application::ID)))
-        );
-        $notices = array_reduce($notices, function ($carry, $notice) {
-            return $carry.$this->buildNotice(json_decode($notice['message'], true));
-        });
-        return apply_filters('site-reviews/notices', $notices);
-    }
+	/**
+	 * @return string
+	 */
+	public function get()
+	{
+		$notices = array_map('unserialize',
+			array_unique(array_map('serialize', get_settings_errors(Application::ID)))
+		);
+		$notices = array_reduce($notices, function ($carry, $notice) {
+			return $carry.$this->buildNotice(json_decode($notice['message'], true));
+		});
+		return apply_filters('site-reviews/notices', $notices);
+	}
 
-    /**
-     * @return string
-     */
-    protected function buildNotice(array $args)
-    {
-        $messages = array_reduce($args['messages'], function ($carry, $message) {
-            return $carry.glsr(Builder::class)->p($message);
-        });
-        $class = 'notice notice-'.$args['type'];
-        if ($args['inline']) {
-            $class.= ' inline';
-        }
-        if ($args['dismissible']) {
-            $class.= ' is-dismissible';
-        }
-        return glsr(Builder::class)->div($messages, [
-            'class' => $class,
-        ]);
-    }
+	/**
+	 * @return string
+	 */
+	protected function buildNotice(array $args)
+	{
+		$messages = array_reduce($args['messages'], function ($carry, $message) {
+			return $carry.glsr(Builder::class)->p($message);
+		});
+		$class = 'notice notice-'.$args['type'];
+		if ($args['inline']) {
+			$class.= ' inline';
+		}
+		if ($args['dismissible']) {
+			$class.= ' is-dismissible';
+		}
+		return glsr(Builder::class)->div($messages, [
+			'class' => $class,
+		]);
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalize(array $args)
-    {
-        $defaults = [
-            'dismissible' => true,
-            'inline' => true,
-            'message' => '',
-            'type' => '',
-        ];
-        $args = shortcode_atts($defaults, $args);
-        if (!in_array($args['type'], ['error', 'warning', 'success'])) {
-            $args['type'] = 'success';
-        }
-        $args['messages'] = is_wp_error($args['message'])
-            ? (array) $args['message']->get_error_message()
-            : (array) $args['message'];
-        unset($args['message']);
-        return $args;
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalize(array $args)
+	{
+		$defaults = [
+			'dismissible' => true,
+			'inline' => true,
+			'message' => '',
+			'type' => '',
+		];
+		$args = shortcode_atts($defaults, $args);
+		if (!in_array($args['type'], ['error', 'warning', 'success'])) {
+			$args['type'] = 'success';
+		}
+		$args['messages'] = is_wp_error($args['message'])
+			? (array) $args['message']->get_error_message()
+			: (array) $args['message'];
+		unset($args['message']);
+		return $args;
+	}
 }

Please login to merge, or discard this patch.

		@@ -8,24 +8,24 @@
		block discarded – undo
8	8
9	9	class TogglePinned
10	10	{
11		- /**
12		- * @return bool
13		- */
14		- public function handle(Command $command)
15		- {
16		- if (!get_post($command->id)) {
17		- return false;
18		- }
19		- if (is_null($command->pinned)) {
20		- $meta = glsr(Database::class)->get($command->id, 'pinned');
21		- $command->pinned = !wp_validate_boolean($meta);
22		- } else {
23		- $notice = $command->pinned
24		- ? __('Review pinned.', 'site-reviews')
25		- : __('Review unpinned.', 'site-reviews');
26		- glsr(Notice::class)->addSuccess($notice);
27		- }
28		- glsr(Database::class)->update($command->id, 'pinned', $command->pinned);
29		- return $command->pinned;
30		- }
	11	+ /**
	12	+ * @return bool
	13	+ */
	14	+ public function handle(Command $command)
	15	+ {
	16	+ if (!get_post($command->id)) {
	17	+ return false;
	18	+ }
	19	+ if (is_null($command->pinned)) {
	20	+ $meta = glsr(Database::class)->get($command->id, 'pinned');
	21	+ $command->pinned = !wp_validate_boolean($meta);
	22	+ } else {
	23	+ $notice = $command->pinned
	24	+ ? __('Review pinned.', 'site-reviews')
	25	+ : __('Review unpinned.', 'site-reviews');
	26	+ glsr(Notice::class)->addSuccess($notice);
	27	+ }
	28	+ glsr(Database::class)->update($command->id, 'pinned', $command->pinned);
	29	+ return $command->pinned;
	30	+ }
31	31	}

		@@ -7,82 +7,82 @@
		block discarded – undo
7	7
8	8	class Akismet
9	9	{
10		- /**
11		- * @return bool
12		- */
13		- public function isSpam(array $review)
14		- {
15		- if (!$this->isActive()) {
16		- return false;
17		- }
18		- $submission = [
19		- 'blog' => glsr(OptionManager::class)->getWP('home'),
20		- 'blog_charset' => glsr(OptionManager::class)->getWP('blog_charset', 'UTF-8'),
21		- 'blog_lang' => get_locale(),
22		- 'comment_author' => $review['name'],
23		- 'comment_author_email' => $review['email'],
24		- 'comment_content' => $review['title']."\n\n".$review['content'],
25		- 'comment_type' => 'review',
26		- 'referrer' => filter_input(INPUT_SERVER, 'HTTP_REFERER'),
27		- 'user_agent' => filter_input(INPUT_SERVER, 'HTTP_USER_AGENT'),
28		- 'user_ip' => $review['ip_address'],
29		- // 'user_role' => 'administrator',
30		- // 'is_test' => 1,
31		- ];
32		- foreach ($_SERVER as $key => $value) {
33		- if (is_array($value) \|\| in_array($key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'])) {
34		- continue;
35		- }
36		- $submission[$key] = $value;
37		- }
38		- return $this->check(apply_filters('site-reviews/akismet/submission', $submission, $review));
39		- }
	10	+ /**
	11	+ * @return bool
	12	+ */
	13	+ public function isSpam(array $review)
	14	+ {
	15	+ if (!$this->isActive()) {
	16	+ return false;
	17	+ }
	18	+ $submission = [
	19	+ 'blog' => glsr(OptionManager::class)->getWP('home'),
	20	+ 'blog_charset' => glsr(OptionManager::class)->getWP('blog_charset', 'UTF-8'),
	21	+ 'blog_lang' => get_locale(),
	22	+ 'comment_author' => $review['name'],
	23	+ 'comment_author_email' => $review['email'],
	24	+ 'comment_content' => $review['title']."\n\n".$review['content'],
	25	+ 'comment_type' => 'review',
	26	+ 'referrer' => filter_input(INPUT_SERVER, 'HTTP_REFERER'),
	27	+ 'user_agent' => filter_input(INPUT_SERVER, 'HTTP_USER_AGENT'),
	28	+ 'user_ip' => $review['ip_address'],
	29	+ // 'user_role' => 'administrator',
	30	+ // 'is_test' => 1,
	31	+ ];
	32	+ foreach ($_SERVER as $key => $value) {
	33	+ if (is_array($value) \|\| in_array($key, ['HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW'])) {
	34	+ continue;
	35	+ }
	36	+ $submission[$key] = $value;
	37	+ }
	38	+ return $this->check(apply_filters('site-reviews/akismet/submission', $submission, $review));
	39	+ }
40	40
41		- /**
42		- * @return bool
43		- */
44		- protected function check(array $submission)
45		- {
46		- $response = AkismetPlugin::http_post($this->buildQuery($submission), 'comment-check');
47		- return apply_filters('site-reviews/akismet/is-spam',
48		- 'true' == $response[1],
49		- $submission,
50		- $response
51		- );
52		- }
	41	+ /**
	42	+ * @return bool
	43	+ */
	44	+ protected function check(array $submission)
	45	+ {
	46	+ $response = AkismetPlugin::http_post($this->buildQuery($submission), 'comment-check');
	47	+ return apply_filters('site-reviews/akismet/is-spam',
	48	+ 'true' == $response[1],
	49	+ $submission,
	50	+ $response
	51	+ );
	52	+ }
53	53
54		- /**
55		- * @return string
56		- */
57		- protected function buildQuery(array $data)
58		- {
59		- $query = [];
60		- foreach ($data as $key => $value) {
61		- if (is_array($value) \|\| is_object($value)) {
62		- continue;
63		- }
64		- if (false === $value) {
65		- $value = '0';
66		- }
67		- $value = trim($value);
68		- if (!strlen($value)) {
69		- continue;
70		- }
71		- $query[] = urlencode($key).'='.urlencode($value);
72		- }
73		- return implode('&', $query);
74		- }
	54	+ /**
	55	+ * @return string
	56	+ */
	57	+ protected function buildQuery(array $data)
	58	+ {
	59	+ $query = [];
	60	+ foreach ($data as $key => $value) {
	61	+ if (is_array($value) \|\| is_object($value)) {
	62	+ continue;
	63	+ }
	64	+ if (false === $value) {
	65	+ $value = '0';
	66	+ }
	67	+ $value = trim($value);
	68	+ if (!strlen($value)) {
	69	+ continue;
	70	+ }
	71	+ $query[] = urlencode($key).'='.urlencode($value);
	72	+ }
	73	+ return implode('&', $query);
	74	+ }
75	75
76		- /**
77		- * @return bool
78		- */
79		- protected function isActive()
80		- {
81		- $check = !glsr(OptionManager::class)->getBool('settings.submissions.akismet')
82		- \|\| !is_callable(['Akismet', 'get_api_key'])
83		- \|\| !is_callable(['Akismet', 'http_post'])
84		- ? false
85		- : (bool) AkismetPlugin::get_api_key();
86		- return apply_filters('site-reviews/akismet/is-active', $check);
87		- }
	76	+ /**
	77	+ * @return bool
	78	+ */
	79	+ protected function isActive()
	80	+ {
	81	+ $check = !glsr(OptionManager::class)->getBool('settings.submissions.akismet')
	82	+ \|\| !is_callable(['Akismet', 'get_api_key'])
	83	+ \|\| !is_callable(['Akismet', 'http_post'])
	84	+ ? false
	85	+ : (bool) AkismetPlugin::get_api_key();
	86	+ return apply_filters('site-reviews/akismet/is-active', $check);
	87	+ }
88	88	}

		@@ -6,54 +6,54 @@
		block discarded – undo
6	6
7	7	class Blacklist
8	8	{
9		- /**
10		- * @return bool
11		- */
12		- public function isBlacklisted(array $review)
13		- {
14		- $target = implode("\n", array_filter([
15		- $review['name'],
16		- $review['content'],
17		- $review['email'],
18		- $review['ip_address'],
19		- $review['title'],
20		- ]));
21		- return (bool) apply_filters('site-reviews/blacklist/is-blacklisted',
22		- $this->check($target),
23		- $review,
24		- $target
25		- );
26		- }
	9	+ /**
	10	+ * @return bool
	11	+ */
	12	+ public function isBlacklisted(array $review)
	13	+ {
	14	+ $target = implode("\n", array_filter([
	15	+ $review['name'],
	16	+ $review['content'],
	17	+ $review['email'],
	18	+ $review['ip_address'],
	19	+ $review['title'],
	20	+ ]));
	21	+ return (bool) apply_filters('site-reviews/blacklist/is-blacklisted',
	22	+ $this->check($target),
	23	+ $review,
	24	+ $target
	25	+ );
	26	+ }
27	27
28		- /**
29		- * @param string $target
30		- * @return bool
31		- */
32		- protected function check($target)
33		- {
34		- $blacklist = $this->getBlacklist();
35		- if (empty($blacklist)) {
36		- return false;
37		- }
38		- $lines = explode("\n", $blacklist);
39		- foreach ((array) $lines as $line) {
40		- $line = trim($line);
41		- if (empty($line) \|\| 256 < strlen($line)) {
42		- continue;
43		- }
44		- $pattern = sprintf('#%s#i', preg_quote($line, '#'));
45		- if (preg_match($pattern, $target)) {
46		- return true;
47		- }
48		- }
49		- return false;
50		- }
	28	+ /**
	29	+ * @param string $target
	30	+ * @return bool
	31	+ */
	32	+ protected function check($target)
	33	+ {
	34	+ $blacklist = $this->getBlacklist();
	35	+ if (empty($blacklist)) {
	36	+ return false;
	37	+ }
	38	+ $lines = explode("\n", $blacklist);
	39	+ foreach ((array) $lines as $line) {
	40	+ $line = trim($line);
	41	+ if (empty($line) \|\| 256 < strlen($line)) {
	42	+ continue;
	43	+ }
	44	+ $pattern = sprintf('#%s#i', preg_quote($line, '#'));
	45	+ if (preg_match($pattern, $target)) {
	46	+ return true;
	47	+ }
	48	+ }
	49	+ return false;
	50	+ }
51	51
52		- protected function getBlacklist()
53		- {
54		- $option = glsr(OptionManager::class)->get('settings.submissions.blacklist.integration');
55		- return $option == 'comments'
56		- ? trim(glsr(OptionManager::class)->getWP('blacklist_keys'))
57		- : trim(glsr(OptionManager::class)->get('settings.submissions.blacklist.entries'));
58		- }
	52	+ protected function getBlacklist()
	53	+ {
	54	+ $option = glsr(OptionManager::class)->get('settings.submissions.blacklist.integration');
	55	+ return $option == 'comments'
	56	+ ? trim(glsr(OptionManager::class)->getWP('blacklist_keys'))
	57	+ : trim(glsr(OptionManager::class)->get('settings.submissions.blacklist.entries'));
	58	+ }
59	59	}

		@@ -8,164 +8,164 @@
		block discarded – undo
8	8
9	9	class Notification
10	10	{
11		- /**
12		- * @var bool
13		- */
14		- protected $email;
	11	+ /**
	12	+ * @var bool
	13	+ */
	14	+ protected $email;
15	15
16		- /**
17		- * @var Review
18		- */
19		- protected $review;
	16	+ /**
	17	+ * @var Review
	18	+ */
	19	+ protected $review;
20	20
21		- /**
22		- * @var bool
23		- */
24		- protected $slack;
	21	+ /**
	22	+ * @var bool
	23	+ */
	24	+ protected $slack;
25	25
26		- /**
27		- * @var array
28		- */
29		- protected $types;
	26	+ /**
	27	+ * @var array
	28	+ */
	29	+ protected $types;
30	30
31		- public function __construct()
32		- {
33		- $types = glsr(OptionManager::class)->get('settings.general.notifications', []);
34		- $this->email = count(array_intersect(['admin', 'custom'], $types)) > 0;
35		- $this->slack = in_array('slack', $types);
36		- $this->types = $types;
37		- }
	31	+ public function __construct()
	32	+ {
	33	+ $types = glsr(OptionManager::class)->get('settings.general.notifications', []);
	34	+ $this->email = count(array_intersect(['admin', 'custom'], $types)) > 0;
	35	+ $this->slack = in_array('slack', $types);
	36	+ $this->types = $types;
	37	+ }
38	38
39		- /**
40		- * @return void
41		- */
42		- public function send(Review $review)
43		- {
44		- if (empty($this->types)) {
45		- return;
46		- }
47		- $this->review = $review;
48		- $args = [
49		- 'link' => $this->getLink(),
50		- 'title' => $this->getTitle(),
51		- ];
52		- $this->sendToEmail($args);
53		- $this->sendToSlack($args);
54		- }
	39	+ /**
	40	+ * @return void
	41	+ */
	42	+ public function send(Review $review)
	43	+ {
	44	+ if (empty($this->types)) {
	45	+ return;
	46	+ }
	47	+ $this->review = $review;
	48	+ $args = [
	49	+ 'link' => $this->getLink(),
	50	+ 'title' => $this->getTitle(),
	51	+ ];
	52	+ $this->sendToEmail($args);
	53	+ $this->sendToSlack($args);
	54	+ }
55	55
56		- /**
57		- * @return Email
58		- */
59		- protected function buildEmail(array $args)
60		- {
61		- return glsr(Email::class)->compose([
62		- 'to' => $this->getEmailAddresses(),
63		- 'subject' => $args['title'],
64		- 'template' => 'email-notification',
65		- 'template-tags' => [
66		- 'review_author' => $this->review->author ?: __('Anonymous', 'site-reviews'),
67		- 'review_content' => $this->review->content,
68		- 'review_email' => $this->review->email,
69		- 'review_ip' => $this->review->ip_address,
70		- 'review_link' => sprintf('<a href="%1$s">%1$s</a>', $args['link']),
71		- 'review_rating' => $this->review->rating,
72		- 'review_title' => $this->review->title,
73		- ],
74		- ]);
75		- }
	56	+ /**
	57	+ * @return Email
	58	+ */
	59	+ protected function buildEmail(array $args)
	60	+ {
	61	+ return glsr(Email::class)->compose([
	62	+ 'to' => $this->getEmailAddresses(),
	63	+ 'subject' => $args['title'],
	64	+ 'template' => 'email-notification',
	65	+ 'template-tags' => [
	66	+ 'review_author' => $this->review->author ?: __('Anonymous', 'site-reviews'),
	67	+ 'review_content' => $this->review->content,
	68	+ 'review_email' => $this->review->email,
	69	+ 'review_ip' => $this->review->ip_address,
	70	+ 'review_link' => sprintf('<a href="%1$s">%1$s</a>', $args['link']),
	71	+ 'review_rating' => $this->review->rating,
	72	+ 'review_title' => $this->review->title,
	73	+ ],
	74	+ ]);
	75	+ }
76	76
77		- /**
78		- * @return Slack
79		- */
80		- protected function buildSlackNotification(array $args)
81		- {
82		- return glsr(Slack::class)->compose($this->review, [
83		- 'button_url' => $args['link'],
84		- 'fallback' => $this->buildEmail($args)->read('plaintext'),
85		- 'pretext' => $args['title'],
86		- ]);
87		- }
	77	+ /**
	78	+ * @return Slack
	79	+ */
	80	+ protected function buildSlackNotification(array $args)
	81	+ {
	82	+ return glsr(Slack::class)->compose($this->review, [
	83	+ 'button_url' => $args['link'],
	84	+ 'fallback' => $this->buildEmail($args)->read('plaintext'),
	85	+ 'pretext' => $args['title'],
	86	+ ]);
	87	+ }
88	88
89		- /**
90		- * @return array
91		- */
92		- protected function getEmailAddresses()
93		- {
94		- $emails = [];
95		- if (in_array('admin', $this->types)) {
96		- $emails[] = glsr(OptionManager::class)->getWP('admin_email');
97		- }
98		- if (in_array('author', $this->types)) {
99		- $assignedPost = get_post(intval($this->review->assigned_to));
100		- if ($assignedPost instanceof WP_Post) {
101		- $this->email = true;
102		- $emails[] = get_the_author_meta('user_email', intval($assignedPost->post_author));
103		- }
104		- }
105		- if (in_array('custom', $this->types)) {
106		- $customEmails = glsr(OptionManager::class)->get('settings.general.notification_email');
107		- $customEmails = str_replace([' ', ',', ';'], ',', $customEmails);
108		- $customEmails = explode(',', $customEmails);
109		- $emails = array_merge($emails, $customEmails);
110		- }
111		- $emails = array_filter(array_keys(array_flip($emails)));
112		- return apply_filters('site-reviews/notification/emails', $emails, $this->review);
113		- }
	89	+ /**
	90	+ * @return array
	91	+ */
	92	+ protected function getEmailAddresses()
	93	+ {
	94	+ $emails = [];
	95	+ if (in_array('admin', $this->types)) {
	96	+ $emails[] = glsr(OptionManager::class)->getWP('admin_email');
	97	+ }
	98	+ if (in_array('author', $this->types)) {
	99	+ $assignedPost = get_post(intval($this->review->assigned_to));
	100	+ if ($assignedPost instanceof WP_Post) {
	101	+ $this->email = true;
	102	+ $emails[] = get_the_author_meta('user_email', intval($assignedPost->post_author));
	103	+ }
	104	+ }
	105	+ if (in_array('custom', $this->types)) {
	106	+ $customEmails = glsr(OptionManager::class)->get('settings.general.notification_email');
	107	+ $customEmails = str_replace([' ', ',', ';'], ',', $customEmails);
	108	+ $customEmails = explode(',', $customEmails);
	109	+ $emails = array_merge($emails, $customEmails);
	110	+ }
	111	+ $emails = array_filter(array_keys(array_flip($emails)));
	112	+ return apply_filters('site-reviews/notification/emails', $emails, $this->review);
	113	+ }
114	114
115		- /**
116		- * @return string
117		- */
118		- protected function getLink()
119		- {
120		- return admin_url('post.php?post='.$this->review->ID.'&action=edit');
121		- }
	115	+ /**
	116	+ * @return string
	117	+ */
	118	+ protected function getLink()
	119	+ {
	120	+ return admin_url('post.php?post='.$this->review->ID.'&action=edit');
	121	+ }
122	122
123		- /**
124		- * @return string
125		- */
126		- protected function getTitle()
127		- {
128		- $assignedTitle = get_the_title(intval($this->review->assigned_to));
129		- $title = _nx(
130		- 'New %s-star review',
131		- 'New %s-star review of: %s',
132		- intval(empty($assignedTitle)),
133		- 'This string differs depending on whether or not the review has been assigned to a post.',
134		- 'site-reviews'
135		- );
136		- $title = sprintf('[%s] %s',
137		- wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES),
138		- sprintf($title, $this->review->rating, $assignedTitle)
139		- );
140		- return apply_filters('site-reviews/notification/title', $title, $this->review);
141		- }
	123	+ /**
	124	+ * @return string
	125	+ */
	126	+ protected function getTitle()
	127	+ {
	128	+ $assignedTitle = get_the_title(intval($this->review->assigned_to));
	129	+ $title = _nx(
	130	+ 'New %s-star review',
	131	+ 'New %s-star review of: %s',
	132	+ intval(empty($assignedTitle)),
	133	+ 'This string differs depending on whether or not the review has been assigned to a post.',
	134	+ 'site-reviews'
	135	+ );
	136	+ $title = sprintf('[%s] %s',
	137	+ wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES),
	138	+ sprintf($title, $this->review->rating, $assignedTitle)
	139	+ );
	140	+ return apply_filters('site-reviews/notification/title', $title, $this->review);
	141	+ }
142	142
143		- /**
144		- * @return void
145		- */
146		- protected function sendToEmail(array $args)
147		- {
148		- $email = $this->buildEmail($args);
149		- if (empty($email->to)) {
150		- glsr_log()->error('Email notification was not sent (missing email address)');
151		- return;
152		- }
153		- $email->send();
154		- }
	143	+ /**
	144	+ * @return void
	145	+ */
	146	+ protected function sendToEmail(array $args)
	147	+ {
	148	+ $email = $this->buildEmail($args);
	149	+ if (empty($email->to)) {
	150	+ glsr_log()->error('Email notification was not sent (missing email address)');
	151	+ return;
	152	+ }
	153	+ $email->send();
	154	+ }
155	155
156		- /**
157		- * @return void
158		- */
159		- protected function sendToSlack(array $args)
160		- {
161		- if (!$this->slack) {
162		- return;
163		- }
164		- $notification = $this->buildSlackNotification($args);
165		- $result = $notification->send();
166		- if (is_wp_error($result)) {
167		- $notification->review = null;
168		- glsr_log()->error($result->get_error_message())->debug($notification);
169		- }
170		- }
	156	+ /**
	157	+ * @return void
	158	+ */
	159	+ protected function sendToSlack(array $args)
	160	+ {
	161	+ if (!$this->slack) {
	162	+ return;
	163	+ }
	164	+ $notification = $this->buildSlackNotification($args);
	165	+ $result = $notification->send();
	166	+ if (is_wp_error($result)) {
	167	+ $notification->review = null;
	168	+ glsr_log()->error($result->get_error_message())->debug($notification);
	169	+ }
	170	+ }
171	171	}

		@@ -7,84 +7,84 @@
		block discarded – undo
7	7
8	8	class Customization
9	9	{
10		- /**
11		- * @return array
12		- */
13		- public function filterEditorSettings(array $settings)
14		- {
15		- if ($this->isReviewEditable()) {
16		- $settings = [
17		- 'media_buttons' => false,
18		- 'quicktags' => false,
19		- 'textarea_rows' => 12,
20		- 'tinymce' => false,
21		- ];
22		- }
23		- return $settings;
24		- }
	10	+ /**
	11	+ * @return array
	12	+ */
	13	+ public function filterEditorSettings(array $settings)
	14	+ {
	15	+ if ($this->isReviewEditable()) {
	16	+ $settings = [
	17	+ 'media_buttons' => false,
	18	+ 'quicktags' => false,
	19	+ 'textarea_rows' => 12,
	20	+ 'tinymce' => false,
	21	+ ];
	22	+ }
	23	+ return $settings;
	24	+ }
25	25
26		- /**
27		- * @param string $html
28		- * @return string
29		- */
30		- public function filterEditorTextarea($html)
31		- {
32		- if ($this->isReviewEditable()) {
33		- $html = str_replace('<textarea', '<div id="ed_toolbar"></div><textarea', $html);
34		- }
35		- return $html;
36		- }
	26	+ /**
	27	+ * @param string $html
	28	+ * @return string
	29	+ */
	30	+ public function filterEditorTextarea($html)
	31	+ {
	32	+ if ($this->isReviewEditable()) {
	33	+ $html = str_replace('<textarea', '<div id="ed_toolbar"></div><textarea', $html);
	34	+ }
	35	+ return $html;
	36	+ }
37	37
38		- /**
39		- * @return void
40		- */
41		- public function removeAutosave()
42		- {
43		- if (!$this->isReviewEditor() \|\| $this->isReviewEditable()) {
44		- return;
45		- }
46		- wp_deregister_script('autosave');
47		- }
	38	+ /**
	39	+ * @return void
	40	+ */
	41	+ public function removeAutosave()
	42	+ {
	43	+ if (!$this->isReviewEditor() \|\| $this->isReviewEditable()) {
	44	+ return;
	45	+ }
	46	+ wp_deregister_script('autosave');
	47	+ }
48	48
49		- /**
50		- * @return void
51		- */
52		- public function removeMetaBoxes()
53		- {
54		- remove_meta_box('slugdiv', Application::POST_TYPE, 'advanced');
55		- }
	49	+ /**
	50	+ * @return void
	51	+ */
	52	+ public function removeMetaBoxes()
	53	+ {
	54	+ remove_meta_box('slugdiv', Application::POST_TYPE, 'advanced');
	55	+ }
56	56
57		- /**
58		- * @return void
59		- */
60		- public function removePostTypeSupport()
61		- {
62		- if (!$this->isReviewEditor() \|\| $this->isReviewEditable()) {
63		- return;
64		- }
65		- remove_post_type_support(Application::POST_TYPE, 'title');
66		- remove_post_type_support(Application::POST_TYPE, 'editor');
67		- }
	57	+ /**
	58	+ * @return void
	59	+ */
	60	+ public function removePostTypeSupport()
	61	+ {
	62	+ if (!$this->isReviewEditor() \|\| $this->isReviewEditable()) {
	63	+ return;
	64	+ }
	65	+ remove_post_type_support(Application::POST_TYPE, 'title');
	66	+ remove_post_type_support(Application::POST_TYPE, 'editor');
	67	+ }
68	68
69		- /**
70		- * @return bool
71		- */
72		- protected function isReviewEditable()
73		- {
74		- $postId = intval(filter_input(INPUT_GET, 'post'));
75		- return $postId > 0
76		- && 'local' == glsr(Database::class)->get($postId, 'review_type')
77		- && $this->isReviewEditor();
78		- }
	69	+ /**
	70	+ * @return bool
	71	+ */
	72	+ protected function isReviewEditable()
	73	+ {
	74	+ $postId = intval(filter_input(INPUT_GET, 'post'));
	75	+ return $postId > 0
	76	+ && 'local' == glsr(Database::class)->get($postId, 'review_type')
	77	+ && $this->isReviewEditor();
	78	+ }
79	79
80		- /**
81		- * @return bool
82		- */
83		- protected function isReviewEditor()
84		- {
85		- $screen = glsr_current_screen();
86		- return 'post' == $screen->base
87		- && Application::POST_TYPE == $screen->id
88		- && Application::POST_TYPE == $screen->post_type;
89		- }
	80	+ /**
	81	+ * @return bool
	82	+ */
	83	+ protected function isReviewEditor()
	84	+ {
	85	+ $screen = glsr_current_screen();
	86	+ return 'post' == $screen->base
	87	+ && Application::POST_TYPE == $screen->id
	88	+ && Application::POST_TYPE == $screen->post_type;
	89	+ }
90	90	}

		@@ -4,29 +4,29 @@
		block discarded – undo
4	4
5	5	interface MultilingualContract
6	6	{
7		- /**
8		- * @param int\|string $postId
9		- * @return \WP_Post\|void\|null
10		- */
11		- public function getPost($postId);
	7	+ /**
	8	+ * @param int\|string $postId
	9	+ * @return \WP_Post\|void\|null
	10	+ */
	11	+ public function getPost($postId);
12	12
13		- /**
14		- * @return array
15		- */
16		- public function getPostIds(array $postIds);
	13	+ /**
	14	+ * @return array
	15	+ */
	16	+ public function getPostIds(array $postIds);
17	17
18		- /**
19		- * @return bool
20		- */
21		- public function isActive();
	18	+ /**
	19	+ * @return bool
	20	+ */
	21	+ public function isActive();
22	22
23		- /**
24		- * @return bool
25		- */
26		- public function isEnabled();
	23	+ /**
	24	+ * @return bool
	25	+ */
	26	+ public function isEnabled();
27	27
28		- /**
29		- * @return bool
30		- */
31		- public function isSupported();
	28	+ /**
	29	+ * @return bool
	30	+ */
	31	+ public function isSupported();
32	32	}

		@@ -7,40 +7,40 @@
		block discarded – undo
7	7
8	8	class EmailDefaults extends Defaults
9	9	{
10		- /**
11		- * @return array
12		- */
13		- protected function defaults()
14		- {
15		- return [
16		- 'after' => '',
17		- 'attachments' => [],
18		- 'bcc' => '',
19		- 'before' => '',
20		- 'cc' => '',
21		- 'from' => $this->getFromName().' <'.$this->getFromEmail().'>',
22		- 'message' => '',
23		- 'reply-to' => '',
24		- 'subject' => '',
25		- 'template' => '',
26		- 'template-tags' => [],
27		- 'to' => '',
28		- ];
29		- }
	10	+ /**
	11	+ * @return array
	12	+ */
	13	+ protected function defaults()
	14	+ {
	15	+ return [
	16	+ 'after' => '',
	17	+ 'attachments' => [],
	18	+ 'bcc' => '',
	19	+ 'before' => '',
	20	+ 'cc' => '',
	21	+ 'from' => $this->getFromName().' <'.$this->getFromEmail().'>',
	22	+ 'message' => '',
	23	+ 'reply-to' => '',
	24	+ 'subject' => '',
	25	+ 'template' => '',
	26	+ 'template-tags' => [],
	27	+ 'to' => '',
	28	+ ];
	29	+ }
30	30
31		- /**
32		- * @return string
33		- */
34		- protected function getFromEmail()
35		- {
36		- return glsr(OptionManager::class)->getWP('admin_email');
37		- }
	31	+ /**
	32	+ * @return string
	33	+ */
	34	+ protected function getFromEmail()
	35	+ {
	36	+ return glsr(OptionManager::class)->getWP('admin_email');
	37	+ }
38	38
39		- /**
40		- * @return string
41		- */
42		- protected function getFromName()
43		- {
44		- return wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES);
45		- }
	39	+ /**
	40	+ * @return string
	41	+ */
	42	+ protected function getFromName()
	43	+ {
	44	+ return wp_specialchars_decode(glsr(OptionManager::class)->getWP('blogname'), ENT_QUOTES);
	45	+ }
46	46	}

		@@ -18,11 +18,11 @@
		block discarded – undo
18	18	defined('WPINC') \|\| die;
19	19
20	20	if (!class_exists('GL_Plugin_Check_v3')) {
21		- require_once __DIR__.'/activate.php';
	21	+ require_once __DIR__.'/activate.php';
22	22	}
23	23	$check = new GL_Plugin_Check_v3(__FILE__);
24	24	if (!$check->canProceed()) {
25		- return;
	25	+ return;
26	26	}
27	27	require_once __DIR__.'/autoload.php';
28	28	require_once __DIR__.'/compatibility.php';

		@@ -8,112 +8,112 @@
		block discarded – undo
8	8
9	9	class Notice
10	10	{
11		- /**
12		- * @param string $type
13		- * @param string\|array\|WP_Error $message
14		- * @return void
15		- */
16		- public function add($type, $message, array $args = [])
17		- {
18		- if (empty(array_filter([$message, $type]))) {
19		- return;
20		- }
21		- $args['message'] = $message;
22		- $args['type'] = $type;
23		- add_settings_error(Application::ID, '', json_encode($this->normalize($args)));
24		- }
	11	+ /**
	12	+ * @param string $type
	13	+ * @param string\|array\|WP_Error $message
	14	+ * @return void
	15	+ */
	16	+ public function add($type, $message, array $args = [])
	17	+ {
	18	+ if (empty(array_filter([$message, $type]))) {
	19	+ return;
	20	+ }
	21	+ $args['message'] = $message;
	22	+ $args['type'] = $type;
	23	+ add_settings_error(Application::ID, '', json_encode($this->normalize($args)));
	24	+ }
25	25
26		- /**
27		- * @param string\|array\|WP_Error $message
28		- * @return void
29		- */
30		- public function addError($message, array $args = [])
31		- {
32		- $this->add('error', $message, $args);
33		- }
	26	+ /**
	27	+ * @param string\|array\|WP_Error $message
	28	+ * @return void
	29	+ */
	30	+ public function addError($message, array $args = [])
	31	+ {
	32	+ $this->add('error', $message, $args);
	33	+ }
34	34
35		- /**
36		- * @param string\|array\|WP_Error $message
37		- * @return void
38		- */
39		- public function addSuccess($message, array $args = [])
40		- {
41		- $this->add('success', $message, $args);
42		- }
	35	+ /**
	36	+ * @param string\|array\|WP_Error $message
	37	+ * @return void
	38	+ */
	39	+ public function addSuccess($message, array $args = [])
	40	+ {
	41	+ $this->add('success', $message, $args);
	42	+ }
43	43
44		- /**
45		- * @param string\|array\|WP_Error $message
46		- * @return void
47		- */
48		- public function addWarning($message, array $args = [])
49		- {
50		- $this->add('warning', $message, $args);
51		- }
	44	+ /**
	45	+ * @param string\|array\|WP_Error $message
	46	+ * @return void
	47	+ */
	48	+ public function addWarning($message, array $args = [])
	49	+ {
	50	+ $this->add('warning', $message, $args);
	51	+ }
52	52
53		- /**
54		- * @return static
55		- */
56		- public function clear()
57		- {
58		- global $wp_settings_errors;
59		- $wp_settings_errors = [];
60		- delete_transient('settings_errors');
61		- return $this;
62		- }
	53	+ /**
	54	+ * @return static
	55	+ */
	56	+ public function clear()
	57	+ {
	58	+ global $wp_settings_errors;
	59	+ $wp_settings_errors = [];
	60	+ delete_transient('settings_errors');
	61	+ return $this;
	62	+ }
63	63
64		- /**
65		- * @return string
66		- */
67		- public function get()
68		- {
69		- $notices = array_map('unserialize',
70		- array_unique(array_map('serialize', get_settings_errors(Application::ID)))
71		- );
72		- $notices = array_reduce($notices, function ($carry, $notice) {
73		- return $carry.$this->buildNotice(json_decode($notice['message'], true));
74		- });
75		- return apply_filters('site-reviews/notices', $notices);
76		- }
	64	+ /**
	65	+ * @return string
	66	+ */
	67	+ public function get()
	68	+ {
	69	+ $notices = array_map('unserialize',
	70	+ array_unique(array_map('serialize', get_settings_errors(Application::ID)))
	71	+ );
	72	+ $notices = array_reduce($notices, function ($carry, $notice) {
	73	+ return $carry.$this->buildNotice(json_decode($notice['message'], true));
	74	+ });
	75	+ return apply_filters('site-reviews/notices', $notices);
	76	+ }
77	77
78		- /**
79		- * @return string
80		- */
81		- protected function buildNotice(array $args)
82		- {
83		- $messages = array_reduce($args['messages'], function ($carry, $message) {
84		- return $carry.glsr(Builder::class)->p($message);
85		- });
86		- $class = 'notice notice-'.$args['type'];
87		- if ($args['inline']) {
88		- $class.= ' inline';
89		- }
90		- if ($args['dismissible']) {
91		- $class.= ' is-dismissible';
92		- }
93		- return glsr(Builder::class)->div($messages, [
94		- 'class' => $class,
95		- ]);
96		- }
	78	+ /**
	79	+ * @return string
	80	+ */
	81	+ protected function buildNotice(array $args)
	82	+ {
	83	+ $messages = array_reduce($args['messages'], function ($carry, $message) {
	84	+ return $carry.glsr(Builder::class)->p($message);
	85	+ });
	86	+ $class = 'notice notice-'.$args['type'];
	87	+ if ($args['inline']) {
	88	+ $class.= ' inline';
	89	+ }
	90	+ if ($args['dismissible']) {
	91	+ $class.= ' is-dismissible';
	92	+ }
	93	+ return glsr(Builder::class)->div($messages, [
	94	+ 'class' => $class,
	95	+ ]);
	96	+ }
97	97
98		- /**
99		- * @return array
100		- */
101		- protected function normalize(array $args)
102		- {
103		- $defaults = [
104		- 'dismissible' => true,
105		- 'inline' => true,
106		- 'message' => '',
107		- 'type' => '',
108		- ];
109		- $args = shortcode_atts($defaults, $args);
110		- if (!in_array($args['type'], ['error', 'warning', 'success'])) {
111		- $args['type'] = 'success';
112		- }
113		- $args['messages'] = is_wp_error($args['message'])
114		- ? (array) $args['message']->get_error_message()
115		- : (array) $args['message'];
116		- unset($args['message']);
117		- return $args;
118		- }
	98	+ /**
	99	+ * @return array
	100	+ */
	101	+ protected function normalize(array $args)
	102	+ {
	103	+ $defaults = [
	104	+ 'dismissible' => true,
	105	+ 'inline' => true,
	106	+ 'message' => '',
	107	+ 'type' => '',
	108	+ ];
	109	+ $args = shortcode_atts($defaults, $args);
	110	+ if (!in_array($args['type'], ['error', 'warning', 'success'])) {
	111	+ $args['type'] = 'success';
	112	+ }
	113	+ $args['messages'] = is_wp_error($args['message'])
	114	+ ? (array) $args['message']->get_error_message()
	115	+ : (array) $args['message'];
	116	+ unset($args['message']);
	117	+ return $args;
	118	+ }
119	119	}

pryley / site-reviews

Push — master ( 7f1e4d...e207fc )

Status

Category

Indentation +20 added lines, -20 removed lines patch added patch discarded remove patch

Indentation +75 added lines, -75 removed lines patch added patch discarded remove patch

Indentation +48 added lines, -48 removed lines patch added patch discarded remove patch

Indentation +148 added lines, -148 removed lines patch added patch discarded remove patch

Indentation +74 added lines, -74 removed lines patch added patch discarded remove patch

Indentation +21 added lines, -21 removed lines patch added patch discarded remove patch

Indentation +34 added lines, -34 removed lines patch added patch discarded remove patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +101 added lines, -101 removed lines patch added patch discarded remove patch