Completed
Push — developer ( dcd43f...1946c6 )
by Błażej
48:56 queued 21:47
created
modules/Vtiger/dashboards/AssignedOverdueCalendarTasks.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -26,10 +26,11 @@
 block discarded – undo
26 26
 		$data = $request->getAll();
27 27
 
28 28
 		$widget = Vtiger_Widget_Model::getInstance($linkId, $currentUser->getId());
29
-		if (!$request->has('owner'))
30
-			$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget);
31
-		else
32
-			$owner = $request->get('owner');
29
+		if (!$request->has('owner')) {
30
+					$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget);
31
+		} else {
32
+					$owner = $request->get('owner');
33
+		}
33 34
 
34 35
 		$pagingModel = new Vtiger_Paging_Model();
35 36
 		$pagingModel->set('page', $page);
Please login to merge, or discard this patch.
modules/Vtiger/dashboards/Calendar.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -23,10 +23,11 @@
 block discarded – undo
23 23
 		$linkId = $request->get('linkid');
24 24
 
25 25
 		$widget = Vtiger_Widget_Model::getInstance($linkId, $currentUser->getId());
26
-		if (!$request->has('owner'))
27
-			$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget);
28
-		else
29
-			$owner = $request->get('owner');
26
+		if (!$request->has('owner')) {
27
+					$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget);
28
+		} else {
29
+					$owner = $request->get('owner');
30
+		}
30 31
 
31 32
 		$pagingModel = new Vtiger_Paging_Model();
32 33
 		$pagingModel->set('page', $page);
Please login to merge, or discard this patch.
modules/Vtiger/dashboards/AssignedUpcomingCalendarTasks.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -26,10 +26,11 @@
 block discarded – undo
26 26
 		$data = $request->getAll();
27 27
 
28 28
 		$widget = Vtiger_Widget_Model::getInstance($linkId, $currentUser->getId());
29
-		if (!$request->has('owner'))
30
-			$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget);
31
-		else
32
-			$owner = $request->get('owner');
29
+		if (!$request->has('owner')) {
30
+					$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget);
31
+		} else {
32
+					$owner = $request->get('owner');
33
+		}
33 34
 		$pagingModel = new Vtiger_Paging_Model();
34 35
 		$pagingModel->set('page', $page);
35 36
 		$pagingModel->set('limit', (int) $widget->get('limit'));
Please login to merge, or discard this patch.
modules/Vtiger/dashboards/AssignedUpcomingProjectsTasks.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -24,10 +24,11 @@
 block discarded – undo
24 24
 		$data = $request->getAll();
25 25
 
26 26
 		$widget = Vtiger_Widget_Model::getInstance($linkId, $currentUser->getId());
27
-		if (!$request->has('owner'))
28
-			$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget, 'Leads');
29
-		else
30
-			$owner = $request->get('owner');
27
+		if (!$request->has('owner')) {
28
+					$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget, 'Leads');
29
+		} else {
30
+					$owner = $request->get('owner');
31
+		}
31 32
 
32 33
 		$pagingModel = new Vtiger_Paging_Model();
33 34
 		$pagingModel->set('page', $page);
Please login to merge, or discard this patch.
modules/Vtiger/helpers/ShortURL.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,8 +28,9 @@  discard block
 block discarded – undo
28 28
 	static function generateURL(array $options)
29 29
 	{
30 30
 		$site_URL = AppConfig::main('site_URL');
31
-		if (!isset($options['onetime']))
32
-			$options['onetime'] = 0;
31
+		if (!isset($options['onetime'])) {
32
+					$options['onetime'] = 0;
33
+		}
33 34
 		$uid = self::generate($options);
34 35
 		return rtrim($site_URL, '/') . "/shorturl.php?id=" . $uid;
35 36
 	}
@@ -75,8 +76,9 @@  discard block
 block discarded – undo
75 76
 			$handler = new $handlerClass();
76 77
 
77 78
 			// Delete onetime URL
78
-			if ($record['onetime'])
79
-				$db->pquery('DELETE FROM vtiger_shorturls WHERE id=?', array($record['id']));
79
+			if ($record['onetime']) {
80
+							$db->pquery('DELETE FROM vtiger_shorturls WHERE id=?', array($record['id']));
81
+			}
80 82
 			call_user_func(array($handler, $handlerFn), $handlerData);
