Skip to content
This repository was archived by the owner on Jun 8, 2023. It is now read-only.

Commit c1b6040

Browse files
committed
Fix multiple ambushes not showing
Get rid of unnecessary `if...else` in favor of `or=` and make multiple ambushes being pushed consistent.
1 parent 28c5238 commit c1b6040

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/scripts/ambush.coffee

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,10 @@
1414
# jmoses
1515

1616
appendAmbush = (data, toUser, fromUser, message) ->
17-
if data[toUser.name]
18-
data[toUser.name].push message
19-
else
20-
data[toUser.name] = [[fromUser.name, message]]
21-
17+
data[toUser.name] or= []
18+
19+
data[toUser.name].push [fromUser.name, message]
20+
2221
module.exports = (robot) ->
2322
robot.brain.on 'loaded', =>
2423
robot.brain.data.ambushes ||= {}

0 commit comments

Comments
 (0)