Issue Details (XML | Word | Printable)

Key: MDL-17492
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: David Mudrak
Reporter: Dennis Meyer
Votes: 3
Watchers: 2
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Reset Flag (Manage Tags)

Created: 02/Dec/08 09:01 PM   Updated: 04/Mar/09 05:30 PM
Component/s: Tags
Affects Version/s: 1.9.3
Fix Version/s: 1.9.5

Database: PostgreSQL
Participants: David Mudrak, Dennis Meyer and Eloy Lafuente (stronk7)
Security Level: None
QA Assignee: Eloy Lafuente (stronk7)
Resolved date: 03/Mar/09
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE


 Description  « Hide
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 :-/

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
David Mudrak added a comment - 03/Mar/09 05:43 AM
The same observed at my site running the latest Moodle 1.9.4+. Assigning to myself - I am going to look at it tommorow.

David Mudrak added a comment - 03/Mar/09 06:27 AM
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


David Mudrak added a comment - 03/Mar/09 06:53 AM
Fixed in MOODLE_19_STABLE and merged into HEAD.

Eloy Lafuente (stronk7) added a comment - 03/Mar/09 07:33 AM
Verified. Closing. Thanks David! B-)

Dennis Meyer added a comment - 04/Mar/09 05:30 PM
Thanks a lot!