Completed
Branch develop (1d589f)
by
unknown
21:18
created
htdocs/core/modules/reception/doc/doc_generic_reception_odt.modules.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 		$maxfilesizearray = getMaxFileSizeArray();
180 180
 		$maxmin = $maxfilesizearray['maxmin'];
181 181
 		if ($maxmin > 0) {
182
-			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">';	// MAX_FILE_SIZE must precede the field type=file
182
+			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
183 183
 		}
184 184
 		$texte .= ' <input type="file" name="uploadfile">';
185 185
 		$texte .= '<input type="hidden" value="RECEPTION_ADDON_PDF_ODT_PATH" name="keyforuploaddir">';
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
 				$newfiletmp = preg_replace('/\.od[ts]/i', '', $newfile);
274 274
 				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
275 275
 				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
276
-				$newfiletmp = $objectref . '_' . $newfiletmp;
276
+				$newfiletmp = $objectref.'_'.$newfiletmp;
277 277
 				//$file=$dir.'/'.$newfiletmp.'.'.dol_print_date(dol_now(),'%Y%m%d%H%M%S').'.odt';
278 278
 				// Get extension (ods or odt)
279 279
 				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
@@ -282,11 +282,11 @@  discard block
 block discarded – undo
282 282
 					if ($format == '1') {
283 283
 						$format = '%Y%m%d%H%M%S';
284 284
 					}
285
-					$filename = $newfiletmp . '-' . dol_print_date(dol_now(), $format) . '.' . $newfileformat;
285
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
286 286
 				} else {
287
-					$filename = $newfiletmp . '.' . $newfileformat;
287
+					$filename = $newfiletmp.'.'.$newfileformat;
288 288
 				}
289
-				$file = $dir . '/' . $filename;
289
+				$file = $dir.'/'.$filename;
290 290
 				//print "newdir=".$dir;
291 291
 				//print "newfile=".$newfile;
292 292
 				//print "file=".$file;
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
 				dol_mkdir($conf->reception->dir_temp);
296 296
 				if (!is_writable($conf->reception->dir_temp)) {
297 297
 					$this->error = $langs->transnoentities("ErrorFailedToWriteInTempDirectory", $conf->reception->dir_temp);
298
-					dol_syslog('Error in write_file: ' . $this->error, LOG_ERR);
298
+					dol_syslog('Error in write_file: '.$this->error, LOG_ERR);
299 299
 					return -1;
300 300
 				}
301 301
 
Please login to merge, or discard this patch.
htdocs/core/modules/expedition/doc/doc_generic_shipment_odt.modules.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
 		$maxfilesizearray = getMaxFileSizeArray();
187 187
 		$maxmin = $maxfilesizearray['maxmin'];
188 188
 		if ($maxmin > 0) {
189
-			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">';	// MAX_FILE_SIZE must precede the field type=file
189
+			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
190 190
 		}
191 191
 		$texte .= ' <input type="file" name="uploadfile">';
192 192
 		$texte .= '<input type="hidden" value="EXPEDITION_ADDON_PDF_ODT_PATH" name="keyforuploaddir">';
@@ -280,7 +280,7 @@  discard block
 block discarded – undo
280 280
 				$newfiletmp = preg_replace('/\.od[ts]/i', '', $newfile);
281 281
 				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
282 282
 				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
283
-				$newfiletmp = $objectref . '_' . $newfiletmp;
283
+				$newfiletmp = $objectref.'_'.$newfiletmp;
284 284
 				//$file=$dir.'/'.$newfiletmp.'.'.dol_print_date(dol_now(),'%Y%m%d%H%M%S').'.odt';
285 285
 				// Get extension (ods or odt)
286 286
 				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
@@ -289,11 +289,11 @@  discard block
 block discarded – undo
289 289
 					if ($format == '1') {
290 290
 						$format = '%Y%m%d%H%M%S';
291 291
 					}
292
-					$filename = $newfiletmp . '-' . dol_print_date(dol_now(), $format) . '.' . $newfileformat;
292
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
293 293
 				} else {
294
-					$filename = $newfiletmp . '.' . $newfileformat;
294
+					$filename = $newfiletmp.'.'.$newfileformat;
295 295
 				}
296
-				$file = $dir . '/' . $filename;
296
+				$file = $dir.'/'.$filename;
297 297
 				//print "newdir=".$dir;
298 298
 				//print "newfile=".$newfile;
299 299
 				//print "file=".$file;
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
 				dol_mkdir($conf->expedition->dir_temp);
303 303
 				if (!is_writable($conf->expedition->dir_temp)) {
304 304
 					$this->error = $langs->transnoentities("ErrorFailedToWriteInTempDirectory", $conf->expedition->dir_temp);
305
-					dol_syslog('Error in write_file: ' . $this->error, LOG_ERR);
305
+					dol_syslog('Error in write_file: '.$this->error, LOG_ERR);
306 306
 					return -1;
307 307
 				}
308 308
 
Please login to merge, or discard this patch.
htdocs/core/modules/bom/doc/doc_generic_bom_odt.modules.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 		$maxfilesizearray = getMaxFileSizeArray();
181 181
 		$maxmin = $maxfilesizearray['maxmin'];
182 182
 		if ($maxmin > 0) {
183
-			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">';	// MAX_FILE_SIZE must precede the field type=file
183
+			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
184 184
 		}
185 185
 		$texte .= ' <input type="file" name="uploadfile">';
186 186
 		$texte .= '<input type="hidden" value="BOM_ADDON_PDF_ODT_PATH" name="keyforuploaddir">';
