Refactoring - cleaning up whitespace

[timetracker.git] / cf_custom_field_edit.php

diff --git a/cf_custom_field_edit.php b/cf_custom_field_edit.php
index 71ffd39..b6a1f46 100644 (file)
--- a/cf_custom_field_edit.php
+++ b/cf_custom_field_edit.php
@@ -42,16 +42,15 @@ if (false === $field)
   $errors->add($i18n->getKey('error.db'));
 
 $form = new Form('fieldForm');
-if ($errors->isEmpty()) {
+if ($errors->no()) {
   $form->addInput(array('type'=>'text','maxlength'=>'100','name'=>'name','value'=>$field['label']));
   $form->addInput(array('type'=>'hidden','name'=>'id','value'=>$cl_id));
   $form->addInput(array('type'=>'checkbox','name'=>'required','data'=>1,'value'=>$field['required']));
   $form->addInput(array('type'=>'combobox','name'=>'type','value'=>$field['type'],
     'data'=>array(CustomFields::TYPE_TEXT=>$i18n->getKey('label.type_text'),
-                  CustomFields::TYPE_DROPDOWN=>$i18n->getKey('label.type_dropdown'))
-  ));
-  $form->addInput(array('type'=>'submit','name'=>'btn_save','value'=>$i18n->getKey('button.save')));   
-}    
+                  CustomFields::TYPE_DROPDOWN=>$i18n->getKey('label.type_dropdown'))));
+  $form->addInput(array('type'=>'submit','name'=>'btn_save','value'=>$i18n->getKey('button.save')));
+}
 
 if ($request->getMethod() == 'POST') {
   $cl_name = trim($request->getParameter('name'));
@@ -59,20 +58,19 @@ if ($request->getMethod() == 'POST') {
   $cl_required = $request->getParameter('required');
   if (!$cl_required)
     $cl_required = 0;
-  
+
   // Validate user input.
   if (!ttValidString($cl_name)) $errors->add($i18n->getKey('error.field'), $i18n->getKey('label.thing_name'));
 
-  if ($errors->isEmpty()) {
+  if ($errors->no()) {
     $res = CustomFields::updateField($cl_id, $cl_name, $cl_type, $cl_required);
     if ($res) {
       header('Location: cf_custom_fields.php');
       exit();
-    } else {
+    } else
       $errors->add($i18n->getKey('error.db'));
-    }
   }
-}
+} // POST
 
 $smarty->assign('forms', array($form->getName()=>$form->toArray()));
 $smarty->assign('onload', 'onLoad="document.fieldForm.name.focus()"');