[Fusionforge-commits] FusionForge branch master updated. ad9a19a2fae06abecef2647f528c1b5ccac678cd

Franck VILLAUME nerville at fusionforge.org
Fri Aug 23 10:47:46 CEST 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "FusionForge".

The branch, master has been updated
       via  ad9a19a2fae06abecef2647f528c1b5ccac678cd (commit)
      from  bbbf59c656e915f98f4fb0285f073f86a96e19e4 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit ad9a19a2fae06abecef2647f528c1b5ccac678cd
Author: Franck Villaume <franck.villaume at trivialdev.com>
Date:   Fri Aug 23 10:47:40 2013 +0200

    docman: cleanup, use tabs, space where is needed

diff --git a/src/common/docman/Document.class.php b/src/common/docman/Document.class.php
index f66172d..8018904 100644
--- a/src/common/docman/Document.class.php
+++ b/src/common/docman/Document.class.php
@@ -49,17 +49,17 @@ class Document extends Error {
 	 */
 	var $Group;
 
-    /**
-     * Constructor.
-     *
-     * @param $Group
-     * @param bool $docid
-     * @param bool $arr
-     * @internal param \The $object Group object to which this document is associated.
-     * @internal param \The $int docid.
-     * @internal param \The $array associative array of data.
-     * @return \Document
-     */
+	/**
+	 * Constructor.
+	 *
+	 * @param	$Group
+	 * @param	bool	$docid
+	 * @param	bool	$arr
+	 * @internal	param	\The $object Group object to which this document is associated.
+	 * @internal	param	\The $int docid.
+	 * @internal	param	\The $array associative array of data.
+	 * @return	\Document
+	 */
 	function __construct(&$Group, $docid = false, $arr = false) {
 		$this->Error();
 		if (!$Group || !is_object($Group)) {
@@ -98,16 +98,16 @@ class Document extends Error {
 	}
 
 	/**
-	 *	create - use this function to create a new entry in the database.
+	 * create - use this function to create a new entry in the database.
 	 *
-	 *	@param	string	The filename of this document. Can be a URL.
-	 *	@param	string	The filetype of this document. If filename is URL, this should be 'URL';
-	 *	@param	string	The absolute path file itself.
-	 *	@param	int	The doc_group id of the doc_groups table.
-	 *	@param	string	The title of this document.
-	 *	@param	string	The description of this document.
-	 *	@param	int	The state id of the document. At creation, can not be deleted status.
-	 *	@return	boolean	success.
+	 * @param	string	The filename of this document. Can be a URL.
+	 * @param	string	The filetype of this document. If filename is URL, this should be 'URL';
+	 * @param	string	The absolute path file itself.
+	 * @param	int	The doc_group id of the doc_groups table.
+	 * @param	string	The title of this document.
+	 * @param	string	The description of this document.
+	 * @param	int	The state id of the document. At creation, can not be deleted status.
+	 * @return	boolean	success.
 	 */
 	function create($filename, $filetype, $data, $doc_group, $title, $description, $stateid = 0) {
 		if (strlen($title) < 5) {
@@ -546,13 +546,13 @@ class Document extends Error {
 		return $values;
 	}
 
-    /**
-     * isMonitoredBy - get the monitored status of this document for a specific user id.
-     *
-     * @param string $userid
-     * @internal param \User $int ID
-     * @return    boolean    true if monitored by this user
-     */
+	/**
+	 * isMonitoredBy - get the monitored status of this document for a specific user id.
+	 *
+	 * @param	string $userid
+	 * @internal	param \User $int ID
+	 * @return	boolean    true if monitored by this user
+	 */
 	function isMonitoredBy($userid = 'ALL') {
 		if ( $userid == 'ALL' ) {
 			$condition = '';
@@ -643,16 +643,16 @@ class Document extends Error {
 		return $this->setValueinDB('doc_group', $newdocgroupid);
 	}
 
-    /**
-     * setLock - set the locking status of the document.
-     *
-     * @param $stateLock
-     * @param    time    the epoch time.
-     * @param int $thistime
-     * @internal param \The $int status of the lock.
-     * @internal param \The $int userid who set the lock.
-     * @return    boolean    success or not.
-     */
+	/**
+	 * setLock - set the locking status of the document.
+	 *
+	 * @param	$stateLock
+	 * @param	time		the epoch time.
+	 * @param	int		$thistime
+	 * @internal	param		\The $int status of the lock.
+	 * @internal	param		\The $int userid who set the lock.
+	 * @return	boolean		success or not.
+	 */
 	function setLock($stateLock, $userid = NULL, $thistime = 0) {
 		$res = db_query_params('UPDATE doc_data SET
 					locked = $1,
@@ -919,8 +919,8 @@ class Document extends Error {
 	 * sendNotice - Notifies of document submissions
 	 *
 	 * @param	boolean	true = new document (default value)
-     * @return bool
-     */
+	 * @return	bool
+	 */
 	function sendNotice($new = true) {
 		$BCC = $this->Group->getDocEmailAddress();
 		if ($this->isMonitoredBy('ALL')) {
diff --git a/src/common/docman/DocumentFactory.class.php b/src/common/docman/DocumentFactory.class.php
index c489ee0..79733b8 100644
--- a/src/common/docman/DocumentFactory.class.php
+++ b/src/common/docman/DocumentFactory.class.php
@@ -7,7 +7,7 @@
  * Copyright 2009, Roland Mas
  * Copyright 2010-2011, Franck Villaume - Capgemini
  * Copyright (C) 2012 Alain Peyrat - Alcatel-Lucent
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012-2013, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -78,14 +78,14 @@ class DocumentFactory extends Error {
 	 */
 	var $limit = 0;
 
-    /**
-     * Constructor.
-     *
-     * @param $Group
-     * @internal param \The $object Group object to which this DocumentFactory is associated.
-     * @return \DocumentFactory
-    @access	public
-     */
+	/**
+	 * Constructor.
+	 *
+	 * @param	$Group
+	 * @internal	param	\The $object Group object to which this DocumentFactory is associated.
+	 * @return	\DocumentFactory
+	 * @access	public
+	 */
 	function __construct(&$Group) {
 		$this->Error();
 		if (!$Group || !is_object($Group)) {
@@ -259,14 +259,14 @@ class DocumentFactory extends Error {
 		$this->limit = $limit;
 	}
 
-    /**
-     * getDocuments - returns an array of Document objects.
-     *
-     * @param int $nocache
-     * @internal param \no $integer cache : force reinit $this->Documents : default: cache is used
-     * @return    array    Document objects.
-     * @access    public
-     */
+	/**
+	 * getDocuments - returns an array of Document objects.
+	 *
+	 * @param	int	$nocache
+	 * @internal	param	\no $integer cache : force reinit $this->Documents : default: cache is used
+	 * @return	array	Document objects.
+	 * @access	public
+	 */
 	function &getDocuments($nocache = 0) {
 		if (!$this->Documents || $nocache) {
 			$this->getFromStorage();
@@ -370,15 +370,18 @@ class DocumentFactory extends Error {
 		return true;
 	}
 
-    function isTrashEmpty() {
-        $res = db_query_params('select count(*) as c from docdata_vw where group_id = $1 and stateid = 2',
-            array($this->Group->getID()));
-        if (!$res) {
-            return false;
-        }
-        return (db_result($res, 0, 'c') == 0);
-    }
-
+	/**
+	 * isTrashEmpty - check if the trash is empty
+	 * @return	boolean	success or not
+	 */
+	function isTrashEmpty() {
+		$res = db_query_params('select count(*) as c from docdata_vw where group_id = $1 and stateid = 2',
+					array($this->Group->getID()));
+		if (!$res) {
+			return false;
+		}
+		return (db_result($res, 0, 'c') == 0);
+	}
 }
 
 // Local Variables:
diff --git a/src/common/docman/DocumentGroup.class.php b/src/common/docman/DocumentGroup.class.php
index 03b465e..88a978a 100644
--- a/src/common/docman/DocumentGroup.class.php
+++ b/src/common/docman/DocumentGroup.class.php
@@ -44,18 +44,18 @@ class DocumentGroup extends Error {
 	 */
 	var $data_array;
 
-    /**
-     * DocumentGroup - constructor.
-     *
-     * Use this constructor if you are modifying an existing doc_group.
-     *
-     * @param $Group
-     * @param bool $data
-     * @internal param \Group $object object.
-     * @internal param array $OR doc_group id from database.
-     * @return \DocumentGroup
-    @access	public
-     */
+	/**
+	 * DocumentGroup - constructor.
+	 *
+	 * Use this constructor if you are modifying an existing doc_group.
+	 *
+	 * @param	$Group
+	 * @param	bool	$data
+	 * @internal	param	\Group $object object.
+	 * @internal	param	array $OR doc_group id from database.
+	 * @return	\DocumentGroup
+	 * @access	public
+	 */
 	function __construct(&$Group, $data = false) {
 		$this->Error();
 
@@ -84,15 +84,15 @@ class DocumentGroup extends Error {
 		}
 	}
 
-    /**
-     * create - create a new item in the database.
-     *
-     * @param $name
-     * @param int $parent_doc_group
-     * @internal param \Item $string name.
-     * @return    boolean    on success / false on failure.
-     * @access    public
-     */
+	/**
+	 * create - create a new item in the database.
+	 *
+	 * @param	$name
+	 * @param	int	$parent_doc_group
+	 * @internal	param	\Item $string name.
+	 * @return	boolean	true on success / false on failure.
+	 * @access	public
+	 */
 	function create($name, $parent_doc_group = 0) {
 		//
 		//	data validation
@@ -164,15 +164,16 @@ class DocumentGroup extends Error {
 		return true;
 	}
 
-    /**
-     * delete - delete a DocumentGroup.
-     * WARNING delete is recursive and permanent
-     * @param $doc_groupid
-     * @param $project_group_id
-     * @internal param \Document $integer Group Id, integer Project Group Id
-     * @return    boolean    success
-     * @access    public
-     */
+	/**
+	 * delete - delete a DocumentGroup.
+	 * WARNING delete is recursive and permanent
+	 *
+	 * @param	$doc_groupid
+	 * @param	$project_group_id
+	 * @internal	param		\Document $integer Group Id, integer Project Group Id
+	 * @return	boolean		success
+	 * @access	public
+	 */
 	function delete($doc_groupid, $project_group_id) {
 		$perm =& $this->Group->getPermission();
 		if (!$perm || !$perm->isDocEditor()) {
@@ -373,13 +374,13 @@ class DocumentGroup extends Error {
 		return $values;
 	}
 
-    /**
-     * isMonitoredBy - get the monitored status of this document directory for a specific user id.
-     *
-     * @param string $userid
-     * @internal param \User $int ID
-     * @return    boolean    true if monitored by this user
-     */
+	/**
+	 * isMonitoredBy - get the monitored status of this document directory for a specific user id.
+	 *
+	 * @param	string	$userid
+	 * @internal	param	\User $int ID
+	 * @return	boolean	true if monitored by this user
+	 */
 	function isMonitoredBy($userid = 'ALL') {
 		if ( $userid == 'ALL' ) {
 			$condition = '';
@@ -526,21 +527,21 @@ class DocumentGroup extends Error {
 		}
 	}
 
-    /**
-     * hasDocuments - Recursive function that checks if this group or any of it childs has documents associated to it
-     *
-     * A group has associated documents if and only if there are documents associated to this
-     * group or to any of its childs
-     *
-     * @param $nested_groups
-     * @param $document_factory
-     * @param int $stateid
-     * @internal param Array $array of nested groups information, fetched from DocumentGroupFactory class
-     * @internal param \The $object DocumentFactory object
-     * @internal param int $State of the documents
-     * @return    boolean    success
-     * @access    public
-     */
+	/**
+	 * hasDocuments - Recursive function that checks if this group or any of it childs has documents associated to it
+	 *
+	 * A group has associated documents if and only if there are documents associated to this
+	 * group or to any of its childs
+	 *
+	 * @param	$nested_groups
+	 * @param	$document_factory
+	 * @param	int		$stateid
+	 * @internal	param		Array $array of nested groups information, fetched from DocumentGroupFactory class
+	 * @internal	param		\The $object DocumentFactory object
+	 * @internal	param		int $State of the documents
+	 * @return	boolean		success
+	 * @access	public
+	*/
 	function hasDocuments(&$nested_groups, &$document_factory, $stateid = 0) {
 		$doc_group_id = $this->getID();
 		static $result = array();	// this function will probably be called several times so we better store results in order to speed things up
@@ -622,6 +623,13 @@ class DocumentGroup extends Error {
 		return false;
 	}
 
+	/**
+	 * getSubgroup - Return the ids of any sub folders (first level only) in specific folder
+	 *
+	 * @param	int	ID of the specific folder
+	 * @param	int	the state id of this specific folder (default is 1)
+	 * @return	array	the ids of any sub folders
+	 */
 	function getSubgroup($docGroupId, $stateId = 1) {
 		$returnArr = array();
 		$res = db_query_params('SELECT doc_group from doc_groups where parent_doc_group = $1 and stateid = $2 and group_id = $3 order by groupname',
@@ -696,8 +704,8 @@ class DocumentGroup extends Error {
 	 * sendNotice - Notifies of directory submissions
 	 *
 	 * @param	boolean	true = new directory (default value)
-     * @return bool
-     */
+	 * @return	bool
+	 */
 	function sendNotice($new = true) {
 		$BCC = $this->Group->getDocEmailAddress();
 		if ($this->isMonitoredBy('ALL')) {
diff --git a/src/common/docman/DocumentGroupFactory.class.php b/src/common/docman/DocumentGroupFactory.class.php
index b03b224..b4cf6df 100644
--- a/src/common/docman/DocumentGroupFactory.class.php
+++ b/src/common/docman/DocumentGroupFactory.class.php
@@ -6,7 +6,7 @@
  * Copyright 2002-2003, Tim Perdue/GForge, LLC
  * Copyright 2009, Roland Mas
  * Copyright 2010, Franck Villaume - Capgemini
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012-2013, Franck Villaume - TrivialDev
  * Copyright (C) 2012 Alain Peyrat - Alcatel-Lucent
  * http://fusionforge.org
  *
@@ -48,12 +48,12 @@ class DocumentGroupFactory extends Error {
 	 */
 	var $Group;
 
-    /**
-     * Constructor.
-     *
-     * @param $Group
-     * @return \DocumentGroupFactory
-     */
+	/**
+	 * Constructor.
+	 *
+	 * @param	$Group
+	 * @return	\DocumentGroupFactory
+	 */
 	function __construct(&$Group) {
 		$this->Error();
 
diff --git a/src/common/docman/DocumentManager.class.php b/src/common/docman/DocumentManager.class.php
index 600e5e5..e4bb4ca 100644
--- a/src/common/docman/DocumentManager.class.php
+++ b/src/common/docman/DocumentManager.class.php
@@ -2,7 +2,7 @@
 /**
  * FusionForge document manager
  *
- * Copyright 2011-2012, Franck Villaume - TrivialDev
+ * Copyright 2011-2013, Franck Villaume - TrivialDev
  * Copyright (C) 2012 Alain Peyrat - Alcatel-Lucent
  * Copyright 2013, French Ministry of National Education
  * http://fusionforge.org
@@ -43,13 +43,13 @@ class DocumentManager extends Error {
 	 */
 	var $Group;
 
-    /**
-     * Constructor.
-     *
-     * @param $Group
-     * @internal param \The $object Group object to which this document is associated.
-     * @return \DocumentManager
-     */
+	/**
+	 * Constructor.
+	 *
+	 * @param	$Group
+	 * @internal	param	\The $object Group object to which this document is associated.
+	 * @return	\DocumentManager
+	 */
 	function __construct(&$Group) {
 		$this->Error();
 		if (!$Group || !is_object($Group)) {
@@ -213,8 +213,8 @@ class DocumentManager extends Error {
 	 *
 	 * @param	string	format of the return values. json returns : { name: id, }. Default is DB object.
 	 * @param	string	skipped status id
-     * @return resource|string
-     */
+	 * @return resource|string
+	 */
 	function getStatusNameList($format = '', $removedval = '') {
 		if (!empty($removedval)) {
 			$stateQuery = db_query_params('select * from doc_states where stateid not in ($1) order by stateid', array($removedval));
@@ -237,6 +237,16 @@ class DocumentManager extends Error {
 		}
 	}
 
+	/**
+	 * getDocGroupList - Returns as a string used in javascript the list of available folders
+	 *
+	 * @param	$nested_groups
+	 * @param	string		must be json which is wrong, this function does not return any json object
+	 * @param	bool		allow the "None" which is the "/"
+	 * @param	int		the selected folder id
+	 * @param	array		folders id to not display
+	 * @return	string
+	 */
 	function getDocGroupList($nested_groups, $format = '', $allow_none = true, $selected_id = 0, $dont_display = array()) {
 		$id_array = array();
 		$text_array = array();
diff --git a/src/common/docman/Parsedata.class.php b/src/common/docman/Parsedata.class.php
index 1a52cc0..c348977 100644
--- a/src/common/docman/Parsedata.class.php
+++ b/src/common/docman/Parsedata.class.php
@@ -4,9 +4,8 @@
  *
  * Copyright 2005, Fabio Bertagnin
  * Copyright 2009-2010, Franck Villaume - Capgemini
- * Copyright 2011-2012, Franck Villaume - TrivialDev
+ * Copyright 2011-2013, Franck Villaume - TrivialDev
  * Copyright (C) 2011-2012 Alain Peyrat - Alcatel-Lucent
- * Copyright 2012, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -35,13 +34,23 @@ class Parsedata {
 	 * Constructor.
 	 *
 	 * @param	string	path to the parser list file
-	 * @return	boolean	true
+	 * @return	\Parsedata
 	 */
 	function __construct() {
 		$this->p_path = dirname(__FILE__).'/engine/';
 		$this->parsers = $this->get_parser_list($this->p_path);
 	}
 
+	/**
+	 * get_parse_data - analyse content and metadata
+	 *
+	 * @param	string	the path of the file to be analysed
+	 * @param	string	the file title
+	 * @param	string	the file description
+	 * @param	string	the file type
+	 * @param	string	the filename
+	 * @return	string	the analysed content
+	 */
 	function get_parse_data($data, $title, $description, $filetype, $filename) {
 		$parser = "";
 		$rep = "";
@@ -68,6 +77,12 @@ class Parsedata {
 		return preg_replace("/\n/", " ", "$rep $rep1");
 	}
 
+	/**
+	 * get_parser_list - get the list of available parsers
+	 *
+	 * @param	string	the path where are located the parsers
+	 * @return	array	available parsers
+	 */
 	function get_parser_list($parser_path) {
 		$file = $parser_path.'parser_list.txt';
 		$rep = array();
diff --git a/src/common/docman/actions/monitorfile.php b/src/common/docman/actions/monitorfile.php
index 87e1b51..024ff59 100644
--- a/src/common/docman/actions/monitorfile.php
+++ b/src/common/docman/actions/monitorfile.php
@@ -3,7 +3,7 @@
  * FusionForge Documentation Manager
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
- * Copyright 2012, Franck Villaume - TrivialDev
+ * Copyright 2012-2013, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -33,7 +33,7 @@ if (!forge_check_perm('docman', $group_id, 'read')) {
 	session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($return_msg));
 }
 
-$arr_fileid = explode(',',getStringFromRequest('fileid'));
+$arr_fileid = explode(',', getStringFromRequest('fileid'));
 $option = getStringFromRequest('option');
 $return_msg = _('Document(s)').' ';
 switch ($option) {
@@ -49,11 +49,11 @@ switch ($option) {
 				if (!$d->addMonitoredBy($LUSER->getID()))
 					session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&error_msg='.urlencode($d->getErrorMessage()));
 			} else {
-				$warning_msg = _('No action to perform');
+				$warning_msg = _('No action to perform.');
 				session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($warning_msg));
 			}
 		}
-		$return_msg .= _('Monitoring Started');
+		$return_msg .= _('Monitoring started.');
 		break;
 	}
 	case "remove": {
@@ -73,7 +73,7 @@ switch ($option) {
 				session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($warning_msg));
 			}
 		}
-		$return_msg .= _('Monitoring Stopped');
+		$return_msg .= _('Monitoring stopped.');
 		break;
 	}
 	default: {
diff --git a/src/common/docman/actions/releasefile.php b/src/common/docman/actions/releasefile.php
index fa8c1d4..a3d18fd 100644
--- a/src/common/docman/actions/releasefile.php
+++ b/src/common/docman/actions/releasefile.php
@@ -3,6 +3,7 @@
  * FusionForge Documentation Manager
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
+ * Copyright 2013, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -32,7 +33,7 @@ if (!forge_check_perm('docman', $group_id, 'approve')) {
 	session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($return_msg));
 }
 
-$arr_fileid = explode(',',getStringFromRequest('fileid'));
+$arr_fileid = explode(',', getStringFromRequest('fileid'));
 $return_msg = _('Document(s)').' ';
 foreach ($arr_fileid as $fileid) {
 	$d= new Document($g, $fileid);
diff --git a/src/common/docman/actions/reservefile.php b/src/common/docman/actions/reservefile.php
index 6ed7746..a776edb 100644
--- a/src/common/docman/actions/reservefile.php
+++ b/src/common/docman/actions/reservefile.php
@@ -3,6 +3,7 @@
  * FusionForge Documentation Manager
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
+ * Copyright 2013, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -33,7 +34,7 @@ if (!forge_check_perm('docman', $group_id, 'approve')) {
 	session_redirect('/docman/?group_id='.$group_id.'&view=listfile&dirid='.$dirid.'&warning_msg='.urlencode($return_msg));
 }
 
-$arr_fileid = explode(',',getStringFromRequest('fileid'));
+$arr_fileid = explode(',', getStringFromRequest('fileid'));
 $return_msg = _('Document(s)').' ';
 foreach ($arr_fileid as $fileid) {
 	$d = new Document($g, $fileid);
diff --git a/src/common/docman/actions/validatefile.php b/src/common/docman/actions/validatefile.php
index caeaedc..924b255 100644
--- a/src/common/docman/actions/validatefile.php
+++ b/src/common/docman/actions/validatefile.php
@@ -3,6 +3,7 @@
  * FusionForge Documentation Manager
  *
  * Copyright 2010-2011, Franck Villaume - Capgemini
+ * Copyright 2013, Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -40,7 +41,7 @@ if (!forge_check_perm('docman', $g->getID(), 'approve')) {
 	session_redirect($urlredirect.'&warning_msg='.urlencode($return_msg));
 }
 
-$arr_fileid = explode(',',getStringFromRequest('fileid'));
+$arr_fileid = explode(',', getStringFromRequest('fileid'));
 $return_msg = _('Document(s)').' ';
 foreach ($arr_fileid as $fileid) {
 	if (!empty($fileid)) {
@@ -53,7 +54,7 @@ foreach ($arr_fileid as $fileid) {
 		if (!$d->setState('1'))
 			session_redirect($urlredirect.'&error_msg='.urlencode($d->getErrorMessage()));
 	} else {
-		$warning_msg = _('No action to perform');
+		$warning_msg = _('No action to perform.');
 		session_redirect($urlredirect.'&warning_msg='.urlencode($warning_msg));
 	}
 }
diff --git a/src/common/docman/include/DocumentGroupHTML.class.php b/src/common/docman/include/DocumentGroupHTML.class.php
index 0514cb1..56bda69 100644
--- a/src/common/docman/include/DocumentGroupHTML.class.php
+++ b/src/common/docman/include/DocumentGroupHTML.class.php
@@ -5,6 +5,7 @@
  * Copyright 2002 GForge, LLC
  * Copyright 2010, Franck Villaume - Capgemini
  * Copyright (C) 2012 Alain Peyrat - Alcatel-Lucent
+ * Copyright 2013 Franck Villaume - TrivialDev
  * http://fusionforge.org
  *
  * This file is part of FusionForge. FusionForge is free software;
@@ -32,6 +33,12 @@ require_once $gfwww.'include/note.php';
 class DocumentGroupHTML extends Error {
 	var $Group;
 
+	/**
+	 * Constructor.
+	 *
+	 * @param	$Group
+	 * @return	\DocumentGroupHTML
+	 */
 	function __construct(&$Group) {
 		$this->Error();
 

-----------------------------------------------------------------------

Summary of changes:
 src/common/docman/Document.class.php               |   78 ++++++-------
 src/common/docman/DocumentFactory.class.php        |   55 +++++-----
 src/common/docman/DocumentGroup.class.php          |  116 +++++++++++---------
 src/common/docman/DocumentGroupFactory.class.php   |   14 +--
 src/common/docman/DocumentManager.class.php        |   30 +++--
 src/common/docman/Parsedata.class.php              |   21 +++-
 src/common/docman/actions/monitorfile.php          |   10 +-
 src/common/docman/actions/releasefile.php          |    3 +-
 src/common/docman/actions/reservefile.php          |    3 +-
 src/common/docman/actions/validatefile.php         |    5 +-
 .../docman/include/DocumentGroupHTML.class.php     |    7 ++
 11 files changed, 194 insertions(+), 148 deletions(-)


hooks/post-receive
-- 
FusionForge



More information about the Fusionforge-commits mailing list