Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.3
    • Fix Version/s: 1.9.5
    • Component/s: Tags
    • Labels:
      None
    • Database:
      PostgreSQL
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      Nothing happens when I choose some flagged tags and try to reset the flags. Moodle shows:
      "tag-name" – Reset
      But the tag is still flagged :-/

        Gliffy Diagrams

          Activity

          Hide
          mudrd8mz David Mudrak added a comment -

          The same observed at my site running the latest Moodle 1.9.4+. Assigning to myself - I am going to look at it tommorow.

          Show
          mudrd8mz David Mudrak added a comment - The same observed at my site running the latest Moodle 1.9.4+. Assigning to myself - I am going to look at it tommorow.
          Hide
          mudrd8mz David Mudrak added a comment -

          This seems to be PostgreSQL specific bug. Postgres does not seem to support table alias in UPDATE statement. Therefore the SQL like

          UPDATE mdl_tag tg SET tg.flag = 0, tg.timemodified = 1236027984 WHERE tg.id IN (4)

          ends with an error
          ERROR: column "tg" of relation "mdl_tag" does not exist LINE 1: UPDATE mdl_tag tg SET tg.flag = 0, tg.timemodified = 1236027...

          The fix is quite easy - just do not use table alias as it makes no sense here anyway

          Show
          mudrd8mz David Mudrak added a comment - This seems to be PostgreSQL specific bug. Postgres does not seem to support table alias in UPDATE statement. Therefore the SQL like UPDATE mdl_tag tg SET tg.flag = 0, tg.timemodified = 1236027984 WHERE tg.id IN (4) ends with an error ERROR: column "tg" of relation "mdl_tag" does not exist LINE 1: UPDATE mdl_tag tg SET tg.flag = 0, tg.timemodified = 1236027... The fix is quite easy - just do not use table alias as it makes no sense here anyway
          Hide
          mudrd8mz David Mudrak added a comment -

          Fixed in MOODLE_19_STABLE and merged into HEAD.

          Show
          mudrd8mz David Mudrak added a comment - Fixed in MOODLE_19_STABLE and merged into HEAD.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Verified. Closing. Thanks David! B-)

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Verified. Closing. Thanks David! B-)
          Hide
          schorsch101 Dennis Meyer added a comment -

          Thanks a lot!

          Show
          schorsch101 Dennis Meyer added a comment - Thanks a lot!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/May/09