diff --git a/SeedDMS_Core/Core/inc.ClassGroup.php b/SeedDMS_Core/Core/inc.ClassGroup.php
index 54fc3c88f..bd092e226 100644
--- a/SeedDMS_Core/Core/inc.ClassGroup.php
+++ b/SeedDMS_Core/Core/inc.ClassGroup.php
@@ -36,6 +36,13 @@ class SeedDMS_Core_Group { /* {{{ */
 	 */
 	protected $_name;
 
+	/**
+	 * The comment of the user group
+	 *
+	 * @var string
+	 */
+	protected $_comment;
+
 	/**
 	 * Back reference to DMS this user group belongs to
 	 *
@@ -145,8 +152,14 @@ class SeedDMS_Core_Group { /* {{{ */
 		return true;
 	} /* }}} */
 
-	// $asManager=false: verify if user is in group
-	// $asManager=true : verify if user is in group as manager
+	/**
+	 * Check if user is member of group
+	 *
+	 * @param object $user user to be checked
+	 * @param boolean $asManager also check whether user is manager of group if
+	 * set to true, otherwise does not care about manager status
+	 * @return boolean true if user is member, otherwise false
+	 */
 	function isMember($user,$asManager=false) { /* {{{ */
 		if (isset($this->_users)&&!$asManager) {
 			foreach ($this->_users as $usr)
@@ -167,6 +180,12 @@ class SeedDMS_Core_Group { /* {{{ */
 		return true;
 	} /* }}} */
 
+	/**
+	 * Toggle manager status of user
+	 *
+	 * @param object $user
+	 * @return boolean true if operation was successful, otherwise false
+	 */
 	function toggleManager($user) { /* {{{ */
 		$db = $this->_dms->getDB();