Revert "Ability to give or update the points to a reputation record"

This reverts commit a2f43c3.

Author: abetwothree

src/HasReputations.php

@@ -23,26 +23,6 @@ public function givePoint(PointType $pointType)
         }
     }
 
-    /**
-     * Give or update reputation points to payee
-     * 
-     * @param PointType $pointType
-     * @return bool
-     */
-    public function giveOrUpdatePoint(PointType $pointType)
-    {
-        if (!$pointType->qualifier()) {
-            return false;
-        }
-
-        if ($pointType->reputationExists()) {
-            $originalPoints = $pointType->syncPointsChange();
-            return $pointType->payee()->syncPointsChange($originalPoints, $pointType->getPoints());
-        }else if ($this->storeReputation($pointType)) {
-            return $pointType->payee()->addPoint($pointType->getPoints());
-        }
-    }
-
     /**
      * Undo last given point for a subject model
      *
@@ -102,22 +82,6 @@ public function addPoint($point = 1)
         return $this;
     }
 
-    /**
-     * Sync points changed for a reputation record to a payee
-     * 
-     * @param int $point
-     * @return HasReputations|\Illuminate\Database\Eloquent\Model
-     */
-    public function syncPointsChange($originalPoints, $newPoints = 1)
-    {
-        $this->decrement($this->getReputationField(), $originalPoints);
-        $this->increment($this->getReputationField(), $newPoints);
-
-        ReputationChanged::dispatch($this, $newPoints, true);
-
-        return $this;
-    }
-
     /**
      * Reduce a user point
      *

src/PointType.php

@@ -134,20 +134,6 @@ public function storeReputation($meta = [])
         ]);
     }
 
-    /**
-     * Sync updates to points to an existing reputation record
-     * @return int $originalPoints
-     */
-    public function syncPointsChange()
-    {
-        $dbPointType = $this->firstReputation();
-        $originalPoints = $dbPointType->point;
-        $dbPointType->update([
-            'point' => $this->getPoints()
-        ]);
-        return $originalPoints;
-    }
-
     /**
      * Get reputation query for this point
      *

src/helpers.php

@@ -22,26 +22,6 @@ function givePoint(PointType $pointType, $payee = null)
     }
 }
 
-if (!function_exists('giveOrUpdatePoint')) {
-    
-    /**
-     * Give point to user
-     *
-     * @param PointType $pointType
-     * @param null $payee
-     */
-    function giveOrUpdatePoint(PointType $pointType, $payee = null)
-    {
-        $payee = $payee ?? auth()->user();
-
-        if (!$payee) {
-            return;
-        }
-
-        $payee->giveOrUpdatePoint($pointType);
-    }
-}
-
 if (!function_exists('undoPoint')) {
 
     /**

tests/PointTest.php

@@ -69,41 +69,6 @@ public function it_gives_point_to_a_user()
         ]);
     }
 
-    /**
-     * it gives point to a user, then updates it validating that that change
-     *
-     * @test
-     */
-    public function it_gives_and_updates_points_to_a_user()
-    {
-        $user = $this->createUser();
-        $post = $this->createPost(['user_id' => $user->id]);
-
-        $user->giveOrUpdatePoint(new FakeUpdateablePostPoint($post));
-
-        $this->assertEquals(1, $user->fresh()->getPoints());
-        $this->assertCount(1, $user->reputations);
-        $this->assertDatabaseHas('reputations', [
-            'payee_id' => $user->id,
-            'subject_type' => $post->getMorphClass(),
-            'subject_id' => $post->id,
-            'point' => 1,
-            'name' => 'FakeUpdateablePostPoint'
-        ]);
-
-        $user->giveOrUpdatePoint(new FakeUpdateablePostPoint($post, 5));
-
-        $this->assertEquals(5, $user->fresh()->getPoints());
-        $this->assertCount(1, $user->reputations);
-        $this->assertDatabaseHas('reputations', [
-            'payee_id' => $user->id,
-            'subject_type' => $post->getMorphClass(),
-            'subject_id' => $post->id,
-            'point' => 5,
-            'name' => 'FakeUpdateablePostPoint'
-        ]);
-    }
-
     /**
      * it can access a reputation payee and subject
      *
@@ -304,24 +269,6 @@ public function payee()
     }
 }
 
-class FakeUpdateablePostPoint extends PointType
-{
-    protected $points = 1;
-
-    public $allowDuplicates = false;
-
-    public function __construct($subject, $points = 1)
-    {
-        $this->subject = $subject;
-        $this->points = $points;
-    }
-
-    public function payee()
-    {
-        return $this->getSubject()->user;
-    }
-}
-
 class FakeWelcomeUserWithNamePoint extends PointType
 {
     protected $name = 'FakeName';