code cleanup

include/class.dept.php

@@ -548,8 +548,7 @@ implements TemplateVariable {
             $query->order_by('name')
                  ->values('id', 'pid', 'flags', 'name', 'parent');
 
-            foreach ($query as $row)
-            {
+            foreach ($query as $row) {
               $display = ($row['flags'] & self::FLAG_ACTIVE);
 
               $depts[$row['id']] = array('id' => $row['id'], 'pid'=>$row['pid'], 'name'=>$row['name'],

include/class.filter.php

@@ -457,8 +457,7 @@ class Filter {
     }
 
     function save($id,$vars,&$errors) {
-      if ($this)
-      {
+      if ($this) {
         foreach ($this->getActions() as $A) {
           if ($A->type == 'dept')
               $dept = Dept::lookup($A->parseConfiguration($vars)['dept_id']);

include/staff/filter.inc.php

@@ -240,8 +240,7 @@ $info=Format::htmlchars(($errors && $_POST)?$_POST:$info);
                 if ($filter) { foreach ($filter->getActions() as $A) {
                     $_warn = '';
                     $existing[] = $A->type;
-                    if($A->type == 'dept')
-                    {
+                    if($A->type == 'dept') {
                       $errors['topic_id'] = '';
                       // $info=Format::htmlchars(($errors && $_POST)?$_POST:$info);
                       $dept_config = $A->parseConfiguration($_POST);
@@ -249,8 +248,7 @@ $info=Format::htmlchars(($errors && $_POST)?$_POST:$info);
                       if($dept && !$dept->isActive())
                         $_warn = sprintf(__('%s must be active'), __('Department'));
                     }
-                    elseif($A->type == 'topic')
-                    {
+                    elseif($A->type == 'topic') {
                       $errors['dept_id'] = '';
                       $topic_config = $A->parseConfiguration($_POST);
                       $topic = Topic::lookup($topic_config['topic_id']);