@@ -276,7 +276,7 @@  discard block
 block discarded – undo
276 276
 				$newfiletmp = preg_replace('/\.od[ts]/i', '', $newfile);
277 277
 				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
278 278
 				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
279
-				$newfiletmp = $objectref . '_' . $newfiletmp;
279
+				$newfiletmp = $objectref.'_'.$newfiletmp;
280 280
 				//$file=$dir.'/'.$newfiletmp.'.'.dol_print_date(dol_now(),'%Y%m%d%H%M%S').'.odt';
281 281
 				// Get extension (ods or odt)
282 282
 				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
@@ -285,11 +285,11 @@  discard block
 block discarded – undo
285 285
 					if ($format == '1') {
286 286
 						$format = '%Y%m%d%H%M%S';
287 287
 					}
288
-					$filename = $newfiletmp . '-' . dol_print_date(dol_now(), $format) . '.' . $newfileformat;
288
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
289 289
 				} else {
290
-					$filename = $newfiletmp . '.' . $newfileformat;
290
+					$filename = $newfiletmp.'.'.$newfileformat;
291 291
 				}
292
-				$file = $dir . '/' . $filename;
292
+				$file = $dir.'/'.$filename;
293 293
 				//print "newdir=".$dir;
294 294
 				//print "newfile=".$newfile;
295 295
 				//print "file=".$file;
@@ -298,7 +298,7 @@  discard block
 block discarded – undo
298 298
 				dol_mkdir($conf->bom->dir_temp);
299 299
 				if (!is_writable($conf->bom->dir_temp)) {
300 300
 					$this->error = $langs->transnoentities("ErrorFailedToWriteInTempDirectory", $conf->bom->dir_temp);
301
-					dol_syslog('Error in write_file: ' . $this->error, LOG_ERR);
301
+					dol_syslog('Error in write_file: '.$this->error, LOG_ERR);
302 302
 					return -1;
303 303
 				}
304 304
 
Please login to merge, or discard this patch.
htdocs/core/modules/mrp/doc/doc_generic_mo_odt.modules.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
 				$newfiletmp = preg_replace('/\.od[ts]/i', '', $newfile);
270 270
 				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
271 271
 				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
272
-				$newfiletmp = $objectref . '_' . $newfiletmp;
272
+				$newfiletmp = $objectref.'_'.$newfiletmp;
273 273
 				//$file=$dir.'/'.$newfiletmp.'.'.dol_print_date(dol_now(),'%Y%m%d%H%M%S').'.odt';
274 274
 				// Get extension (ods or odt)
275 275
 				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
@@ -278,11 +278,11 @@  discard block
 block discarded – undo
278 278
 					if ($format == '1') {
279 279
 						$format = '%Y%m%d%H%M%S';
280 280
 					}
281
-					$filename = $newfiletmp . '-' . dol_print_date(dol_now(), $format) . '.' . $newfileformat;
281
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
282 282
 				} else {
283
-					$filename = $newfiletmp . '.' . $newfileformat;
283
+					$filename = $newfiletmp.'.'.$newfileformat;
284 284
 				}
285
-				$file = $dir . '/' . $filename;
285
+				$file = $dir.'/'.$filename;
286 286
 				//print "newdir=".$dir;
287 287
 				//print "newfile=".$newfile;
288 288
 				//print "file=".$file;
@@ -291,7 +291,7 @@  discard block
 block discarded – undo
291 291
 				dol_mkdir($conf->mrp->dir_temp);
292 292
 				if (!is_writable($conf->mrp->dir_temp)) {
293 293
 					$this->error = $langs->transnoentities("ErrorFailedToWriteInTempDirectory", $conf->mrp->dir_temp);
294
-					dol_syslog('Error in write_file: ' . $this->error, LOG_ERR);
294
+					dol_syslog('Error in write_file: '.$this->error, LOG_ERR);
295 295
 					return -1;
296 296
 				}
297 297
 
Please login to merge, or discard this patch.
htdocs/core/modules/commande/doc/doc_generic_order_odt.modules.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
 		$maxfilesizearray = getMaxFileSizeArray();
188 188
 		$maxmin = $maxfilesizearray['maxmin'];
189 189
 		if ($maxmin > 0) {
190
-			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">';	// MAX_FILE_SIZE must precede the field type=file
190
+			$texte .= '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
191 191
 		}
192 192
 		$texte .= ' <input type="file" name="uploadfile">';
193 193
 		$texte .= '<input type="hidden" value="COMMANDE_ADDON_PDF_ODT_PATH" name="keyforuploaddir">';
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
 				$newfiletmp = preg_replace('/\.od[ts]/i', '', $newfile);
283 283
 				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
284 284
 				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
285
-				$newfiletmp = $objectref . '_' . $newfiletmp;
285
+				$newfiletmp = $objectref.'_'.$newfiletmp;
286 286
 
287 287
 				// Get extension (ods or odt)
288 288
 				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
@@ -291,11 +291,11 @@  discard block
 block discarded – undo
291 291
 					if ($format == '1') {
292 292
 						$format = '%Y%m%d%H%M%S';
293 293
 					}
294
-					$filename = $newfiletmp . '-' . dol_print_date(dol_now(), $format) . '.' . $newfileformat;
294
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
295 295
 				} else {
296
-					$filename = $newfiletmp . '.' . $newfileformat;
296
+					$filename = $newfiletmp.'.'.$newfileformat;
297 297
 				}
