From 74967c041c5b44da110f04998b9961ffe54854b2 Mon Sep 17 00:00:00 2001
From: David Rans <david@ifixit.com>
Date: Tue, 20 Aug 2024 11:51:57 -0700
Subject: [PATCH] Remove return by reference from methods that return self

---
 QueryGenerator.php | 6 +++---
 psalm-baseline.xml | 2 --
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/QueryGenerator.php b/QueryGenerator.php
index 9c3abb4..4776866 100644
--- a/QueryGenerator.php
+++ b/QueryGenerator.php
@@ -250,7 +250,7 @@ public function __construct(
     * Append the given clause components and parameters to their existing
     * counterparts for the specified clause.
     */
-   public function &__call($method, $args) {
+   public function __call($method, $args) {
       $method = strtolower($method);
 
       if (!isset(self::$methods[$method])) {
@@ -335,7 +335,7 @@ public function build($skipClauses = false) {
    /**
     * Bypass query validation when building.
     */
-   public function &skipValidation() {
+   public function skipValidation(): self {
       $this->validateQuery = false;
       return $this;
    }
@@ -343,7 +343,7 @@ public function &skipValidation() {
    /**
     * Use OR when joining where conditions
     */
-   public function &useOr() {
+   public function useOr(): self {
       $this->useOr = true;
       return $this;
    }
diff --git a/psalm-baseline.xml b/psalm-baseline.xml
index e3503ca..0d967d5 100644
--- a/psalm-baseline.xml
+++ b/psalm-baseline.xml
@@ -17,8 +17,6 @@
       <code><![CDATA[getPrimaryClauses]]></code>
       <code><![CDATA[getPrimaryMethod]]></code>
       <code><![CDATA[getSetMethods]]></code>
-      <code><![CDATA[skipValidation]]></code>
-      <code><![CDATA[useOr]]></code>
     </MissingReturnType>
     <PossiblyUnusedMethod>
       <code><![CDATA[useOr]]></code>