search

Pierre-Sassoulas / grey-handling

Helps you get rid or exchange the right objects when your bags are full in World of Warcraft
https://www.curseforge.com/wow/addons/greyhandling
MIT License
3 stars 2 forks source link

This addon seems to hijack the print() function #22

Closed thefuzzy0ne closed 5 years ago

thefuzzy0ne commented 5 years ago

With an addon I am working on, I was trying to debug a chat filter . After about 2 hours of not getting the output I was expecting, I disabled Grey Handling (as I noticed the name pop up a lot in chat when it shouldn't) and I finally got the output I was expecting.

This addon seems to cause lots of problems when trying to print to a chat frame using print. It seems to gobble up everything after the first argument to print. Just running /run print("test") outputs GreyHandling: test to the chat frame. GreyHandling should not be intercepting these calls.

Pierre-Sassoulas commented 5 years ago

Ho, sorry I thought it was doing this only in my local context. I'm going to fix it ASAP. Thanks for the report !

Pierre-Sassoulas commented 5 years ago

Should be fixed if you upgrade to 0.9.2c. I'm really sorry about the debug you had to do, I thought that the open-closed principle was enforced somewhere by LUA or Wow and evidently, it is not.

thefuzzy0ne commented 5 years ago

Hey, no problem. Shit happens. Thanks for sorting it out so quickly. :-)