Moodle

Workshop cron produces invalid SQL statement

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Won't Fix
  • Affects Version/s: 1.6.4
  • Fix Version/s: 1.9.5
  • Component/s: Forum, General, Workshop
  • Labels:
    None
  • Database:
    PostgreSQL
  • Affected Branches:
    MOODLE_16_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

I can see these error messages in my database log file. It seems that at least cron.php is using invalid SQL statement and my database server complains about this:

2007-02-19 10:25:01 CET moodle_cz SELECTERROR: invalid input syntax for integer: ""
2007-02-19 10:25:01 CET moodle_cz SELECTSTATEMENT: SELECT * FROM mdl_workshop_assessments WHERE id = '' LIMIT 1
...
2007-02-19 10:28:25 CET moodle_cz SELECTERROR: invalid input syntax for integer: ""
2007-02-19 10:28:25 CET moodle_cz SELECTSTATEMENT: SELECT * FROM mdl_forum_subscriptions WHERE userid = '' AND forum = '5' LIMIT 1
...
2007-02-19 10:30:01 CET moodle_cz SELECTERROR: invalid input syntax for integer: ""
2007-02-19 10:30:01 CET moodle_cz SELECTSTATEMENT: SELECT * FROM mdl_workshop_assessments WHERE id = '' LIMIT 1
...
2007-02-19 10:35:01 CET moodle_cz SELECTERROR: invalid input syntax for integer: ""
2007-02-19 10:35:01 CET moodle_cz SELECTSTATEMENT: SELECT * FROM mdl_workshop_assessments WHERE id = '' LIMIT 1

Thus Moodle should check if it is providing valid integers before executing the query.

Activity

Hide
Martin Dougiamas added a comment -

Assigning to me temporarily because Vy-Shane no longer works for Moodle HQ.

Show
Martin Dougiamas added a comment - Assigning to me temporarily because Vy-Shane no longer works for Moodle HQ.
Hide
Petr Škoda (skodak) added a comment -

another for you

Show
Petr Škoda (skodak) added a comment - another for you
Hide
David Mudrak added a comment -

As I am the only one reporting this and given that the module is not standard one at the moment, I am closing.

Show
David Mudrak added a comment - As I am the only one reporting this and given that the module is not standard one at the moment, I am closing.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: