openmeetings / openmeetings-moodle-plugin

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

using module error #30

Closed mafzalzadeh closed 6 years ago

mafzalzadeh commented 6 years ago

Hi.

When I want add an openmeetings action it shows these errors:

line 35 of \mod\openmeetings\api\OmRestService.php: ArgumentCountError thrown
line 82 of \mod\openmeetings\api\OmGateway.php: call to OmRestService->__construct()
line 52 of \mod\openmeetings\mod_form.php: call to OmGateway->login()
line 135 of \course\modedit.php: call to require_once()

Thanks.

solomax commented 6 years ago

What version of Moodle are you using? What version of Moodle plugin are you using? What version of PHP do you have?

mafzalzadeh commented 6 years ago

Moodle 3.5.2 Moodle plugin Release 3.0.0.6 PHP Version 7.1.10

Thanks

solomax commented 6 years ago

Just have tested with:

Moodle 3.5.2 OM plugin 3.0.0.6 PHP 7.0.32 (under Ubuntu 16)

Everything seems to work What OS do you have?

mafzalzadeh commented 6 years ago

I'm Testing on windows10 with xampp. If need to change OS or anything I can do it.

thanks

solomax commented 6 years ago

I have tested under Apache Web Server so far .... there also was nginx installation some time ago

Will try to set up VM with Win10 and xampp, not sure how long it will take :(

mafzalzadeh commented 6 years ago

No Problem for wating. I'll also try to do that and will tell you the result Thanks.

solomax commented 6 years ago

Reproducible using XAMPP+PHP 7.1 on Win10 Will see what I can do

Thanks for the report

solomax commented 6 years ago

Should be fixed, next version will be released right after 4.0.6

mafzalzadeh commented 6 years ago

after the changes I still have problem with connecting:

Exception - Too few arguments to function OmRestService::__construct(), 0 passed in [dirroot]\mod\openmeetings\api\OmGateway.php on line 78 and exactly 1 expected

solomax commented 6 years ago

here is the line #78: https://github.com/openmeetings/openmeetings-moodle-plugin/blob/master/api/OmGateway.php#L78 there is no OmRestService call ...

How have you applied changes to your server?

mafzalzadeh commented 6 years ago

I just copied and replaced all of the new version files from the git.

solomax commented 6 years ago

I'm afraid it might not work this way :( You need to clone git repo and run ant in cloned folder then replace moodle/mod/openmeetings folder contents with contents of dist folder created by build script

mafzalzadeh commented 6 years ago

OK. Thanks. I'll do it and tell you the result.

mafzalzadeh commented 6 years ago

@solomax The problem solved. Thanks.

solomax commented 6 years ago

Great! Thanks for report and for checking Plugin will be released next week