Changeset 2549

Timestamp:

07/19/10 15:24:46 (8 weeks ago)

Author:

mattlevine

Message:

1. Adding logic to make sure that siteIDs are valid cf variables.

Location:

branches/5.2/www/requirements/mura

Files:

• settings/settingsBean.cfc (modified) (1 diff)
• utility.cfc (modified) (1 diff)

branches/5.2/www/requirements/mura/settings/settingsBean.cfc

@@ -142 +142 @@
         
         <cfif getSiteID() eq "admin" or getSiteID() eq "tasks">
+                <cfset variables.instance.errors.siteid="The 'SiteID' variable is invalid." />
+        </cfif>
+        
+        <cfif not getBean('utility').isValidCFVariableName(getSiteID())>
                 <cfset variables.instance.errors.siteid="The 'SiteID' variable is invalid." />
         </cfif>

branches/5.2/www/requirements/mura/utility.cfc

@@ -437 +437 @@
                 </cfscript>
 </cffunction>
-        
+
+<!---
+Author: John Mason, mason@fusionlink.com
+Blog: www.codfusion.com--->
+<cffunction name="isValidCFVariableName" output="false" access="public" returntype="Any">
+                <cfargument name="text" required="true" type="String">
+                <cfset var local = StructNew()/>        
+                <cfset local.result = true/>
+
+                <cfif len(arguments.text) eq 0>
+                        <cfset local.result = false/>
+                <cfelseif FindNoCase(".",arguments.text) gt 0>
+                        <cfset local.result = false/>
+                <cfelseif FindNoCase(" ",arguments.text) gt 0>
+                        <cfset local.result = false/>
+                <cfelseif ReFindNoCase("^[A-Za-z][A-Za-z0-9_]*",arguments.text) eq 0>
+                        <cfset local.result = false/>
+                </cfif>
+
+                <cfreturn local.result/>
+        </cffunction>
 </cfcomponent>