Почему я получаю ошибку синтаксиса SQL в Moodle 3.3? - PullRequest
Новая
       31

Почему я получаю ошибку синтаксиса SQL в Moodle 3.3?

0 голосов
/ 07 июля 2019

Я установил плагин на Moodle 3.3 и теперь получаю ошибку синтаксиса SQL и Error reading from database. Я сделал некоторую отладку PHP, дав следующее описание: 

Debug info: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'OR target = LIMIT 0, 1' at line 1
SELECT 'x' FROM mdl_dasis_relations WHERE source = OR target = LIMIT 0, 1 [array ( )]
Error code: dmlreadexception
Stack trace:
line 486 of /lib/dml/moodle_database.php: dml_read_exception thrown
line 1184 of /lib/dml/mysqli_native_moodle_database.php: call to moodle_database->query_end()
line 1889 of /lib/dml/moodle_database.php: call to mysqli_native_moodle_database->get_recordset_sql()
line 1874 of /lib/dml/moodle_database.php: call to moodle_database->record_exists_sql()
line 76 of /blocks/semantic_web/block_semantic_web.php: call to moodle_database->record_exists_select()
line 709 of /blocks/moodleblock.class.php: call to block_semantic_web->get_content()
line 230 of /blocks/moodleblock.class.php: call to block_list->formatted_contents()
line 1205 of /lib/blocklib.php: call to block_base->get_content_for_output()
line 1257 of /lib/blocklib.php: call to block_manager->create_block_contents()
line 579 of /lib/outputrenderers.php: call to block_manager->ensure_content_created()
line 39 of /theme/bootstrapbase/renderers/core_renderer.php: call to core_renderer->standard_head_html()
line 44 of /theme/clean/layout/columns3.php: call to theme_bootstrapbase_core_renderer->standard_head_html()
line 1162 of /lib/outputrenderers.php: call to include()
line 1092 of /lib/outputrenderers.php: call to core_renderer->render_page_layout()
line 68 of /course/index.php: call to core_renderer->header()

Я также провел некоторые исследования, и я думаю, что эта часть файла block_semantic_web.php выдает ошибку: 

    function setLastActivity() {
    global $USER, $PAGE, $DB;
    if($PAGE->cm) {
        if($DB->record_exists_select("dasis_relations", "source = ".$PAGE->cm->id." OR target = ".$PAGE->cm->id)) {
            $lastActivity = new object();
            $lastActivity->userid = $USER->id;
            $lastActivity->courseid = $PAGE->cm->course;
            $lastActivity->course_module = $PAGE->cm->id;

            if($rec = $DB->get_record("dasis_last_activity", array("userid" => $lastActivity->userid, "courseid" => $lastActivity->courseid))) {
                $lastActivity->id = $rec->id;
                $lastActivity->last_access = date("Y-m-d H:i:s", time());
                $DB->update_record("dasis_last_activity", $lastActivity);
            }else{
                $DB->insert_record("dasis_last_activity", $lastActivity);
            }
        }
    }
}

Я не знаю, что это за синтаксическая ошибка.

PHP: 7.0.33-8+ubuntu18.04.1

MySQL: Ver 14.14 Distrib 5.7.26

1 Ответ

1 голос
/ 07 июля 2019

$PAGE->cm->id не может быть определено, например, оно не определено на страницах вне контекста модуля действия / курса (представление курса, домашняя страница, профиль и т. Д.).Я рекомендую вам проверить, не является ли он пустым, прежде чем звонить setLastActivity или $DB->record_exists_select.Например: 

global $PAGE;
if (!empty($PAGE->cm->id)) {
    // ...
}

Затем (как кто-то заметил ранее) вы можете вызвать его, передав параметры отдельно, чтобы избежать какой-либо уязвимости в SQL-инъекции: 

$select = 'source = :source OR target = :target';
$params = ['source' => $PAGE->cm->id, 'target' => $PAGE->cm->id];
$DB->record_exists_select('dasis_relations', $select, $params);
...