Skip to content

Commit e182dd7

Browse files
committed
Fix shadowing of "name" property + hidden rocks
Fixes #189.
1 parent 5116684 commit e182dd7

File tree

10 files changed

+222
-193
lines changed

10 files changed

+222
-193
lines changed

networking/multiplayer_bomber/bomb.gd

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
extends Area2D
22

33
var in_area = []
4-
var owner
4+
var from_player
55

66
# Called from the animation
77
func explode():
@@ -10,7 +10,7 @@ func explode():
1010
return
1111
for p in in_area:
1212
if (p.has_method("exploded")):
13-
p.rpc("exploded", owner) # Exploded has a master keyword, so it will only be received by the master
13+
p.rpc("exploded", from_player) # Exploded has a master keyword, so it will only be received by the master
1414

1515
func done():
1616
queue_free()

networking/multiplayer_bomber/bomb.tscn

Lines changed: 15 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,11 +23,13 @@ _data = [ Vector2( 0.00150494, 0.398437 ), 0.0, 0.0, 0, 0, Vector2( 0.0152287, 1
2323

2424
[sub_resource type="CurveTexture" id=4]
2525

26+
flags = 4
2627
width = 2048
2728
curve = SubResource( 3 )
2829

2930
[sub_resource type="ParticlesMaterial" id=5]
3031

32+
render_priority = 0
3133
trail_divisor = 1
3234
emission_shape = 2
3335
emission_box_extents = Vector3( 80, 1, 1 )
@@ -76,6 +78,7 @@ tracks/0/path = NodePath("sprite:self_modulate")
7678
tracks/0/interp = 1
7779
tracks/0/loop_wrap = true
7880
tracks/0/imported = false
81+
tracks/0/enabled = true
7982
tracks/0/keys = {
8083
"times": PoolRealArray( 0, 0.4, 0.6, 0.8, 1.1, 1.3, 1.5, 1.8, 1.9, 2, 2.2, 2.3, 2.4, 2.5, 2.6, 2.7, 2.8, 3 ),
8184
"transitions": PoolRealArray( 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1 ),
@@ -87,6 +90,7 @@ tracks/1/path = NodePath(".")
8790
tracks/1/interp = 1
8891
tracks/1/loop_wrap = true
8992
tracks/1/imported = false
93+
tracks/1/enabled = true
9094
tracks/1/keys = {
9195
"times": PoolRealArray( 2.8, 3.4 ),
9296
"transitions": PoolRealArray( 1, 1 ),
@@ -103,6 +107,7 @@ tracks/2/path = NodePath("explosion1:emitting")
103107
tracks/2/interp = 1
104108
tracks/2/loop_wrap = true
105109
tracks/2/imported = false
110+
tracks/2/enabled = true
106111
tracks/2/keys = {
107112
"times": PoolRealArray( 0, 2.8 ),
108113
"transitions": PoolRealArray( 1, 1 ),
@@ -114,6 +119,7 @@ tracks/3/path = NodePath("explosion2:emitting")
114119
tracks/3/interp = 1
115120
tracks/3/loop_wrap = true
116121
tracks/3/imported = false
122+
tracks/3/enabled = true
117123
tracks/3/keys = {
118124
"times": PoolRealArray( 0, 2.8 ),
119125
"transitions": PoolRealArray( 1, 1 ),
@@ -132,23 +138,23 @@ audio_bus_override = false
132138
audio_bus_name = "Master"
133139
script = ExtResource( 1 )
134140

135-
[node name="sprite" type="Sprite" parent="."]
141+
[node name="sprite" type="Sprite" parent="." index="0"]
136142

137143
position = Vector2( -2.92606, -2.92606 )
138144
texture = ExtResource( 2 )
139145
region_enabled = true
140146
region_rect = Rect2( 144, 0, 48, 48 )
141147
_sections_unfolded = [ "Region" ]
142148

143-
[node name="shape1" type="CollisionShape2D" parent="."]
149+
[node name="shape1" type="CollisionShape2D" parent="." index="1"]
144150

145151
shape = SubResource( 1 )
146152

147-
[node name="shape2" type="CollisionShape2D" parent="."]
153+
[node name="shape2" type="CollisionShape2D" parent="." index="2"]
148154

149155
shape = SubResource( 2 )
150156

151-
[node name="explosion1" type="Particles2D" parent="."]
157+
[node name="explosion1" type="Particles2D" parent="." index="3"]
152158

153159
emitting = false
154160
amount = 8
@@ -170,7 +176,7 @@ h_frames = 1
170176
v_frames = 1
171177
_sections_unfolded = [ "Process Material", "Textures", "Time" ]
172178

173-
[node name="explosion2" type="Particles2D" parent="."]
179+
[node name="explosion2" type="Particles2D" parent="." index="4"]
174180

175181
rotation = 1.57162
176182
emitting = false
@@ -193,14 +199,14 @@ h_frames = 1
193199
v_frames = 1
194200
_sections_unfolded = [ "Process Material", "Textures", "Time" ]
195201

196-
[node name="anim" type="AnimationPlayer" parent="."]
202+
[node name="anim" type="AnimationPlayer" parent="." index="5"]
197203

204+
root_node = NodePath("..")
205+
autoplay = "anim"
198206
playback_process_mode = 1
199207
playback_default_blend_time = 0.0
200-
root_node = NodePath("..")
208+
playback_speed = 1.0
201209
anims/anim = SubResource( 6 )
202-
playback/active = true
203-
playback/speed = 1.0
204210
blend_times = [ ]
205211
autoplay = "anim"
206212

networking/multiplayer_bomber/gamestate.gd

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -56,15 +56,15 @@ func _connected_fail():
5656

5757
# Lobby management functions
5858

59-
remote func register_player(id, name):
59+
remote func register_player(id, new_player_name):
6060
if (get_tree().is_network_server()):
6161
# If we are the server, let everyone know about the new player
6262
rpc_id(id, "register_player", 1, player_name) # Send myself to new dude
6363
for p_id in players: # Then, for each remote player
6464
rpc_id(id, "register_player", p_id, players[p_id]) # Send player to new dude
65-
rpc_id(p_id, "register_player", id, name) # Send new dude to player
65+
rpc_id(p_id, "register_player", id, new_player_name) # Send new dude to player
6666

67-
players[id] = name
67+
players[id] = new_player_name
6868
emit_signal("player_list_changed")
6969

7070
remote func unregister_player(id):
@@ -124,14 +124,14 @@ remote func ready_to_start(id):
124124
rpc_id(p, "post_start_game")
125125
post_start_game()
126126

127-
func host_game(name):
128-
player_name = name
127+
func host_game(new_player_name):
128+
player_name = new_player_name
129129
var host = NetworkedMultiplayerENet.new()
130130
host.create_server(DEFAULT_PORT, MAX_PEERS)
131131
get_tree().set_network_peer(host)
132132

133-
func join_game(ip, name):
134-
player_name = name
133+
func join_game(ip, new_player_name):
134+
player_name = new_player_name
135135
var host = NetworkedMultiplayerENet.new()
136136
host.create_client(ip, DEFAULT_PORT)
137137
get_tree().set_network_peer(host)

networking/multiplayer_bomber/lobby.gd

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ func _on_host_pressed():
1717
get_node("players").show()
1818
get_node("connect/error_label").text=""
1919

20-
var name = get_node("connect/name").text
21-
gamestate.host_game(name)
20+
var player_name = get_node("connect/name").text
21+
gamestate.host_game(player_name)
2222
refresh_lobby()
2323

2424
func _on_join_pressed():
@@ -35,8 +35,8 @@ func _on_join_pressed():
3535
get_node("connect/host").disabled=true
3636
get_node("connect/join").disabled=true
3737

38-
var name = get_node("connect/name").text
39-
gamestate.join_game(ip, name)
38+
var player_name = get_node("connect/name").text
39+
gamestate.join_game(ip, player_name)
4040
# refresh_lobby() gets called by the player_list_changed signal
4141

4242
func _on_connection_success():
@@ -56,7 +56,7 @@ func _on_game_ended():
5656
get_node("connect/join").disabled
5757

5858
func _on_game_error(errtxt):
59-
get_node("error").text=errtxt
59+
get_node("error").dialog_text = errtxt
6060
get_node("error").popup_centered_minsize()
6161

6262
func refresh_lobby():

0 commit comments

Comments
 (0)