Chatter

243 - lua error blocking channel text when player name - level is used

What steps will reproduce the problem?
1. using player names with levels turned on
2.
3.

What is the expected output? What do you see instead?
I expect to see players talk in channel

What version of the product are you using?
chatter 1.3.4-2

Do you have an error log of what happened?
Interface\AddOns\Chatter\Modules\PlayerNames.lua:532 bad argument #2 to 'format' (string expected, got no value)
Time: 08/30/12 08:19:58
Count: 3
Stack: [C]: ?
Interface\AddOns\Chatter\Modules\PlayerNames.lua:532 in function <Interface\AddOns\Chatter\Modules\PlayerNames.lua:531
[C]: ?
Interface\AddOns\Chatter\Modules\PlayerNames.lua:531 in function <Interface\AddOns\Chatter\Modules\PlayerNames.lua:500
[C]: ?
Interface\AddOns\Chatter\Modules\PlayerNames.lua:571 in function <Interface\AddOns\Chatter\Modules\PlayerNames.lua:569
(tail call): ?
Interface\FrameXML\ChatFrame.lua:3271 in function <Interface\FrameXML\ChatFrame.lua:2929
(tail call): ?
(tail call): ?
Interface\FrameXML\ChatFrame.lua:2741 in function `ChatFrame_OnEvent'
[string "*:OnEvent"]:1: in function <[string "*:OnEvent"]:1>

Locals: (*temporary) = "%s%s"
(*temporary) = "string expected, got no value"

Please provide any additional information below.
Basically, with player levels turned on in the player name module, i get that lua error and nothing in the channel when myself or anyone else speaks.
Turning off the levels after their names, and chat works fine.

User When Change
kagaro Sep 02, 2012 at 14:42 UTC
glayde Aug 30, 2012 at 15:33 UTC Changed name from chatter 1.3.4-2 to lua error blocking channel text when player name - level is used
glayde Aug 30, 2012 at 12:22 UTC Create

You must login to post a comment. Don't have an account? Register to get one!

  • 2 comments
  • Avatar of brykrys brykrys Aug 31, 2012 at 08:47 UTC - 0 likes

    Suggestion:

    	-- If we already have a color -- steal it and use it to color the level
    	local color = strmatch(displayName, "|cff%x%x%x%x%x%x")
    	if color then
    		level = format("%s%s|r", color, level)
    	end
    	displayName = format("%s%s%s", displayName, separator, level)
    
  • Avatar of mikk mikk Aug 30, 2012 at 17:55 UTC - 0 likes

    532: return format("%s%s|r")

    Will obviously never work

  • 2 comments

Facts

Last updated
Sep 02, 2012
Reported
Aug 30, 2012
Status
Fixed - Developer made requested changes. QA should verify.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.
Votes
2

Reported by

Possible assignees