add possibility to remove items from collections

Author: anlutro

src/Collection.php

@@ -183,6 +183,31 @@ public function getItem($id)
 		return array_key_exists($id, $this->ids) ? $this->ids[$id] : null;
 	}
 
+	/**
+	 * Remove an item from the collection.
+	 *
+	 * @param  string $id
+	 *
+	 * @return boolean
+	 */
+	public function removeItem($id)
+	{
+		if (!array_key_exists($id, $this->ids)) {
+			return false;
+		}
+
+		$item = $this->ids[$id];
+		unset($this->ids[$id]);
+
+		foreach ($this->items as $k1 => $v1) {
+			foreach ($v1 as $k2 => $v2) {
+				if ($item === $v2) {
+					unset($this->items[$k1][$k2]);
+				}
+			}
+		}
+	}
+
 	/**
 	 * Determine whether the menu collection is currently empty or not.
 	 *

tests/MenuCollectionTest.php

@@ -50,4 +50,19 @@ public function testAddWithPriorities()
 		$this->assertTrue(strpos($str, 'First Item') > strpos($str, 'Second Item'),
 			'Second Item should come before First Item.'.PHP_EOL.$str);
 	}
+
+	public function testCanRemoveItem()
+	{
+		$coll = $this->makeCollection();
+		$coll->addItem('First Item', '/foo', [], 20);
+		$coll->addItem('Second Item', '/foo', [], 10);
+		$coll->removeItem('first-item');
+
+		$this->assertNull($coll->getItem('first-item'));
+		$this->assertNotNull($coll->getItem('second-item'));
+
+		$str = $coll->render();
+		$this->assertSame(false, strpos($str, 'First Item'));
+		$this->assertNotSame(false, strpos($str, 'Second Item'));
+	}
 }