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

Course report logs link broken in logs

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2.2
    • Component/s: Administration
    • Labels:
    • Testing Instructions:
      Hide

      1/ visit some report
      2/ go to logs and click on the relevant link

      (ignore any problems on the target page caused by invalid params, this fixes only the urls - the rest should be fixed during log subsystem rewrite in future)

      Show
      1/ visit some report 2/ go to logs and click on the relevant link (ignore any problems on the target page caused by invalid params, this fixes only the urls - the rest should be fixed during log subsystem rewrite in future)
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w03_MDL-31011_m23_loglinks

      Description

      Goto course>anycourse>report>logs
      Find entries with "course report log" in the action filed
      try clicking on the link, would generate an 404 error
      the link is
      /course/report/log/index.php?id=x
      should be
      /report/log/index.php?id=x

      PS:- I guess we should have a component "logs" in the tracker

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            thanks for the report

            Show
            skodak Petr Skoda added a comment - thanks for the report
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Petr - integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Petr - integrated now
            Hide
            phalacee Jason Fowler added a comment -

            worked well

            Show
            phalacee Jason Fowler added a comment - worked well
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This is now available in the git and cvs repositories.

            Consider the responsibility of your fingerprints engraved there for future generations!

            Thanks for the work, closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This is now available in the git and cvs repositories. Consider the responsibility of your fingerprints engraved there for future generations! Thanks for the work, closing, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Mar/12