BasePermissions.to_json() issue?

[dksdarkness]

Hi there! 👋

First of all, thank you for this great library — it's been very helpful!

While migrating roles from one SharePoint site to another, I encountered some unexpected behavior. Upon investigating, I noticed the following method in base_permissions.py:

https://github.com/vgrem/Office365-REST-Python-Client/blob/master/office365/sharepoint/permissions/base_permissions.py:

def to_json(self, json_format=None): return {"Low": str(self.High), "High": str(self.Low)}

Shouldn't it be the other way around?:

{"Low": str(self.Low), "High": str(self.High)}

Best Regards!

[vgrem]

Greetings!

Thank you for catching it, it appears variables themselves are misnamed here: self.High actually stores the Low value, and self.Low stores the High value. Thanks, it will be addresed in the upcoming version.

[dksdarkness]

I'm happy to help!

Thank yoy for all your efforts!!