Consistent documentation and no return value for removeAttribute()

Author: clue

src/Attribute/AttributeAware.php

@@ -27,8 +27,9 @@ public function getAttribute($name, $default = null);
     public function setAttribute($name, $value);
 
     /**
-     * @param $name
-     * @return mixed
+     * Removes a single attribute with the given $name
+     *
+     * @param string $name
      */
     public function removeAttribute($name);
 

src/Attribute/AttributeBag.php

@@ -9,6 +9,7 @@ interface AttributeBag extends AttributeAware
 {
     // public function getAttribute($name, $default);
     // public function setAttribute($name, $value);
+    // public function removeAttribute();
     // public function getAttributeBag();
 
     /**

src/Attribute/AttributeBagContainer.php

@@ -42,18 +42,13 @@ public function setAttribute($name, $value)
     }
 
     /**
-     * remove a single attribute with the given $name
+     * Removes a single attribute with the given $name
      *
-     * @param $name
-     * @return $this
+     * @param string $name
      */
     public function removeAttribute($name)
     {
-        if (isset($this->attributes[$name]) === true) {
-            unset($this->attributes[$name]);
-        }
-
-        return $this;
+        unset($this->attributes[$name]);
     }
 
     /**

src/Attribute/AttributeBagNamespaced.php

@@ -69,18 +69,13 @@ public function setAttribute($name, $value)
     }
 
     /**
-     * remove a single attribute with the given $name
+     * Removes a single attribute with the given $name
      *
-     * @param $name
-     * @return $this
+     * @param string $name
      */
     public function removeAttribute($name)
     {
-        if (isset($this->attributes[$name]) === true) {
-            unset($this->attributes[$name]);
-        }
-
-        return $this;
+        $this->bag->removeAttribute($this->prefix . $name);
     }
 
     /**

src/Attribute/AttributeBagReference.php

@@ -56,18 +56,13 @@ public function setAttribute($name, $value)
     }
 
     /**
-     * remove a single attribute with the given $name
+     * Removes a single attribute with the given $name
      *
-     * @param $name
-     * @return $this
+     * @param string $name
      */
     public function removeAttribute($name)
     {
-        if (isset($this->attributes[$name]) === true) {
-            unset($this->attributes[$name]);
-        }
-
-        return $this;
+        unset($this->attributes[$name]);
     }
 
     /**

src/Edge/Base.php

@@ -305,9 +305,7 @@ public function setAttribute($name, $value)
 
     public function removeAttribute($name)
     {
-        if (isset($this->attributes[$name]) === true) {
-            unset($this->attributes[$name]);
-        }
+        unset($this->attributes[$name]);
     }
 
     public function getAttributeBag()

src/Graph.php

@@ -464,9 +464,7 @@ public function setAttribute($name, $value)
 
     public function removeAttribute($name)
     {
-        if (isset($this->attributes[$name]) === true) {
-            unset($this->attributes[$name]);
-        }
+        unset($this->attributes[$name]);
     }
 
     public function getAttributeBag()

src/Vertex.php

@@ -384,9 +384,7 @@ public function setAttribute($name, $value)
 
     public function removeAttribute($name)
     {
-        if (isset($this->attributes[$name]) === true) {
-            unset($this->attributes[$name]);
-        }
+        unset($this->attributes[$name]);
     }
 
     public function getAttributeBag()

tests/Attribute/AttributeBagContainerTest.php

@@ -2,8 +2,13 @@
 
 use Fhaculty\Graph\Attribute\AttributeBagContainer;
 
-class AttributeBagContainerTest extends TestCase
+class AttributeBagContainerTest extends AbstractAttributeAwareTest
 {
+    protected function createAttributeAware()
+    {
+        return new AttributeBagContainer();
+    }
+
     public function testEmpty()
     {
         $bag = new AttributeBagContainer();

tests/Attribute/AttributeBagNamespacedTest.php

@@ -5,8 +5,13 @@
 use Fhaculty\Graph\Attribute\AttributeAware;
 use Fhaculty\Graph\Attribute\AttributeBagContainer;
 
-class AtributeBagNamespacedTest extends TestCase
+class AtributeBagNamespacedTest extends AbstractAttributeAwareTest
 {
+    protected function createAttributeAware()
+    {
+        return new AttributeBagNamespaced(new AttributeBagContainer(), 'test.');
+    }
+
     public function testBagContainer()
     {
         $container = new AttributeBagContainer();