CFFORM and CFINPUT issue

Feb 23, 2008

I've been neglecting my blog because I have been working on several large eCommerce projects over the last two months, but I found an issue that I think is worth asking the CF community about.

With the new features in CF8, I find myself wanting to use cfform and cfinput more for binding and Ajax features, but I keep running into issues with errors generated by ColdFusion.

 Like most developers, I like to make my code modular for use in various parts of an application.  I have traditionally done this with forms, where I will set up a CFC that contains a method for displaying part of a form.  This worked great when my forms were straight HTML/JavaScript.  But take a look at this simplified example...

My CFM file...
<cfset formlibrary = createObject('component','formlib') />
<cfform action="myscript.cfm" method="post">
    <cfset formlib.printNameFields() />   
    <cfset formlib.printAddressField() />
</cfform>

My CFC file...
<cfcomponent>
    <cffunction name="printNameFields">
       First name: <cfinput type="text" name="firstname" required="yes" message="First name is required." />
       Last name: <cfinput type="text" name="lastname" required="yes" message="Last name is required." />
       etc...
    </cfcomponent>
    <cffunction name="printAddressFields">
       Line 1: <cfinput type="text" name="addr1"  />
       Line 2:  <cfinput type="text" name="addr2"  />
          etc...
    </cfcomponent>
</cfcomponent>

This code will generate a CF error because you cannot have a CFINPUT tag that is not nested inside a CFFORM tag.  However, if the error did not fire, the generated source code sent to the browser would work!  The error message that ColdFusion generates isn't really an error.  ColdFusion just doesn't understand that I am outputting the form fields from a CFC that is inside the CFFORM tag.  

This is a simple example of the problem, but what if I also wanted to use the CF8 type="datefield" CFINPUT?  I can't have that in a CFC either.  I also can't use any CF8 ajax binding on with CFINPUT tags.  

 For now, I am just putting the CFINPUT tags in the CFM file and not making the code as modular as I would like.  It is frustrating, though, that I have to repeat code on various forms for name and address form fields. 

Does anyone have a suggestion for this?  Is there a way to suppress the ColdFusion error and let it just generate the code?  Or am I missing something? 

Comments

Todd Rafferty

Todd Rafferty wrote on 02/24/08 10:27 AM

I would highly recommend against CFCs being used to handle display logic like this. If you must have something, then use &lt;cfsavecontent variable="myoutput"&gt; to save the content and &lt;cfreturn myoutput&gt;. That being said, even what I recommended won't work because there's still no cfform around the block of code and it'll throw the same error.

That being said, things to try:
1.) Have you tried putting output="Yes" in the function?
2.) Have you tried keeping the code modular by using cfinclude instead?
Michael Sprague

Michael Sprague wrote on 03/02/08 11:16 AM

Thanks Todd. For suggestion #1, the output=true doesn't work because the issue is with ColdFusion's built-in validation. It is looking for a cfform tag around the cfinput, and whether there is output or not the cfform tag just isn't in the CFC.

Suggestion #2 will work, but it strays from the way all of my apps are structured. I have a cfc with all of my re-usable display methods, which I think is a fairly common method for handling modular displays because I see it frequently in most of the major open source CF apps on the market. It works REALLY well and efficiently except in this case when CF's validation is actually incorrect.

I will probably just use includes for cases when I need to have modular code with CFFORM, so that is a good suggestion. Overall, though, I don't see why using a CFC display library is a bad idea. Seems to make more sense to me than having a whole series of include files.
Cyrill Gross

Cyrill Gross wrote on 08/17/08 9:21 AM

Hi there
It is quite simple to go around that:
Just add a <cfform> tag qithout any attributes as an HTML-Comment into you inner cfc (the one with the form fields):

<cffunction name="printAddressFields">
<!-- <cfform> -->
Line 1: <cfinput type="text" name="addr1" />
Line 2: <cfinput type="text" name="addr2" />
etc...
<!-- </cfform> -->
</cfcomponent>

=> Take care using HTML comments (<!-- -->) and not ColdFusion comments (<!--- --->).

The ColdFusion compiler finds the <cfform> tag and therefore recognizes the code as valid but the browser does not interpretes the form (if you load it asynchronously it will be filtered by Ext at all :-) ).
Works fine in my application...

Regards, Cyrill

Write your comment



(it will not be displayed)





About Michael Sprague

Mike is a Web developer and manager from Central New York, specializing in ColdFusion, CSS, JavaScript, and Ajax development. More ...

Categories

Monthly Archives

Favorite Bloggers

Web Resources