cdmichaelb / Outfitter

Outfitter - Classic
MIT License
10 stars 22 forks source link

Journalator/Outfitter conflict? #124

Open soalicious opened 1 year ago

soalicious commented 1 year ago

Appreciate your maintaining Outfitter, been a personal favorite for a long time- Thanks!

Using a new addon Journalator and Outfitter throws a "nil" value error. I merely hovered my cursor over a sold item (in a Journalator frame) no longer in my inventory, though it's not even an equip-able item??? This does not happen if it's in the auctionhouse., so I can post to Journaltor as it IS cited below as well. I'll await your feedback.

Version info: Outfitter-3.0.1-wrath.zip from Curseforge, though your TOC says 3.0.0

Error 1: Message: Interface\AddOns\Outfitter\OutfitterInventory.lua:357: attempt to compare number with nil Time: Wed Nov 30 07:30:11 2022 Count: 1 Stack: Interface\AddOns\Outfitter\OutfitterInventory.lua:357: attempt to compare number with nil [string "=[C]"]: in function SetHyperlink' [string "@Interface\AddOns\Journalator\Source\ResultsListing\ResultsRow.lua"]:11: in functionOnEnter' [string "@Interface\AddOns\Auctionator\Source\Components\ResultsListing\Mixins\AuctionatorCell.lua"]:10: in function <...Components\ResultsListing\Mixins\AuctionatorCell.lua:8>

Locals: (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = nil (temporary) = defined =[C]:-1 = defined =[C]:-1 = defined @Interface\AddOns\Outfitter\Outfitter.lua:6813

Error 2 Message: Interface\AddOns\Outfitter\OutfitterInventory.lua:357: attempt to compare number with nil Time: Wed Nov 30 07:48:03 2022 Count: 99 Stack: Interface\AddOns\Outfitter\OutfitterInventory.lua:357: attempt to compare number with nil

Locals:

GovtGeek commented 1 year ago

I know this was a long time ago, but do you still have this error?

soalicious commented 1 year ago

Not sure, I removed outfitter due to other problems. Using the Blizz "Equipment NManager", which is not great, but does not toss errors.

On Mon, Apr 17, 2023 at 2:31 PM GovtGeek @.***> wrote:

I know this was a long time ago, but do you still have this error?

— Reply to this email directly, view it on GitHub https://github.com/cdmichaelb/Outfitter/issues/124#issuecomment-1512111412, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQEPTGNQZ4ES7XJVTV57YTDXBWZDBANCNFSM6AAAAAASPZG4F4 . You are receiving this because you authored the thread.Message ID: @.***>

--

Tom