Merge pull request #8 from RomainLvr/fix/by-visible

Fix By visible

Author: Rom1-B

inc/itilsolution.class.php

@@ -90,7 +90,7 @@ public static function pdfForItem(PluginPdfSimplePDF $pdf, CommonDBTM $item)
                 } else {
                     $text = $textapprove = '';
                 }
-                if (isset($row['date_approval']) || isset($row['users_id_approval'])) {
+                if (isset($row['date_approval']) || !empty($row['users_id_approval'])) {
                     $textapprove = '<br /><br /><br /><i>' .
                                     sprintf(
                                         __('%1$s %2$s'),
@@ -103,9 +103,8 @@ public static function pdfForItem(PluginPdfSimplePDF $pdf, CommonDBTM $item)
                                         Toolbox::stripTags($dbu->getUserName($row['users_id_approval'])),
                                     )
                                     . '</i>';
-                    $pdf->displayText('<b><i>' . sprintf(__('%1$s: %2$s'), $title . '</i></b>', ''), $sol .
-                                      $textapprove);
                 }
+                $pdf->displayText('<b><i>' . sprintf(__('%1$s: %2$s'), $title . '</i></b>', ''), $sol . $textapprove);
             }
         }