Merge pull request #92 from qandidate-labs/php-cs-fixer

Apply coding standards
qandidate-labs · Dec 20, 2023 · 27fd02e · 27fd02e
2 parents 82e1b31 + 0843312
commit 27fd02e
Show file tree

Hide file tree

Showing 17 changed files with 2 additions and 73 deletions.
diff --git a/lib/Qandidate/Toggle/Context.php b/lib/Qandidate/Toggle/Context.php
@@ -26,8 +26,6 @@ class Context
 
     /**
      * @param int|string $key
-     *
-     * @return mixed
      */
     public function get($key)
     {
@@ -36,7 +34,6 @@ public function get($key)
 
     /**
      * @param int|string $key
-     * @param mixed      $value
      *
      * @return $this
      */

diff --git a/lib/Qandidate/Toggle/ExpressionCondition.php b/lib/Qandidate/Toggle/ExpressionCondition.php
@@ -37,9 +37,6 @@ public function __construct(string $expression, ExpressionLanguage $language)
         $this->language = $language;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function holdsFor(Context $context): bool
     {
         return true === $this->language->evaluate($this->expression, $context->toArray());

diff --git a/lib/Qandidate/Toggle/Operator.php b/lib/Qandidate/Toggle/Operator.php
@@ -19,8 +19,6 @@
 abstract class Operator
 {
     /**
-     * @param mixed $argument
-     *
      * @return bool True, if the operator applies to the argument
      */
     abstract public function appliesTo($argument): bool;

diff --git a/lib/Qandidate/Toggle/Operator/EqualTo.php b/lib/Qandidate/Toggle/Operator/EqualTo.php
@@ -15,9 +15,6 @@
 
 class EqualTo extends EqualityOperator
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return $argument === $this->value;

diff --git a/lib/Qandidate/Toggle/Operator/EqualityOperator.php b/lib/Qandidate/Toggle/Operator/EqualityOperator.php
@@ -20,14 +20,8 @@
  */
 abstract class EqualityOperator extends Operator
 {
-    /**
-     * @var mixed
-     */
     protected $value;
 
-    /**
-     * @param mixed $value
-     */
     public function __construct($value)
     {
         $this->value = $value;

diff --git a/lib/Qandidate/Toggle/Operator/GreaterThan.php b/lib/Qandidate/Toggle/Operator/GreaterThan.php
@@ -15,9 +15,6 @@
 
 class GreaterThan extends EqualityOperator
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return $argument > $this->value;

diff --git a/lib/Qandidate/Toggle/Operator/GreaterThanEqual.php b/lib/Qandidate/Toggle/Operator/GreaterThanEqual.php
@@ -15,9 +15,6 @@
 
 class GreaterThanEqual extends EqualityOperator
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return $argument >= $this->value;

diff --git a/lib/Qandidate/Toggle/Operator/HasIntersection.php b/lib/Qandidate/Toggle/Operator/HasIntersection.php
@@ -25,9 +25,6 @@ public function __construct(array $values)
         $this->values = $values;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return null !== $argument

diff --git a/lib/Qandidate/Toggle/Operator/InSet.php b/lib/Qandidate/Toggle/Operator/InSet.php
@@ -27,9 +27,6 @@ public function __construct(array $values)
         $this->values = $values;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return null !== $argument

diff --git a/lib/Qandidate/Toggle/Operator/LessThan.php b/lib/Qandidate/Toggle/Operator/LessThan.php
@@ -15,9 +15,6 @@
 
 class LessThan extends EqualityOperator
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return $argument < $this->value;

diff --git a/lib/Qandidate/Toggle/Operator/LessThanEqual.php b/lib/Qandidate/Toggle/Operator/LessThanEqual.php
@@ -15,9 +15,6 @@
 
 class LessThanEqual extends EqualityOperator
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return $argument <= $this->value;

diff --git a/lib/Qandidate/Toggle/Operator/MatchesRegex.php b/lib/Qandidate/Toggle/Operator/MatchesRegex.php
@@ -15,9 +15,6 @@
 
 class MatchesRegex extends EqualityOperator
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return (bool) preg_match($this->value, $argument);

diff --git a/lib/Qandidate/Toggle/Operator/NotInSet.php b/lib/Qandidate/Toggle/Operator/NotInSet.php
@@ -15,9 +15,6 @@
 
 class NotInSet extends InSet
 {
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         return !parent::appliesTo($argument);

diff --git a/lib/Qandidate/Toggle/Operator/Percentage.php b/lib/Qandidate/Toggle/Operator/Percentage.php
@@ -33,15 +33,12 @@ public function __construct(int $percentage, int $shift = 0)
         $this->shift = $shift;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function appliesTo($argument): bool
     {
         $asPercentage = (int) $argument % 100;
 
-        return $asPercentage >= $this->shift &&
-            $asPercentage < ($this->percentage + $this->shift);
+        return $asPercentage >= $this->shift
+            && $asPercentage < ($this->percentage + $this->shift);
     }
 
     public function getPercentage(): int

diff --git a/lib/Qandidate/Toggle/OperatorCondition.php b/lib/Qandidate/Toggle/OperatorCondition.php
@@ -34,9 +34,6 @@ public function __construct(string $key, Operator $operator)
         $this->operator = $operator;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function holdsFor(Context $context): bool
     {
         if (!$context->has($this->key)) {

diff --git a/lib/Qandidate/Toggle/ToggleCollection/InMemoryCollection.php b/lib/Qandidate/Toggle/ToggleCollection/InMemoryCollection.php
@@ -27,17 +27,11 @@ class InMemoryCollection extends ToggleCollection
      */
     private $toggles = [];
 
-    /**
-     * {@inheritdoc}
-     */
     public function all(): array
     {
         return $this->toggles;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function get(string $name): ?Toggle
     {
         if (!array_key_exists($name, $this->toggles)) {
@@ -47,17 +41,11 @@ public function get(string $name): ?Toggle
         return $this->toggles[$name];
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function set(string $name, Toggle $toggle): void
     {
         $this->toggles[$name] = $toggle;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function remove(string $name): void
     {
         if (array_key_exists($name, $this->toggles)) {

diff --git a/lib/Qandidate/Toggle/ToggleCollection/PredisCollection.php b/lib/Qandidate/Toggle/ToggleCollection/PredisCollection.php
@@ -48,9 +48,6 @@ public function getNamespace(): string
         return $this->namespace;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function all(): array
     {
         $keys = $this->client->keys($this->namespace.'__TOGGLE__*');
@@ -66,25 +63,16 @@ public function all(): array
         return $toggles;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function get(string $name): ?Toggle
     {
         return $this->getFromKey($this->namespace.'__TOGGLE__'.$name);
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function set(string $name, Toggle $toggle): void
     {
         $this->client->set($this->namespace.'__TOGGLE__'.$name, serialize($toggle));
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function remove(string $name): void
     {
         $this->client->del([$this->namespace.'__TOGGLE__'.$name]);