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

          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