Skip to content

Commit 5361b70

Browse files
committed
internetradio: stylistic improvements
remove whitespaces, use camelCase for params/args
1 parent 254700c commit 5361b70

File tree

4 files changed

+32
-30
lines changed

4 files changed

+32
-30
lines changed

[gameplay]/internetradio/logic/gui/CRadioGUI.lua

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ local function initializeRadioGUI()
2929
RADIO_GUI["Toggle remote speakers checkbox"] = guiCreateCheckBox(15, 345, 180, 17, "Allow other players speakers", allowRemoteSpeakers, false, RADIO_GUI["Radio window"])
3030
RADIO_GUI["Toggle remote speakers label"] = guiCreateLabel(167, 345, 150, 17, "", false, RADIO_GUI["Radio window"])
3131
RADIO_GUI["Radio station URL column"] = guiGridListAddColumn(RADIO_GUI["Stream URLs gridlist"], "Radio station", 0.8)
32-
32+
3333
guiSetVisible(RADIO_GUI["Radio window"], false)
3434
guiEditSetMaxLength(RADIO_GUI["Stream URL edit"], RADIO_STREAM_URL_MAX_LENGTH)
3535
guiWindowSetSizable(RADIO_GUI["Radio window"], false)

[gameplay]/internetradio/logic/handle_radio/CHandleRadio.lua

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -368,8 +368,8 @@ end
368368
addEvent("onClientCreateSpeaker", true)
369369
addEventHandler("onClientCreateSpeaker", root, onClientCreateSpeaker)
370370

371-
function onClientUpdateVolume(volume)
372-
setPlayerSpeakerVolume(source, volume)
371+
function onClientUpdateVolume(speakerVolume)
372+
setPlayerSpeakerVolume(source, speakerVolume)
373373
end
374374
addEvent("onClientUpdateVolume", true)
375375
addEventHandler("onClientUpdateVolume", root, onClientUpdateVolume)

[gameplay]/internetradio/logic/handle_radio/SHandleRadio.lua

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,17 @@ function setPlayerSpeakerData(playerElement, speakerData)
1919
return true
2020
end
2121

22-
function updateSpeakerVolume(playerElement, volume)
22+
function updateSpeakerVolume(playerElement, speakerVolume)
2323
local playerSpeakerData = getPlayerSpeakerData(playerElement)
2424

2525
if (not playerSpeakerData) then
2626
return false
2727
end
2828

29-
playerSpeakerData.speakerVolume = volume
30-
triggerClientEvent(root, "onClientUpdateVolume", playerElement, volume)
29+
playerSpeakerData.speakerVolume = speakerVolume
30+
triggerClientEvent(root, "onClientUpdateVolume", playerElement, speakerVolume)
31+
32+
return true
3133
end
3234

3335
function getPlayerSpeakerData(playerElement)
@@ -73,7 +75,7 @@ function isObjectSpeaker(objectElement)
7375
return false
7476
end
7577

76-
for playerElement, speakerData in pairs(playerSpeakers) do
78+
for _, speakerData in pairs(playerSpeakers) do
7779
local speakerBox = speakerData.speakerBox
7880
local matchingElement = (speakerBox == objectElement)
7981

@@ -85,7 +87,7 @@ function isObjectSpeaker(objectElement)
8587
return false
8688
end
8789

88-
function onServerCreateSpeaker(streamURL, volume)
90+
function onServerCreateSpeaker(streamURL, speakerVolume)
8991
if (not client) then
9092
return false
9193
end
@@ -102,7 +104,7 @@ function onServerCreateSpeaker(streamURL, volume)
102104
return false
103105
end
104106

105-
local validVolume = verifyRadioVolume(volume)
107+
local validVolume = verifyRadioVolume(speakerVolume)
106108

107109
if (not validVolume) then
108110
return false
@@ -134,15 +136,15 @@ function onServerCreateSpeaker(streamURL, volume)
134136
speakerStreamURL = streamURL,
135137
speakerSoundMaxDistance = RADIO_MAX_SOUND_DISTANCE,
136138
speakerPaused = false,
137-
speakerVolume = volume
139+
speakerVolume = speakerVolume,
138140
}
139141

140142
setPlayerSpeakerData(client, speakerData)
141143
end
142144
addEvent("onServerCreateSpeaker", true)
143145
addEventHandler("onServerCreateSpeaker", root, onServerCreateSpeaker)
144146

145-
function onServerEditVolume(volume)
147+
function onServerEditVolume(speakerVolume)
146148
if (not client or client ~= source) then
147149
return false
148150
end
@@ -153,19 +155,19 @@ function onServerEditVolume(volume)
153155
return false
154156
end
155157

156-
local validVolume = verifyRadioVolume(volume)
158+
local validVolume = verifyRadioVolume(speakerVolume)
157159

158160
if (not validVolume) then
159161
return false
160162
end
161-
163+
162164
local speakerData = playerSpeakers[client]
163165

164166
if (not speakerData) then
165167
return false
166168
end
167169

168-
updateSpeakerVolume(client, volume)
170+
updateSpeakerVolume(client, speakerVolume)
169171
end
170172
addEvent("onServerEditVolume", true)
171173
addEventHandler("onServerEditVolume", root, onServerEditVolume)

[gameplay]/internetradio/logic/handle_radio/ShHandleRadio.lua

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,21 @@ function verifyRadioStreamURL(streamURL)
5050
return true
5151
end
5252

53+
function verifyRadioVolume(radioVolume)
54+
local radioVolumeType = type(radioVolume)
55+
local radioVolumeNumber = (radioVolumeType == "number")
56+
57+
if (not radioVolumeNumber) then
58+
return false
59+
end
60+
61+
local radioVolumeMin = 0
62+
local radioVolumeMax = 1
63+
local radioVolumeInRange = (radioVolume >= radioVolumeMin) and (radioVolume <= radioVolumeMax)
64+
65+
return radioVolumeInRange
66+
end
67+
5368
function getPlayerFromPartialName(partialName)
5469
if (not partialName) then
5570
return false
@@ -127,19 +142,4 @@ end
127142
function clearPlayersDelay()
128143
playerDelays[source] = nil
129144
end
130-
addEventHandler(isServer and "onPlayerQuit" or "onClientPlayerQuit", root, clearPlayersDelay)
131-
132-
function verifyRadioVolume(radioVolume)
133-
local radioVolumeType = type(radioVolume)
134-
local radioVolumeNumber = (radioVolumeType == "number")
135-
136-
if (not radioVolumeNumber) then
137-
return false
138-
end
139-
140-
local radioVolumeMin = 0
141-
local radioVolumeMax = 1
142-
local radioVolumeInRange = (radioVolume >= radioVolumeMin) and (radioVolume <= radioVolumeMax)
143-
144-
return radioVolumeInRange
145-
end
145+
addEventHandler(isServer and "onPlayerQuit" or "onClientPlayerQuit", root, clearPlayersDelay)

0 commit comments

Comments
 (0)