Check that a section has fields first, fixes #406

brendo · Sep 25, 2010 · 7c37dc0 · 7c37dc0
1 parent 80c0153
commit 7c37dc0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/symphony/content/content.blueprintsevents.php b/symphony/content/content.blueprintsevents.php
@@ -320,11 +320,16 @@ function __formAction(){
 
 				$sectionManager = new SectionManager($this->_Parent);
 				$section = $sectionManager->fetch($fields['source']);
-				$markup = NULL;
-				foreach($section->fetchFields() as $f){
-					if ($f->getExampleFormMarkup() instanceof XMLElement)
-						$container->appendChild($f->getExampleFormMarkup());
+
+				$section_fields = $section->fetchFields();
+				if(is_array($section_fields) && !empty($section_fields)) {
+					foreach($section_fields as $f) {
+						if ($f->getExampleFormMarkup() instanceof XMLElement) {
+							$container->appendChild($f->getExampleFormMarkup());
+						}
+					}
 				}
+
 				$container->appendChild(Widget::Input('action['.$rootelement.']', __('Submit'), 'submit'));
 
 				$code = $container->generate(true);
@@ -442,4 +447,4 @@ function __injectAboutInformation(&$shell, $details){
 		}
 	}
 
-?>
+?>
diff --git a/symphony/lib/toolkit/class.eventmanager.php b/symphony/lib/toolkit/class.eventmanager.php
@@ -85,14 +85,10 @@ function listAll(){
 							$f = $f = self::__getHandleFromFilename($f);
 
 							if($about = $this->about($f)){
-
 								$classname = $this->__getClassName($f);
-
-								$can_parse = false;
-								$type = NULL;
-
-								$about['can_parse'] = $can_parse;
-								$result[$f] = $about;	
+
+								$about['can_parse'] = false;
+								$result[$f] = $about;
 							}
 						}						
 					}					
@@ -119,7 +115,6 @@ function &create($name, $environment=NULL){
 
 			return new $classname($this->_Parent, $environment);	
 
-        }       
+        }
 
     }
-