Hah, Nice one Neo

Welcome back Freddy =)

I'm about to give this a run also.  Kind of wish it had Bank support since that is where I store all of my Scrolls... however that could be kind of dangerous unless you use a isolated bank to recall in and out of.

Edit:
Same issues here as Xanderyum.  I select what I want to keep, hit play and It skips the section where I need to select my runebooks & secure containers.

No Magic Resist is a bad idea for a Mage, IMO it's a bad idea period for pvp.  I suppose I'm just oldschool though, I have used, and continue to use, the same mage template since I started to PvP.

120 Magery
120 Meditation
120 Eval Intel
120 Magic Resist
120 Wrestling
100 Inscription

Sometimes I'll switch it up and go with 100 Poisoning vs 100 Inscription, but it depends on the situation.  (will I be dropping mad poison fields?  Yes... go with Poisoning, No?  Bonus SD wins.)  If I'm dueling, I always go with poisoning.

People still play on Sonoma?  What is it like one guild (the entire server at this point) throwing a challenge at another guild (the entire other server) via purple books dropped at the newbie starting cities?

Still the same issue.  I'm dumbfounded at this point.  I've gone line by line from the Multi-skill trainer to see how it was done there, and I have it practically copied, yet getting two different reactions.

Here's what I changed (Taking your advice)

