fix phpstan

Dolibarr · Nov 13, 2024 · 6a8feca · 6a8feca
1 parent 8bce9dc
commit 6a8feca
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/htdocs/compta/paiement/rapport.php b/htdocs/compta/paiement/rapport.php
@@ -72,7 +72,7 @@
  * Actions
  */
 
-if ($action == 'builddoc' && $permissiontoread) {
+if ($action == 'builddoc' && !empty($permissiontoread)) {
 	$rap = new pdf_paiement($db);
 
 	$outputlangs = $langs;
@@ -95,7 +95,7 @@
 }
 
 // Delete file from disk
-if ($action == 'removedoc' && $permissiontoread && $fileToRemove) {
+if ($action == 'removedoc' && !empty($permissiontoread) && $fileToRemove) {
 	$fileDirectory = dirname($dir.'/'.$fileToRemove);
 	if (dol_delete_file($dir.'/'.$fileToRemove)) {
 		// Delete empty directory after file deletion

diff --git a/htdocs/fourn/facture/rapport.php b/htdocs/fourn/facture/rapport.php
@@ -74,7 +74,7 @@
  * Actions
  */
 
-if ($action == 'builddoc' && $permissiontoread) {
+if ($action == 'builddoc' && !empty($permissiontoread)) {
 	$rap = new pdf_paiement_fourn($db);
 
 	$outputlangs = $langs;
@@ -97,7 +97,7 @@
 }
 
 // Delete file from disk
-if ($action == 'removedoc' && $permissiontoread && $fileToRemove) {
+if ($action == 'removedoc' && !empty($permissiontoread) && $fileToRemove) {
 	$fileDirectory = dirname($dir.'/'.$fileToRemove);
 	if (dol_delete_file($dir.'/'.$fileToRemove)) {
 		// Delete empty directory after file deletion