pryley / site-reviews

plugin/Modules/Schema/LocalBusiness.php

Indentation +14 added lines, -14 removed lines

@@ -15,19 +15,19 @@
  */
 class LocalBusiness extends BaseType
 {
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $allowed = [
-        'branchOf', 'currenciesAccepted', 'openingHours', 'paymentAccepted', 'priceRange',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $allowed = [
+		'branchOf', 'currenciesAccepted', 'openingHours', 'paymentAccepted', 'priceRange',
+	];
 
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $parents = [
-        'Organization', 'Place',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $parents = [
+		'Organization', 'Place',
+	];
 }

plugin/Modules/Schema/Person.php

Indentation +22 added lines, -22 removed lines

@@ -64,27 +64,27 @@
  */
 class Person extends BaseType
 {
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $allowed = [
-        'additionalName', 'address', 'affiliation', 'alumniOf', 'award', 'awards', 'birthDate',
-        'birthPlace', 'brand', 'children', 'colleague', 'colleagues', 'contactPoint',
-        'contactPoints', 'deathDate', 'deathPlace', 'duns', 'email', 'familyName', 'faxNumber',
-        'follows', 'funder', 'gender', 'givenName', 'globalLocationNumber', 'hasOfferCatalog',
-        'hasPOS', 'height', 'homeLocation', 'honorificPrefix', 'honorificSuffix', 'isicV4',
-        'jobTitle', 'knows', 'makesOffer', 'memberOf', 'naics', 'nationality', 'netWorth', 'owns',
-        'parent', 'parents', 'performerIn', 'publishingPrinciples', 'relatedTo', 'seeks', 'sibling',
-        'siblings', 'sponsor', 'spouse', 'taxID', 'telephone', 'vatID', 'weight', 'workLocation',
-        'worksFor',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $allowed = [
+		'additionalName', 'address', 'affiliation', 'alumniOf', 'award', 'awards', 'birthDate',
+		'birthPlace', 'brand', 'children', 'colleague', 'colleagues', 'contactPoint',
+		'contactPoints', 'deathDate', 'deathPlace', 'duns', 'email', 'familyName', 'faxNumber',
+		'follows', 'funder', 'gender', 'givenName', 'globalLocationNumber', 'hasOfferCatalog',
+		'hasPOS', 'height', 'homeLocation', 'honorificPrefix', 'honorificSuffix', 'isicV4',
+		'jobTitle', 'knows', 'makesOffer', 'memberOf', 'naics', 'nationality', 'netWorth', 'owns',
+		'parent', 'parents', 'performerIn', 'publishingPrinciples', 'relatedTo', 'seeks', 'sibling',
+		'siblings', 'sponsor', 'spouse', 'taxID', 'telephone', 'vatID', 'weight', 'workLocation',
+		'worksFor',
+	];
 
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $parents = [
-        'Thing',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $parents = [
+		'Thing',
+	];
 }

plugin/Modules/Schema/Offer.php

Indentation +21 added lines, -21 removed lines

@@ -54,26 +54,26 @@
  */
 class Offer extends BaseType
 {
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $allowed = [
-        'acceptedPaymentMethod', 'addOn', 'advanceBookingRequirement', 'aggregateRating',
-        'areaServed', 'availability', 'availabilityEnds', 'availabilityStarts', 'availableAtOrFrom',
-        'availableDeliveryMethod', 'businessFunction', 'category', 'deliveryLeadTime',
-        'eligibleCustomerType', 'eligibleDuration', 'eligibleQuantity', 'eligibleRegion',
-        'eligibleTransactionVolume', 'gtin12', 'gtin13', 'gtin14', 'gtin8', 'includesObject',
-        'ineligibleRegion', 'inventoryLevel', 'itemCondition', 'itemOffered', 'mpn', 'price',
-        'priceCurrency', 'priceSpecification', 'priceValidUntil', 'review', 'reviews', 'seller',
-        'serialNumber', 'sku', 'validFrom', 'validThrough', 'warranty',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $allowed = [
+		'acceptedPaymentMethod', 'addOn', 'advanceBookingRequirement', 'aggregateRating',
+		'areaServed', 'availability', 'availabilityEnds', 'availabilityStarts', 'availableAtOrFrom',
+		'availableDeliveryMethod', 'businessFunction', 'category', 'deliveryLeadTime',
+		'eligibleCustomerType', 'eligibleDuration', 'eligibleQuantity', 'eligibleRegion',
+		'eligibleTransactionVolume', 'gtin12', 'gtin13', 'gtin14', 'gtin8', 'includesObject',
+		'ineligibleRegion', 'inventoryLevel', 'itemCondition', 'itemOffered', 'mpn', 'price',
+		'priceCurrency', 'priceSpecification', 'priceValidUntil', 'review', 'reviews', 'seller',
+		'serialNumber', 'sku', 'validFrom', 'validThrough', 'warranty',
+	];
 
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $parents = [
-        'Intangible',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $parents = [
+		'Intangible',
+	];
 }

plugin/Modules/Schema/CreativeWork.php

Indentation +29 added lines, -29 removed lines

@@ -92,34 +92,34 @@
  */
 class CreativeWork extends BaseType
 {
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $allowed = [
-        'about', 'accessMode', 'accessModeSufficient', 'accessibilityAPI', 'accessibilityControl',
-        'accessibilityFeature', 'accessibilityHazard', 'accessibilitySummary', 'accountablePerson',
-        'aggregateRating', 'alternativeHeadline', 'associatedMedia', 'audience', 'audio', 'author',
-        'award', 'awards', 'character', 'citation', 'comment', 'commentCount', 'contentLocation',
-        'contentRating', 'contributor', 'copyrightHolder', 'copyrightYear', 'creator',
-        'dateCreated', 'dateModified', 'datePublished', 'discussionUrl', 'editor',
-        'educationalAlignment', 'educationalUse', 'encoding', 'encodingFormat', 'encodings',
-        'exampleOfWork', 'expires', 'fileFormat', 'funder', 'genre', 'hasPart', 'headline',
-        'inLanguage', 'interactionStatistic', 'interactivityType', 'isAccessibleForFree',
-        'isBasedOn', 'isBasedOnUrl', 'isFamilyFriendly', 'isPartOf', 'keywords',
-        'learningResourceType', 'license', 'locationCreated', 'mainEntity', 'material', 'mentions',
-        'offers', 'position', 'producer', 'provider', 'publication', 'publisher',
-        'publishingPrinciples', 'recordedAt', 'releasedEvent', 'review', 'reviews', 'schemaVersion',
-        'sourceOrganization', 'spatialCoverage', 'sponsor', 'temporalCoverage', 'text',
-        'thumbnailUrl', 'timeRequired', 'translator', 'typicalAgeRange', 'version', 'video',
-        'workExample',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $allowed = [
+		'about', 'accessMode', 'accessModeSufficient', 'accessibilityAPI', 'accessibilityControl',
+		'accessibilityFeature', 'accessibilityHazard', 'accessibilitySummary', 'accountablePerson',
+		'aggregateRating', 'alternativeHeadline', 'associatedMedia', 'audience', 'audio', 'author',
+		'award', 'awards', 'character', 'citation', 'comment', 'commentCount', 'contentLocation',
+		'contentRating', 'contributor', 'copyrightHolder', 'copyrightYear', 'creator',
+		'dateCreated', 'dateModified', 'datePublished', 'discussionUrl', 'editor',
+		'educationalAlignment', 'educationalUse', 'encoding', 'encodingFormat', 'encodings',
+		'exampleOfWork', 'expires', 'fileFormat', 'funder', 'genre', 'hasPart', 'headline',
+		'inLanguage', 'interactionStatistic', 'interactivityType', 'isAccessibleForFree',
+		'isBasedOn', 'isBasedOnUrl', 'isFamilyFriendly', 'isPartOf', 'keywords',
+		'learningResourceType', 'license', 'locationCreated', 'mainEntity', 'material', 'mentions',
+		'offers', 'position', 'producer', 'provider', 'publication', 'publisher',
+		'publishingPrinciples', 'recordedAt', 'releasedEvent', 'review', 'reviews', 'schemaVersion',
+		'sourceOrganization', 'spatialCoverage', 'sponsor', 'temporalCoverage', 'text',
+		'thumbnailUrl', 'timeRequired', 'translator', 'typicalAgeRange', 'version', 'video',
+		'workExample',
+	];
 
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $parents = [
-        'Thing',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $parents = [
+		'Thing',
+	];
 }

plugin/Modules/Schema/AggregateOffer.php

Indentation +14 added lines, -14 removed lines

@@ -13,19 +13,19 @@
  */
 class AggregateOffer extends BaseType
 {
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $allowed = [
-        'highPrice', 'lowPrice', 'offerCount', 'offers',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $allowed = [
+		'highPrice', 'lowPrice', 'offerCount', 'offers',
+	];
 
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $parents = [
-        'Offer',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $parents = [
+		'Offer',
+	];
 }

plugin/Modules/Schema/BaseType.php

Indentation +221 added lines, -221 removed lines

@@ -12,248 +12,248 @@
 
 abstract class BaseType implements ArrayAccess, JsonSerializable, Type
 {
-    /**
-     * @var array
-     */
-    public $allowed = [];
+	/**
+	 * @var array
+	 */
+	public $allowed = [];
 
-    /**
-     * @var array
-     */
-    public $parents = [];
+	/**
+	 * @var array
+	 */
+	public $parents = [];
 
-    /**
-     * @var array
-     */
-    protected $properties = [];
+	/**
+	 * @var array
+	 */
+	protected $properties = [];
 
-    /**
-     * @var string
-     */
-    protected $type;
+	/**
+	 * @var string
+	 */
+	protected $type;
 
-    /**
-     * @param string $method
-     * @return static
-     */
-    public function __call($method, array $arguments)
-    {
-        return $this->setProperty($method, glsr_get($arguments, 0));
-    }
+	/**
+	 * @param string $method
+	 * @return static
+	 */
+	public function __call($method, array $arguments)
+	{
+		return $this->setProperty($method, glsr_get($arguments, 0));
+	}
 
-    /**
-     * @param string $type
-     */
-    public function __construct($type = null)
-    {
-        $this->type = !is_string($type)
-            ? (new ReflectionClass($this))->getShortName()
-            : $type;
-        $this->setAllowedProperties();
-    }
+	/**
+	 * @param string $type
+	 */
+	public function __construct($type = null)
+	{
+		$this->type = !is_string($type)
+			? (new ReflectionClass($this))->getShortName()
+			: $type;
+		$this->setAllowedProperties();
+	}
 
-    /**
-     * @return string
-     */
-    public function __toString()
-    {
-        return $this->toScript();
-    }
+	/**
+	 * @return string
+	 */
+	public function __toString()
+	{
+		return $this->toScript();
+	}
 
-    /**
-     * @return static
-     */
-    public function addProperties(array $properties)
-    {
-        foreach ($properties as $property => $value) {
-            $this->setProperty($property, $value);
-        }
-        return $this;
-    }
+	/**
+	 * @return static
+	 */
+	public function addProperties(array $properties)
+	{
+		foreach ($properties as $property => $value) {
+			$this->setProperty($property, $value);
+		}
+		return $this;
+	}
 
-    /**
-     * @return string
-     */
-    public function getContext()
-    {
-        return 'https://schema.org';
-    }
+	/**
+	 * @return string
+	 */
+	public function getContext()
+	{
+		return 'https://schema.org';
+	}
 
-    /**
-     * @return array
-     */
-    public function getProperties()
-    {
-        return $this->properties;
-    }
+	/**
+	 * @return array
+	 */
+	public function getProperties()
+	{
+		return $this->properties;
+	}
 
-    /**
-     * @param string $property
-     * @param mixed $default
-     * @return mixed
-     */
-    public function getProperty($property, $default = null)
-    {
-        return glsr_get($this->properties, $property, $default);
-    }
+	/**
+	 * @param string $property
+	 * @param mixed $default
+	 * @return mixed
+	 */
+	public function getProperty($property, $default = null)
+	{
+		return glsr_get($this->properties, $property, $default);
+	}
 
-    /**
-     * @return string
-     */
-    public function getType()
-    {
-        return $this->type;
-    }
+	/**
+	 * @return string
+	 */
+	public function getType()
+	{
+		return $this->type;
+	}
 
-    /**
-     * @param bool $condition
-     * @param mixed $callback
-     * @return static
-     */
-    public function doIf($condition, $callback)
-    {
-        if ($condition) {
-            $callback($this);
-        }
-        return $this;
-    }
+	/**
+	 * @param bool $condition
+	 * @param mixed $callback
+	 * @return static
+	 */
+	public function doIf($condition, $callback)
+	{
+		if ($condition) {
+			$callback($this);
+		}
+		return $this;
+	}
 
-    /**
-     * @return array
-     */
-    public function jsonSerialize()
-    {
-        return $this->toArray();
-    }
+	/**
+	 * @return array
+	 */
+	public function jsonSerialize()
+	{
+		return $this->toArray();
+	}
 
-    /**
-     * @param mixed $offset
-     * @return bool
-     */
-    public function offsetExists($offset)
-    {
-        return array_key_exists($offset, $this->properties);
-    }
+	/**
+	 * @param mixed $offset
+	 * @return bool
+	 */
+	public function offsetExists($offset)
+	{
+		return array_key_exists($offset, $this->properties);
+	}
 
-    /**
-     * @param string $offset
-     * @return mixed
-     */
-    public function offsetGet($offset)
-    {
-        return $this->getProperty($offset);
-    }
+	/**
+	 * @param string $offset
+	 * @return mixed
+	 */
+	public function offsetGet($offset)
+	{
+		return $this->getProperty($offset);
+	}
 
-    /**
-     * @param string $offset
-     * @param mixed $value
-     * @return void
-     */
-    public function offsetSet($offset, $value)
-    {
-        $this->setProperty($offset, $value);
-    }
+	/**
+	 * @param string $offset
+	 * @param mixed $value
+	 * @return void
+	 */
+	public function offsetSet($offset, $value)
+	{
+		$this->setProperty($offset, $value);
+	}
 
-    /**
-     * @param string $offset
-     * @return void
-     */
-    public function offsetUnset($offset)
-    {
-        unset($this->properties[$offset]);
-    }
+	/**
+	 * @param string $offset
+	 * @return void
+	 */
+	public function offsetUnset($offset)
+	{
+		unset($this->properties[$offset]);
+	}
 
-    /**
-     * @param string $property
-     * @param mixed $value
-     * @return static
-     */
-    public function setProperty($property, $value)
-    {
-        if (!in_array($property, $this->allowed)
-            && 'UnknownType' != (new ReflectionClass($this))->getShortName()) {
-            glsr_log()->warning($this->getType().' does not allow the "'.$property.'" property');
-            return $this;
-        }
-        $this->properties[$property] = $value;
-        return $this;
-    }
+	/**
+	 * @param string $property
+	 * @param mixed $value
+	 * @return static
+	 */
+	public function setProperty($property, $value)
+	{
+		if (!in_array($property, $this->allowed)
+			&& 'UnknownType' != (new ReflectionClass($this))->getShortName()) {
+			glsr_log()->warning($this->getType().' does not allow the "'.$property.'" property');
+			return $this;
+		}
+		$this->properties[$property] = $value;
+		return $this;
+	}
 
-    /**
-     * @return array
-     */
-    public function toArray()
-    {
-        return [
-            '@context' => $this->getContext(),
-            '@type' => $this->getType(),
-        ] + $this->serializeProperty($this->getProperties());
-    }
+	/**
+	 * @return array
+	 */
+	public function toArray()
+	{
+		return [
+			'@context' => $this->getContext(),
+			'@type' => $this->getType(),
+		] + $this->serializeProperty($this->getProperties());
+	}
 
-    /**
-     * @return string
-     */
-    public function toScript()
-    {
-        return sprintf('<script type="application/ld+json">%s</script>',
-            json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
-        );
-    }
+	/**
+	 * @return string
+	 */
+	public function toScript()
+	{
+		return sprintf('<script type="application/ld+json">%s</script>',
+			json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
+		);
+	}
 
-    /**
-     * @param array|null $parents
-     * @return array
-     */
-    protected function getParents($parents = null)
-    {
-        if (!isset($parents)) {
-            $parents = $this->parents;
-        }
-        $newParents = $parents;
-        foreach ($parents as $parent) {
-            $parentClass = glsr(Helper::class)->buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
-                continue;
-            }
-            $newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
-        }
-        return array_values(array_unique($newParents));
-    }
+	/**
+	 * @param array|null $parents
+	 * @return array
+	 */
+	protected function getParents($parents = null)
+	{
+		if (!isset($parents)) {
+			$parents = $this->parents;
+		}
+		$newParents = $parents;
+		foreach ($parents as $parent) {
+			$parentClass = glsr(Helper::class)->buildClassName($parent, __NAMESPACE__);
+			if (!class_exists($parentClass)) {
+				continue;
+			}
+			$newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
+		}
+		return array_values(array_unique($newParents));
+	}
 
-    /**
-     * @return void
-     */
-    protected function setAllowedProperties()
-    {
-        $parents = $this->getParents();
-        foreach ($parents as $parent) {
-            $parentClass = glsr(Helper::class)->buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
-                continue;
-            }
-            $this->allowed = array_values(array_unique(array_merge((new $parentClass())->allowed, $this->allowed)));
-        }
-    }
+	/**
+	 * @return void
+	 */
+	protected function setAllowedProperties()
+	{
+		$parents = $this->getParents();
+		foreach ($parents as $parent) {
+			$parentClass = glsr(Helper::class)->buildClassName($parent, __NAMESPACE__);
+			if (!class_exists($parentClass)) {
+				continue;
+			}
+			$this->allowed = array_values(array_unique(array_merge((new $parentClass())->allowed, $this->allowed)));
+		}
+	}
 
-    /**
-     * @param mixed $property
-     * @return array|string
-     */
-    protected function serializeProperty($property)
-    {
-        if (is_array($property)) {
-            return array_map([$this, 'serializeProperty'], $property);
-        }
-        if ($property instanceof Type) {
-            $property = $property->toArray();
-            unset($property['@context']);
-        }
-        if ($property instanceof DateTimeInterface) {
-            $property = $property->format(DateTime::ATOM);
-        }
-        if (is_object($property)) {
-            throw new InvalidProperty();
-        }
-        return $property;
-    }
+	/**
+	 * @param mixed $property
+	 * @return array|string
+	 */
+	protected function serializeProperty($property)
+	{
+		if (is_array($property)) {
+			return array_map([$this, 'serializeProperty'], $property);
+		}
+		if ($property instanceof Type) {
+			$property = $property->toArray();
+			unset($property['@context']);
+		}
+		if ($property instanceof DateTimeInterface) {
+			$property = $property->format(DateTime::ATOM);
+		}
+		if (is_object($property)) {
+			throw new InvalidProperty();
+		}
+		return $property;
+	}
 }

plugin/Modules/Schema/Rating.php

Indentation +14 added lines, -14 removed lines

@@ -12,19 +12,19 @@
  */
 class Rating extends BaseType
 {
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $allowed = [
-        'author', 'bestRating', 'ratingValue', 'worstRating',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $allowed = [
+		'author', 'bestRating', 'ratingValue', 'worstRating',
+	];
 
-    /**
-     * @var array
-     * @see http://schema.org/{property_name}
-     */
-    public $parents = [
-        'Intangible',
-    ];
+	/**
+	 * @var array
+	 * @see http://schema.org/{property_name}
+	 */
+	public $parents = [
+		'Intangible',
+	];
 }

plugin/Modules/Validator.php

Indentation +274 added lines, -274 removed lines

@@ -12,302 +12,302 @@
  */
 class Validator
 {
-    use ValidationRules;
+	use ValidationRules;
 
-    /**
-     * @var array
-     */
-    public $errors = [];
+	/**
+	 * @var array
+	 */
+	public $errors = [];
 
-    /**
-     * The data under validation.
-     * @var array
-     */
-    protected $data = [];
+	/**
+	 * The data under validation.
+	 * @var array
+	 */
+	protected $data = [];
 
-    /**
-     * The failed validation rules.
-     * @var array
-     */
-    protected $failedRules = [];
+	/**
+	 * The failed validation rules.
+	 * @var array
+	 */
+	protected $failedRules = [];
 
-    /**
-     * The rules to be applied to the data.
-     * @var array
-     */
-    protected $rules = [];
+	/**
+	 * The rules to be applied to the data.
+	 * @var array
+	 */
+	protected $rules = [];
 
-    /**
-     * The size related validation rules.
-     * @var array
-     */
-    protected $sizeRules = [
-        'Between', 'Max', 'Min',
-    ];
+	/**
+	 * The size related validation rules.
+	 * @var array
+	 */
+	protected $sizeRules = [
+		'Between', 'Max', 'Min',
+	];
 
-    /**
-     * The validation rules that imply the field is required.
-     * @var array
-     */
-    protected $implicitRules = [
-        'Required',
-    ];
+	/**
+	 * The validation rules that imply the field is required.
+	 * @var array
+	 */
+	protected $implicitRules = [
+		'Required',
+	];
 
-    /**
-     * The numeric related validation rules.
-     * @var array
-     */
-    protected $numericRules = [
-        'Number',
-    ];
+	/**
+	 * The numeric related validation rules.
+	 * @var array
+	 */
+	protected $numericRules = [
+		'Number',
+	];
 
-    /**
-     * Run the validator's rules against its data.
-     * @param mixed $data
-     * @return array
-     */
-    public function validate($data, array $rules = [])
-    {
-        $this->normalizeData($data);
-        $this->setRules($rules);
-        foreach ($this->rules as $attribute => $rules) {
-            foreach ($rules as $rule) {
-                $this->validateAttribute($attribute, $rule);
-                if ($this->shouldStopValidating($attribute)) {
-                    break;
-                }
-            }
-        }
-        return $this->errors;
-    }
+	/**
+	 * Run the validator's rules against its data.
+	 * @param mixed $data
+	 * @return array
+	 */
+	public function validate($data, array $rules = [])
+	{
+		$this->normalizeData($data);
+		$this->setRules($rules);
+		foreach ($this->rules as $attribute => $rules) {
+			foreach ($rules as $rule) {
+				$this->validateAttribute($attribute, $rule);
+				if ($this->shouldStopValidating($attribute)) {
+					break;
+				}
+			}
+		}
+		return $this->errors;
+	}
 
-    /**
-     * Validate a given attribute against a rule.
-     * @param string $attribute
-     * @param string $rule
-     * @return void
-     * @throws BadMethodCallException
-     */
-    public function validateAttribute($attribute, $rule)
-    {
-        list($rule, $parameters) = $this->parseRule($rule);
-        if ('' == $rule) {
-            return;
-        }
-        $value = $this->getValue($attribute);
-        if (!method_exists($this, $method = 'validate'.$rule)) {
-            throw new BadMethodCallException("Method [$method] does not exist.");
-        }
-        if (!$this->$method($value, $attribute, $parameters)) {
-            $this->addFailure($attribute, $rule, $parameters);
-        }
-    }
+	/**
+	 * Validate a given attribute against a rule.
+	 * @param string $attribute
+	 * @param string $rule
+	 * @return void
+	 * @throws BadMethodCallException
+	 */
+	public function validateAttribute($attribute, $rule)
+	{
+		list($rule, $parameters) = $this->parseRule($rule);
+		if ('' == $rule) {
+			return;
+		}
+		$value = $this->getValue($attribute);
+		if (!method_exists($this, $method = 'validate'.$rule)) {
+			throw new BadMethodCallException("Method [$method] does not exist.");
+		}
+		if (!$this->$method($value, $attribute, $parameters)) {
+			$this->addFailure($attribute, $rule, $parameters);
+		}
+	}
 
-    /**
-     * Add an error message to the validator's collection of errors.
-     * @param string $attribute
-     * @param string $rule
-     * @return void
-     */
-    protected function addError($attribute, $rule, array $parameters)
-    {
-        $message = $this->getMessage($attribute, $rule, $parameters);
-        $this->errors[$attribute][] = $message;
-    }
+	/**
+	 * Add an error message to the validator's collection of errors.
+	 * @param string $attribute
+	 * @param string $rule
+	 * @return void
+	 */
+	protected function addError($attribute, $rule, array $parameters)
+	{
+		$message = $this->getMessage($attribute, $rule, $parameters);
+		$this->errors[$attribute][] = $message;
+	}
 
-    /**
-     * Add a failed rule and error message to the collection.
-     * @param string $attribute
-     * @param string $rule
-     * @return void
-     */
-    protected function addFailure($attribute, $rule, array $parameters)
-    {
-        $this->addError($attribute, $rule, $parameters);
-        $this->failedRules[$attribute][$rule] = $parameters;
-    }
+	/**
+	 * Add a failed rule and error message to the collection.
+	 * @param string $attribute
+	 * @param string $rule
+	 * @return void
+	 */
+	protected function addFailure($attribute, $rule, array $parameters)
+	{
+		$this->addError($attribute, $rule, $parameters);
+		$this->failedRules[$attribute][$rule] = $parameters;
+	}
 
-    /**
-     * Get the data type of the given attribute.
-     * @param string $attribute
-     * @return string
-     */
-    protected function getAttributeType($attribute)
-    {
-        return !$this->hasRule($attribute, $this->numericRules)
-            ? 'length'
-            : '';
-    }
+	/**
+	 * Get the data type of the given attribute.
+	 * @param string $attribute
+	 * @return string
+	 */
+	protected function getAttributeType($attribute)
+	{
+		return !$this->hasRule($attribute, $this->numericRules)
+			? 'length'
+			: '';
+	}
 
-    /**
-     * Get the validation message for an attribute and rule.
-     * @param string $attribute
-     * @param string $rule
-     * @return string|null
-     */
-    protected function getMessage($attribute, $rule, array $parameters)
-    {
-        if (in_array($rule, $this->sizeRules)) {
-            return $this->getSizeMessage($attribute, $rule, $parameters);
-        }
-        $lowerRule = glsr(Helper::class)->snakeCase($rule);
-        return $this->translator($lowerRule, $parameters);
-    }
+	/**
+	 * Get the validation message for an attribute and rule.
+	 * @param string $attribute
+	 * @param string $rule
+	 * @return string|null
+	 */
+	protected function getMessage($attribute, $rule, array $parameters)
+	{
+		if (in_array($rule, $this->sizeRules)) {
+			return $this->getSizeMessage($attribute, $rule, $parameters);
+		}
+		$lowerRule = glsr(Helper::class)->snakeCase($rule);
+		return $this->translator($lowerRule, $parameters);
+	}
 
-    /**
-     * Get a rule and its parameters for a given attribute.
-     * @param string $attribute
-     * @param string|array $rules
-     * @return array|null
-     */
-    protected function getRule($attribute, $rules)
-    {
-        if (!array_key_exists($attribute, $this->rules)) {
-            return;
-        }
-        $rules = (array) $rules;
-        foreach ($this->rules[$attribute] as $rule) {
-            list($rule, $parameters) = $this->parseRule($rule);
-            if (in_array($rule, $rules)) {
-                return [$rule, $parameters];
-            }
-        }
-    }
+	/**
+	 * Get a rule and its parameters for a given attribute.
+	 * @param string $attribute
+	 * @param string|array $rules
+	 * @return array|null
+	 */
+	protected function getRule($attribute, $rules)
+	{
+		if (!array_key_exists($attribute, $this->rules)) {
+			return;
+		}
+		$rules = (array) $rules;
+		foreach ($this->rules[$attribute] as $rule) {
+			list($rule, $parameters) = $this->parseRule($rule);
+			if (in_array($rule, $rules)) {
+				return [$rule, $parameters];
+			}
+		}
+	}
 
-    /**
-     * Get the size of an attribute.
-     * @param string $attribute
-     * @param mixed $value
-     * @return mixed
-     */
-    protected function getSize($attribute, $value)
-    {
-        $hasNumeric = $this->hasRule($attribute, $this->numericRules);
-        if (is_numeric($value) && $hasNumeric) {
-            return $value;
-        } elseif (is_array($value)) {
-            return count($value);
-        }
-        return function_exists('mb_strlen')
-            ? mb_strlen($value)
-            : strlen($value);
-    }
+	/**
+	 * Get the size of an attribute.
+	 * @param string $attribute
+	 * @param mixed $value
+	 * @return mixed
+	 */
+	protected function getSize($attribute, $value)
+	{
+		$hasNumeric = $this->hasRule($attribute, $this->numericRules);
+		if (is_numeric($value) && $hasNumeric) {
+			return $value;
+		} elseif (is_array($value)) {
+			return count($value);
+		}
+		return function_exists('mb_strlen')
+			? mb_strlen($value)
+			: strlen($value);
+	}
 
-    /**
-     * Get the proper error message for an attribute and size rule.
-     * @param string $attribute
-     * @param string $rule
-     * @return string|null
-     */
-    protected function getSizeMessage($attribute, $rule, array $parameters)
-    {
-        $type = $this->getAttributeType($attribute);
-        $lowerRule = glsr(Helper::class)->snakeCase($rule.$type);
-        return $this->translator($lowerRule, $parameters);
-    }
+	/**
+	 * Get the proper error message for an attribute and size rule.
+	 * @param string $attribute
+	 * @param string $rule
+	 * @return string|null
+	 */
+	protected function getSizeMessage($attribute, $rule, array $parameters)
+	{
+		$type = $this->getAttributeType($attribute);
+		$lowerRule = glsr(Helper::class)->snakeCase($rule.$type);
+		return $this->translator($lowerRule, $parameters);
+	}
 
-    /**
-     * Get the value of a given attribute.
-     * @param string $attribute
-     * @return mixed
-     */
-    protected function getValue($attribute)
-    {
-        if (isset($this->data[$attribute])) {
-            return $this->data[$attribute];
-        }
-    }
+	/**
+	 * Get the value of a given attribute.
+	 * @param string $attribute
+	 * @return mixed
+	 */
+	protected function getValue($attribute)
+	{
+		if (isset($this->data[$attribute])) {
+			return $this->data[$attribute];
+		}
+	}
 
-    /**
-     * Determine if the given attribute has a rule in the given set.
-     * @param string $attribute
-     * @param string|array $rules
-     * @return bool
-     */
-    protected function hasRule($attribute, $rules)
-    {
-        return !is_null($this->getRule($attribute, $rules));
-    }
+	/**
+	 * Determine if the given attribute has a rule in the given set.
+	 * @param string $attribute
+	 * @param string|array $rules
+	 * @return bool
+	 */
+	protected function hasRule($attribute, $rules)
+	{
+		return !is_null($this->getRule($attribute, $rules));
+	}
 
-    /**
-     * Normalize the provided data to an array.
-     * @param mixed $data
-     * @return void
-     */
-    protected function normalizeData($data)
-    {
-        $this->data = is_object($data)
-            ? get_object_vars($data)
-            : $data;
-    }
+	/**
+	 * Normalize the provided data to an array.
+	 * @param mixed $data
+	 * @return void
+	 */
+	protected function normalizeData($data)
+	{
+		$this->data = is_object($data)
+			? get_object_vars($data)
+			: $data;
+	}
 
-    /**
-     * Parse a parameter list.
-     * @param string $rule
-     * @param string $parameter
-     * @return array
-     */
-    protected function parseParameters($rule, $parameter)
-    {
-        return 'regex' == strtolower($rule)
-            ? [$parameter]
-            : str_getcsv($parameter);
-    }
+	/**
+	 * Parse a parameter list.
+	 * @param string $rule
+	 * @param string $parameter
+	 * @return array
+	 */
+	protected function parseParameters($rule, $parameter)
+	{
+		return 'regex' == strtolower($rule)
+			? [$parameter]
+			: str_getcsv($parameter);
+	}
 
-    /**
-     * Extract the rule name and parameters from a rule.
-     * @param string $rule
-     * @return array
-     */
-    protected function parseRule($rule)
-    {
-        $parameters = [];
-        if (false !== strpos($rule, ':')) {
-            list($rule, $parameter) = explode(':', $rule, 2);
-            $parameters = $this->parseParameters($rule, $parameter);
-        }
-        $rule = glsr(Helper::class)->camelCase($rule);
-        return [$rule, $parameters];
-    }
+	/**
+	 * Extract the rule name and parameters from a rule.
+	 * @param string $rule
+	 * @return array
+	 */
+	protected function parseRule($rule)
+	{
+		$parameters = [];
+		if (false !== strpos($rule, ':')) {
+			list($rule, $parameter) = explode(':', $rule, 2);
+			$parameters = $this->parseParameters($rule, $parameter);
+		}
+		$rule = glsr(Helper::class)->camelCase($rule);
+		return [$rule, $parameters];
+	}
 
-    /**
-     * Set the validation rules.
-     * @return void
-     */
-    protected function setRules(array $rules)
-    {
-        foreach ($rules as $key => $rule) {
-            $rules[$key] = is_string($rule)
-                ? explode('|', $rule)
-                : $rule;
-        }
-        $this->rules = $rules;
-    }
+	/**
+	 * Set the validation rules.
+	 * @return void
+	 */
+	protected function setRules(array $rules)
+	{
+		foreach ($rules as $key => $rule) {
+			$rules[$key] = is_string($rule)
+				? explode('|', $rule)
+				: $rule;
+		}
+		$this->rules = $rules;
+	}
 
-    /**
-     * Check if we should stop further validations on a given attribute.
-     * @param string $attribute
-     * @return bool
-     */
-    protected function shouldStopValidating($attribute)
-    {
-        return $this->hasRule($attribute, $this->implicitRules)
-            && isset($this->failedRules[$attribute])
-            && array_intersect(array_keys($this->failedRules[$attribute]), $this->implicitRules);
-    }
+	/**
+	 * Check if we should stop further validations on a given attribute.
+	 * @param string $attribute
+	 * @return bool
+	 */
+	protected function shouldStopValidating($attribute)
+	{
+		return $this->hasRule($attribute, $this->implicitRules)
+			&& isset($this->failedRules[$attribute])
+			&& array_intersect(array_keys($this->failedRules[$attribute]), $this->implicitRules);
+	}
 
-    /**
-     * Returns a translated message for the attribute.
-     * @param string $key
-     * @param string $attribute
-     * @return void|string
-     */
-    protected function translator($key, array $parameters)
-    {
-        $strings = glsr(ValidationStringsDefaults::class)->defaults();
-        if (isset($strings[$key])) {
-            return $this->replace($strings[$key], $parameters);
-        }
-        return 'error';
-    }
+	/**
+	 * Returns a translated message for the attribute.
+	 * @param string $key
+	 * @param string $attribute
+	 * @return void|string
+	 */
+	protected function translator($key, array $parameters)
+	{
+		$strings = glsr(ValidationStringsDefaults::class)->defaults();
+		if (isset($strings[$key])) {
+			return $this->replace($strings[$key], $parameters);
+		}
+		return 'error';
+	}
 }

plugin/Modules/Blacklist.php

Indentation +40 added lines, -40 removed lines

@@ -6,45 +6,45 @@
 
 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
-        );
-    }
+	/**
+	 * @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
+		);
+	}
 
-    /**
-     * @param string $target
-     * @return bool
-     */
-    protected function check($target)
-    {
-        $blacklist = trim(glsr(OptionManager::class)->get('settings.submissions.blacklist.entries'));
-        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 = trim(glsr(OptionManager::class)->get('settings.submissions.blacklist.entries'));
+		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;
+	}
 }