Improved validations for Pages

raulriera · Apr 23, 2010 · b623aea · b623aea
1 parent fd40c7e
commit b623aea
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 12 deletions.
diff --git a/models/Page.cfc b/models/Page.cfc
@@ -1,14 +1,17 @@
 <cfcomponent extends="Model" output="false">
 
     <cffunction name="init">
-        <cfset belongsTo(name='pageLayout', class="layout", foreignKey="layoutID")>
-        <cfset hasMany('PageParts')>
+		<cfset hasMany('PageParts')>
+        <cfset belongsTo(name='pageLayout', class="layout", foreignKey="layoutID")>      
         <cfset belongsTo('PageClass')>
         <cfset belongsTo(name="author", class="user", foreignKey="createdByID")>
 
         <cfset validatesPresenceOf(property="title" , message="Your page must have a title.")>
+		<cfset validatesLengthOf(properties="title,keywords,description", message="You have exceeded the maximun length", allowBlank="true", maximum="255")>
         <cfset validatesPresenceOf(property="slug" , message="Your page must have a slug." )>
         <cfset validatesUniquenessOf(property="slug", message="The slug you entered already exists.")>
+		<cfset validatesLengthOf(property="slug", message="You have exceeded the maximun length for the slug", maximum="100")>
+		<cfset validatesLengthOf(property="breadcrumb", message="You have exceeded the maximun length for the breadcrumb", allowBlank="true", maximum="160")>
 
         <cfset beforeCreate('setCreatedByID')>
         <cfset beforeUpdate('setUpdatedByID')>

diff --git a/views/pages/edit.cfm b/views/pages/edit.cfm
@@ -29,21 +29,21 @@
 	</cfif>
 
 		<p><label class="label">Page Title</label>
-			#textField(objectName='page', property='title', class="text_field")#</p>
+			#textField(objectName='page', property='title', class="text_field", maxlength="255")#</p>
 
 		<div class="more">
 
 		  	<p><label class="label">Slug</label>
-					#textField(objectName='page', property='slug', class="text_field slug")#</p>
+					#textField(objectName='page', property='slug', class="text_field slug", maxlength="100")#</p>
 
 			<p><label class="label">Breadcrumb</label>
-					#textField(objectName='page', property='breadcrumb', class="text_field")#</p>
+					#textField(objectName='page', property='breadcrumb', class="text_field", maxlength="160")#</p>
 
 			<p><label class="label">Description</label>
-					#textField(objectName='page', property='description', class="text_field")#</p>	
+					#textField(objectName='page', property='description', class="text_field", maxlength="255")#</p>	
 
 			<p><label class="label">Keywords</label>
-					#textField(objectName='page', property='keywords', class="text_field")#</p>
+					#textField(objectName='page', property='keywords', class="text_field", maxlength="255")#</p>
 		</div>
 	  </cfoutput>
 	  <div id="tabs">

diff --git a/views/pages/new.cfm b/views/pages/new.cfm
@@ -26,21 +26,21 @@
 		#hiddenField(objectName="page", property="parentid")#
 
 		<p><label class="label">Page Title</label>
-			#textField(objectName='page', property='title', class="text_field")#</p>
+			#textField(objectName='page', property='title', class="text_field", maxlength="255")#</p>
 
 		<div class="more">
 
 		  	<p><label class="label">Slug</label>
-						#textField(objectName='page', property='slug', class="text_field slug")#</p>
+						#textField(objectName='page', property='slug', class="text_field slug", maxlength="100")#</p>
 
 				<p><label class="label">Breadcrumb</label>
-						#textField(objectName='page', property='breadcrumb', class="text_field")#</p>
+						#textField(objectName='page', property='breadcrumb', class="text_field", maxlength="160")#</p>
 
 				<p><label class="label">Description</label>
-						#textField(objectName='page', property='description', class="text_field")#</p>	
+						#textField(objectName='page', property='description', class="text_field", maxlength="255")#</p>	
 
 				<p><label class="label">Keywords</label>
-						#textField(objectName='page', property='keywords', class="text_field")#</p>
+						#textField(objectName='page', property='keywords', class="text_field", maxlength="255")#</p>
 		</div>
 
 	  <div id="tabs">