Enochen / bastmush

Automatically exported from code.google.com/p/bastmush
0 stars 0 forks source link

Quest monitor/miniwin plugin fails on linkdead if quest is active #78

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Request a quest, don't finish it yet.
2. Disconnect MUSHclient, or otherwise go linkdead
3. Restart MUSHclient, or reconnect to Aardwolf
4. Plugin fails due to runtime exception.

What is the expected output? What do you see instead?
Expect it to reload quest info, instead it 'attempts
to use non-existant field in arithmetic'.

What version of the product are you using? On what operating system?
miniwin_quest.xml 1272 2011-03-08
using MUSHclient version r956
on Windows 7

Please provide any additional information below.

Original issue reported on code.google.com by Vak...@gmail.com on 8 Jun 2011 at 4:08

GoogleCodeExporter commented 9 years ago
comm.quest { "action": "status", "targ": "Lizardman Priest", "room": "Inside 
the Temple", "area": "Jungles of Verume", "timer": 39 }
Run-time error
Plugin: xBroadcast_Quest (called from world: Aardwolf)
Function/Sub: OnPluginBroadcast called by Plugin xBroadcast_Quest
Reason: Executing plugin xBroadcast_Quest sub OnPluginBroadcast
[string "Plugin"]:133: attempt to perform arithmetic on field 'wait' (a nil 
value)
stack traceback:
        [string "Plugin"]:133: in function <[string "Plugin"]:79>
Error context in script:
 129 :       elseif stuff.action == 'status' then
 130 :         if stuff.status == 'ready' then
 131 :           quest_available()
 132 :         else
 133*:           when_required = os.time() + (stuff.wait * 60)
 134 :           update_timer()
 135 :         end
 136 :       elseif stuff.action == 'reset' then
 137 :         when_required = os.time() + (stuff.timer * 60)

Original comment by enda...@gmail.com on 8 Jun 2011 at 5:04

GoogleCodeExporter commented 9 years ago
This should be fixed in the r1401 release.

Original comment by enda...@gmail.com on 17 Jun 2011 at 3:47