add more documentation

SeedDMS_Core/Core/inc.ClassDMS.php

@@ -155,8 +155,8 @@ class SeedDMS_Core_DMS {
 	 * two objects, which isn't required. The method will first check
 	 * if the objects are instances of the same class.
 	 *
-	 * @param object $object1
+	 * @param object $object1 first object to be compared
-	 * @param object $object2
+	 * @param object $object2 second object to be compared
 	 * @return boolean true if objects are equal, otherwise false
 	 */
 	static function checkIfEqual($object1, $object2) { /* {{{ */
@@ -246,6 +246,14 @@ class SeedDMS_Core_DMS {
 			$this->version = '4.3.11';
 	} /* }}} */
 
+	/**
+	 * Return database where meta data is stored
+	 *
+	 * This method returns the database object as it was set by the first
+	 * parameter of the constructor.
+	 *
+	 * @return object database
+	 */
 	function getDB() { /* {{{ */
 		return $this->db;
 	} /* }}} */
@@ -309,14 +317,20 @@ class SeedDMS_Core_DMS {
 	/**
 	 * Set maximum number of subdirectories per directory
 	 *
-	 * The value of maxDirID is quite crucial because, all documents are
+	 * The value of maxDirID is quite crucial, because each document is
-	 * associated with a directory in the filesystem. Consequently, there is
+	 * stored within a directory in the filesystem. Consequently, there can be
-	 * maximum number of documents, because depending on the file system
+	 * a maximum number of documents, because depending on the file system
 	 * the maximum number of subdirectories is limited. Since version 3.3.0 of
-	 * SeedDMS an additional directory level has been introduced. All documents
+	 * SeedDMS an additional directory level has been introduced, which
+	 * will be created when maxDirID is not 0. All documents
 	 * from 1 to maxDirID-1 will be saved in 1/<docid>, documents from maxDirID
 	 * to 2*maxDirID-1 are stored in 2/<docid> and so on.
 	 *
+	 * Modern file systems like ext4 do not have any restrictions on the number
+	 * of subdirectories anymore. Therefore it is best if this parameter is
+	 * set to 0. Never change this parameter if documents has already been
+	 * created.
+	 *
 	 * This function must be called right after creating an instance of
 	 * {@link SeedDMS_Core_DMS}
 	 *
@@ -1726,6 +1740,7 @@ class SeedDMS_Core_DMS {
 	/**
 	 * Return workflow by its Id
 	 *
+	 * @param integer $id internal id of workflow
 	 * @return object of instances of {@link SeedDMS_Core_Workflow} or false
 	 */
 	function getWorkflow($id) { /* {{{ */
@@ -1749,6 +1764,7 @@ class SeedDMS_Core_DMS {
 	/**
 	 * Return workflow by its name
 	 *
+	 * @param string $name name of workflow
 	 * @return object of instances of {@link SeedDMS_Core_Workflow} or false
 	 */
 	function getWorkflowByName($name) { /* {{{ */
@@ -1771,6 +1787,12 @@ class SeedDMS_Core_DMS {
 		return $workflow;
 	} /* }}} */
 
+	/**
+	 * Add a new workflow
+	 *
+	 * @param string $name name of workflow
+	 * @param string $initstate initial state of workflow
+	 */
 	function addWorkflow($name, $initstate) { /* {{{ */
 		$db = $this->db;
 		if (is_object($this->getWorkflowByName($name))) {

SeedDMS_Core/Core/inc.ClassNotification.php

@@ -56,6 +56,17 @@ class SeedDMS_Core_Notification { /* {{{ */
 	 */
 	protected $_dms;
 
+	/**
+	 * Constructor
+	 *
+	 * @param integer $target id of document/folder this notification is
+	 * attached to.
+	 * @param integer $targettype 1 = target is document, 2 = target is a folder
+	 * @param integer $userid id of user. The id is -1 if the notification is
+	 * for a group.
+	 * @param integer $groupid id of group. The id is -1 if the notification is
+	 * for a user.
+	 */
 	function SeedDMS_Core_Notification($target, $targettype, $userid, $groupid) { /* {{{ */
 		$this->_target = $target;
 		$this->_targettype = $targettype;
@@ -63,16 +74,43 @@ class SeedDMS_Core_Notification { /* {{{ */
 		$this->_groupid = $groupid;
 	} /* }}} */
 
+	/**
+	 * Set instance of dms this object belongs to
+	 *
+	 * @param object $dms instance of dms
+	 */
 	function setDMS($dms) { /* {{{ */
 		$this->_dms = $dms;
 	} /* }}} */
 
+	/**
+	 * Get id of target (document/object) this notification is attachted to
+	 *
+	 * @return integer id of target
+	 */
 	function getTarget() { return $this->_target; }
 
+	/**
+	 * Get type of target
+	 *
+	 * @return integer type of target (1=document/2=object)
+	 */
 	function getTargetType() { return $this->_targettype; }
 
+	/**
+	 * Get user for this notification
+	 *
+	 * @return integer id of user or -1 if this notification does not belong
+	 * to a user
+	 */
 	function getUser() { return $this->_dms->getUser($this->_userid); }
 
+	/**
+	 * Get group for this notification
+	 *
+	 * @return integer id of group or -1 if this notification does not belong
+	 * to a group
+	 */
 	function getGroup() { return $this->_dms->getGroup($this->_groupid); }
 } /* }}} */
 ?>