function delete in dataobjects now don't break strict syntax

diff --git a/classes/Avatar.php b/classes/Avatar.php
index c7f6a56b0420bcca6cbd5825404e85afb5e9b12d..02874b902b5f267e8f94e6e742981c884cb4c07c 100644 (file)
--- a/classes/Avatar.php
+++ b/classes/Avatar.php
@@ -51,12 +51,14 @@ class Avatar extends Managed_DataObject
     }
 
     // We clean up the file, too
-    function delete()
+    function delete($useWhere=false)
     {
         $filename = $this->filename;
-        if (parent::delete() && file_exists(Avatar::path($filename))) {
+        if (file_exists(Avatar::path($filename))) {
             @unlink(Avatar::path($filename));
         }
+
+        return parent::delete($useWhere);
     }
 
     /*

diff --git a/classes/Config.php b/classes/Config.php
index 33a556e579752324a6af0fef5e640691522fc8e3..899396d710e13823270698903eaf5dcdfc422ea5 100644 (file)
--- a/classes/Config.php
+++ b/classes/Config.php
@@ -115,10 +115,10 @@ class Config extends Managed_DataObject
         return $result;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
-        $result = parent::delete();
-        if ($result) {
+        $result = parent::delete($useWhere);
+        if ($result !== false) {
             Config::_blowSettingsCache();
         }
         return $result;

diff --git a/classes/Consumer.php b/classes/Consumer.php
index b1ff6d9e1a2175405769edea6c91725b5e2ff1ae..68e973bfb095bcfe55180c539026856b04c1f087 100644 (file)
--- a/classes/Consumer.php
+++ b/classes/Consumer.php
@@ -54,14 +54,14 @@ class Consumer extends Managed_DataObject
      * XXX: Should this happen in an OAuthDataStore instead?
      *
      */
-    function delete()
+    function delete($useWhere=false)
     {
         // XXX: Is there any reason NOT to do this kind of cleanup?
 
         $this->_deleteTokens();
         $this->_deleteNonces();
 
-        parent::delete();
+        return parent::delete($useWhere);
     }
 
     function _deleteTokens()

diff --git a/classes/Fave.php b/classes/Fave.php
index 53d60b3a8202138ef58cf2cd6bbf6f8dbeb67444..beeda9aafd314795414ad8c51712de94c2f03a46 100644 (file)
--- a/classes/Fave.php
+++ b/classes/Fave.php
@@ -78,7 +78,7 @@ class Fave extends Managed_DataObject
         return $fave;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         $profile = Profile::getKV('id', $this->user_id);
         $notice  = Notice::getKV('id', $this->notice_id);
@@ -87,7 +87,7 @@ class Fave extends Managed_DataObject
 
         if (Event::handle('StartDisfavorNotice', array($profile, $notice, &$result))) {
 
-            $result = parent::delete();
+            $result = parent::delete($useWhere);
             self::blow('fave:list-ids:notice_id:%d', $this->notice_id);
             self::blow('popular');
 

diff --git a/classes/File_to_post.php b/classes/File_to_post.php
index e53b1ebb4cfa38dc14b6a629ff7cd9f2487d1712..1d2733738fb277f2d13f9b1e392cf390dafae678 100644 (file)
--- a/classes/File_to_post.php
+++ b/classes/File_to_post.php
@@ -84,12 +84,12 @@ class File_to_post extends Managed_DataObject
         }
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         $f = File::getKV('id', $this->file_id);
         if (!empty($f)) {
             $f->blowCache();
         }
-        return parent::delete();
+        return parent::delete($useWhere);
     }
 }

diff --git a/classes/Memcached_DataObject.php b/classes/Memcached_DataObject.php
index f9ace16ce6b1bf626f175e11c6ddb9d7270c7cec..55eecd807e58ee5d49c08392cdd6412784efa545 100644 (file)
--- a/classes/Memcached_DataObject.php
+++ b/classes/Memcached_DataObject.php
@@ -401,10 +401,10 @@ class Memcached_DataObject extends Safe_DataObject
         return $result;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         $this->decache(); # while we still have the values!
