From 8d62572ff8c2b5bde2c2a933c5b0268d5984a879 Mon Sep 17 00:00:00 2001 From: Brett Profitt Date: Sun, 22 Jan 2012 13:02:53 -0800 Subject: Fixes #4321. Limit alone is no longer valid for batch delete operations for metadata or annotations. --- engine/tests/api/metastrings.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'engine/tests') diff --git a/engine/tests/api/metastrings.php b/engine/tests/api/metastrings.php index a96388217..0a8945084 100644 --- a/engine/tests/api/metastrings.php +++ b/engine/tests/api/metastrings.php @@ -132,7 +132,7 @@ class ElggCoreMetastringsTest extends ElggCoreUnitTest { public function testKeepMeFromDeletingEverything() { foreach ($this->metastringTypes as $type) { $required = array( - 'guid', 'guids', 'limit' + 'guid', 'guids' ); switch ($type) { @@ -160,6 +160,10 @@ class ElggCoreMetastringsTest extends ElggCoreUnitTest { $options = array(); $this->assertFalse(elgg_is_valid_options_for_batch_operation($options, $type)); + // limit alone isn't valid: + $options = array('limit' => 10); + $this->assertFalse(elgg_is_valid_options_for_batch_operation($options, $type)); + foreach ($required as $key) { $options = array(); -- cgit v1.2.3