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

Coding error detected in comments report

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      This bug is a fringe case, hence it is important to replicate it without patch. Please do the following on a branch without the patch:-

      1. Create a Moodle instance with at least two separate courses.
      2. Add a comment block to an activity(Ex:- forum) within Course A.
      3. Add some comments.
      4. View the Comments Report.
      5. Add a comment block to an activity (Ex:- book) Course B.
      6. Add some comments.
      7. View the Comments Report (admin->reports->comment) make sure you see the error
      8. Apply the patch
      9. Refresh the comments report page and make sure no error is generated.
      Show
      This bug is a fringe case, hence it is important to replicate it without patch. Please do the following on a branch without the patch:- Create a Moodle instance with at least two separate courses. Add a comment block to an activity(Ex:- forum) within Course A. Add some comments. View the Comments Report. Add a comment block to an activity (Ex:- book) Course B. Add some comments. View the Comments Report (admin->reports->comment) make sure you see the error Apply the patch Refresh the comments report page and make sure no error is generated.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull Master Branch:
      MDL-31010-master

      Description

      As found in http://moodle.org/comment/

      Coding error detected, it must be fixed by a programmer: Invalid component used in plugin_callback():mod_

      Whilst the QA testing site is not affected by this problem, I can't think of any reason why the problem should be specific to moodle.org and so I am creating a MDL issue. Feel free to change it to MDLSITE if I am wrong.

        Attachments

        1. MDL-31010.diff
          1 kB
        2. patch.diff
          1.0 kB

          Issue Links

            Activity

              People

              Assignee:
              ankit_frenz Ankit Agarwal
              Reporter:
              tsala Helen Foster
              Peer reviewer:
              Rossiani Wijaya
              Integrator:
              Aparup Banerjee
              Tester:
              Michael de Raadt
              Participants:
              Component watchers:
              Jake Dallimore, Jun Pataleta
              Votes:
              18 Vote for this issue
              Watchers:
              15 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Nov/12