@@ -179,7 +179,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -186,7 +186,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -180,7 +180,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -269,7 +269,7 @@ discard block |
||
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 |
||
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 |
||
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 |
@@ -187,7 +187,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -264,7 +264,7 @@ discard block |
||
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 |
||
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 |
||
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 |
@@ -609,7 +609,9 @@ |
||
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 |
@@ -102,13 +102,13 @@ discard block |
||
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 |
||
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 |
||
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 |
@@ -51,8 +51,12 @@ discard block |
||
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 |
||
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 |
||
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 . '"> </span> |
@@ -329,8 +341,11 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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"; |
@@ -33,13 +33,13 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 . '"> </span> |
|
299 | + $out .= '<li fk_skill="'.$id.'" class="'.$class.'" style="text-align:center;"> |
|
300 | + <span class="' . $class.'"> </span> |
|
301 | 301 | </li>'; |
302 | 302 | } |
303 | 303 | |
@@ -332,8 +332,8 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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; |
@@ -69,8 +69,8 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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; |