-        return parent::delete();
+        return parent::delete($useWhere);
     }
 
     static function memcache() {

diff --git a/classes/Notice.php b/classes/Notice.php
index df05ce465061dd0eacf2fe8f5bf9e2c0e9f04da3..4e5a83a85970ab4fae894d29100b761be97ca363 100644 (file)
--- a/classes/Notice.php
+++ b/classes/Notice.php
@@ -162,7 +162,7 @@ class Notice extends Managed_DataObject
         $this->_profile = $profile;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         // For auditing purposes, save a record that the notice
         // was deleted.
@@ -171,11 +171,11 @@ class Notice extends Managed_DataObject
         // insert fails.
         $deleted = Deleted_notice::getKV('id', $this->id);
 
-        if (!$deleted) {
+        if (!$deleted instanceof Deleted_notice) {
             $deleted = Deleted_notice::getKV('uri', $this->uri);
         }
 
-        if (!$deleted) {
+        if (!$deleted instanceof Deleted_notice) {
             $deleted = new Deleted_notice();
 
             $deleted->id         = $this->id;
@@ -202,7 +202,7 @@ class Notice extends Managed_DataObject
             // NOTE: we don't clear queue items
         }
 
-        $result = parent::delete();
+        $result = parent::delete($useWhere);
 
         $this->blowOnDelete();
         return $result;

diff --git a/classes/Oauth_application.php b/classes/Oauth_application.php
index 90218e93cbd5c6d3019d23dbc214e92c23cfc874..b29fdfe41b1fb059cb79ae409010950a44be1650 100644 (file)
--- a/classes/Oauth_application.php
+++ b/classes/Oauth_application.php
@@ -139,14 +139,14 @@ class Oauth_application extends Managed_DataObject
         }
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         $this->_deleteAppUsers();
 
         $consumer = $this->getConsumer();
         $consumer->delete();
 
-        parent::delete();
+        return parent::delete($useWhere);
     }
 
     function _deleteAppUsers()

diff --git a/classes/Profile.php b/classes/Profile.php
index b2dab519a1599940733f0c1a9853860db52eeb3c..50cbeb3ac4fdc16b422aa1b8df93fff9910577c0 100644 (file)
--- a/classes/Profile.php
+++ b/classes/Profile.php
@@ -869,7 +869,7 @@ class Profile extends Managed_DataObject
         return parent::update($dataObject);
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         $this->_deleteNotices();
         $this->_deleteSubscriptions();
@@ -891,7 +891,7 @@ class Profile extends Managed_DataObject
             $inst->delete();
         }
 
-        parent::delete();
+        return parent::delete($useWhere);
     }
 
     function _deleteNotices()

diff --git a/classes/Profile_list.php b/classes/Profile_list.php
index 509a1df242498fd546abce672f2c2b2ed945dba9..0ad8106e3187d10b88da3dcfe0d45857d146c2f5 100644 (file)
--- a/classes/Profile_list.php
+++ b/classes/Profile_list.php
@@ -334,7 +334,7 @@ class Profile_list extends Managed_DataObject
      * @return boolean success
      */
 
