Moodle
  1. Moodle
  2. MDL-37375

Cannot access related blog post from activity report

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.3.5, 2.4.2
    • Component/s: Blog, Reports
    • Labels:
    • Testing Instructions:
      Hide
      1. Create assignment
      2. Add the Blog menu block to the assignment page
      3. Post an associated visible blog post for the assignment.
      4. From blog menu click View all entries about this Assignment - post is visible (by any user).
      5. Go to Navigation > Reports > Activity report /report/outline/index.php?id=XX for the course.
      6. Click associated blog entries link for the assignment.

      Expected result: New blog post should be visible.

      Show
      Create assignment Add the Blog menu block to the assignment page Post an associated visible blog post for the assignment. From blog menu click View all entries about this Assignment - post is visible (by any user). Go to Navigation > Reports > Activity report /report/outline/index.php?id=XX for the course. Click associated blog entries link for the assignment. Expected result: New blog post should be visible.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Replication steps:

      1. Create assignment
      2. Post an associated visible blog post.
      3. From blog menu click View all entries about this Assignment - post is visible (by any user).
      4. Go to Navigation > Reports > Activity report /report/outline/index.php?id=XX for the course.
      5. Click associated blog entries link for the assignment.

      Expected result: New blog post should be visible.

      Actual result: Blog post is not displayed.

        Gliffy Diagrams

          Activity

          Hide
          Michael de Raadt added a comment -

          Thanks for reporting that.

          Show
          Michael de Raadt added a comment - Thanks for reporting that.
          Hide
          Ankit Agarwal added a comment - - edited

          Hi Rosie,
          The patch looks great.

          [y] Syntax
          [y] Output
          [y] Whitespace
          [-] Language
          [-] Databases
          [y] Testing
          [-] Security
          [-] Documentation
          [y] Git
          [y] Sanity check

          Feel free to send it for integration.
          Thanks

          Show
          Ankit Agarwal added a comment - - edited Hi Rosie, The patch looks great. [y] Syntax [y] Output [y] Whitespace [-] Language [-] Databases [y] Testing [-] Security [-] Documentation [y] Git [y] Sanity check Feel free to send it for integration. Thanks
          Hide
          Rossiani Wijaya added a comment -

          Thanks Ankit.

          Submitting for integration review.

          Show
          Rossiani Wijaya added a comment - Thanks Ankit. Submitting for integration review.
          Hide
          Dan Poltawski added a comment -

          Integrated to master, 24 and 23 - thanks Rosie.

          Show
          Dan Poltawski added a comment - Integrated to master, 24 and 23 - thanks Rosie.
          Hide
          Michael de Raadt added a comment -

          Test result: Success!

          Tested in 2.3, 2.4 and Master.

          Show
          Michael de Raadt added a comment - Test result: Success! Tested in 2.3, 2.4 and Master.
          Hide
          Ray Lawrence added a comment -

          Thanks all.

          Show
          Ray Lawrence added a comment - Thanks all.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Surely you will be happy to know that your code is now part of Moodle upstream. Thanks, thanks!

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Surely you will be happy to know that your code is now part of Moodle upstream. Thanks, thanks! Closing as fixed, ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: