How would I make sure the person who clicked it is in the group and change text with a TextButton?
Basically I asked a previous question which I thought when it was answered would answer the problem to what I'm trying to do.
01 | GroupID = 3713839 |
02 | Button = script.Parent.Screen.SurfaceGui.Frame.Button |
03 | Button.MouseButton1Click:Connect(function(Clicked) |
04 | local Player = game.Players:GetPlayerFromCharacter(Clicked.Parent.Parent) |
05 | local ServerStorage = game.ServerStorage |
06 | if Player:IsInGroup(GroupID) then |
07 | Button.Text = "Currently in use" |
08 | Button.TextSize = 55 |
09 | end |
10 | end) |
I don't know what to do. The group Id is correct I just can't get the player.
2 answers
Everything is correct except for the fact that you're trying to get the player from the nothing. There is no argument from MouseButton1Click. If you do insert a click detector, however, you can do this:
01 | local groupId = 3713839 |
02 | local button = script.Parent.Screen.ClickDetector |
03 |
04 | button.MouseClick:(function(player) |
05 | local serverStorage = game:GetService("ServerStorage') |
06 |
07 | if player:IsInGroup(groupId) then |
08 | button.Text = "Currently in use." |
09 | button.TextSize = 55 |
10 | end |
11 | end) |
0
0
The first question you asked was about when you touch a part. If your doing it via a GUI then you do not need to use GetPlayerFromCharacter as you are using a local script for a GUI.
A local script already knows who the player is. Therefore you can reference the player as:
1 | player = game.Players.LocalPlayer |
Therefore you can adjust your script as such (put the local script in the button):
01 | GroupID = 3713839 |
02 |
03 | --Function that runs when the button is clicked. |
04 | script.Parent.MouseButton1Click:Connect(function() |
05 | --References the player (you can only reference the player like this in a local script.) |
06 | local Player = game.Players.LocalPlayer |
07 | --Checks if the player is in the group and changes the text if so. |
08 | if Player:IsInGroup(GroupID) then |
09 | script.Parent.Text = "Currently in use" |
10 | script.Parent.TextSize = 55 |
11 | end |
12 | end) |
0
0