Better destruction of PhameBlog, BadgesBadge

Summary: Allows proper destruction of Badge Awards and Phame Posts.

Test Plan: bin/remove destroy PHID...

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin

Differential Revision: https://secure.phabricator.com/D16182

src/applications/badges/storage/PhabricatorBadgesBadge.php

@@ -186,7 +186,7 @@ final class PhabricatorBadgesBadge extends PhabricatorBadgesDAO
       ->execute();
 
     foreach ($awards as $award) {
-      $engine->destroyObjectPermanently($award);
+      $engine->destroyObject($award);
     }
 
     $this->openTransaction();

src/applications/phame/storage/PhameBlog.php

@@ -322,10 +322,12 @@ final class PhameBlog extends PhameDAO
 
     $this->openTransaction();
 
-      $posts = id(new PhamePost())
-        ->loadAllWhere('blogPHID = %s', $this->getPHID());
+      $posts = id(new PhamePostQuery())
+        ->setViewer($engine->getViewer())
+        ->withBlogPHIDs(array($this->getPHID()))
+        ->execute();
       foreach ($posts as $post) {
-        $post->delete();
+        $engine->destroyObject($post);
       }
       $this->delete();
 

src/applications/phame/storage/PhamePost.php

@@ -258,15 +258,15 @@ final class PhamePost extends PhameDAO
     return $timeline;
   }
 
+
 /* -(  PhabricatorDestructibleInterface  )----------------------------------- */
 
+
   public function destroyObjectPermanently(
     PhabricatorDestructionEngine $engine) {
 
     $this->openTransaction();
-
       $this->delete();
-
     $this->saveTransaction();
   }