Neuron icon indicating copy to clipboard operation
Neuron copied to clipboard

Dark iron paladin mount not dragging to action bar

Open Trajin opened this issue 6 years ago • 16 comments

I just created a dark iron paladin and was unable to drag the paladin mount to the action bar. I even tried to macro it and that didn't work either. I don't get any errors, it just doesn't work. It only happens with this mount. Other mounts and paladin mounts drag and work normally. The name of the mount is Darkforge Ram

Trajin avatar Mar 19 '19 04:03 Trajin

Hrm.... Interesting. When you drag it does it attempt to write a macro for it?

brittyazel avatar Mar 19 '19 04:03 brittyazel

I editted the button and checked. It looks like it's creating the macro and checking the box for mount. There isn't an icon and clicking the button doesn't do anything.
Edit: Tried creating a new bar and dragging the mount to it. It creates the macro, but doesn't put the check in the mount checkbox. The icon is the question mark icon. Logging back into the game, the question mark icon appears on the other bar where I drug the mount there originally.

Trajin avatar Mar 19 '19 11:03 Trajin

I figured out the problem. I turned off all the addons and drug the Darkforge Ram icon to the blizzard action bar and then moused over it. It says Summon Darkforge Ram. So I went back to the neuron button and edited the macro to include the word summon and that fixed the problem. Don't know why blizzard did this one mount this way.

Trajin avatar Mar 19 '19 19:03 Trajin

Ugh, wtf Blizz. I can add in a hard coded check for this mount, but I hate adding in 1 off hacks like that

brittyazel avatar Mar 19 '19 20:03 brittyazel

I agree. Thanks

Sent from my iPhone

On Mar 19, 2019, at 3:25 PM, Britt Yazel <[email protected]mailto:[email protected]> wrote:

Ugh, wtf Blizz. I can add in a hard coded check for this mount, but I hate adding in 1 off hacks like that

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/brittyazel/Neuron/issues/281#issuecomment-474567531, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Af2t-4AdbYNBfd23uaHXwKX24LnYbe6_ks5vYUemgaJpZM4b7PZx.

Trajin avatar Mar 19 '19 21:03 Trajin

Just to mention that there were previous similar fixes for the heirloom mounts names as well, so this can happen from time to time it seems :( On Tuesday, March 19, 2019, 1:38:29 p.m. CST, Trajin [email protected] wrote:

I figured out the problem. I turned off all the addons and drug the Darkforge Ram icon to the blizzard action bar and then moused over it. It says Summon Darkforge Ram. So I went back to the neuron button and edited the macro to include the word summon and that fixed the problem. Don't know why blizzard did this one mount this way.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

teh-fixerer avatar Mar 19 '19 23:03 teh-fixerer

Does /summon work for all mounts? Right now I think we use /cast for mounts, but maybe we should use a new syntax for our macros

brittyazel avatar Mar 19 '19 23:03 brittyazel

The macro for the darkforge ram is /cast Summon Darkforge Ram;

Sent from my iPhone

On Mar 19, 2019, at 6:26 PM, Britt Yazel <[email protected]mailto:[email protected]> wrote:

Does /summon work for all mounts? Right now I think we use /cast for mounts, but maybe we should use a new syntax for our macros

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/brittyazel/Neuron/issues/281#issuecomment-474623276, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Af2t-_fLhW8ILYkwdpbueiFuveR-1M61ks5vYXI8gaJpZM4b7PZx.

Trajin avatar Mar 20 '19 00:03 Trajin

FYI - /summon doesn't seem to work for mounts at all in limited testing, either in a macro or just trying to type it in manually.

raysmith59 avatar Mar 20 '19 20:03 raysmith59

Summon is part the name of the dark iron paladin mount. The new guy must’ve named it.

Sent from my iPhone

On Mar 20, 2019, at 3:44 PM, raysmith59 <[email protected]mailto:[email protected]> wrote:

FYI - /summon doesn't seem to work for mounts at all in limited testing, either in a macro or just trying to type it in manually.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/brittyazel/Neuron/issues/281#issuecomment-475020537, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Af2t-2iDVwm-toNc2A5cY9z44EoTeOHJks5vYp2egaJpZM4b7PZx.

Trajin avatar Mar 20 '19 21:03 Trajin

Is this still an issue? Has it been fixed? I was kinda hoping it would just sort itself out

brittyazel avatar May 22 '19 20:05 brittyazel

I haven’t had to redo the bar once i fixed it. Ive unsubbed for the summer so I can’t test it currently.

Sent from my iPhone

On May 22, 2019, at 3:44 PM, Britt Yazel <[email protected]mailto:[email protected]> wrote:

Is this still an issue? Has it been fixed? I was kinda hoping it would just sort itself out

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/brittyazel/Neuron/issues/281?email_source=notifications&email_token=AH62367YHCZTB6IULRGAUCLPWWWDRA5CNFSM4G7M6ZY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWAJE3Y#issuecomment-494965359, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH62366UHTQFOVCCCQEQNNLPWWWDRANCNFSM4G7M6ZYQ.

Trajin avatar May 23 '19 02:05 Trajin

Unfortunately It is still an issue. You still have to edit the spell to be /cast Summon Darkforge Ram On Wednesday, May 22, 2019, 2:44:40 p.m. CST, Britt Yazel [email protected] wrote:

Is this still an issue? Has it been fixed? I was kinda hoping it would just sort itself out

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

teh-fixerer avatar May 23 '19 04:05 teh-fixerer

I suggest to add a few exceptions for these special types of mounts. They also behave differently in that they have to be /cast by protected button and can't be summoned by addon. Similar to druid travel form or running wild which are also spells. Or maybe there is some way to detect the spell name for the dragged ability correctly.

/cast Summon Chauffeur (replace /cast Chauffeured Mekgineer's Chopper) /cast Summon Charger (replace /cast Charger) /cast Summon Warhorse /cast Summon Darkforge Ram (/cast Darkforge Ram) Different paladin mounts

DejayRezme avatar Dec 18 '20 02:12 DejayRezme

You could also use the API function that returns the correct spellID.

https://wow.gamepedia.com/API_C_MountJournal.GetMountInfoByID

spellID number - The ID of the spell that summons the mount.

DejayRezme avatar Dec 18 '20 08:12 DejayRezme

Adding this line at 385 in "Neuron\Objects\ACTIONBUTTON_DragAndDrop.lua" fixes the issue for me:

	local mountSpellName = GetSpellInfo(mountSpellID) -- the journal name is not always the same as the spell name (paladin mount and chauffeured mount)
	self.data.macro_Text = "#autowrite\n/cast "..mountSpellName..";"

DejayRezme avatar Dec 18 '20 08:12 DejayRezme