[pLog-svn] r6626 - plog/branches/lifetype-1.2/class/action/admin

[jondaley at devel.lifetype.net]

Author: jondaley
Date: 2008-06-21 14:53:10 -0400 (Sat, 21 Jun 2008)
New Revision: 6626

Modified:
   plog/branches/lifetype-1.2/class/action/admin/adminupdateglobalsettingsaction.class.php
Log:
need to call parent::validate.  Comments about more work to do

Modified: plog/branches/lifetype-1.2/class/action/admin/adminupdateglobalsettingsaction.class.php
===================================================================
--- plog/branches/lifetype-1.2/class/action/admin/adminupdateglobalsettingsaction.class.php	2008-06-21 18:50:50 UTC (rev 6625)
+++ plog/branches/lifetype-1.2/class/action/admin/adminupdateglobalsettingsaction.class.php	2008-06-21 18:53:10 UTC (rev 6626)
@@ -25,11 +25,17 @@
 
 			$this->registerFieldValidator( "blogId", new IntegerValidator(), true);
 			$this->registerFieldValidator( "show", new StringValidator(), true);
+                // TODO: how do we validate the data inside the array?
 			$this->registerFieldValidator( "config", new ArrayValidator() );
+
+                // TODO: need an error view here
         }
 
         function validate()
         {
+            if(!parent::validate())
+                return false;
+            
 	    	// all the seettings come from a very nice array from the html form
             $this->_newConfigOpts = Array();
             $this->_newConfigOpts = $this->_request->getValue( "config" );