Talk to Game Master window not working with addon enabled

[aleluzu]

Is there an existing issue for this?

• I have searched the existing open and closed issues.

Description

When creating a new ticket in-game with the addon enabled I get a LUA error and the "Talk to Game Master" window stops working so you cannot see when a GM has replied to your ticket or respond to them. Other players with the same version of the addon also have this problem.

WeakAuras Version

WeakAuras 5.21.1

World of Warcraft Flavor

WotLK 3.3.5a

World of Warcraft Language

enGB/enUS

Server

Ascension-Bronzebeard

Tested with only WeakAuras

• I got this issue with only WeakAuras enabled

Lua Error

2x WeakAuras\Compatibility.lua:138: attempt to compare string with number
WeakAuras\Compatibility.lua:138: in function `Clamp'
SharedXML\LayoutFrame.lua:191: in function <SharedXML\LayoutFrame.lua:190>
SharedXML\LayoutFrame.lua:448: in function `Layout'
SharedXML\LayoutFrame.lua:247: in function `LayoutChildren'
SharedXML\LayoutFrame.lua:217: in function `Layout'
Ascension_TicketUI\ViewTicketPanelMixin.lua:81: in function `RefreshMessages'
Ascension_TicketUI\ViewTicketPanelMixin.lua:170: in function `PLAYER_TICKET_UPDATE'
Ascension_TicketUI\ViewTicketPanelMixin.lua:36: in function <Ascension_TicketUI\ViewTicketPanelMixin.lua:35>
[C]: in function `Show'
SharedXML\TypeExtensions\Region.lua:30: in function `SetShown'
Ascension_TicketUI\TicketUI.lua:42: in function `UpdateShownPanel'
Ascension_TicketUI\TicketUI.lua:88: in function `?'
SharedXML\SharedConstants.lua:88: in function <SharedXML\SharedConstants.lua:86>

Locals (No Functions or Userdata):
value = 285.00000677034
min = "260"
max = "596"
(*temporary) = "attempt to compare string with number"


Realm: Bronzebeard - Warcraft Reborn
Locale: enUS

Reproduction Steps

1. Enable addon
2. Create new ticket
3. Open ticket window
4. It doesn't work
5. Disable addon
6. Open ticket window again
7. Now it works

Last Good Version

Weakauras 4.0

Screenshots

With WeakAuras addon

Without addon

Export String

[NoM0Re]

This is not a WeakAuras bug.
The error is caused by the Ascension Ticket UI calling Blizzard’s global Clamp() function with string values.

From the error output:

value = 285.00000677034
min   = "260"
max   = "596"

The Blizzard helper looks like this:

function Clamp(value, min, max)
	if value > max then
		return max
	elseif value < min then
		return min
	end
	return value
end

Clamp() expects numbers for min and max.
Passing strings causes:

attempt to compare string with number

The stack trace shows that this happens inside:

Ascension_TicketUI\ViewTicketPanelMixin.lua:RefreshMessages()

WeakAuras only provides compatibility layers so Blizzard UI code can run on 3.3.5.
It does not change the behavior of Clamp() and does not pass these values.

This function has been unchanged in Blizzard’s UI code for many years.

Correct fix on Ascension side

The Ticket UI must ensure numeric arguments are passed, for example:

Clamp(value, tonumber(min), tonumber(max))

(or fix the upstream values so they are already numbers).

Important note

If Ascension needs a custom clamping behavior, they should not rely on or override the global Blizzard Clamp() helper.
They should use a local helper function instead.

Conclusion

This is an Ascension Ticket UI bug.
WeakAuras is only triggering the code path but is not the cause of the error.

https://github.com/Gethe/wow-ui-source/blob/d6dffb48b2c0d7a7ab5d37dad2817ccee710e0ef/Interface/AddOns/Blizzard_SharedXMLBase/MathUtil.lua#L27-L34