WowAce.com

Grid2

180 - frame is nil in GridHeader_InitialConfigFunction

In r535, GridLayout.lua, the second argument of function GridHeader_InitialConfigFunction, "frame", is being passed in as nil, and thus throwing an error when nil is passed to RegisterFrame.

The problem seems to be with the initialConfigFunction attribute, and I was able to fix it with the following changes to GridLayout.lua:

Line 14:

	Grid2Frame:RegisterFrame(loadstring("return " .. frame)())

Line 50:

				header:CallMethod("initialConfigFunction", self:GetName())

The change passes the name of the frame as a string instead of the actual frame itself, and then the frame is retrieved the GridHeader_InitialConfigFunction via the name that was passed in.

This, along with clearing the saved variables file of old Paladin spells, got Grid2 working for me again. HUGE THANKS FOR THE UPDATE!!!!

User When Change
jerry Oct 19, 2010 at 06:36 UTC Changed status from New to Fixed
roncli Oct 18, 2010 at 21:57 UTC Changed description:
- In r535, GridLayout.lua, the second argument of function GridHeader_InitialConfigFunction, "frame", is returning nil, and thus throwing an error when nil is passed to RegisterFrame.
+ In r535, GridLayout.lua, the second argument of function GridHeader_InitialConfigFunction, "frame", is being passed in as nil, and thus throwing an error when nil is passed to RegisterFrame.
  The problem seems to be with the initialConfigFunction attribute, and I was able to fix it with the following changes to GridLayout.lua:
roncli Oct 18, 2010 at 21:56 UTC Create

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

  • 1 comment
  • Avatar of jerry jerry Oct 19, 2010 at 06:36 UTC - 0 likes

    Fixed in rev536. Thanks for the report.

  • 1 comment

Facts

Last updated
Oct 19, 2010
Reported
Oct 18, 2010
Status
Fixed - Developer made requested changes. QA should verify.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees