Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Database SQL/XMLDB
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      It seems that the migration of admin/report/courseoverview to support new DDL API was wrong:

      http://cvs.moodle.org/moodle/admin/report/courseoverview/index.php?r1=1.20&r2=1.21

      apart from the "$pram" typo in line 80, those >params doesn't exist at all and also, there is one $param>timeafter hardcoded in the query.

      Ciao

        Gliffy Diagrams

          Attachments

            Activity

            stronk7 Eloy Lafuente (stronk7) created issue -
            Hide
            skodak Petr Skoda added a comment -

            I hate this spaghetti reports sql, I hope I fixed it but not sure

            Show
            skodak Petr Skoda added a comment - I hate this spaghetti reports sql, I hope I fixed it but not sure
            skodak Petr Skoda made changes -
            Field Original Value New Value
            Assignee Petr ?koda [ skodak ] Eloy Lafuente (stronk7) [ stronk7 ]
            Hide
            skodak Petr Skoda added a comment -

            closing, thanks

            Show
            skodak Petr Skoda added a comment - closing, thanks
            skodak Petr Skoda made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            Assignee Eloy Lafuente (stronk7) [ stronk7 ] Petr ?koda [ skodak ]
            dougiamas Martin Dougiamas made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            dougiamas Martin Dougiamas made changes -
            Workflow jira [ 27195 ] MDL Workflow [ 60180 ]
            dougiamas Martin Dougiamas made changes -
            Workflow MDL Workflow [ 60180 ] MDL Full Workflow [ 89348 ]

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10