Uploaded image for project: 'Plugins'
  1. Plugins
  2. CONTRIB-3686

Error reading from database in function get_strings of HotPot

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.3
    • Fix Version/s: 2.0.10
    • Component/s: Module: HotPot
    • Labels:
      None
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      When reviewing attempts on mod/hotpot/review.php?id=...

      Debug info: ERROR: invalid input syntax for integer: "35,36"
      SELECT id,string FROM mdl_hotpot_strings WHERE id = $1
      [array (
      0 => '35,36',
      )]
      Stack trace:
      line 394 of /lib/dml/moodle_database.php: dml_read_exception thrown
      line 232 of /lib/dml/pgsql_native_moodle_database.php: call to moodle_database->query_end()
      line 678 of /lib/dml/pgsql_native_moodle_database.php: call to pgsql_native_moodle_database->query_end()
      line 1129 of /lib/dml/moodle_database.php: call to pgsql_native_moodle_database->get_records_sql()
      line 837 of /mod/hotpot/locallib.php: call to moodle_database->get_records_select()
      line 257 of /mod/hotpot/attempt/review.php: call to hotpot::get_strings()
      line ? of unknownfile: call to mod_hotpot_attempt_review::review()
      line 79 of /mod/hotpot/review.php: call to call_user_func()

      It occurs because in hotpot_responses table has a comma sepparated value in the column correct.

        Attachments

          Activity

            People

            • Assignee:
              xxxxxxx Gordon Bateson
              Reporter:
              pferre22 Pau Ferrer
              Participants:
              Component watchers:
              Gordon Bateson
            • Votes:
              2 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Jul/12