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

          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