298
-				$file = $dir . '/' . $filename;
298
+				$file = $dir.'/'.$filename;
299 299
 				//print "newdir=".$dir;
300 300
 				//print "newfile=".$newfile;
301 301
 				//print "file=".$file;
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
 				dol_mkdir($conf->commande->dir_temp);
305 305
 				if (!is_writable($conf->commande->dir_temp)) {
306 306
 					$this->error = $langs->transnoentities("ErrorFailedToWriteInTempDirectory", $conf->commande->dir_temp);
307
-					dol_syslog('Error in write_file: ' . $this->error, LOG_ERR);
307
+					dol_syslog('Error in write_file: '.$this->error, LOG_ERR);
308 308
 					return -1;
309 309
 				}
310 310
 
Please login to merge, or discard this patch.
core/modules/supplier_order/doc/doc_generic_supplier_order_odt.modules.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
 				$newfiletmp = preg_replace('/\.od[ts]/i', '', $newfile);
265 265
 				$newfiletmp = preg_replace('/template_/i', '', $newfiletmp);
266 266
 				$newfiletmp = preg_replace('/modele_/i', '', $newfiletmp);
267
-				$newfiletmp = $objectref . '_' . $newfiletmp;
267
+				$newfiletmp = $objectref.'_'.$newfiletmp;
268 268
 				//$file=$dir.'/'.$newfiletmp.'.'.dol_print_date(dol_now(),'%Y%m%d%H%M%S').'.odt';
269 269
 				// Get extension (ods or odt)
270 270
 				$newfileformat = substr($newfile, strrpos($newfile, '.') + 1);
@@ -273,11 +273,11 @@  discard block
 block discarded – undo
273 273
 					if ($format == '1') {
274 274
 						$format = '%Y%m%d%H%M%S';
275 275
 					}
276
-					$filename = $newfiletmp . '-' . dol_print_date(dol_now(), $format) . '.' . $newfileformat;
276
+					$filename = $newfiletmp.'-'.dol_print_date(dol_now(), $format).'.'.$newfileformat;
277 277
 				} else {
278
-					$filename = $newfiletmp . '.' . $newfileformat;
278
+					$filename = $newfiletmp.'.'.$newfileformat;
279 279
 				}
280
-				$file = $dir . '/' . $filename;
280
+				$file = $dir.'/'.$filename;
281 281
 				//print "newdir=".$dir;
282 282
 				//print "newfile=".$newfile;
283 283
 				//print "file=".$file;
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
 				dol_mkdir($conf->fournisseur->commande->dir_temp);
287 287
 				if (!is_writable($conf->fournisseur->commande->dir_temp)) {
288 288
 					$this->error = $langs->transnoentities("ErrorFailedToWriteInTempDirectory", $conf->fournisseur->commande->dir_temp);
289
-					dol_syslog('Error in write_file: ' . $this->error, LOG_ERR);
289
+					dol_syslog('Error in write_file: '.$this->error, LOG_ERR);
290 290
 					return -1;
291 291
 				}
292 292
 
Please login to merge, or discard this patch.
htdocs/hrm/class/job.class.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -609,7 +609,9 @@
 block discarded – undo
