Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-70494

Forum: deprecate unused string 'privacy:request:delete:post:message'

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.11
    • Fix Version/s: None
    • Component/s: Forum
    • Labels:
      None
    • Affected Branches:
      MOODLE_311_STABLE

      Description

      In addressing the test_mod_forum_get_forum_discussion_posts_deleted() method in MDL-65252, I noted that the 'privacy:request:delete:post:message' string can probably now be deprecated, since it's no longer used. The string 'privacy:request:delete:post:subject' appears to be in use still, in mod/forum/rsslib.php, however. Not sure why this was being used in the regular deletion stuff, but a quick grep revealed it wasn't in use once that test method was updated.

        Attachments

          Issue Links

          Discovered while testing

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-65252 Final deprecations of forum_count_replies and get_forum_discussion_posts

          • Minor - Minor loss of function where workaround is possible
          • Peer review in progress
          is blocked by

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-65252 Final deprecations of forum_count_replies and get_forum_discussion_posts

          • Minor - Minor loss of function where workaround is possible
          • Peer review in progress

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              jaked Jake Dallimore
              Participants:
              Component watchers:
              Andrew Nicols, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated: