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

MP groups and permissions #2783

Merged
merged 19 commits into from Jan 24, 2016
Merged

MP groups and permissions #2783

merged 19 commits into from Jan 24, 2016

Conversation

zsilencer
Copy link
Contributor

mp

@IntelOrca
Copy link
Contributor

Impressive.
Currently anyone with full access can set other clients including themselves to spectator / full access. This should probably only be allowed by host. Alternatively there could just an additional "moderator" permission level.

Once this gets merged, we can all start making the interface read-only when player is a spectator. E.g. changing drop downs to just plain text.

@zsilencer
Copy link
Contributor Author

I plan on expanding the group management tab so that you can create new groups with fine grained control, such as allowing only certain game commands, and boundaries on the map where things can be placed, etc. Right now just wanted these two defaults, and I made it so the host always has full access, with a setting of a default group of any players joining. So you can just have you and your buddies working on a park without needing a password, and can let anyone join, watch and chat.

Making everything in the interfaces read-only might be a big pain, and they get a error box saying permission denied when they try things, so it shouldn't be too hard for users to understand.

@IntelOrca
Copy link
Contributor

Currently clients with full access can change the permissions of other clients, this shouldn't be allowed for the time being.

@janisozaur
Copy link
Member

Cool stuff. Out of curiosity – does OpenTTD has something like that?

@zsilencer zsilencer force-pushed the multiplayer branch 2 times, most recently from 31f7be7 to 14ae348 Compare January 21, 2016 04:36
@zsilencer zsilencer force-pushed the multiplayer branch 3 times, most recently from 71384a9 to 02f4359 Compare January 22, 2016 07:27
@zsilencer
Copy link
Contributor Author

@IntelOrca K I have this mostly complete now, and have added a default User group that does not have permission to modify or assign groups to address your concern. Maybe some people can test this.

@IntelOrca
Copy link
Contributor

@LRFLEW could you xcode this PR?

}break;
case 3:{ // set group name
size_t nameChunkOffset = nameChunkIndex - 1;
if (nameChunkOffset < 0)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

size_t is unsigned, so this is always false.

@LRFLEW
Copy link
Contributor

LRFLEW commented Jan 22, 2016

@zsilencer I made a PR against your branch to fix the Xcode project. I also fixed one of @janisozaur's comments, as it was causing a warning in Xcode.

@zsilencer
Copy link
Contributor Author

still have some things to change so do not merge yet

@IntelOrca
Copy link
Contributor

This reminds me of Locomotion which showed you what other players were looking at.

@IntelOrca
Copy link
Contributor

Clients can still pick up and move guests even though the guest permission is disabled.

@zsilencer
Copy link
Contributor Author

alright i fixed that, let me know if anyone finds any other issues.

@IntelOrca
Copy link
Contributor

Does it save the groups and permissions?

IntelOrca added a commit that referenced this pull request Jan 24, 2016
@IntelOrca IntelOrca merged commit 6eaa93c into OpenRCT2:develop Jan 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants