YetiForceCompany / YetiForceCRM

libraries/SabreDAV/Xml/LibXMLException.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -2,8 +2,7 @@
 
 namespace Sabre\Xml;
 
-use
-    LibXMLError;
+use LibXMLError;
 
 /**
  * This exception is thrown when the Readers runs into a parsing error.

Indentation +34 added lines, -34 removed lines

@@ -3,7 +3,7 @@ 
 namespace Sabre\Xml;
 
 use
-    LibXMLError;
+	LibXMLError;
 
 /**
  * This exception is thrown when the Readers runs into a parsing error.
@@ -16,38 +16,38 @@ 
  */
 class LibXMLException extends ParseException {
 
-    /**
-     * The error list.
-     *
-     * @var LibXMLError[]
-     */
-    protected $errors;
-
-    /**
-     * Creates the exception.
-     *
-     * You should pass a list of LibXMLError objects in its constructor.
-     *
-     * @param LibXMLError[] $errors
-     * @param int $code
-     * @param Exception $previousException
-     */
-    public function __construct(array $errors, $code = null, Exception $previousException = null) {
-
-        $this->errors = $errors;
-        parent::__construct($errors[0]->message . ' on line ' . $errors[0]->line . ', column ' . $errors[0]->column, $code, $previousException);
-
-    }
-
-    /**
-     * Returns the LibXML errors
-     *
-     * @return void
-     */
-    public function getErrors() {
-
-        return $this->errors;
-
-    }
+	/**
+	 * The error list.
+	 *
+	 * @var LibXMLError[]
+	 */
+	protected $errors;
+
+	/**
+	 * Creates the exception.
+	 *
+	 * You should pass a list of LibXMLError objects in its constructor.
+	 *
+	 * @param LibXMLError[] $errors
+	 * @param int $code
+	 * @param Exception $previousException
+	 */
+	public function __construct(array $errors, $code = null, Exception $previousException = null) {
+
+		$this->errors = $errors;
+		parent::__construct($errors[0]->message . ' on line ' . $errors[0]->line . ', column ' . $errors[0]->column, $code, $previousException);
+
+	}
+
+	/**
+	 * Returns the LibXML errors
+	 *
+	 * @return void
+	 */
+	public function getErrors() {
+
+		return $this->errors;
+
+	}
 
 }

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
     /**
      * Returns the LibXML errors
      *
-     * @return void
+     * @return LibXMLError[]
      */
     public function getErrors() {
 

libraries/SabreDAV/Yeti/Debug.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php namespace Yeti;
 
 use Sabre\DAV,
-	Sabre\HTTP\RequestInterface,
-	Sabre\HTTP\ResponseInterface;
+    Sabre\HTTP\RequestInterface,
+    Sabre\HTTP\ResponseInterface;
 
 class Debug extends DAV\ServerPlugin
 {

libraries/SabreDAV/Yeti/WebDAV_Node.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 namespace Yeti;
 
 use Sabre\DAV,
-	Sabre\HTTP\URLUtil;
+    Sabre\HTTP\URLUtil;
 
 /**
  * Base node-class

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 		$stmt = $this->exData->pdo->prepare('SELECT * FROM dav_users WHERE username = ?;');
 		$stmt->execute([$user['username']]);
 		$row = $stmt->fetch(\PDO::FETCH_ASSOC);
-		if($row){
+		if ($row) {
 			$this->exData->username = $user['username'];
 			$this->exData->userId = $row['id'];
 			$this->exData->crmUserId = $row['userid'];

modules/Calendar/CalendarCommon.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
  * To get hour,minute and format
  * @param $starttime -- The date&time :: Type string
  * @param $endtime -- The date&time :: Type string
- * @param $format -- The format :: Type string
+ * @param string $format -- The format :: Type string
  * @returns $timearr :: Type Array
  */
 function getaddEventPopupTime($starttime, $endtime, $format)

Braces +31 added lines, -23 removed lines

@@ -21,8 +21,9 @@ 
 	$result = $adb->pquery($query, array($sharedid));
 	if ($adb->num_rows($result) != 0) {
 		$countResult = $adb->num_rows($result);
-		for ($j = 0; $j < $countResult; $j++)
-			$userid[] = $adb->query_result($result, $j, 'userid');
+		for ($j = 0; $j < $countResult; $j++) {
+					$userid[] = $adb->query_result($result, $j, 'userid');
+		}
 		$shared_ids = implode(",", $userid);
 	}
 	return $shared_ids;
@@ -43,15 +44,17 @@ 
 	if ($format == 'am/pm' || $format == '12') {
 		$hr = $sthr + 0;
 		$timearr['startfmt'] = ($hr >= 12) ? "pm" : "am";
-		if ($hr == 0)
-			$hr = 12;
+		if ($hr == 0) {
+					$hr = 12;
+		}
 		$timearr['starthour'] = twoDigit(($hr > 12) ? ($hr - 12) : $hr);
 		$timearr['startmin'] = $stmin;
 
 		$edhr = $edhr + 0;
 		$timearr['endfmt'] = ($edhr >= 12) ? "pm" : "am";
-		if ($edhr == 0)
-			$edhr = 12;
+		if ($edhr == 0) {
+					$edhr = 12;
+		}
 		$timearr['endhour'] = twoDigit(($edhr > 12) ? ($edhr - 12) : $edhr);
 		$timearr['endmin'] = $edmin;
 		return $timearr;
@@ -99,10 +102,11 @@ 
 	$startDate = new DateTimeField($description['st_date_time']);
 	$endDate = new DateTimeField($description['end_date_time']);
 
-	if ($from == "invite")
-		$msg = \App\Language::translate($mod_strings['LBL_ACTIVITY_INVITATION']);
-	else
-		$msg = \App\Language::translate($mod_strings['LBL_ACTIVITY_NOTIFICATION']);
+	if ($from == "invite") {
+			$msg = \App\Language::translate($mod_strings['LBL_ACTIVITY_INVITATION']);
+	} else {
+			$msg = \App\Language::translate($mod_strings['LBL_ACTIVITY_NOTIFICATION']);
+	}
 
 	$currentUsername = \App\Fields\Owner::getUserLabel($currentUser->id);
 	$status = \App\Language::translate($description['status'], 'Calendar');
@@ -116,8 +120,9 @@ 
 	$list .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;' . $mod_strings["Related To"] . ': ' . \App\Language::translate($description['relatedto']);
 	if (!empty($description['contact_name'])) {
 		$list .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;' . $mod_strings["LBL_CONTACT_LIST"] . ' ' . $description['contact_name'];
-	} else
-		$list .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;' . $mod_strings["Location"] . ' : ' . $description['location'];
+	} else {
+			$list .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;' . $mod_strings["Location"] . ' : ' . $description['location'];
+	}
 
 	$list .= '<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;' . $mod_strings["LBL_APP_DESCRIPTION"] . ': ' . $description['description'];
 	$list .= '<br><br>' . $mod_strings["LBL_REGARDS_STRING"] . ' ,';
@@ -129,11 +134,12 @@ 
 
 function twoDigit($no)
 {
-	if ($no < 10 && strlen(trim($no)) < 2)
-		return "0" . $no;
-	else
-		return "" . $no;
-}
+	if ($no < 10 && strlen(trim($no)) < 2) {
+			return "0" . $no;
+	} else {
+			return "" . $no;
+	}
+	}
 
 // User Select Customization
 /**
@@ -144,8 +150,9 @@ 
 {
 	$currentUser = Users_Privileges_Model::getCurrentUserModel();
 	$onlyForUser = htmlspecialchars(strip_tags(AppRequest::getForSql('onlyforuser')), ENT_QUOTES, AppConfig::main('default_charset'));
-	if ($onlyForUser == '')
-		$onlyForUser = $currentUser->id;
+	if ($onlyForUser == '') {
+			$onlyForUser = $currentUser->id;
+	}
 	return $onlyForUser;
 }
 
@@ -161,10 +168,11 @@ 
 			if ($onlyForUser == $currentUser->id) {
 				$userGroupIds = fetchUserGroupids($currentUser->id);
 				// User does not belong to any group? Let us reset to non-existent group
-				if (!empty($userGroupIds))
-					$userGroupIds .= ',';
-				else
-					$userGroupIds = '';
+				if (!empty($userGroupIds)) {
+									$userGroupIds .= ',';
+				} else {
+									$userGroupIds = '';
+				}
 				$userGroupIds .= $currentUser->id;
 				$qcondition = " && vtiger_crmentity.smownerid IN (" . $userGroupIds . ")";
 			} else {

modules/Calendar/iCal/ical-parser-class.php

Doc Comments +6 added lines

@@ -10,12 +10,18 @@
 		$this->folders = 'cache/import/';
 	}
 
+	/**
+	 * @param string $filename
+	 */
 	public function iCalReader($filename, $rootDirectory = '')
 	{
 		$iCaltoArray = $this->iCalDecoder($filename, $rootDirectory);
 		return $iCaltoArray;
 	}
 
+	/**
+	 * @param string $rootDirectory
+	 */
 	public function iCalDecoder($file, $rootDirectory)
 	{
 		$ical = file_get_contents($rootDirectory . $this->folders . $file);

modules/Calendar/iCal/iCalendar_parameters.php

Doc Comments +6 added lines

@@ -31,6 +31,9 @@ 
 		}
 	}
 
+	/**
+	 * @param iCalendar_property $parent_property
+	 */
 	public function is_valid_value(&$parent_property, $parameter, $value)
 	{
 		switch ($parameter) {
@@ -183,6 +186,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $parameter
+	 */
 	public function do_value_formatting($parameter, $value)
 	{
 		switch ($parameter) {

modules/Calendar/iCal/iCalendar_properties.php

Doc Comments +7 added lines

@@ -27,6 +27,10 @@ 
 
 	// If some property needs extra care with its parameters, override this
 	// IMPORTANT: the parameter name MUST BE CAPITALIZED!
+
+	/**
+	 * @param string $parameter
+	 */
 	public function is_valid_parameter($parameter, $value)
 	{
 
@@ -1218,6 +1222,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function set_name($name)
 	{
 

modules/Calendar/iCal/iCalendar_rfc2445.php

Doc Comments +6 added lines

@@ -40,6 +40,9 @@ 
 define('RFC2445_TYPE_URI', 12); // CAL_ADDRESS === URI
 define('RFC2445_TYPE_UTC_OFFSET', 13);
 
+/**
+ * @param string $string
+ */
 function rfc2445_fold($string)
 {
 	if (strlen($string) <= RFC2445_FOLDED_LINE_LENGTH) {
@@ -67,6 +70,9 @@ 
 	return $string;
 }
 
+/**
+ * @param string $name
+ */
 function rfc2445_is_xname($name)
 {
 

modules/Calendar/views/Edit.php

Doc Comments +4 added lines

@@ -155,6 +155,10 @@
 		$viewer->view('EditView.tpl', $moduleName);
 	}
 
+	/**
+	 * @param Vtiger_Request $request
+	 * @param string $moduleName
+	 */
 	public function Calendar($request, $moduleName)
 	{
 		parent::process($request);