WowAce

95051, -- The Brassiest Knuckle (Alliance)

95050, -- The Brassiest Knuckle (Horde)

There are a number of smaller portal/teleport items that could be added to this....

Is there a reason an update hasnt been put out?

Looks like CrazyBenny isn't updating any of his addons anymore. Likely he's left the game or had other complications.

Several folks have posted code to help you add teleport items to the addon. Don't know how long this will keep working. I do wish someone would take the addon over, but I'm sure we're all grateful to CrazyBenny for his efforts, and wish him well.

Thats sad when a popular addon dies. If I wasnt already up to my eyes in handling 2 addons (and trying to further my knowledge to be able to add features and not just maintaine them) I would ask if they were willing to let someone else maintaine this addon =/

for Kirin Tor Beacon/Sunreaver Beacon replace in portals.lua

  63353  -- Shroud of Cooperation (Horde)

with

63353, -- Shroud of Cooperation (Horde)
95567, -- Kirin Tor Beacon
95568 -- Sunreaver Beacon

IMPORTANT REPLACE!!!

MfG

Will this be updated for 5.2?

For some reason the addon is not announcing when i cast portals...

Minor display issue.  Every time you open the menu from Titan Bar the width of the drop down expands by one pixel.

I am using the latest release and somehow I still don't see the Old Dalaran teleport + portal in the drop down window.

Any ETA on Old Dalaran Teleport + Portal and Gold Challenge Dungeons Teleports ?

i also like to up the vote on adding:

Ancient Tome of Teleport: Dalaran and Ancient Tome of Portal: Dalaran

These bring you to the crater of old Dalaran.

+1 to that !

Your latest alpha push R171 broke the addon.

Also, please add the new Ancient Teleport: Dalaran and Ancient Portal: Dalaran.

Yes, please add the new portal spells

Awesome addon, I have been using it for years with my mage.

Could you please take a look and add the two new mage portal spells to Vale of Eternal Blossoms as well as Wormhole Generator : Pandaria to the drop down window? 

Great addon, can only recommend it even if you don't play a mage.

Your mod (as of v1.8.27) generates a bug for lower level mages because of how GetBookItemName works now (it no longer return nil if a spell does not exist or is not learned yet, it will throw an invalid spell slot exception instead).

You may want to change your UpdateSpells() function to use IsPlayerSpell before calling findSpell to check if the player does have the spell.

Use something like:

local function UpdateSpells()
         if not portals then
               SetupSpells()
          end

           if portals then
                  for _,unTransSpell in ipairs(portals) do

                           local spell, _, spellIcon = GetSpellInfo(unTransSpell[1])
                           local spellid = nil
                           if(IsPlayerSpell(unTransSpell[1])) then
                                   spellid = findSpell(spell)
                           end

                           if spellid then
                                    methods[spell] = {
                                            spellid = spellid,
                                            text = spell,
                                           spellIcon = spellIcon,
                                            isPortal = unTransSpell[2] == 'P_RUNE',
                                            secure = {
                                               type = 'spell',
                                              spell = spell
                                           }
                                    }
                           end
                 end
         end
end

PS: WTB identation preserving on copy-paste operations.

Logged in last night with a lvl35 mage and got the following error, guessing that's the one you're referring to.

["message"] = "Interface\\AddOns\\Broker_Portals\\portals.lua:121: GetSpellBookItemName(): Invalid spell slot",

Yea that's the one. As I said, it's because they changed the behaviour of GetSpellBookItemName(index,type) from returning nil to throwing an exception if there's noting at that index. Using IsPlayerSpell(spellid)  before calling findSpell(spellid) fixes that problem (and would also probably fix anny spell removal problems in the future).

PS: in this post, spellid is the global identification of a spell and not the spellbook's index of that spell