Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: 1.8
    • Component/s: Groups
    • Labels:
      None
    • Environment:
      Oracle
    • Database:
      Oracle
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE

      Description

      I was looking groups code when I've detected that the "AS" keyword is being used to alias tables.

      This totally forbidden by rule number 11 in http://docs.moodle.org/en/Coding#Database_structures is order to allow Oracle to work properly.

      Easy to fix

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Assigning to Nick

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Assigning to Nick
              Hide
              dougiamas Martin Dougiamas added a comment -

              Nick, sorry I missed you on Skype just now. Do you have time to clear this blocker?

              Show
              dougiamas Martin Dougiamas added a comment - Nick, sorry I missed you on Skype just now. Do you have time to clear this blocker?
              Hide
              nfreear Nick Freear added a comment -

              Fixed with this patch, which removes 'AS', fixes missing quotes and moves some SQL from lib/ to db/ directory.

              Show
              nfreear Nick Freear added a comment - Fixed with this patch, which removes 'AS', fixes missing quotes and moves some SQL from lib/ to db/ directory.
              Hide
              nfreear Nick Freear added a comment -

              Fixed, committed.

              Show
              nfreear Nick Freear added a comment - Fixed, committed.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    31/Mar/07