Refactor group permission function.

[Jonas-Sander]

I stumbled on the requestPermission function in our code which was used to check if a user can perform a certain action, e.g. adding homework or changing course settings.

This function was weirdly named and had logic for enum values which were not in use anymore.

I changed the function to an extension method on MemberRole and removed the unused enums.
Additionally I moved the files from the additional to groups folder.
Lastly I renamed the function isUserAdminOrOwnerFromCourse to isUserAdminOrOwnerOfGroup in the same file.

[github-actions]

Visit the preview URL for this PR (updated for commit 25287db):

https://sharezone-test--pr332-course-permission-fu-vc3kafwv.web.app

_{(expires Fri, 13 Jan 2023 17:15:54 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 4cb3ae61e1e018abfd9841fd3239f5b49ccc034b}

[nilsreichardt]

LGTM 👍

Would be great to also add a few tests for the group_permission.dart file, but we can also do this later.

[Jonas-Sander]

LGTM 👍

Would be great to also add a few tests for the group_permission.dart file, but we can also do this later.

Since I didn't touch any behavior (just deleted unused code) and the code is really dead simple I don't see that much reason to write tests. They would just mirror the implementation.

I feel like it would make more sense to write tests in the future that test this function through the code that uses it.
This will require more refactoring though.

[nilsreichardt]

⬇️ Generated builds by Codemagic for commit 25287db ⬇️

Note: Only Sharezone team members are able to install the iOS app.

iOS	Android
Download link	Download link