Redstoner/redstoner-utils
Archived
0
Fork 0
Code Issues 6 Pull Requests Wiki Activity

only using recipient list when iterating

Browse Source
This commit is contained in:
jomo
2014-06-21 12:47:25 +02:00
parent 086902a202
commit 5431cd2b90
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
mentio.py
View File

@@ -13,9 +13,9 @@ def onChat(event):
if not event.isCancelled(): if not event.isCancelled():
sender = event.getPlayer() sender = event.getPlayer()
words = event.getMessage().split(" ") words = event.getMessage().split(" ")
recipients = list(event.getRecipients()) recipients = event.getRecipients()
for recipient in recipients[:]: for recipient in recipients.tolist():
rec_words = words[:] # copy rec_words = words[:] # copy
for i in range(len(rec_words)): for i in range(len(rec_words)):
word = rec_words[i] word = rec_words[i]
@@ -25,9 +25,10 @@ def onChat(event):
# player was mentioned # player was mentioned
if rec_words != words: if rec_words != words:
try: # list might not be mutable try:
recipients.remove(recipient) # don't send original message recipients.remove(recipient) # don't send original message
except: except:
# list might not be mutable, ignoring. Receiver will get the message twice
pass pass
message = " ".join([sender.getDisplayName(), arrow] + rec_words) message = " ".join([sender.getDisplayName(), arrow] + rec_words)
msg(recipient, message, usecolor = False) msg(recipient, message, usecolor = False)