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

Course > Grades > Reports > Outcomes does not work at PostgreSQL

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.1
    • Fix Version/s: 1.9.2
    • Component/s: Gradebook
    • Labels:
      None
    • Environment:
      PostgreSQL 8.1.5
    • Database:
      PostgreSQL
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      grade/report/outcomes/index.php contains MySQL-specific query and produces an error at PostgreSQL.

        Gliffy Diagrams

          Activity

          Hide
          mudrd8mz David Mudrak added a comment -

          There were two problems.

          • the patch removes ` ` wrapping column names
          • it fixes the problem known as "id must be aggregated or part of GROUP BY statement"

          AFAIK wrapping column names by ` ` is not necessary in this case ("avg" and "count").
          The "itemid" can be used here as a first returned column (and therefore the array key) because data are grouped by it.

          Waiting for +2

          Show
          mudrd8mz David Mudrak added a comment - There were two problems. the patch removes ` ` wrapping column names it fixes the problem known as "id must be aggregated or part of GROUP BY statement" AFAIK wrapping column names by ` ` is not necessary in this case ("avg" and "count"). The "itemid" can be used here as a first returned column (and therefore the array key) because data are grouped by it. Waiting for +2
          Hide
          skodak Petr Skoda added a comment -

          +2

          Show
          skodak Petr Skoda added a comment - +2
          Hide
          mudrd8mz David Mudrak added a comment -

          Fixed in CVS HEAD and MOODLE_19_STABLE - see revs 1.14 and 1.11.2.3

          Show
          mudrd8mz David Mudrak added a comment - Fixed in CVS HEAD and MOODLE_19_STABLE - see revs 1.14 and 1.11.2.3
          Hide
          skodak Petr Skoda added a comment -

          thanks

          Show
          skodak Petr Skoda added a comment - thanks

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Jul/08