Bug in content.cfm

2 views
Skip to first unread message

Clarke Bishop

unread,
Mar 3, 2010, 2:11:38 PM3/3/10
to spla...@googlegroups.com

In content.cfm, the code starts:

 

<cfif thisTag.executionMode IS "start">

  

    <cfparam name="attributes.part" default="">

    <cfparam name="page" default="#request.page#">

 

    <cfset wheelsProxy=CreateObject("component","controllers.Controller") >

 

    <cfset pagePart = wheelsProxy.wheelsExecute("model('pagePart').findByPage(pageID=#page.ID#, part = '#attributes.part#')")>

 

Shouldn’t this be:

 

<cfif thisTag.executionMode IS "start">

 

    <cfparam name="attributes.part" default="">

    <cfparam name="attributes.page" default="#request.page#">

 

    <cfset wheelsProxy=CreateObject("component","controllers.Controller") >

 

    <cfset pagePart = wheelsProxy.wheelsExecute("model('pagePart').findByPage(pageID=#attributes.page.ID#, part = '#attributes.part#')")>

 

Isn’t the intent to be able to pass the page into the tag?

 

   Clarke

Office: 770-642-1353
eMail: cbi...@resultantsys.com

 

raulriera

unread,
Mar 3, 2010, 10:57:40 PM3/3/10
to Splash CMS
Looks like it, although it could be that Russ was "pretting up" the
variable name

> eMail: cbis...@resultantsys.com

Clarke Bishop

unread,
Mar 4, 2010, 9:11:00 AM3/4/10
to spla...@googlegroups.com
Just curious, but what do you mean by "prettying up" the variable name?

Doesn't it have to specify the attributes scope?

Clarke

raulriera

unread,
Mar 4, 2010, 12:57:32 PM3/4/10
to Splash CMS
In Actionscript people often rename their variables just so they are
shorter... but I am sure that is not the case here :)
Reply all
Reply to author
Forward
0 new messages