Merge pull request #171 from clue-labs/revert-remove-attribute

Revert "Merge pull request #138 from clue-labs/remove-attribute"
graphp · Sep 29, 2019 · b744a33 · b744a33
2 parents 1eafe38 + 09ea558
commit b744a33
Show file tree

Hide file tree

Showing 9 changed files with 0 additions and 66 deletions.
diff --git a/src/Attribute/AttributeAware.php b/src/Attribute/AttributeAware.php
@@ -26,13 +26,6 @@ public function getAttribute($name, $default = null);
      */
     public function setAttribute($name, $value);
 
-    /**
-     * Removes a single attribute with the given $name
-     *
-     * @param string $name
-     */
-    public function removeAttribute($name);
-
     /**
      * get a container for all attributes
      *

diff --git a/src/Attribute/AttributeBag.php b/src/Attribute/AttributeBag.php
@@ -9,7 +9,6 @@ interface AttributeBag extends AttributeAware
 {
     // public function getAttribute($name, $default);
     // public function setAttribute($name, $value);
-    // public function removeAttribute();
     // public function getAttributeBag();
 
     /**

diff --git a/src/Attribute/AttributeBagContainer.php b/src/Attribute/AttributeBagContainer.php
@@ -41,16 +41,6 @@ public function setAttribute($name, $value)
         return $this;
     }
 
-    /**
-     * Removes a single attribute with the given $name
-     *
-     * @param string $name
-     */
-    public function removeAttribute($name)
-    {
-        unset($this->attributes[$name]);
-    }
-
     /**
      * get an array of all attributes
      *

diff --git a/src/Attribute/AttributeBagNamespaced.php b/src/Attribute/AttributeBagNamespaced.php
@@ -68,16 +68,6 @@ public function setAttribute($name, $value)
         $this->bag->setAttribute($this->prefix . $name, $value);
     }
 
-    /**
-     * Removes a single attribute with the given $name
-     *
-     * @param string $name
-     */
-    public function removeAttribute($name)
-    {
-        $this->bag->removeAttribute($this->prefix . $name);
-    }
-
     /**
      * get an array of all attributes
      *

diff --git a/src/Attribute/AttributeBagReference.php b/src/Attribute/AttributeBagReference.php
@@ -55,16 +55,6 @@ public function setAttribute($name, $value)
         return $this;
     }
 
-    /**
-     * Removes a single attribute with the given $name
-     *
-     * @param string $name
-     */
-    public function removeAttribute($name)
-    {
-        unset($this->attributes[$name]);
-    }
-
     /**
      * get an array of all attributes
      *

diff --git a/src/Edge/Base.php b/src/Edge/Base.php
@@ -301,11 +301,6 @@ public function setAttribute($name, $value)
         $this->attributes[$name] = $value;
     }
 
-    public function removeAttribute($name)
-    {
-        unset($this->attributes[$name]);
-    }
-
     public function getAttributeBag()
     {
         return new AttributeBagReference($this->attributes);

diff --git a/src/Graph.php b/src/Graph.php
@@ -457,11 +457,6 @@ public function setAttribute($name, $value)
         $this->attributes[$name] = $value;
     }
 
-    public function removeAttribute($name)
-    {
-        unset($this->attributes[$name]);
-    }
-
     public function getAttributeBag()
     {
         return new AttributeBagReference($this->attributes);

diff --git a/src/Vertex.php b/src/Vertex.php
@@ -399,11 +399,6 @@ public function setAttribute($name, $value)
         $this->attributes[$name] = $value;
     }
 
-    public function removeAttribute($name)
-    {
-        unset($this->attributes[$name]);
-    }
-
     public function getAttributeBag()
     {
         return new AttributeBagReference($this->attributes);

diff --git a/tests/Attribute/AbstractAttributeAwareTest.php b/tests/Attribute/AbstractAttributeAwareTest.php
@@ -31,19 +31,6 @@ public function testAttributeSetGetDefault(AttributeAware $entity)
         $this->assertEquals('default', $entity->getAttribute('unknown', 'default'));
     }
 
-    /**
-     * @depends testAttributeAwareInterface
-     * @param AttributeAware $entity
-     */
-    public function testAttributeSetRemoveGet(AttributeAware $entity)
-    {
-        $entity->setAttribute('test', 'value');
-        $this->assertEquals('value', $entity->getAttribute('test'));
-
-        $entity->removeAttribute('test');
-        $this->assertEquals(null, $entity->getAttribute('test'));
-    }
-
     /**
      * @depends testAttributeAwareInterface
      * @param AttributeAware $entity