Scripting Helpers is winding down operations and is now read-only. More info→
Ad
Log in to vote
0

I can not figure out why both UI elements show when only one should, maybe this is a bug in roblox?

Asked by
AzrefriskDreemurr 8
4 years ago

So, I have two scripts, one in each item, and both are modified versions of the Pistol model from the Toolbox, designed to look more like a laser weapon, and it tells you how much ammo is left using a Gui element. I have the scripts functional, but for some reason, equipping just one of them causes both Labels to appear. I have them separate so the reloading text doesnt display when another weapon is equipped. So the Admin Version of the weapon makes both the Admin weapon's ui and the Normal weapon's ui appear/disappear, but it should only be making the Admin ui appear. Similar story for the normal version, it makes the admin ui appear.

Normal Version:

view source
01equipEvent.OnServerEvent:connect(function(player)
02    player.Character.Humanoid.AutoRotate = false
03    player.PlayerGui.WeaponsUI.AmmoPistol.Visible = true
04    player.PlayerGui.WeaponsUI.AmmoPistol.Text = currentAmmo.."/"..clipSize
05    plr = player
06end)
07unequipEvent.OnServerEvent:connect(function(player)
08    player.Character.Humanoid.AutoRotate = true
09    player.PlayerGui.WeaponsUI.AmmoPistol.Visible = false
10end)

Admin Version:

view source
01equipEvent.OnServerEvent:connect(function(player)
02    player.Character.Humanoid.AutoRotate = false
03    player.PlayerGui.WeaponsUI.AmmoAdminPistol.Visible = true
04    player.PlayerGui.WeaponsUI.AmmoAdminPistol.Text = currentAmmo.."/"..clipSize
05    plr = player
06end)
07unequipEvent.OnServerEvent:connect(function(player)
08    player.Character.Humanoid.AutoRotate = true
09    player.PlayerGui.WeaponsUI.AmmoAdminPistol.Visible = false
10end)

2 answers

Log in to vote
0
Answered by
AlexTheCreator 461 Moderation Voter
4 years ago
Edited 4 years ago

local equipEvent = ?? and local unqeuipEvent ==??

If they're both listening to the same remote and there is nothing being passed which differentiates between which gun is being equiped then the script will run whenever any tool is equiped.

To fix this, maybe pass the name of the gun ...:Connect(function(player, weapon_name)...end) ... and check if the weapon_name == the name of the gun you are listening for if weapon_name == "the name of the weapon here, inside the speach marks" then ...... end

Ad
Log in to vote
0
Answered by
AzrefriskDreemurr 8
4 years ago

Now that I've checked the naming for the events created by the script, they do all have the same names for the same triggers. I'll have to change all that, but thanks for mentioning that. (Sometimes I overlook the simple stuff like names of objects.)

0
No problem :) I think you should be able to just have one event and one listener script though rather than one for each gun, just pass the name of the item you're equiping and find the appropriate UI using the passed name too :) AlexTheCreator 461 — 4y

Answer this question