[notabug] SetText() with gsub operation crashes game

[Phuein]

Originally Posted by votan gsub returns two values: the changed string and the replacement count. SetText takes two parameters. But the second must be boolean. gsub returns int. => boom You can not directly use gsub and SetText in one operation. => local value = gsub(...) -- take first return value only. edit:SetText(value)

This was used as-is up until the last patch without issue. I assume gsub() is the same, so unless SetText() was changed - do you know that it was? - the other option is the string operation. Either way, breaking-changes without a notification is rough.

[sirinsidiator]

Originally Posted by Phuein This was used as-is up until the last patch without issue. I assume gsub() is the same, so unless SetText() was changed - do you know that it was? - the other option is the string operation. Either way, breaking-changes without a notification is rough.

That's not a breaking change - the addon code was just written naively.
Regardless of language or project, one should never pass variable returns directly into functions that are maintained by a third party. Otherwise there will be surprises like that in case the function is updated to accept new arguments.