Convert the event handlers for the "identity provider changed" and "user_deleted" events to use ORM or the Kohana query builder to build the database update calls instead of coding the sql directly.

1 files changed, 14 insertions, 13 deletions

diff --git a/modules/comment/helpers/comment_event.php b/modules/comment/helpers/comment_event.php
index ddf72e3c..a72102b9 100644
--- a/modules/comment/helpers/comment_event.php
+++ b/modules/comment/helpers/comment_event.php
@@ -24,23 +24,24 @@ class comment_event_Core {
 
   static function user_deleted($user) {
     $guest = identity::guest();
-    Database::instance()
-      ->query("UPDATE {comments}
-                  SET author_id = {$guest->id},
-                      guest_email = NULL,
-                      guest_name = 'guest',
-                      guest_url = NULL
-                WHERE author_id = {$user->id}");
+    Database::instance()->from("comments")
+      ->set(array("author_id" => $guest->id,
+                  "guest_email" => null,
+                  "guest_name" => "guest",
+                  "guest_url" => null))
+      ->where(array("author_id" => $user->id))
+      ->update();
   }
 
   static function identity_provider_changed($old_provider, $new_provider) {
     $guest = identity::guest();
-    Database::instance()
-      ->query("UPDATE {comments}
-                  SET author_id = {$guest->id},
-                      guest_email = NULL,
-                      guest_name = 'guest',
-                      guest_url = null");
+    Database::instance()->from("comments")
+      ->set(array("author_id" => $guest->id,
+                  "guest_email" => null,
+                  "guest_name" => "guest",
+                  "guest_url" => null))
+      ->where("1 = 1")
+      ->update();
   }
 
   static function admin_menu($menu, $theme) {