From 8c6fae4e9c4824fb79d1dcdcbed99110fb5a6e7d Mon Sep 17 00:00:00 2001
From: Adrian Greeve <adrian@moodle.com>
Date: Thu, 19 Apr 2018 13:51:17 +0800
Subject: [PATCH] MDL-62035 core_blog: Update to blog with comments.

This makes blog use the updated delete methods in the
comments provider.
---
 blog/classes/privacy/provider.php | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/blog/classes/privacy/provider.php b/blog/classes/privacy/provider.php
index 6c585862a26..398d5a3f22c 100644
--- a/blog/classes/privacy/provider.php
+++ b/blog/classes/privacy/provider.php
@@ -371,9 +371,8 @@ class provider implements
                 $DB->delete_records('blog_association', ['contextid' => $context->id]);
                 break;
         }
-
         // Delete all the comments.
-        \core_comment\privacy\provider::delete_comments_for_all_users_in_context($context);
+        \core_comment\privacy\provider::delete_comments_for_all_users($context, 'blog', 'format_blog');
     }
 
     /**
@@ -383,21 +382,21 @@ class provider implements
      */
     public static function delete_data_for_user(approved_contextlist $contextlist) {
         global $DB;
-
         $userid = $contextlist->get_user()->id;
         $associationcontextids = [];
 
         foreach ($contextlist as $context) {
             if ($context->contextlevel == CONTEXT_USER && $context->instanceid == $userid) {
                 static::delete_all_user_data($context);
-
+                \core_comment\privacy\provider::delete_comments_for_all_users($context, 'blog', 'format_blog');
             } else if ($context->contextlevel == CONTEXT_COURSE) {
                 // Only delete the course associations.
                 $associationcontextids[] = $context->id;
-
             } else if ($context->contextlevel == CONTEXT_MODULE) {
                 // Only delete the module associations.
                 $associationcontextids[] = $context->id;
+            } else {
+                \core_comment\privacy\provider::delete_comments_for_user($contextlist, 'blog', 'format_blog');
             }
         }
 
@@ -416,9 +415,6 @@ class provider implements
             list($insql, $inparams) = $DB->get_in_or_equal($associds, SQL_PARAMS_NAMED, 'param', true);
             $DB->delete_records_select('blog_association', "id $insql", $inparams);
         }
-
-        // Delete the comments.
-        \core_comment\privacy\provider::delete_comments_for_user($contextlist);
     }
 
     /**
@@ -443,9 +439,6 @@ class provider implements
         // Delete all external blogs, and their associated tags.
         $DB->delete_records('blog_external', ['userid' => $userid]);
         core_tag_tag::delete_instances('core', 'blog_external', $usercontext->id);
-
-        // Delete all comments made in this context.
-        $DB->delete_records('comments', ['contextid' => $usercontext->id, 'component' => 'blog', 'commentarea' => 'format_blog']);
     }
 
     /**
-- 
2.43.0