Raymond Camden wrote on 09/22/08 11:13 AM

Yep, this is a good tip. This is two good tips actually. So first, youare right in using the myself VS value to be more 'safe' in the MG framework. The second tip - copying it from the VS into a local var, is also a good one as it saves you a heck of a lot of typing.

Tony Petruzzi wrote on 09/22/08 11:13 AM

Model-Glue should seriously take a hint from Mach-ii and incorporate a BuildUrl(). It's so nice just having to type BuildURL('my.event').

Rob Munn wrote on 09/22/08 11:13 AM

Mark,

Nice tip. I also like the practice of coding events into the ModelGlue.xml file, e.g. :

<event-handler name="cms.showEditObject" access="public">
   <broadcasts >
      <message name="needObject"/>
   </broadcasts>
   <results />
   <views>
      <include name="page" template="cms/dspShowEditObject.cfm">
         <value name="xe.showBinaryObject" value="showBinaryObject"></value>
         <value name="xe.saveObject" value="saveObject"></value>
         <value name="xe.deleteObject" value="deleteObject"></value>      
      </include>
   </views>
</event-handler>

I have also gotten into the habit of using two part event names, like cms.showEditObject, in ModelGlue.xml. Then I leave the first part of the event off of the exit event variables (xe.), and dynamically set the first part in my views using a utility function:

<cfset event_path = "" />
<cfif listlen(viewstate.getValue("event"), ".") eq 2>
   <!--- get the first part of the event to make sure we set the exit events correctly --->
   <cfset event_path = getToken(viewstate.getValue("event"),1,".") & "." />
</cfif>

<cfscript>
   function setPath(basePath,eventPath,varValue){
      if(listLen(varValue,".") eq 2)
         return basePath & varValue;
      if(listLen(varValue,".") eq 1)
         return basePath & eventPath & varValue;
   }
</cfscript>

Then I set the events like so:

<cfset submit = setPath(viewstate.getValue("myself"),event_path,viewState.getValue("xe.saveObject"))/>
<cfset deleteObject = setPath(viewstate.getValue("myself"),event_path,viewState.getValue("xe.deleteObject"))/>
<cfset showBinaryObject = setPath(viewstate.getValue("myself"),event_path,viewState.getValue("xe.showBinaryObject"))/>

In effect, I have created atomic events that I can compose as needed in my application. My cms.showEditObject event could be called through an admin interface, or it could be called inline in the application, each through a separate event, like so:

<event-handler name="admin.showEditObject">
   <broadcasts>
   </broadcasts>
   <results>
      <result do="cms.showEditObject"></result>
      <result do="layout.getAdminLayout"></result>
   </results>
   <views/>
</event-handler>   

In this separate event, I can include other business logic as dictated by the needs of the application. It ends up being a little bit verbose in the XML, but the trade-off in granularity of control is worth it to me.

Sean Corfield wrote on 09/22/08 11:13 AM

The self / myself thing comes from Fusebox (as does the concept of eXit Events - eXit FuseActions in Fusebox) and they're both good techniques for keeping the view abstracted from the actual URLs which in turn makes it easier to rewrite an application when the control flow needs to change.

As for buildUrl(), I'm in two minds about that. Fusebox tackles the problem by allowing XFAs to be constructed in the control layer and automatically handles SES URL generation. Overall, I'm not sure that I prefer href="#buildUrl('my.event')#" over href="#myself##xfa.next#" (where next is an XFA that resolves to my.event).

Jeff Gladnick wrote on 09/22/08 11:14 AM

This is good, but I need something better: Is there a way to set this so its avaliable on EVERY view somehow?

I tried doing this in my dspTemplate.cfm file that creates the header/footer, but unfortunately in this case (although fortunately in other cases) that doesnt work because model glue will load the "body" view BEFORE dspTemplate.

Is there a way to globally define a viewstate variable, like i used to do with a quick cfset in application.cfm?