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 2.4 Branch:
      MDL-37375_m24
    • Pull Master Branch:
    • Rank:
      46998

      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.

        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: