Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Syntax Error when running local_o365\task\usersync #2184

Closed
peta3000 opened this issue Sep 21, 2022 · 4 comments
Closed

Syntax Error when running local_o365\task\usersync #2184

peta3000 opened this issue Sep 21, 2022 · 4 comments
Assignees
Labels
Feature - user sync Issue type - bug Bugs in existing code that needs to be fixed. Plugin - local_o365 Status - PR ready / pending release Dev is done and PR ready. Will be included in the next release.

Comments

@peta3000
Copy link

Hi @weilai-irl
When running the usersync script a client receives the following syntax-error:
usersync_syntax-error

Context:
database: postgres
moodle-version: 4.0.4
local_o365 version: 2022041905 (current master branch, 4.0.1 release)

According to the syntax of the "JOIN" clause the "ON" condition seems to be optional in MySQL but expected in other database-types:
here a reference discussion on stackoverflow:
https://stackoverflow.com/questions/16470942/how-to-use-mysql-join-without-on-condition

the problematic sql-query seems to be at the end of this php-file:
https://github.com/microsoft/o365-moodle/blob/MOODLE_40_STABLE/local/o365/lib.php

Has there already been a patch in the meanwhile?

Thanks a lot!
With best regards!

peta3000

related issue(s):
#2173

@weilai-irl weilai-irl self-assigned this Sep 27, 2022
@weilai-irl weilai-irl added this to the Release 2022-09 milestone Sep 27, 2022
@weilai-irl weilai-irl added Issue type - bug Bugs in existing code that needs to be fixed. Feature - user sync Plugin - local_o365 Status - PR ready / pending release Dev is done and PR ready. Will be included in the next release. labels Sep 27, 2022
@weilai-irl
Copy link
Collaborator

Hi @peta3000,

I have updated the code change fixing the linked issue #2173 to cater for this issue as well. You can try it out in your environment. The fix will be included in the next release.

Regards,
Lai

@peta3000
Copy link
Author

Dear @weilai-irl
Thanks a lot for your reply!
We are going to wait for the next release then :)

Many thanks and best regards

peta

@weilai-irl
Copy link
Collaborator

Hi all,

The fix to this issue has been included in the release from today. Please upgrade your plugins to the latest version to see the changes.

I'm going to close the issue now.

Regards,
Lai

@peta3000
Copy link
Author

peta3000 commented Oct 6, 2022

Thanks a lot for your efforts @weilai-irl !!!

Looking forward to giving it a try! :)
Best regards
peta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature - user sync Issue type - bug Bugs in existing code that needs to be fixed. Plugin - local_o365 Status - PR ready / pending release Dev is done and PR ready. Will be included in the next release.
Projects
None yet
Development

No branches or pull requests

2 participants