WeakAuras

WeakAuras

206M Downloads

Weakauras not importing via chat caused by special symbols.

Melonoway opened this issue ยท 3 comments

commented

Describe the bug

Whenever I try to share a weakaura that has special symbols in it's name such as [ ] in case, it displays an error "Requested display does not exist"

Do you have an error log of what happened?

There are no errors in bug-grabber I use it all the time to analyze this stuff in case shit hits the fan.

To Reproduce

Screenshots

https://i.gyazo.com/dfeb36d5793c885ea6da74a794b8ff4e.png

Did you try having WeakAuras as the only enabled addon and everything else (especially something like ElvUI) disabled?

Yes, base blizz + wa's

Which version of WeakAuras are you using?

2.11.6

Was it working in a previous version? If yes, which was the last good one?

No

Additional context

commented

This is a fairly standard case of pattern matching being used when a true parser would be more appropriate. See here, where the display name is taken to be a string of characters which are not ], but then is followed by ]:

local start, finish, characterName, displayName = remaining:find("%[WeakAuras: ([^%s]+) %- ([^%]]+)%]");

Since there already exists a simple workaround (just don't include ] in your aura names) and we plan to revisit how the database is indexed in the first place (with #1039) which would render this problem moot, I am personally in favor of not fixing this at the moment.

commented

Even if we internally use a uid, the problem would remain. As this is the text that is shown to the user in a hyperlink, that'll still use the displayname.

commented

It seems i've been overruled!