Moodle
  1. Moodle
  2. MDL-15036

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

    Details

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

      Description

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

        Activity

        Hide
        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
        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
        Petr Škoda added a comment -

        +2

        Show
        Petr Škoda added a comment - +2
        Hide
        David Mudrak added a comment -

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

        Show
        David Mudrak added a comment - Fixed in CVS HEAD and MOODLE_19_STABLE - see revs 1.14 and 1.11.2.3
        Hide
        Petr Škoda added a comment -

        thanks

        Show
        Petr Škoda added a comment - thanks

          People

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

            Dates

            • Created:
              Updated:
              Resolved: