fix phpstan (#31182)

* fix phpstan * fix phpstan --------- Co-authored-by: Laurent Destailleur <eldy@destailleur.fr>
Dolibarr · Sep 29, 2024 · 119456c · 119456c
1 parent 9e897b9
commit 119456c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/htdocs/hrm/compare.php b/htdocs/hrm/compare.php
@@ -121,10 +121,10 @@
 $fk_usergroup2 = 0;
 $fk_job = (int) GETPOST('fk_job');
 if ($fk_job <= 0) {
-	$fk_usergroup2 = GETPOST('fk_usergroup2');
+	$fk_usergroup2 = GETPOSTINT('fk_usergroup2');
 }
 
-$fk_usergroup1 = GETPOST('fk_usergroup1');
+$fk_usergroup1 = GETPOSTINT('fk_usergroup1');
 
 ?>
 
@@ -237,11 +237,11 @@
 							$job = new Job($db);
 							$job->fetch($fk_job);
 							$userlist2 = '<ul>
-											  <li>
-												  <h3>' . $job->label . '</h3>
-												  <p>'  . $job->description . '</p>
-											  </li>
-										  </ul>';
+											<li>
+												<h3>' . $job->label . '</h3>
+												<p>' . $job->description . '</p>
+											</li>
+										</ul>';
 						} else {
 							$userlist2 = displayUsersListWithPicto($TUser2, $fk_usergroup2, 'list2');
 							$TSkill2 = getSkillForUsers($TUser2);

diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -84,7 +84,7 @@ parameters:
 		- '# EmailCollector::getpart\(\) expects string#'
 		- '#expects int, float#'
 		- '#::saveboxorder\(\) expects int, array#'
-		- '# (fetchObjectByElement|print_actions_filter|dol_mktime|dol_remove_file_process|displayUsersListWithPicto) expects int, array\|string given.#'
+		- '# (fetchObjectByElement|print_actions_filter|dol_mktime|dol_remove_file_process) expects int, array\|string given.#'
 		- '# (CSMSFile) constructor expects int, array\|string given.#'
 		- '#(?:ProductFournisseur::logPrice\(\)) expects float\|null#'
 		- '#(?:(?:Asset::addDepreciationL|Facture(?:(?:(?:Fournisseur)?::add|Fournisseur::update)l))ine\(\)|calcul_price_total|dol_convertToWord|(?:loanCalcMonthlyPaymen|print_paypal_redirec)t) expects float, string given.#'