-    function delete()
+    function delete($useWhere=false)
     {
         // force delete one item at a time.
         if (empty($this->id)) {
@@ -349,7 +349,7 @@ class Profile_list extends Managed_DataObject
 
         self::blow('profile:lists:%d', $this->tagger);
 
-        return parent::delete();
+        return parent::delete($useWhere);
     }
 
     /**

diff --git a/classes/Profile_tag.php b/classes/Profile_tag.php
index 5256aa13c8029805c4041be53b1f355858027f9d..6d6f49bedc634abf174d6d5d663da38e18d35302 100644 (file)
--- a/classes/Profile_tag.php
+++ b/classes/Profile_tag.php
@@ -322,10 +322,10 @@ class Profile_tag extends Managed_DataObject
         return $result;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
-        $result = parent::delete();
-        if ($result) {
+        $result = parent::delete($useWhere);
+        if ($result !== false) {
             self::blow('profile_list:tagged_count:%d:%s', 
                        $this->tagger,
                        $this->tag);

diff --git a/classes/Profile_tag_subscription.php b/classes/Profile_tag_subscription.php
index b088023fab74dabbd336f239cd87df41eb1c10d9..f7b1553300b52010faae091c0fb69916028cd61a 100644 (file)
--- a/classes/Profile_tag_subscription.php
+++ b/classes/Profile_tag_subscription.php
@@ -127,10 +127,10 @@ class Profile_tag_subscription extends Managed_DataObject
         return $result;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
-        $result = parent::delete();
-        if ($result) {
+        $result = parent::delete($useWhere);
+        if ($result !== false) {
             self::blow('profile_list:subscriber_count:%d', 
                        $this->profile_tag_id);
         }

diff --git a/classes/Safe_DataObject.php b/classes/Safe_DataObject.php
index 622b57f2900a291529a48be2a9442d7f3ea6abfc..b4c251c475f609047c737d401863fd930416e697 100644 (file)
--- a/classes/Safe_DataObject.php
+++ b/classes/Safe_DataObject.php
@@ -108,7 +108,7 @@ class Safe_DataObject extends DB_DataObject
      * @return mixed
      * @throws Exception
      */
-    function __call($method, array $params)
+    function __call($method, $params)
     {
         $return = null;
         // Yes, that's _call with one underscore, which does the

diff --git a/classes/Status_network.php b/classes/Status_network.php
index 863dea98b4949c7956b671a39712d11d160d66ec..9874030ba0472d83177092952991e76d56a4933c 100644 (file)
--- a/classes/Status_network.php
+++ b/classes/Status_network.php
@@ -183,10 +183,10 @@ class Status_network extends Safe_DataObject
         return $result;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         $this->decache(); # while we still have the values!
-        return parent::delete();
+        return parent::delete($useWhere);
     }
 
     /**

diff --git a/classes/Status_network_tag.php b/classes/Status_network_tag.php
index b7ea2cff3a3693ac1161c57f459386a43f8b3143..9e5ff89be352c9863d72a977756c8445ae5da7ea 100644 (file)
--- a/classes/Status_network_tag.php
+++ b/classes/Status_network_tag.php
@@ -125,11 +125,10 @@ class Status_network_tag extends Safe_DataObject
         return $ret;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
-        $ret = parent::delete();
         $this->decache();
-        return $ret;
+        return parent::delete($useWhere);
     }
 
     static function withTag($tag)

diff --git a/classes/User.php b/classes/User.php
index aaf5f91526e778e5ad78668e886b50198ec88f89..24a03b62fd0d0d82941e72a481cc3fa2f8906119 100644 (file)
--- a/classes/User.php
+++ b/classes/User.php
@@ -678,11 +678,11 @@ class User extends Managed_DataObject
         return $this->getProfile()->hasRight($right);
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         if (empty($this->id)) {
             common_log(LOG_WARNING, "Ambiguous User->delete(); skipping related tables.");
-            return parent::delete();
+            return parent::delete($useWhere);
         }
 
         try {
@@ -710,7 +710,7 @@ class User extends Managed_DataObject
         $this->_deleteTags();
         $this->_deleteBlocks();
 
-        parent::delete();
+        return parent::delete($useWhere);
     }
 
     function _deleteTags()

diff --git a/classes/User_group.php b/classes/User_group.php
index efac365da20bafe5c41b33bd647c0efb7bb3be36..6f3c7038a74d25b80c7cc8d8ca80631677cfb771 100644 (file)
--- a/classes/User_group.php
+++ b/classes/User_group.php
@@ -727,11 +727,11 @@ class User_group extends Managed_DataObject
      * are not de-cached in the UI, including the sidebar lists on
      * GroupsAction
      */
-    function delete()
+    function delete($useWhere=false)
     {
         if (empty($this->id)) {
             common_log(LOG_WARNING, "Ambiguous User_group->delete(); skipping related tables.");
-            return parent::delete();
+            return parent::delete($useWhere);
         }
 
         try {
@@ -779,7 +779,7 @@ class User_group extends Managed_DataObject
         // blow the cached ids
         self::blow('user_group:notice_ids:%d', $this->id);
 
-        return parent::delete();
+        return parent::delete($useWhere);
     }
 
     public function update($dataObject=false)

diff --git a/plugins/ActivitySpam/classes/spam_score.php b/plugins/ActivitySpam/classes/spam_score.php
index 2ae32fbfaa4835718ca97b411be479f9ae1978e8..5c583a2e0722ebb3d72d90b0623c3befac1cf39d 100644 (file)
--- a/plugins/ActivitySpam/classes/spam_score.php
+++ b/plugins/ActivitySpam/classes/spam_score.php
@@ -100,11 +100,11 @@ class Spam_score extends Managed_DataObject
         return $score;
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         self::blow('spam_score:notice_ids');
         self::blow('spam_score:notice_ids;last');
-        parent::delete();
+        return parent::delete($useWhere);
     }
 
     /**

diff --git a/plugins/Event/classes/RSVP.php b/plugins/Event/classes/RSVP.php
index cea916c23458bc06025eb0fc3291a9b27df1d95c..521e8110366735886fa678644510dbb55deb535e 100644 (file)
--- a/plugins/Event/classes/RSVP.php
+++ b/plugins/Event/classes/RSVP.php
@@ -399,9 +399,9 @@ class RSVP extends Managed_DataObject
                        $eventTitle);
     }
 
-    function delete()
+    function delete($useWhere=false)
     {
         self::blow('rsvp:for-event:%s', $event->id);
-        parent::delete();
+        return parent::delete($useWhere);
     }
 }