Merged, sorry for PR #8, I was sure I have merged it :( On Sun, May 15, 2016 at 4:47 PM, Michael Wuttke <mwut...@beuth-hochschule.de > wrote:
> 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 > -- WBR Maxim aka solomax
