search

openmeetings / openmeetings-moodle-plugin

GNU General Public License v3.0
16 stars 8 forks source link

two issues caused by upgrading from a previous plugin version #9

Closed mwuttke closed 8 years ago

mwuttke commented 8 years ago

Hello Maxim,

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

  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()

Thanks & Greetings, Michael