gsub and pattern failure

[myrroddin]

I am trying to replace text on the reputation bar. The current text looks like the following, but of course the number and status change with each watched faction update.

Valajar: 7.5K [Exalted]

What I would like to do is replace the [Exalted] portion with Paragon if the faction is paragon-level.

This is what I have so far.

if isParagon then
    local replacement = ""
    local barText = bar.text:GetText()
    local gender = UnitSex("player")
    local exalted = GetText("FACTION_STANDING_LABEL" .. standingID, gender)
    print("My standing is ", exalted)
    if E.db.PCB.reputationBar.textFormat == "P" then
        replacement = L["P"]
    else
        replacement = L["Paragon"]
    end
    local match = strmatch(barText, "^%[(.+)%]")
    print("My match is ", match)
    --[[
    barText = string.gsub(barText, "^%[" .. ("%w+"), replacement)
    barText = string.gsub(barText, "%[", "")
    barText = string.gsub(barText, "%]", "")
    bar.text:SetText(barText)
    ]]--
end

Sure enough, the first test print does say "My standing is Exalted" only if the watched rep is Paragon, but the second test print outputs "My match is nil" so it isn't matching anything. If I could get it to work, the end result would look like the below for Paragon factions. For every other faction it should still say whatever text is on the rep bar by default. This also means that if a faction is Paragon-capable yet the user has not gained Paragon status it should still say [Revered] or whatever.

Valajar: 7.5K Paragon

[myrroddin]

About the commented section, I have tried so many variants of pattern matching I can't even keep them straight anymore. That is just the last version before I added the test prints.

Also, the original author left off the gender portion of FACTION_STANDING_LABEL .. standingID and I don't know if that matters. I put it back in for my code.

[Rainrider]

Remove the ^ anchor from the match pattern, you are only trying to match at the beginning of the string.

[myrroddin]

Whoa. Okay, next question: if I wanted to keep the [] brackets and only wanted to replace the text between them, how would I match that?

Removing the ^ worked great, so thank you for that!

[Fizzlemizz]

replacement = "["..replacement .."]"

But I'm lazy

[Kanegasi]

The gender part is only for some non-English clients, since the reputation words are different based on gender. You can drop it if your client is English.

For example:

English GlobalStrings.lua:
	FACTION_STANDING_LABEL8 = "Exalted";
	FACTION_STANDING_LABEL8_FEMALE = "Exalted";

Spanish (ES) GlobalStrings.lua:
	FACTION_STANDING_LABEL8 = "Exaltado";
	FACTION_STANDING_LABEL8_FEMALE = "Exaltada";

As for your replacement, here's a simplified version that doesn't bother with brackets or anything else, drycoded:

if isParagon then
    local replacement
    if E.db.PCB.reputationBar.textFormat == "P" then
        replacement = L["P"]
    else
        replacement = L["Paragon"]
    end
    bar.text:SetText(gsub(bar.text:GetText(),FACTION_STANDING_LABEL8,replacement))
end

[myrroddin]

Excellent!! Thank you guys

[Rainrider]

If there is a faction with Exalted as part of its name, you might get undesired results:

Exalted Coders: 7.5K [Exalted]

will turn into

Paragon Coders: 7.5K [Paragon]

[myrroddin]

Originally Posted by Rainrider If there is a faction with Exalted as part of its name, you might get undesired results: Exalted Coders: 7.5K [Exalted] will turn into Paragon Coders: 7.5K [Paragon]

I realized that too, although it is a rare case, I should think. However, I decided to stick to a suggestion above and start the gsub at the "[" symbol which ought to prevent that outlier case.

local barText = bar.text:GetText()
barText = gsub(barText, "%[(.+)%]", replacement)

Unless someone finds it not to work when the addon goes live, I'm going with this as the solution.