any programmers out there...whats wrong with my basic enchanting override code?

[churobot]

View attachment enchant test.xml

can somebody with some programming knowledge tell me what i have done wrong?

 

[thephoenix25]

I guess from your script, you are leveling enchanting by casting the same enchant on an item that either you are wearing or is in your bag. I've not tried it, but I'd suggest you need to have your customcode and castspellaction statements in a while loop, as the customcode will only be called after the castspell is complete.

Why not just buy a load of vellumes (or however it's spelt) an cast the enchants on them. At least you can vendor them or put them on the ah. I use the vellumes for high level enchating with PB all the time, no need to complicate it with customcode and lua statements.

hope that helps

 

[churobot]

i think you are on to something phoenix!!! .........could you tell me the script you would use.

thx for the advice on enchanting but i am doing this to improve my programming!

 

[thephoenix25]

Errm, the part in my script that enchants the vellumes is simply this :

<CastSpellAction Entry="74235" Repeat="5" RepeatType="Banker" CastOnItem="True" ItemId="38682" ItemType="Chest" />

But with PB you don't need to write the scripts by hand, they are generated automaticaly from the PB user interface.

But if you are choosing to write it manualy, be sure to know what RepeatType does, as "Banker" may cause you problems.


Good luck<CASTSPELLACTION ItemType="Chest" ItemId="38682" CastOnItem="True" RepeatType="Banker" Repeat="5" Entry="74235" />

 

[churobot]

sweet...thanks for your time

 

[thephoenix25]

I think this should do what you want.

Remove materials needed from your personal bank.
craft 20 enchants to vellume