81 83
 		} else {
82 84
 			echo '<h3>Link you have used is invalid or has expired. .</h3>';
Please login to merge, or discard this patch.
modules/Vtiger/data_access/validate_mandatory.php 1 patch
Braces   +5 added lines, -4 removed lines patch added patch discarded remove patch
@@ -31,8 +31,8 @@  discard block
 block discarded – undo
31 31
 				}
32 32
 			}
33 33
 		}
34
-		if (!$save_record)
35
-			return Array(
34
+		if (!$save_record) {
35
+					return Array(
36 36
 				'save_record' => $save_record,
37 37
 				's' => $moduleName,
38 38
 				'fne' => $fieldName2,
@@ -43,8 +43,9 @@  discard block
 block discarded – undo
43 43
 					'type' => 'info'
44 44
 				)
45 45
 			);
46
-		else
47
-			return Array('save_record' => $save_record);
46
+		} else {
47
+					return Array('save_record' => $save_record);
48
+		}
48 49
 	}
49 50
 
50 51
 	public function getConfig($id, $module, $baseModule)
Please login to merge, or discard this patch.
modules/Vtiger/data_access/check_taskdate.php 1 patch
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -17,13 +17,15 @@
 block discarded – undo
17 17
 	public function process($ModuleName, $ID, $record_form, $config)
18 18
 	{
19 19
 		$projectmilestoneid = $record_form['projectmilestoneid'];
20
-		if (!isset($projectmilestoneid) || $projectmilestoneid == 0 || $projectmilestoneid == '')
21
-			return Array('save_record' => true);
20
+		if (!isset($projectmilestoneid) || $projectmilestoneid == 0 || $projectmilestoneid == '') {
21
+					return Array('save_record' => true);
22
+		}
22 23
 
23 24
 		$moduleModel = Vtiger_Record_Model::getInstanceById($projectmilestoneid, 'ProjectMilestone');
24 25
 		$projectMilestoneDate = $moduleModel->get('projectmilestonedate');
25
-		if (!isset($projectMilestoneDate) || $projectMilestoneDate == 0 || $projectMilestoneDate == '')
26
-			return Array('save_record' => true);
26
+		if (!isset($projectMilestoneDate) || $projectMilestoneDate == 0 || $projectMilestoneDate == '') {
27
+					return Array('save_record' => true);
28
+		}
27 29
 
28 30
 		$dateField = new DateTimeField($projectMilestoneDate);
29 31
 		$projectMilestoneDateUserFormat = $dateField->convertToUserFormat($projectMilestoneDate);
Please login to merge, or discard this patch.
modules/Vtiger/inventoryfields/Name.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,8 +23,9 @@
 block discarded – undo
23 23
 	 */
24 24
 	public function getDisplayValue($value)
25 25
 	{
26
-		if ($value != 0)
27
-			return vtlib\Functions::getCRMRecordLabel($value);
26
+		if ($value != 0) {
27
+					return vtlib\Functions::getCRMRecordLabel($value);
28
+		}
28 29
 		return '';
29 30
 	}
30 31
 
Please login to merge, or discard this patch.
modules/Vtiger/inventoryfields/Basic.php 1 patch
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -72,8 +72,9 @@  discard block
 block discarded – undo
72 72
 
73 73
 	public function getColSpan()
74 74
 	{
75
-		if ($this->has('colspan'))
76
-			return $this->get('colspan');
75
+		if ($this->has('colspan')) {
76
+					return $this->get('colspan');
77
+		}
77 78
 
78 79
 		return $this->colSpan;
79 80
 	}
@@ -128,8 +129,9 @@  discard block
 block discarded – undo
128 129
 	 */
129 130
 	public function getColumnName()
130 131
 	{
131
-		if ($this->has('columnname'))
132
-			return $this->get('columnname');
132
+		if ($this->has('columnname')) {
133
+					return $this->get('columnname');
134
+		}
133 135
 		return $this->columnName;
134 136
 	}
135 137
 
@@ -149,8 +151,9 @@  discard block
 block discarded – undo
149 151
 
150 152
 	public function getDefaultValue()
151 153
 	{
152
-		if ($this->has('defaultvalue'))
153
-			return $this->get('defaultvalue');
154
+		if ($this->has('defaultvalue')) {
155
+					return $this->get('defaultvalue');
156
+		}
154 157
 		return $this->defaultValue;
155 158
 	}
156 159
 
Please login to merge, or discard this patch.