diff --git a/htdocs/core/class/commonobject.class.php b/htdocs/core/class/commonobject.class.php
index 19221febe3c..e246e8ade73 100644
--- a/htdocs/core/class/commonobject.class.php
+++ b/htdocs/core/class/commonobject.class.php
@@ -6744,7 +6744,7 @@ abstract class CommonObject
* Return HTML string to put an input field into a page
* Code very similar with showInputField of extra fields
*
- * @param array $val Array of properties for field to show (used only if ->fields not defined)
+ * @param array|null $val Array of properties for field to show (used only if ->fields not defined)
* @param string $key Key of attribute
* @param string|array $value Preselected value to show (for date type it must be in timestamp format, for amount or price it must be a php numeric value, for array type must be array)
* @param string $moreparam To add more parameters on html input tag
diff --git a/htdocs/core/class/html.formother.class.php b/htdocs/core/class/html.formother.class.php
index 596df87e583..ee12b3b72fb 100644
--- a/htdocs/core/class/html.formother.class.php
+++ b/htdocs/core/class/html.formother.class.php
@@ -782,7 +782,7 @@ class FormOther
*
* @param string $color String with hex (FFFFFF) or comma RGB ('255,255,255')
* @param string $textifnotdefined Text to show if color not defined
- * @return void
+ * @return string Show color string
* @see selectColor()
*/
public static function showColor($color, $textifnotdefined = '')
@@ -796,9 +796,9 @@ class FormOther
$color = colorArrayToHex(colorStringToArray($color, array()), '');
if ($color) {
- print '';
+ return '';
} else {
- print $textifnotdefined;
+ return $textifnotdefined;
}
}
diff --git a/htdocs/core/class/smtps.class.php b/htdocs/core/class/smtps.class.php
index fe1f90dd735..dc58c54f122 100644
--- a/htdocs/core/class/smtps.class.php
+++ b/htdocs/core/class/smtps.class.php
@@ -99,8 +99,10 @@ class SMTPs
/**
* Message Content
+ *
+ * @var array $_msgContent Array of messages
*/
- private $_msgContent = null;
+ private array $_msgContent = array();
/**
* Custom X-Headers
@@ -1841,7 +1843,7 @@ class SMTPs
/**
* Retrieves the Message X-Header Content
*
- * @return string[] $_msgContent Message X-Header Content
+ * @return array $_msgContent Message X-Header Content
*/
public function getXheader()
{
diff --git a/htdocs/core/modules/supplier_payment/doc/pdf_standard.modules.php b/htdocs/core/modules/supplier_payment/doc/pdf_standard.modules.php
index 18cb0796ed0..bede0cc948e 100644
--- a/htdocs/core/modules/supplier_payment/doc/pdf_standard.modules.php
+++ b/htdocs/core/modules/supplier_payment/doc/pdf_standard.modules.php
@@ -603,10 +603,10 @@ class pdf_standard extends ModelePDFSuppliersPayments
/**
* Show top header of page.
*
- * @param TCPDF $pdf Object PDF
- * @param FactureFournisseur $object Object to show
- * @param int $showaddress 0=no, 1=yes
- * @param Translate $outputlangs Object lang for output
+ * @param TCPDF $pdf Object PDF
+ * @param PaiementFourn $object Object to show
+ * @param int $showaddress 0=no, 1=yes
+ * @param Translate $outputlangs Object lang for output
* @return void
*/
protected function _pagehead(&$pdf, $object, $showaddress, $outputlangs)
@@ -814,11 +814,11 @@ class pdf_standard extends ModelePDFSuppliersPayments
/**
* Show footer of page. Need this->emetteur object
*
- * @param TCPDF $pdf PDF
- * @param FactureFournisseur $object Object to show
- * @param Translate $outputlangs Object lang for output
- * @param int $hidefreetext 1=Hide free text
- * @return int Return height of bottom margin including footer text
+ * @param TCPDF $pdf PDF
+ * @param PaiementFourn $object Object to show
+ * @param Translate $outputlangs Object lang for output
+ * @param int $hidefreetext 1=Hide free text
+ * @return int Return height of bottom margin including footer text
*/
protected function _pagefoot(&$pdf, $object, $outputlangs, $hidefreetext = 0)
{