menu get skill
  if ( ( #menuRes = 1 ) && ( !old_menures <> #menuRes ) )
     {
     set !old_menures #menuRes
     set %skill alch
     set %skill_name Alchemy:
     gosub main_menu
     )
  if ( ( #menuRes = 2 ) && ( !old_menures <> #menuRes ) )
     {
     set !old_menures #menuRes
     set %skill blac
     set %skill_name Blacksmithy:
     gosub main_menu
     )
  if ( ( #menuRes = 3 ) && ( !old_menures <> #menuRes ) )
     {
     set !old_menures #menuRes
     set %skill carp
     set %skill_name Carpentry:
     gosub main_menu
     )
  if ( ( #menuRes = 4 ) && ( !old_menures <> #menuRes ) )
     {
     set !old_menures #menuRes
     set %skill tail
     set %skill_name Tailoring:
     gosub main_menu
     )
  if ( ( #menuRes = 5 ) && ( !old_menures <> #menuRes ) )
     {
     set !old_menures #menuRes
     set %skill tink
     set %skill_name Tinkering:
     gosub main_menu
     )
until #false

Maybe I'll just scrap it and start over from scratch.  Starting to get frustrated.

I vaguely remember running into a similar problem years ago when I did the multi-skill trainer, however I can not find the error.  I've even looked back through the multi-trainer and have re-written the menu for this script, but it is still just cycling through each of the 5 option as if they are selected in the pull down.  The way I read it when I look at it is it will only cycle through the option if the Pull Down is selected to (lets use option 2) Blacksmithy AND the !old_menures is NOT #menuRes, which I have set previously in the code.

This is the area I believe my error is in:

menu combo create skill 85 40 120
menu combo add skill Alchemy
menu combo add skill Blacksmithy
menu combo add skill Carpentry
menu combo add skill Tailoring
menu combo add skill Tinkering

set !old_menures #menuRes

repeat
menu get skill
  if #menuRes = 1 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill alch
     set %skill_name Alchemy:
     gosub main_menu
     )
  if #menuRes = 2 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill blac
     set %skill_name Blacksmithy:
     gosub main_menu
     )
  if #menuRes = 3 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill carp
     set %skill_name Carpentry:
     gosub main_menu
     )
  if #menuRes = 4 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill tail
     set %skill_name Tailoring:
     gosub main_menu
     )
  if #menuRes = 5 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill tink
     set %skill_name Tinkering:
     gosub main_menu
     )
until #false

sub main_menu
set %size1 105
set %size1 ( %size1 + 100 )
menu window size 290 %size1
menu shape status_bg 5 105 280 95 3 7 1 Black 7 Gray
menu font align right
menu font color black
menu font size 10
menu text titles 85 115 %skill_name
menu text titles 85 135 Attempts:
menu text titles 85 155 Time:
menu text titles 85 175 Last Gain:
return

And here is the entire script that I've written so far.  I am only on the menu, the actual script to perform (blacksmithing training only at this point) works fine.

set %sewing_kit HAG
set %ingot_type ENK
set %cloth_type CUI
set %oil_cloth RTI
set %size1 105
set %timeout #scnt + 5
gosub TM_AddUnderscore #CHARNAME
      set %charname #RESULT
 
; XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
; XXXXXXXXXXXXXXX MENU DESIGN XXXXXXXXXXXXXXX
; XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
;menu hideeuo
menu window title Guadah's Crafters Trainer
menu window color silver
menu window size 290 %size1
menu font bgcolor gray
menu shape potionbg 5 5 280 65 3 7 1 Black 7 Gray
menu font align right
menu text title 80 12 Resources:
menu text title 80 44 Train Skill:
menu font align left
menu font bgcolor silver
menu font size 10
menu button target_resource_bag 175 10 30 24 Set
menu edit resource_bag 85 10 85 %resource_bag
menu button start 230 10 50 25 Start
menu button quit 230 40 50 25 Quit
; pull down skill selection box
menu combo create skill 85 40 120
menu combo add skill Alchemy
menu combo add skill Blacksmithy
menu combo add skill Carpentry
menu combo add skill Tailoring
menu combo add skill Tinkering
; Status Box
menu shape status_bg 5 75 280 25 3 7 1 Black 7 Gray
menu font bgcolor gray
menu text status_title 10 80 Status:
menu font color maroon
menu font size 9
menu text status 60 80 Please wait, Initializing Script...
menu show

gosub TM_GetPersistantVariable resource_bag
menu set resource_bag #result
menu get resource_bag
if #menures = N/A
   {
   menu set status Please set your Resource Chest
   }
if #menures <> N/A
   {
   set %resource_bag #menures
   set #lobjectid %resource_bag
   event macro 17 0
    wait 10
   contpos 575 400
    wait 10
   }

set !old_menures #menuRes
repeat
if #menuButton = target_resource_bag
   {
   menu set status Please set your Resource Chest
   set #targCurs 1
set_resource_bag:
    if #targCurs = 1
     goto set_resource_bag
   Set %resource_bag #ltargetid
   Set #lobjectid %resource_bag
   Event Macro 17 0
    wait 10
   contpos 610 440
    wait 10
   menu font size 10
   menu set resource_bag %resource_bag
   gosub TM_SetPersistantVariable resource_bag %resource_bag
   set #menubutton N/A
   }
menu set status Please select your skill to train.
if #menubutton = Start
   {
   set #menubutton N/A
   menu delete Start
   menu font bgcolor silver
   menu font size 10
   menu font color black
   menu button Pause 230 10 50 25 Pause
   gosub training
   }
if #menubutton = Quit
   {
   set #menubutton N/A
   display ok Thank you for using my script.$
           +$
           +Script is Halting$
   menu hide
   halt
   }
menu get skill
  if #menuRes = 1 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill alch
     set %skill_name Alchemy:
     gosub main_menu
     )
  if #menuRes = 2 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill blac
     set %skill_name Blacksmithy:
     gosub main_menu
     )
  if #menuRes = 3 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill carp
     set %skill_name Carpentry:
     gosub main_menu
     )
  if #menuRes = 4 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill tail
     set %skill_name Tailoring:
     gosub main_menu
     )
  if #menuRes = 5 && !old_menures <> #menuRes
     {
     set !old_menures #menuRes
     set %skill tink
     set %skill_name Tinkering:
     gosub main_menu
     )
until #false

sub main_menu
set %size1 105
set %size1 ( %size1 + 100 )
menu window size 290 %size1
menu shape status_bg 5 105 280 95 3 7 1 Black 7 Gray
menu font align right
menu font color black
menu font size 10
menu text titles 85 115 %skill_name
menu text titles 85 135 Attempts:
menu text titles 85 155 Time:
menu text titles 85 175 Last Gain:
return

; #################################################
; ########            PAUSE SUB            ########
; #################################################
Sub CheckPause
if #menubutton = pause
   {
   set #menubutton N/A
   menu delete pause
   menu font color maroon
   menu font bgcolor silver
   menu button resume 230 10 50 25 Resume
   menu set status Paused!  Click "Resume" when ready.
   gosub PauseLoop
   }
if #menubutton = quit
   {
   display ok Thank you for using Guadah's Alchemist Assistant.$
          +$
          +Script is Halting$
   menu hide
   halt
   }
return

Sub PauseLoop
Pause_Loop:
If #menubutton = resume
   {
   set #menubutton N/A
   Menu Delete Resume
   Menu Font Color Black
   menu font bgcolor silver
   Menu Button Pause 230 10 50 25 Pause
   Return
   }
if #menuButton = quit
   {
   Display ok Thank you for using Guadah's Alchemist Assistant.$
            +$
            +Script is Halting$
   Menu Hide
   Halt
   }
Goto Pause_Loop
Return
; XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
; XXX TRAILMYX'S PERSISTANT VARIABLE SUBS XXX
; XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
sub TM_GetPersistantVariable
  set #RESULT %1 , _ , #CHARID
  set #RESULT * . #RESULT
return #RESULT
sub TM_SetPersistantVariable
  set #RESULT %1 , _ , #CHARID
  set * . #RESULT %2
return

I've left out the crafting & training subs, since the script doesn't even get to that part... there was no need to bog down extra code.

If any of you see what I've missed, I'd love a pointer on what I did wrong.  I've tried to walk away and come back with fresh eyes, but still don't see the issue.

Appreciate it 12