miniwin_spellup bug

[GoogleCodeExporter]

What is your username on Aardwolf?
Xiucozcatl

What steps will reproduce the problem?
1. dexx spellup
2. sadd true seeing
3. sremove detect hidden

not sure if this will reproduce but this was leadup to problem

What is the expected output? What do you see instead?

What version of the product are you using? On what operating system?
r1451 snapshot, r1443 snapshot, mushclient 4.77, windows vista

Please provide any additional information below.

Existing profile loaded - please enter your password.

Password:
############# Reconnecting to Game #############
cp check

You have no missed tells.

[1479/1479hp 1276/1276mn 1382/1382mv 18qt 943tnl] > 
You are not currently on a campaign.
You may take a campaign at this level.

[1479/1479hp 1276/1276mn 1382/1382mv 18qt 943tnl] > 
Run-time error
Plugin: miniwin_spellup (called from world: Aardwolf)
Function/Sub: OnPluginBroadcast called by Plugin miniwin_spellup
Reason: Executing plugin miniwin_spellup sub OnPluginBroadcast
[string "Plugin"]:811: attempt to index local 'v' (a nil value)
stack traceback:
        [string "Plugin"]:811: in function 'func'
        [string "Plugin"]:394: in function 'build_windows'
        [string "Plugin"]:1631: in function <[string "Plugin"]:1611>
Error context in script:
 807 :       local lstyle = {}
 808 :       
 809 :       local v = askedselfbysn[tonumber(i)]
 810 : 
 811*:       if not v.noprac and not v.nolearn then
 812 :         
 813 :         local stuff = ''
 814 :         local hint = ''
 815 :         local colour = 'white'

Original issue reported on code.google.com by mordai...@gmail.com on 24 Aug 2011 at 2:47

[GoogleCodeExporter]

if you do 'bsst refresh', does it still do it?

Original comment by enda...@gmail.com on 24 Aug 2011 at 1:36

[GoogleCodeExporter]

yes

Original comment by mordai...@gmail.com on 24 Aug 2011 at 1:49

[GoogleCodeExporter]

what if you reset the spells?

do a 'bsp export' first and copy paste the output to save your current list
then do 'bsp srem all
reload both the broadcast_skills and bast_spellup plugins'
the do 'bsp sadd all'

does that fix it?

(sorry for the guessing, but I can't reproduce it)

Original comment by enda...@gmail.com on 24 Aug 2011 at 1:55

[GoogleCodeExporter]

that seems to work:

Enabled plugin bast_spellup (aaaf0c555a44169e0f1d9674)
WARFARE: The preparation room will close in 1 minute.

bsp sremove all
bsp sadd 48 57 72 168 178 525 422 421 427 428 317 333 171 76
bsp oremove all
bsp oadd 

bast_spellup v6.1 
-----------------------------------------------
Run-time error
Plugin: miniwin_spellup (called from world: Aardwolf)
Function/Sub: onspellschange called by Plugin miniwin_spellup
Reason: Executing plugin miniwin_spellup sub onspellschange
[string "Plugin"]:811: attempt to index local 'v' (a nil value)
stack traceback:
        [string "Plugin"]:811: in function 'func'
        [string "Plugin"]:394: in function 'build_windows'
        [string "Plugin"]:253: in function <[string "Plugin"]:250>
Error context in script:
 807 :       local lstyle = {}
 808 :       
 809 :       local v = askedselfbysn[tonumber(i)]
 810 : 
 811*:       if not v.noprac and not v.nolearn then
 812 :         
 813 :         local stuff = ''
 814 :         local hint = ''
 815 :         local colour = 'white'
All spells removed
reload broadcast_skills
Unknown command. Please check the helpfiles.

[1687/1687hp 1425/1425mn 1500/1500mv 37qt 563tnl] > 

bast_spellup v6.1 Spells requested (0)
-----------------------------------------------
#. Name                        P  B  D  NP  NL 
____________________________________________________
A blade being sharpened can be heard in the distance as a bell tolls.  The 
bazaar is open.

bast_spellup v6.1 Spells requested (0)
-----------------------------------------------
#. Name                        P  B  D  NP  NL 
____________________________________________________
WARFARE: There are no enemies to fight each other. Peace declared!

INFO: Gren has plundered the ruins of Nanjiki!

bast_spellup v6.1 
-----------------------------------------------
Added detect hidden
Added detect invis
Added fly
Added night vision
Added invis
Added shield
Added stone skin
Added avoidance
Added true seeing
Added power projection
Added harden body
Added mystic might
Added perception
Added magical rush
Added absorb
Added underwater breathing
Added blur
All spells added
INFO: Farkyss has quelled the rebellion threatening Fort Terramire.

Global Quest: Global quest # 532739 has been declared for levels 61 to 72.
Global Quest: The quest will start in 4 ticks and will last for 97 ticks.
Global Quest: See 'Help Global Quests', or, type 'Gquest Join' to take part.

Refreshing affected spells and resetting events
detect hidden disabled because the affect already exists.
You can already detect hidden.

[1687/1687hp 1410/1425mn 1500/1500mv 35qt 563tnl] > 
detect invis disabled because the affect already exists.
You already see invisible.

[1687/1687hp 1395/1425mn 1500/1500mv 35qt 563tnl] > 
You fade out of existence.

[1687/1687hp 1370/1425mn 1500/1500mv 35qt 563tnl] > 
You feel a strong magical force surround your body.
You have become better at absorb! (81%)

[1687/1687hp 1305/1425mn 1500/1500mv 35qt 557tnl] > 
You feel gills growing on your neck!

Original comment by mordai...@gmail.com on 24 Aug 2011 at 2:02

[GoogleCodeExporter]

great! closing the issue then.

Original comment by enda...@gmail.com on 24 Aug 2011 at 2:25

• Changed state: Fixed