1 files changed, 76 insertions, 3 deletions

diff --git a/engine/classes/ODDDocument.php b/engine/classes/ODDDocument.php
index 834b5b8fe..540c35a3b 100644
--- a/engine/classes/ODDDocument.php
+++ b/engine/classes/ODDDocument.php
@@ -1,8 +1,9 @@
 <?php
 /**
- * @class ODDDocument ODD Document container.
  * This class is used during import and export to construct.
- * @author Curverider Ltd
+ *
+ * @package    Elgg.Core
+ * @subpackage ODD
  */
 class ODDDocument implements Iterator {
 	/**
@@ -22,6 +23,13 @@ class ODDDocument implements Iterator {
 	 */
 	private $wrapperfactory;
 
+	/**
+	 * Create a new ODD Document.
+	 *
+	 * @param array $elements Elements to add
+	 *
+	 * @return void
+	 */
 	public function __construct(array $elements = NULL) {
 		if ($elements) {
 			if (is_array($elements)) {
@@ -43,29 +51,57 @@ class ODDDocument implements Iterator {
 		return $this->ODDSupportedVersion;
 	}
 
+	/**
+	 * Returns the number of elements
+	 *
+	 * @return int
+	 */
 	public function getNumElements() {
 		return count($this->elements);
 	}
 
+	/**
+	 * Add an element
+	 *
+	 * @param ODD $element An ODD element
+	 *
+	 * @return void
+	 */
 	public function addElement(ODD $element) {
 		if (!is_array($this->elements)) {
 			$this->elements = array();
-			$this->elements[] = $element;
 		}
+		$this->elements[] = $element;
 	}
 
+	/**
+	 * Add multiple elements at once
+	 *
+	 * @param array $elements Array of ODD elements
+	 *
+	 * @return void
+	 */
 	public function addElements(array $elements) {
 		foreach ($elements as $element) {
 			$this->addElement($element);
 		}
 	}
 
+	/**
+	 * Return all elements
+	 *
+	 * @return array
+	 */
 	public function getElements() {
 		return $this->elements;
 	}
 
 	/**
 	 * Set an optional wrapper factory to optionally embed the ODD document in another format.
+	 *
+	 * @param ODDWrapperFactory $factory The factory
+	 *
+	 * @return void
 	 */
 	public function setWrapperFactory(ODDWrapperFactory $factory) {
 		$this->wrapperfactory = $factory;
@@ -73,6 +109,8 @@ class ODDDocument implements Iterator {
 
 	/**
 	 * Magic function to generate valid ODD XML for this item.
+	 *
+	 * @return string
 	 */
 	public function __toString() {
 		$xml = "";
@@ -107,22 +145,57 @@ class ODDDocument implements Iterator {
 
 	private $valid = FALSE;
 
+	/**
+	 * Iterator interface
+	 *
+	 * @see Iterator::rewind()
+	 *
+	 * @return void
+	 */
 	function rewind() {
 		$this->valid = (FALSE !== reset($this->elements));
 	}
 
+	/**
+	 * Iterator interface
+	 *
+	 * @see Iterator::current()
+	 *
+	 * @return void
+	 */
 	function current() {
 		return current($this->elements);
 	}
 
+	/**
+	 * Iterator interface
+	 *
+	 * @see Iterator::key()
+	 *
+	 * @return void
+	 */
 	function key() {
 		return key($this->elements);
 	}
 
+	/**
+	 * Iterator interface
+	 *
+	 * @see Iterator::next()
+	 *
+	 * @return void
+	 */
 	function next() {
 		$this->valid = (FALSE !== next($this->elements));
 	}
 
+	/**
+	 * Iterator interface
+	 *
+	 * @see Iterator::valid()
+	 *
+	 * @return void
+	 */
 	function valid() {
 		return $this->valid;
 	}