Allow usage of Microsoft graph to lookup guid to group names on EntraID

[small1]

This patch will allow to lookup group guid on Microsoft Graph and store that as group name in Nextcloud.

It will only lookup groups if "Fetch group names from Microsoft Graph" is checked. Otherwise it will run as normal.

Code has been tested and works against EntraID

Fixes #640 and #1017

[julien-nc]

Thank you for the PR!

• I feel like this change is too specific to one provider to justify adding an option in the UI. It would be fine if the flag can only be set via the occ command. Wdyt?
• Can you rename the new option in the upsert command from entraid to something more explicit like azure-group-names?
• Can you isolate the 2 change blocks in ProvisioningService.php in new methods to avoid making the getSyncGroupsOfToken one too long and more difficult to read?

[small1]

Thank you for the PR!

• I feel like this change is too specific to one provider to justify adding an option in the UI. It would be fine if the flag can only be set via the occ command. Wdyt?
• Can you rename the new option in the upsert command from entraid to something more explicit like azure-group-names?
• Can you isolate the 2 change blocks in ProvisioningService.php in new methods to avoid making the getSyncGroupsOfToken one too long and more difficult to read?

In this case it is EntraID so entraid-group-names yes sure (This was developed when the name was Azure AD). And we can remove the ui option. I actually just added the occ command when i submitted this.

The last one i can check. A bit above my development skills Ill make an effort and ask for some assistance on that one.

[julien-nc]

Sure feel free to ask for assistance. Are you asking me?

About my last change request, it would be replacing each of the 2 blocks you inserted in ProvisioningService.php by a call to a method that contains the block. So everything you added would end up in 2 methods of this class and those methods would be called in getSyncGroupsOfToken.

I can make this change (after you've made the other adjustments) if you ask me to.

[small1]

Sure feel free to ask for assistance. Are you asking me?

About my last change request, it would be replacing each of the 2 blocks you inserted in ProvisioningService.php by a call to a method that contains the block. So everything you added would end up in 2 methods of this class and those methods would be called in getSyncGroupsOfToken.

I can make this change (after you've made the other adjustments) if you ask me to.

I have made changes and tested them. I would appretiate assistance with the method part from you @julien-nc :)

[julien-nc]

@small1 Can you validate the changes i made in ed40f6b ?

[small1]

@small1 Can you validate the changes i made in ed40f6b ?

Tested and verified. It works on my system against EntraID. Thank you very much!