Case insensitive flag for remove in Collections

phalcon/collection.zep

@@ -199,17 +199,22 @@ class Collection implements \ArrayAccess, \Countable, \IteratorAggregate, \JsonS
 	/**
 	 * Delete the element from the collection
 	 */
-	public function remove(string! element) -> void
+	public function remove(string! element, bool insensitive = true) -> void
 	{
-		var data, lowerKeys, key, value;
+		var data, lowerKeys, value;
+
 
 		let data      = this->data,
-			lowerKeys = this->lowerKeys,
-			key       = strtolower(element);
+			lowerKeys = this->lowerKeys;
 
 		if this->has(element) {
-			let value = lowerKeys[key];
-			unset lowerKeys[key];
+			if likely insensitive {
+				let element = strtolower(element);
+			}
+
+			let value = lowerKeys[element];
+
+			unset lowerKeys[element];
 			unset data[value];
 		}
 

phalcon/registry.zep

@@ -207,9 +207,9 @@ final class Registry extends Collection
 	/**
 	 * Delete the element from the collection
 	 */
-	public final function remove(string! element) -> void
+	public final function remove(string! element, bool insensitive = true) -> void
 	{
-		parent::remove(element);
+		parent::remove(element, insensitive);
 	}
 
     /**

phalcon/session/bag.zep

@@ -94,9 +94,9 @@ class Bag extends Collection implements InjectionAwareInterface
 	/**
 	 * Removes a property from the internal bag
 	 */
-	public function remove(string! element) -> void
+	public function remove(string! element, bool insensitive = true) -> void
 	{
-		parent::remove(element);
+		parent::remove(element, insensitive);
 		this->session->set(this->name, this->data);
 	}
 

tests/integration/Session/Bag/RemoveCest.php

@@ -56,6 +56,14 @@ public function sessionBagRemove(IntegrationTester $I)
         $actual   = $collection->toArray();
         $I->assertEquals($expected, $actual);
 
+        $collection->remove('FIVE');
+        $expected = [
+            'one'   => 'two',
+            'three' => 'four',
+        ];
+        $actual   = $collection->toArray();
+        $I->assertEquals($expected, $actual);
+
         $collection->init($data);
         unset($collection['five']);
         $expected = [

tests/unit/Collection/RemoveCest.php

@@ -50,6 +50,14 @@ public function collectionRemove(UnitTester $I)
         $actual   = $collection->toArray();
         $I->assertEquals($expected, $actual);
 
+        $collection->remove('FIVE');
+        $expected = [
+            'one'   => 'two',
+            'three' => 'four',
+        ];
+        $actual   = $collection->toArray();
+        $I->assertEquals($expected, $actual);
+
         $collection->init($data);
         unset($collection['five']);
         $expected = [

tests/unit/Registry/RemoveCest.php

@@ -50,6 +50,14 @@ public function collectionRemove(UnitTester $I)
         $actual   = $registry->toArray();
         $I->assertEquals($expected, $actual);
 
+        $registry->remove('FIVE');
+        $expected = [
+            'one'   => 'two',
+            'three' => 'four',
+        ];
+        $actual   = $registry->toArray();
+        $I->assertEquals($expected, $actual);
+
         $registry->init($data);
         unset($registry['five']);
         $expected = [