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

Cannot access related blog post from activity report

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting that.

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting that.
            Hide
            ankit_frenz 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_frenz 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
            rwijaya Rossiani Wijaya added a comment -

            Thanks Ankit.

            Submitting for integration review.

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

            Integrated to master, 24 and 23 - thanks Rosie.

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

            Test result: Success!

            Tested in 2.3, 2.4 and Master.

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

            Thanks all.

            Show
            ray Ray Lawrence added a comment - Thanks all.
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  11/Mar/13