diff options
| author | cash <cash.costello@gmail.com> | 2011-11-18 18:02:39 -0500 |
|---|---|---|
| committer | cash <cash.costello@gmail.com> | 2011-11-18 18:02:39 -0500 |
| commit | f3b50d11966eb75c648e317ab11045913e28e56a (patch) | |
| tree | 057aa6ecb00e1f68d72de17aaee8bc7d9560fb0d /engine | |
| parent | ff907019418cd1635cba8b1c225827a20eedfe2c (diff) | |
| download | elgg-f3b50d11966eb75c648e317ab11045913e28e56a.tar.gz elgg-f3b50d11966eb75c648e317ab11045913e28e56a.tar.bz2 | |
Fixes #4131 elgg_delete_metastring_based_object_by_id() should return a bool
Diffstat (limited to 'engine')
| -rw-r--r-- | engine/lib/metastrings.php | 2 | ||||
| -rw-r--r-- | engine/tests/api/metastrings.php | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/engine/lib/metastrings.php b/engine/lib/metastrings.php index 9dccec700..9fe9b4bff 100644 --- a/engine/lib/metastrings.php +++ b/engine/lib/metastrings.php @@ -810,7 +810,7 @@ function elgg_delete_metastring_based_object_by_id($id, $type) { } if (($obj->canEdit()) && (elgg_trigger_event('delete', $type, $obj))) { - return delete_data("DELETE from $table where id=$id"); + return (bool)delete_data("DELETE from $table where id=$id"); } } diff --git a/engine/tests/api/metastrings.php b/engine/tests/api/metastrings.php index 0a008c6ce..f6870d34b 100644 --- a/engine/tests/api/metastrings.php +++ b/engine/tests/api/metastrings.php @@ -80,7 +80,7 @@ class ElggCoreMetastringsTest extends ElggCoreUnitTest { $test = get_data($q); $this->assertEqual($test[0]->id, $id); - $this->assertTrue(elgg_delete_metastring_based_object_by_id($id, $type)); + $this->assertIdentical(true, elgg_delete_metastring_based_object_by_id($id, $type)); $this->assertFalse(get_data($q)); } } |