Jump to content

  • [Cataclysm] Visit trainer only if there are spells to be trained

    Hashira

    By Hashira

      • Product: WRobot General Type: Suggestion Status: Unconfirmed
     Share

    Currently the bot may visit a class trainer even if there is no spell to be trained. It would be nice if in Cataclysm the bot would take into account the spell book information while deciding if it should visit the trainer or not:

    image.png.b0cc9001d231ca89ab39d9d591017990.png

     

    I guess the training state could be cancelled via "robotManager.Events.FiniteStateMachineEvents.OnBeforeCheckIfNeedToRunState", but I don't see any method in the Spell/SpellInfo/SpellManger that could return if a method is 'learnable' - is this something that can be added to the API ? (and maybe throw an error on other expansions, since WoTLK and lower would need static lists, and MoP and higher don't have training anymore).

     Share
    Go to bug reports

    User Feedback

    Recommended Comments

    Hashira

    This can be retrieved if you are at a trainer via https://wowwiki-archive.fandom.com/wiki/API_GetTrainerServiceInfo but ideally we won't travel there at all first.

     

    Before visiting the trainer, this is available via the spell book API:

    https://wowpedia.fandom.com/wiki/API_GetSpellBookItemInfo - this requires one to walk the whole book, the link contains information about this. `FUTURESPELL` spells are the ones that may be trained in the future.

    Then the API GetSpellAvailableLevel can retrieve from which level that particular spell can be used. This one can then be used to decide if the spell can be trained.

     

    See https://www.townlong-yak.com/framexml/4.3.4/SpellBookFrame.lua#640 for the client side logic.

    Edited by Hashira
    Link to comment
    Share on other sites


    Create an account or sign in to comment

    You need to be a member in order to leave a comment

    Create an account

    Sign up for a new account in our community. It's easy!

    Register a new account

    Sign in

    Already have an account? Sign in here.

    Sign In Now

×
×
  • Create New...