Hello Maxim,

I have found two issues installing the new openmeetings moodle plugin
(version 2.0.0.1). I tested to upgrade the plugin from version 1.7.5 to
version 2.0.01 on moodle 2.9 and moodle 3.0 on postgresql.

1. the renaming of the settings strings (red5host -> host, red5port ->
port, openmeetingsAdminUser -> user, openmeetingsAdminUserPass -> pass &
openmeetingsModuleKey -> moduleKey) effect that the settings are set to
default during the update process and not taken from the previous
local/costum installation.

2. the change of the type of the field 'type' on mdl_openmeetings from
int to character varying(16): creating new rooms will end up with the
following error:

Debug info: FEHLER: ungültige Eingabesyntax für ganze Zahl: „conference“
INSERT INTO mdl_openmeetings
(name,room_id,type,max_user,language,is_moderated_room,allow_recording,whole_window,room_recording_id,course,intro)
VALUES($1,$2,$3,$4,$5,$6,$7,$8,$9,$10,$11) RETURNING id
[array (
'name' => 'test room moodle29 conference inframe',
'room_id' => 54,
'type' => 'conference',
'max_user' => '2',
'language' => '1',
'is_moderated_room' => '1',
'allow_recording' => '1',
'whole_window' => '0',
'room_recording_id' => '9',
'course' => '2',
'intro' => '<p>test room moodle29 conference inframe<br></p>',
)]
Error code: dmlwriteexception
Stack trace:

    line 446 of /lib/dml/moodle_database.php: dml_write_exception thrown
    line 244 of /lib/dml/pgsql_native_moodle_database.php: call to
moodle_database->query_end()
    line 868 of /lib/dml/pgsql_native_moodle_database.php: call to
pgsql_native_moodle_database->query_end()
    line 924 of /lib/dml/pgsql_native_moodle_database.php: call to
pgsql_native_moodle_database->insert_record_raw()
    line 132 of /mod/openmeetings/lib.php: call to
pgsql_native_moodle_database->insert_record()
    line 121 of /course/modlib.php: call to openmeetings_add_instance()
    line 274 of /course/modedit.php: call to add_moduleinfo()

See also issue #9 on github:
https://github.com/openmeetings/openmeetings-moodle-plugin/issues/9

For the second point I created a pull request on github:
https://github.com/openmeetings/openmeetings-moodle-plugin/pull/10

Thanks & Greetings,
Michael

Reply via email to