@@ -26,10 +26,11 @@ |
||
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); |
@@ -23,10 +23,11 @@ |
||
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); |
@@ -26,10 +26,11 @@ |
||
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')); |
@@ -24,10 +24,11 @@ |
||
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); |
@@ -28,8 +28,9 @@ discard block |
||
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 |
||
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>'; |
@@ -31,8 +31,8 @@ discard block |
||
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 |
||
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) |
@@ -17,13 +17,15 @@ |
||
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); |
@@ -23,8 +23,9 @@ |
||
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 |
@@ -72,8 +72,9 @@ discard block |
||
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 |
||
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 |
||
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 |