Moodle
  1. Moodle
  2. MDL-31011

Course report logs link broken in logs

    Details

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

      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

        Activity

        Hide
        Petr Škoda added a comment -

        thanks for the report

        Show
        Petr Škoda added a comment - thanks for the report
        Hide
        Sam Hemelryk added a comment -

        Thanks Petr - integrated now

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

        worked well

        Show
        Jason Fowler added a comment - worked well
        Hide
        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
        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: