Merge pull request #19 from alwaysblank/12-allow-removing-data

Author: alwaysblank

src/Brief.php

@@ -903,4 +903,23 @@ protected function resolveInternalKey($key) {
 
         return null;
     }
+
+    /**
+     * Remove data from the Brief.
+     *
+     * This will resolve aliases and delete the data they point to,
+     * but *not* the aliases themselves.
+     *
+     * @param $key
+     *
+     * @return void
+     */
+    public function delete($key) {
+        $internalKey = $this->resolveInternalKey($key);
+        if ($internalKey === null) {
+            // Doesn't exist, so nothing to do.
+            return;
+        }
+        unset($this->store[$internalKey]);
+    }
 }

tests/BriefTest.php

@@ -480,6 +480,18 @@ public function testCanUserCustomEmptyFunction(): void
         ], ['isEmpty' => $test]);
         $this->assertFalse($Brieful->isEmpty());
     }
+
+    public function testCanDeleteDataFromBrief(): void {
+        $Brief = Brief::make(['key' => 'value']);
+        $this->assertEquals('value', $Brief->key);
+        $Brief->delete('key');
+        $this->assertNull($Brief->value);
+        $NumericBrief = Brief::make(['value', 'not-value']);
+        $this->assertEquals('value', $NumericBrief->_0);
+        $NumericBrief->delete(0);
+        $this->assertNull($NumericBrief->_0);
+
+    }
 }
 
 /**