609 609
 	{
610 610
 		$Tab = $this->getForUser($fk_user);
611 611
 
612
-		if (empty($Tab)) return '';
612
+		if (empty($Tab)) {
613
+			return '';
614
+		}
613 615
 
614 616
 		$lastpos = array_shift($Tab);
615 617
 
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -102,13 +102,13 @@  discard block
 block discarded – undo
102 102
 	/**
103 103
 	 * @var array  Array with all fields and their property. Do not use it as a static var. It may be modified by constructor.
104 104
 	 */
105
-	public $fields=array(
105
+	public $fields = array(
106 106
 		'rowid' => array('type'=>'integer', 'label'=>'TechnicalID', 'enabled'=>'1', 'position'=>1, 'notnull'=>1, 'visible'=>0, 'noteditable'=>'1', 'index'=>1, 'css'=>'left', 'comment'=>"Id"),
107 107
 		'label' => array('type'=>'varchar(128)', 'label'=>'Label', 'enabled'=>'1', 'position'=>20, 'notnull'=>1, 'visible'=>1, 'index'=>1, 'searchall'=>1, 'showoncombobox'=>'1', 'comment'=>"Label of object"),
108 108
 		'description' => array('type'=>'text', 'label'=>'Description', 'enabled'=>'1', 'position'=>21, 'notnull'=>0, 'visible'=>1,),
109 109
 		'date_creation' => array('type'=>'datetime', 'label'=>'DateCreation', 'enabled'=>'1', 'position'=>500, 'notnull'=>1, 'visible'=>2,),
110 110
 		'tms' => array('type'=>'timestamp', 'label'=>'DateModification', 'enabled'=>'1', 'position'=>501, 'notnull'=>0, 'visible'=>2,),
111
-		'deplacement' => array('type'=>'select', 'required'=> 1,'label'=> 'NeedBusinessTravels', 'enabled'=> 1, 'position'=> 90, 'notnull'=> 1, 'visible'=> 1, 'arrayofkeyval'=> array(0 =>"No", 1=>"Yes"), 'default'=>0),
111
+		'deplacement' => array('type'=>'select', 'required'=> 1, 'label'=> 'NeedBusinessTravels', 'enabled'=> 1, 'position'=> 90, 'notnull'=> 1, 'visible'=> 1, 'arrayofkeyval'=> array(0 =>"No", 1=>"Yes"), 'default'=>0),
112 112
 		'note_public' => array('type'=>'html', 'label'=>'NotePublic', 'enabled'=>'1', 'position'=>70, 'notnull'=>0, 'visible'=>0,),
113 113
 		'note_private' => array('type'=>'html', 'label'=>'NotePrivate', 'enabled'=>'1', 'position'=>71, 'notnull'=>0, 'visible'=>0,),
114 114
 		'fk_user_creat' => array('type'=>'integer:User:user/class/user.class.php', 'label'=>'UserAuthor', 'enabled'=>'1', 'position'=>510, 'notnull'=>1, 'visible'=>-2, 'foreignkey'=>'user.rowid',),
@@ -851,7 +851,7 @@  discard block
 block discarded – undo
851 851
 	public function LibStatut($status, $mode = 0)
852 852
 	{
853 853
 		// phpcs:enable
854
-		return '';		// There is no status on job profile for the moment
854
+		return ''; // There is no status on job profile for the moment
855 855
 
856 856
 		/*
857 857
 		if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -960,7 +960,7 @@  discard block
 block discarded – undo
960 960
 		if (getDolGlobalString('hrm_JOB_ADDON')) {
961 961
 			$mybool = false;
962 962
 
963
-			$file = getDolGlobalString('hrm_JOB_ADDON') . ".php";
963
+			$file = getDolGlobalString('hrm_JOB_ADDON').".php";
964 964
 			$classname = $conf->global->hrm_JOB_ADDON;
965 965
 
966 966
 			// Include file with class
Please login to merge, or discard this patch.
htdocs/hrm/compare.php 2 patches
Braces   +46 added lines, -17 removed lines patch added patch discarded remove patch
@@ -51,8 +51,12 @@  discard block
 block discarded – undo
51 51
 $permissiontoread = $user->rights->hrm->evaluation->read || $user->rights->hrm->compare_advance->read;
52 52
 $permissiontoadd = 0;
53 53
 
54
-if (empty($conf->hrm->enabled)) accessforbidden();
55
-if (!$permissiontoread || ($action === 'create' && !$permissiontoadd)) accessforbidden();
54
+if (empty($conf->hrm->enabled)) {
55
+	accessforbidden();
56
+}
57
+if (!$permissiontoread || ($action === 'create' && !$permissiontoadd)) {
58
+	accessforbidden();
59
+}
56 60
 
57 61
 
58 62
 /*
@@ -113,7 +117,9 @@  discard block
 block discarded – undo
113 117
 
114 118
 $fk_usergroup2 = 0;
115 119
 $fk_job = (int) GETPOST('fk_job');
116
-if ($fk_job <= 0) $fk_usergroup2 = GETPOST('fk_usergroup2');
120
+if ($fk_job <= 0) {
121
+	$fk_usergroup2 = GETPOST('fk_usergroup2');
122
+}
117 123
 
118 124
 $fk_usergroup1 = GETPOST('fk_usergroup1');
119 125
 
@@ -290,11 +296,17 @@  discard block
 block discarded – undo
290 296
 	foreach ($TMergedSkills as $id => &$sk) {
291 297
 		$class = 'diffnote';
292 298
 
293
-		if (empty($sk->rate2)) $class .= ' toohappy';
294
-		elseif (empty($sk->rate1)) $class .= ' toosad';
295
-		elseif ($sk->rate1 == $sk->rate2) $class .= ' happy';
296
-		elseif ($sk->rate2 < $sk->rate1) $class .= ' veryhappy';
297
-		elseif ($sk->rate2 > $sk->rate1) $class .= ' sad';
299
+		if (empty($sk->rate2)) {
300
+			$class .= ' toohappy';
301
+		} elseif (empty($sk->rate1)) {
302
+			$class .= ' toosad';
303
+		} elseif ($sk->rate1 == $sk->rate2) {
304
+			$class .= ' happy';
305
+		} elseif ($sk->rate2 < $sk->rate1) {
306
+			$class .= ' veryhappy';
307
+		} elseif ($sk->rate2 > $sk->rate1) {
308
+			$class .= ' sad';
309
+		}
298 310
 
299 311
 		$out .= '<li fk_skill="' . $id . '" class="' . $class . '" style="text-align:center;">
300 312
 	      <span class="' . $class . '">&nbsp;</span>
@@ -329,8 +341,11 @@  discard block
 block discarded – undo
329 341
 			$how_many = ($field === 'rate1') ? $sk->how_many_max1 : $sk->how_many_max2;
330 342
 		}
331 343
 
332
-		if ($field === 'rate2' && $fk_job > 0) $trad = $langs->trans('RequiredRank');
333
-		else $trad = $langs->trans('HighestRank');
344
+		if ($field === 'rate2' && $fk_job > 0) {
345
+			$trad = $langs->trans('RequiredRank');
346
+		} else {
347
+			$trad = $langs->trans('HighestRank');
348
+		}
334 349
 
335 350
 		$out .= '<li fk_skill="' . $id . '" style="text-align:center;">
336 351
 	      <p><span class="' . $class . ' classfortooltip" title="' . $trad . '">' . $note . '</span>' . ($how_many > 0 ? '<span class="bubble classfortooltip" title="' . $langs->trans('HowManyUserWithThisMaxNote') . '">' . $how_many . '</span>' : '') . '</p>
@@ -378,7 +393,9 @@  discard block
 block discarded – undo
378 393
 	$Tab = array();
379 394
 
380 395
 	foreach ($TSkill1 as &$sk) {
381
-			if (empty($Tab[$sk->fk_skill])) $Tab[$sk->fk_skill] = new stdClass;
396
+			if (empty($Tab[$sk->fk_skill])) {
397
+				$Tab[$sk->fk_skill] = new stdClass;
398
+			}
382 399
 
383 400
 			$Tab[$sk->fk_skill]->rate1 = $sk->rankorder;
384 401
 			$Tab[$sk->fk_skill]->how_many_max1 = $sk->how_many_max;
@@ -387,7 +404,9 @@  discard block
 block discarded – undo
387 404
 	}
388 405
 
389 406
 	foreach ($TSkill2 as &$sk) {
390
-			if (empty($Tab[$sk->fk_skill])) $Tab[$sk->fk_skill] = new stdClass;
407
+			if (empty($Tab[$sk->fk_skill])) {
408
+				$Tab[$sk->fk_skill] = new stdClass;
409
+			}
391 410
 			$Tab[$sk->fk_skill]->rate2 = $sk->rankorder;
392 411
 			$Tab[$sk->fk_skill]->label = $sk->label;
393 412
 			$Tab[$sk->fk_skill]->description = $sk->description;
@@ -436,12 +455,16 @@  discard block
 block discarded – undo
436 455
 			$user->fetch($obj->rowid);
437 456
 
438 457
 			$name = $user->getFullName($langs);
439
-			if (empty($name)) $name = $user->login;
458
+			if (empty($name)) {
459
+				$name = $user->login;
460
+			}
440 461
 
441 462
 			if (in_array($user->id, $TExcludedId)) {
442 463
 				$class .= ' disabled';
443 464
 			} else {
444
-				if (!in_array($user->id, $TUser)) $TUser[] = $user->id;
465
+				if (!in_array($user->id, $TUser)) {
466
+					$TUser[] = $user->id;
467
+				}
445 468
 			}
446 469
 
447 470
 			$desc = '';
@@ -458,7 +481,9 @@  discard block
 block discarded – undo
458 481
 				$desc .= $langs->trans('NoEval');
459 482
 			}
460 483
 
461
-			if (!empty($user->array_options['options_DDA'])) $desc .= '<br>' . $langs->trans('Anciennete') . ' : ' . dol_print_date(strtotime($user->array_options['options_DDA']));
484
+			if (!empty($user->array_options['options_DDA'])) {
485
+				$desc .= '<br>' . $langs->trans('Anciennete') . ' : ' . dol_print_date(strtotime($user->array_options['options_DDA']));
486
+			}
462 487
 
463 488
 			$out .= '<li fk_user="' . $user->id . '" class="' . $class . '">
464 489
 		      ' . $form->showphoto('userphoto', $user, 0, 0, 0, 'photoref', 'small', 1, 0, 1) . '
@@ -486,7 +511,9 @@  discard block
 block discarded – undo
486 511
 	global $db;
487 512
 
488 513
 	//I go back to the user with the highest score in a given group for all the skills assessed in that group
489
-	if (empty($TUser)) return array();
514
+	if (empty($TUser)) {
515
+		return array();
516
+	}
490 517
 
491 518
 	$sql = 'SELECT sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill, ';
492 519
 	$sql.= ' MAX(sr.rankorder) as rankorder';
@@ -541,7 +568,9 @@  discard block
 block discarded – undo
541 568
 {
542 569
 	global $db;
543 570
 
544
-	if (empty($fk_job)) return array();
571
+	if (empty($fk_job)) {
572
+		return array();
573
+	}
545 574
 
546 575
 	$sql = 'SELECT sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill,';
547 576
 	$sql.= " MAX(sr.rankorder) as rankorder";
Please login to merge, or discard this patch.
Spacing   +50 added lines, -50 removed lines patch added patch discarded remove patch
@@ -33,13 +33,13 @@  discard block
 block discarded – undo
33 33
 
34 34
 // Load Dolibarr environment
35 35
 require_once '../main.inc.php';
36
-require_once DOL_DOCUMENT_ROOT . '/core/lib/functions.lib.php';
37
-require_once DOL_DOCUMENT_ROOT . '/core/lib/functions2.lib.php';
38
-require_once DOL_DOCUMENT_ROOT . '/hrm/class/skill.class.php';
39
-require_once DOL_DOCUMENT_ROOT . '/hrm/class/job.class.php';
40
-require_once DOL_DOCUMENT_ROOT . '/hrm/class/evaluation.class.php';
41
-require_once DOL_DOCUMENT_ROOT . '/hrm/class/position.class.php';
42
-require_once DOL_DOCUMENT_ROOT . '/hrm/lib/hrm.lib.php';
36
+require_once DOL_DOCUMENT_ROOT.'/core/lib/functions.lib.php';
37
+require_once DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
38
+require_once DOL_DOCUMENT_ROOT.'/hrm/class/skill.class.php';
39
+require_once DOL_DOCUMENT_ROOT.'/hrm/class/job.class.php';
40
+require_once DOL_DOCUMENT_ROOT.'/hrm/class/evaluation.class.php';
41
+require_once DOL_DOCUMENT_ROOT.'/hrm/class/position.class.php';
42
+require_once DOL_DOCUMENT_ROOT.'/hrm/lib/hrm.lib.php';
43 43
 
44 44
 
45 45
 // Load translation files required by the page
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
 						</tr>
147 147
 						<tr>
148 148
 							<td><?php
149
-									echo $langs->trans('OrJobToCompare') . '</td><td>';
149
+									echo $langs->trans('OrJobToCompare').'</td><td>';
150 150
 									$j = new Job($db);
151 151
 									$jobs = $j->fetchAll();
152 152
 									$TJobs = array();
@@ -229,8 +229,8 @@  discard block
 block discarded – undo
229 229
 							$job->fetch($fk_job);
230 230
 							$userlist2 = '<ul>
231 231
 											  <li>
232
-												  <h3>' . $job->label . '</h3>
233
-												  <p>'  . $job->description . '</p>
232
+												  <h3>' . $job->label.'</h3>
233
+												  <p>'  . $job->description.'</p>
234 234
 											  </li>
235 235
 										  </ul>';
236 236
 						} else {
@@ -244,10 +244,10 @@  discard block
 block discarded – undo
244 244
 
245 245
 						echo '</td>';
246 246
 
247
-						echo '<td id="" style="width:20%" valign="top">' . skillList($TMergedSkills) . '</td>';
248
-						echo '<td id="" style="width:5%" valign="top">' . rate($TMergedSkills, 'rate1') . '</td>';
249
-						echo '<td id="" style="width:10%" valign="top">' . diff($TMergedSkills) . '</td>';
250
-						echo '<td id="" style="width:5%" valign="top">' . rate($TMergedSkills, 'rate2') . '</td>';
247
+						echo '<td id="" style="width:20%" valign="top">'.skillList($TMergedSkills).'</td>';
248
+						echo '<td id="" style="width:5%" valign="top">'.rate($TMergedSkills, 'rate1').'</td>';
249
+						echo '<td id="" style="width:10%" valign="top">'.diff($TMergedSkills).'</td>';
250
+						echo '<td id="" style="width:5%" valign="top">'.rate($TMergedSkills, 'rate2').'</td>';
251 251
 
252 252
 						echo '<td id="list-user-right" style="width:30%" valign="top">';
253 253
 
@@ -296,8 +296,8 @@  discard block
 block discarded – undo
296 296
 		elseif ($sk->rate2 < $sk->rate1) $class .= ' veryhappy';
297 297
 		elseif ($sk->rate2 > $sk->rate1) $class .= ' sad';
298 298
 
299
-		$out .= '<li fk_skill="' . $id . '" class="' . $class . '" style="text-align:center;">
300
-	      <span class="' . $class . '">&nbsp;</span>
299
+		$out .= '<li fk_skill="'.$id.'" class="'.$class.'" style="text-align:center;">
300
+	      <span class="' . $class.'">&nbsp;</span>
301 301
 	    </li>';
302 302
 	}
303 303
 
@@ -332,8 +332,8 @@  discard block
 block discarded – undo
332 332
 		if ($field === 'rate2' && $fk_job > 0) $trad = $langs->trans('RequiredRank');
333 333
 		else $trad = $langs->trans('HighestRank');
334 334
 
335
-		$out .= '<li fk_skill="' . $id . '" style="text-align:center;">
336
-	      <p><span class="' . $class . ' classfortooltip" title="' . $trad . '">' . $note . '</span>' . ($how_many > 0 ? '<span class="bubble classfortooltip" title="' . $langs->trans('HowManyUserWithThisMaxNote') . '">' . $how_many . '</span>' : '') . '</p>
335
+		$out .= '<li fk_skill="'.$id.'" style="text-align:center;">
336
+	      <p><span class="' . $class.' classfortooltip" title="'.$trad.'">'.$note.'</span>'.($how_many > 0 ? '<span class="bubble classfortooltip" title="'.$langs->trans('HowManyUserWithThisMaxNote').'">'.$how_many.'</span>' : '').'</p>
337 337
 	    </li>';
338 338
 	}
339 339
 
@@ -354,9 +354,9 @@  discard block
 block discarded – undo
354 354
 	$out = '<ul class="competence">';
355 355
 
356 356
 	foreach ($TMergedSkills as $id => &$sk) {
357
-		$out .= '<li fk_skill="' . $id . '">
358
-	      <h3>' . $sk->label . '</h3>
359
-	      <p>' . $sk->description . '</p>
357
+		$out .= '<li fk_skill="'.$id.'">
358
+	      <h3>' . $sk->label.'</h3>
359
+	      <p>' . $sk->description.'</p>
360 360
 	    </li>';
361 361
 	}
362 362
 
@@ -411,17 +411,17 @@  discard block
 block discarded – undo
411 411
 
412 412
 	$out = '';
413 413
 	if ($fk_usergroup > 0) {
414
-		$list = $namelist . '_excluded_id';
414
+		$list = $namelist.'_excluded_id';
415 415
 
416 416
 		$excludedIdsList = GETPOST($list);
417 417
 
418
-		$sql = "SELECT u.rowid FROM " . MAIN_DB_PREFIX . "user u
419
-		LEFT JOIN " . MAIN_DB_PREFIX . "usergroup_user as ugu ON (u.rowid = ugu.fk_user)
418
+		$sql = "SELECT u.rowid FROM ".MAIN_DB_PREFIX."user u
419
+		LEFT JOIN " . MAIN_DB_PREFIX."usergroup_user as ugu ON (u.rowid = ugu.fk_user)
420 420
 		WHERE u.statut > 0 AND ugu.entity = ".((int) $conf->entity);
421
-		$sql .= " AND ugu.fk_usergroup=" . ((int) $fk_usergroup);
421
+		$sql .= " AND ugu.fk_usergroup=".((int) $fk_usergroup);
422 422
 
423 423
 		$res = $db->query($sql);
424
-		$out .= '<ul name="' . $namelist . '">';
424
+		$out .= '<ul name="'.$namelist.'">';
425 425
 
426 426
 		$TExcludedId = explode(',', $excludedIdsList);
427 427
 
@@ -453,17 +453,17 @@  discard block
 block discarded – undo
453 453
 			$evaluation = $static_eval->getLastEvaluationForUser($user->id);
454 454
 
455 455
 			if (!empty($evaluation) && !empty($evaluation->date_eval)) {
456
-				$desc .= $langs->trans('DateLastEval') . ' : ' . dol_print_date($evaluation->date_eval);
456
+				$desc .= $langs->trans('DateLastEval').' : '.dol_print_date($evaluation->date_eval);
457 457
 			} else {
458 458
 				$desc .= $langs->trans('NoEval');
459 459
 			}
460 460
 
461
-			if (!empty($user->array_options['options_DDA'])) $desc .= '<br>' . $langs->trans('Anciennete') . ' : ' . dol_print_date(strtotime($user->array_options['options_DDA']));
461
+			if (!empty($user->array_options['options_DDA'])) $desc .= '<br>'.$langs->trans('Anciennete').' : '.dol_print_date(strtotime($user->array_options['options_DDA']));
462 462
 
463
-			$out .= '<li fk_user="' . $user->id . '" class="' . $class . '">
464
-		      ' . $form->showphoto('userphoto', $user, 0, 0, 0, 'photoref', 'small', 1, 0, 1) . '
465
-		      <h3>' . $name . '</h3>
466
-		      <p>' . $desc . '</p>
463
+			$out .= '<li fk_user="'.$user->id.'" class="'.$class.'">
464
+		      ' . $form->showphoto('userphoto', $user, 0, 0, 0, 'photoref', 'small', 1, 0, 1).'
465
+		      <h3>' . $name.'</h3>
466
+		      <p>' . $desc.'</p>
467 467
 		    </li>';
468 468
 		}
469 469
 
@@ -489,12 +489,12 @@  discard block
 block discarded – undo
489 489
 	if (empty($TUser)) return array();
490 490
 
491 491
 	$sql = 'SELECT sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill, ';
492
-	$sql.= ' MAX(sr.rankorder) as rankorder';
493
-	$sql.= ' FROM '.MAIN_DB_PREFIX.'hrm_skill sk';
494
-	$sql.= ' LEFT JOIN '.MAIN_DB_PREFIX.'hrm_skillrank sr ON (sk.rowid = sr.fk_skill)';
495
-	$sql.= " WHERE sr.objecttype = '".$db->escape(SkillRank::SKILLRANK_TYPE_USER)."'";
496
-	$sql.= ' AND sr.fk_object IN ('.$db->sanitize(implode(',', $TUser)).')';
497
-	$sql.= " GROUP BY sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill "; // group par competence
492
+	$sql .= ' MAX(sr.rankorder) as rankorder';
493
+	$sql .= ' FROM '.MAIN_DB_PREFIX.'hrm_skill sk';
494
+	$sql .= ' LEFT JOIN '.MAIN_DB_PREFIX.'hrm_skillrank sr ON (sk.rowid = sr.fk_skill)';
495
+	$sql .= " WHERE sr.objecttype = '".$db->escape(SkillRank::SKILLRANK_TYPE_USER)."'";
496
+	$sql .= ' AND sr.fk_object IN ('.$db->sanitize(implode(',', $TUser)).')';
497
+	$sql .= " GROUP BY sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill "; // group par competence
498 498
 
499 499
 	$resql = $db->query($sql);
500 500
 	$Tab = array();
@@ -502,12 +502,12 @@  discard block
 block discarded – undo
502 502
 	if ($resql) {
503 503
 		//For each skill, we count the number of times that the max score has been reached within a given group
504 504
 		$num = 0;
505
-		while ($obj = $db->fetch_object($resql) ) {
505
+		while ($obj = $db->fetch_object($resql)) {
506 506
 			$sql1 = "SELECT COUNT(rowid) as how_many_max FROM ".MAIN_DB_PREFIX."hrm_skillrank as sr";
507
-			$sql1.=" WHERE sr.rankorder = ".((int) $obj->rankorder);
508
-			$sql1.=" AND sr.objecttype = '".$db->escape(SkillRank::SKILLRANK_TYPE_USER)."'";
509
-			$sql1.=" AND sr.fk_skill = ".((int) $obj->fk_skill);
510
-			$sql1.=" AND sr.fk_object IN (".$db->sanitize(implode(',', $TUser)).")";
507
+			$sql1 .= " WHERE sr.rankorder = ".((int) $obj->rankorder);
508
+			$sql1 .= " AND sr.objecttype = '".$db->escape(SkillRank::SKILLRANK_TYPE_USER)."'";
509
+			$sql1 .= " AND sr.fk_skill = ".((int) $obj->fk_skill);
510
+			$sql1 .= " AND sr.fk_object IN (".$db->sanitize(implode(',', $TUser)).")";
511 511
 			$resql1 = $db->query($sql1);
512 512
 
513 513
 			$objMax = $db->fetch_object($resql1);
@@ -544,19 +544,19 @@  discard block
 block discarded – undo
544 544
 	if (empty($fk_job)) return array();
545 545
 
546 546
 	$sql = 'SELECT sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill,';
547
-	$sql.= " MAX(sr.rankorder) as rankorder";
548
-	$sql.=' FROM '.MAIN_DB_PREFIX.'hrm_skill as sk';
549
-	$sql.='	LEFT JOIN '.MAIN_DB_PREFIX.'hrm_skillrank as sr ON (sk.rowid = sr.fk_skill)';
550
-	$sql.="	WHERE sr.objecttype = '".SkillRank::SKILLRANK_TYPE_JOB."'";
551
-	$sql.=' AND sr.fk_object = '.((int) $fk_job);
552
-	$sql.=' GROUP BY sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill'; // group par competence*/
547
+	$sql .= " MAX(sr.rankorder) as rankorder";
548
+	$sql .= ' FROM '.MAIN_DB_PREFIX.'hrm_skill as sk';
549
+	$sql .= '	LEFT JOIN '.MAIN_DB_PREFIX.'hrm_skillrank as sr ON (sk.rowid = sr.fk_skill)';
550
+	$sql .= "	WHERE sr.objecttype = '".SkillRank::SKILLRANK_TYPE_JOB."'";
551
+	$sql .= ' AND sr.fk_object = '.((int) $fk_job);
552
+	$sql .= ' GROUP BY sk.rowid, sk.label, sk.description, sk.skill_type, sr.fk_object, sr.objecttype, sr.fk_skill'; // group par competence*/
553 553
 
554 554
 	$resql = $db->query($sql);
555 555
 	$Tab = array();
556 556
 
557 557
 	if ($resql) {
558 558
 		$num = 0;
559
-		while ($obj = $db->fetch_object($resql) ) {
559
+		while ($obj = $db->fetch_object($resql)) {
560 560
 			$Tab[$num] = new stdClass();
561 561
 			$Tab[$num]->fk_skill = $obj->fk_skill;
562 562
 			$Tab[$num]->label = $obj->label;
Please login to merge, or discard this patch.
htdocs/variants/class/ProductCombination2ValuePair.class.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -69,8 +69,8 @@  discard block
 block discarded – undo
69 69
 	 */
70 70
 	public function __toString()
71 71
 	{
72
-		require_once DOL_DOCUMENT_ROOT . '/variants/class/ProductAttributeValue.class.php';
73
-		require_once DOL_DOCUMENT_ROOT . '/variants/class/ProductAttribute.class.php';
72
+		require_once DOL_DOCUMENT_ROOT.'/variants/class/ProductAttributeValue.class.php';
73
+		require_once DOL_DOCUMENT_ROOT.'/variants/class/ProductAttribute.class.php';
74 74
 
75 75
 		$prodattr = new ProductAttribute($this->db);
76 76
 		$prodattrval = new ProductAttributeValue($this->db);
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 		$prodattr->fetch($this->fk_prod_attr);
79 79
 		$prodattrval->fetch($this->fk_prod_attr_val);
80 80
 
81
-		return $prodattr->label . ': ' . $prodattrval->value;
81
+		return $prodattr->label.': '.$prodattrval->value;
82 82
 	}
83 83
 
84 84
 	/**
@@ -89,14 +89,14 @@  discard block
 block discarded – undo
89 89
 	 */
90 90
 	public function create($user)
91 91
 	{
92
-		$sql = "INSERT INTO " . MAIN_DB_PREFIX . "product_attribute_combination2val
92
+		$sql = "INSERT INTO ".MAIN_DB_PREFIX."product_attribute_combination2val
93 93
 		(fk_prod_combination, fk_prod_attr, fk_prod_attr_val)
94
-		VALUES(" . (int) $this->fk_prod_combination . ", " . (int) $this->fk_prod_attr . ", " . (int) $this->fk_prod_attr_val . ")";
94
+		VALUES(" . (int) $this->fk_prod_combination.", ".(int) $this->fk_prod_attr.", ".(int) $this->fk_prod_attr_val.")";
95 95
 
96 96
 		$query = $this->db->query($sql);
97 97
 
98 98
 		if ($query) {
99
-			$this->id = $this->db->last_insert_id(MAIN_DB_PREFIX . 'product_attribute_combination2val');
99
+			$this->id = $this->db->last_insert_id(MAIN_DB_PREFIX.'product_attribute_combination2val');
100 100
 
101 101
 			return 1;
102 102
 		}
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
         c2v.fk_prod_attr_val,
118 118
         c2v.fk_prod_attr,
119 119
         c2v.fk_prod_combination
120
-        FROM " . MAIN_DB_PREFIX . "product_attribute c LEFT JOIN " . MAIN_DB_PREFIX . "product_attribute_combination2val c2v ON c.rowid = c2v.fk_prod_attr
120
+        FROM " . MAIN_DB_PREFIX."product_attribute c LEFT JOIN ".MAIN_DB_PREFIX."product_attribute_combination2val c2v ON c.rowid = c2v.fk_prod_attr
121 121
         WHERE c2v.fk_prod_combination = " . (int) $fk_combination;
122 122
 
123 123
 		$sql .= $this->db->order('c.position', 'asc');
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 	 */
152 152
 	public function deleteByFkCombination($fk_combination)
153 153
 	{
154
-		$sql = "DELETE FROM " . MAIN_DB_PREFIX . "product_attribute_combination2val WHERE fk_prod_combination = " . (int) $fk_combination;
154
+		$sql = "DELETE FROM ".MAIN_DB_PREFIX."product_attribute_combination2val WHERE fk_prod_combination = ".(int) $fk_combination;
155 155
 
156 156
 		if ($this->db->query($sql)) {
157 157
 			return 1;
Please login to merge, or discard this patch.