Thanks! done
On Mon, Mar 25, 2013 at 10:01 PM, <michael.wut...@eledia.de> wrote: > Hello Maxim, > > sorry, but I found one more error message in moodle. > > Could you add to ~/mod/openmeetings/lang/en/openmeetings.php and the other > language-files: > $string['openmeetings:becomemoderator'] = 'Become moderator in an > Openmeetings Room'; > > And please could you also count up the version in > ~/mod/openmeetings/version.php? > f.e.: > $module->version = 2013032500; > > Thank you! > Michael > > > done! > > Committed revision 10. > > > > The build will be available at > > > https://builds.apache.org/view/M-R/view/OpenMeetings/job/OpenMeetings%20Moodle%20Plugin/ > > (130 or later) > > > > Thanks for your contribution! > > > > > > On Fri, Mar 22, 2013 at 10:45 PM, <michael.wut...@eledia.de> wrote: > > > >> Hello Maxim, > >> > >> just one thing, please! > >> > >> If I start a online session/open a conference-room from the moodle site > >> I > >> get the following error message in the debug mode on the moodle site: > >> > >> --- > >> This page did not call $PAGE->set_url(...). Using > >> http://localhost/mod/openmeetings/view.php?id=745 > >> > >> line 557 of /lib/pagelib.php: call to debugging() > >> line 717 of /lib/pagelib.php: call to moodle_page->magic_get_url() > >> line 530 of /lib/outputrenderers.php: call to moodle_page->__get() > >> line 62 of /theme/base/layout/general.php: call to > >> core_renderer->login_info() > >> line 804 of /lib/outputrenderers.php: call to include() > >> line 734 of /lib/outputrenderers.php: call to > >> core_renderer->render_page_layout() > >> line 1232 of /lib/deprecatedlib.php: call to core_renderer->header() > >> line 75 of /mod/openmeetings/view.php: call to print_header() > >> --- > >> > >> Please can you add in the file ~/mod/openmeetings/view.php the following > >> line behind require_login($course->id); > >> > >> $PAGE->set_url('/mod/openmeetings/view.php', array('id' => $cm->id)); > >> > >> according to the Page API of moodle: > >> http://docs.moodle.org/dev/Page_API > >> > >> Thank you! > >> Michael > >> > >> > Hello Maxim, > >> > > >> > yes this is ok. > >> > > >> > Thank you! > >> > Michael > >> > > >> >> I did all changes (not commited yet) > >> >> except for the change in version.php > >> >> > >> >> according http://docs.moodle.org/dev/version.php > >> >> it is strongly recommended > >> >> I have changed $plugin with $module is it OK? > >> >> > >> >> > >> >> > >> >> On Fri, Mar 22, 2013 at 9:23 PM, <michael.wut...@eledia.de> wrote: > >> >> > >> >>> Hello Maxim, > >> >>> > >> >>> I tried the plugin with the moodle version 2.4.3 in the debug mode > >> in > >> >>> moodle and I got some errors. > >> >>> > >> >>> Please could you change (add/remove) the following strings? > >> >>> > >> >>> diff -r /tmp/openmeetings/lang/de/openmeetings.php > >> >>> /var/www/moodle24/mod/openmeetings/lang/de/openmeetings.php > >> >>> 66a67,70 > >> >>> > > >> >>> > $string['pluginadministration'] = 'OpenMeetings Administration'; > >> >>> > $string['pluginname'] = 'OpenMeetings'; > >> >>> > > >> >>> diff -r /tmp/openmeetings/lang/en/openmeetings.php > >> >>> /var/www/moodle24/mod/openmeetings/lang/en/openmeetings.php > >> >>> 66a67,71 > >> >>> > > >> >>> > $string['pluginadministration'] = 'OpenMeetings administration'; > >> >>> > $string['pluginname'] = 'OpenMeetings'; > >> >>> > > >> >>> > > >> >>> diff -r /tmp/openmeetings/version.php > >> >>> /var/www/moodle243/mod/openmeetings/version.php > >> >>> 24d23 > >> >>> < $plugin->component = 'mod_openmeetings'; > >> >>> 29,30c28 > >> >>> < > >> >>> > >> >>> Thank you for your help! > >> >>> Michael > >> >>> > >> >>> > sure > >> >>> > can you create issue here > >> >>> > > >> >>> > >> > http://code.google.com/a/apache-extras.org/p/openmeetings-moodle-plugin/issues/list > >> >>> > > >> >>> > not sure how soon I'll have time to implement this > >> >>> > do not hesitate to ping me regarding this :) > >> >>> > > >> >>> > > >> >>> > On Tue, Feb 19, 2013 at 9:57 PM, <michael.wut...@eledia.de> > wrote: > >> >>> > > >> >>> >> Hello, > >> >>> >> > >> >>> >> I tried this plugin to and it worked fine. > >> >>> >> > >> >>> >> Is it possible to expand the plugin with the possibility to > >> select > >> >>> >> popup/full screen mode for using a OM-room from a moodle-course > >> >>> site? > >> >>> >> > >> >>> >> Thank you for your work! > >> >>> >> Michael > >> >>> >> > >> >>> >> > Bravo Maxim! > >> >>> >> > > >> >>> >> > it works perfect. > >> >>> >> > > >> >>> >> > Thanks > >> >>> >> > > >> >>> >> > Alvaro > >> >>> >> > > >> >>> >> > ............. > >> >>> >> > > >> >>> >> > El mar, 19-02-2013 a las 01:25 +0700, Maxim Solodovnik > >> escribió: > >> >>> >> >> seems to be fixed > >> >>> >> >> please try new version from > >> >>> >> >> here > >> >>> >> >> > >> >>> >> > >> >>> > >> > https://builds.apache.org/view/M-R/view/OpenMeetings/job/OpenMeetings%20Moodle%20Plugin/ > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> according to my tests works for both 2.3 and 2.4.1 > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> On Tue, Feb 19, 2013 at 12:18 AM, Alvaro <zurca...@gmail.com> > >> >>> wrote: > >> >>> >> >> Hi, > >> >>> >> >> > >> >>> >> >> After install Moodle 2.41 (the last until now) and the > >> OM > >> >>> >> >> plugin 1.4 > >> >>> >> >> version, when try to build a room from Moodle, get > >> this > >> >>> >> error: > >> >>> >> >> > >> >>> >> >> ----- > >> >>> >> >> Can not find data record in database table course. > >> >>> >> >> More information about this error > >> >>> >> >> ----- > >> >>> >> >> > >> >>> >> >> Somes others spanish Apache OM list users get the same > >> >>> error. > >> >>> >> >> > >> >>> >> >> Could you please tell something about? > >> >>> >> >> > >> >>> >> >> Thank you > >> >>> >> >> > >> >>> >> >> Alvaro > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> > >> >>> >> >> -- > >> >>> >> >> WBR > >> >>> >> >> Maxim aka solomax > >> >>> >> > > >> >>> >> > > >> >>> >> > > >> >>> >> > >> >>> >> > >> >>> > > >> >>> > > >> >>> > -- > >> >>> > WBR > >> >>> > Maxim aka solomax > >> >>> > > >> >>> > >> >> > >> >> > >> >> > >> >> -- > >> >> WBR > >> >> Maxim aka solomax > >> >> > >> > > >> > > >> > >> > > > > > > -- > > WBR > > Maxim aka solomax > > > > -- WBR Maxim aka solomax