Fix CI
This commit is contained in:
parent
a8173ff888
commit
9e4eb43dbc
1 changed files with 13 additions and 13 deletions
|
|
@ -1152,16 +1152,16 @@ class FormOther
|
|||
/**
|
||||
* Return HTML combo list of years
|
||||
*
|
||||
* @param string $selected Preselected value (''=current year, -1=none, year otherwise)
|
||||
* @param string $htmlname Name of HTML select object
|
||||
* @param int $useempty Affiche valeur vide dans liste
|
||||
* @param int $min_year Offset of minimum year into list (by default current year -10)
|
||||
* @param int $max_year Offset of maximum year into list (by default current year + 5)
|
||||
* @param int $offset Offset
|
||||
* @param int $invert Invert
|
||||
* @param string $option Option
|
||||
* @param string $morecss More css
|
||||
* @param bool $addjscombo Add js combo
|
||||
* @param int|string $selected Preselected value (''=current year, -1=none, year otherwise)
|
||||
* @param string $htmlname Name of HTML select object
|
||||
* @param int $useempty Affiche valeur vide dans liste
|
||||
* @param int $min_year Offset of minimum year into list (by default current year -10)
|
||||
* @param int $max_year Offset of maximum year into list (by default current year + 5)
|
||||
* @param int $offset Offset
|
||||
* @param int $invert Invert
|
||||
* @param string $option Option
|
||||
* @param string $morecss More css
|
||||
* @param bool $addjscombo Add js combo
|
||||
* @return string
|
||||
*/
|
||||
public function selectyear($selected = '', $htmlname = 'yearid', $useempty = 0, $min_year = 10, $max_year = 5, $offset = 0, $invert = 0, $option = '', $morecss = 'valignmiddle width75', $addjscombo = false)
|
||||
|
|
@ -1178,7 +1178,7 @@ class FormOther
|
|||
$out .= '<select class="flat'.($morecss ? ' '.$morecss : '').'" id="'.$htmlname.'" name="'.$htmlname.'"'.$option.' >';
|
||||
if ($useempty) {
|
||||
$selected_html = '';
|
||||
if ($selected == '') {
|
||||
if ((string) $selected == '') {
|
||||
$selected_html = ' selected';
|
||||
}
|
||||
$out .= '<option value=""'.$selected_html.'> </option>';
|
||||
|
|
@ -1186,7 +1186,7 @@ class FormOther
|
|||
if (!$invert) {
|
||||
for ($y = $max_year; $y >= $min_year; $y--) {
|
||||
$selected_html = '';
|
||||
if ($selected > 0 && $y == $selected) {
|
||||
if ((int) $selected > 0 && $y == (int) $selected) {
|
||||
$selected_html = ' selected';
|
||||
}
|
||||
$out .= '<option value="'.$y.'"'.$selected_html.' >'.$y.'</option>';
|
||||
|
|
@ -1194,7 +1194,7 @@ class FormOther
|
|||
} else {
|
||||
for ($y = $min_year; $y <= $max_year; $y++) {
|
||||
$selected_html = '';
|
||||
if ($selected > 0 && $y == $selected) {
|
||||
if ((int) $selected > 0 && $y == (int) $selected) {
|
||||
$selected_html = ' selected';
|
||||
}
|
||||
$out .= '<option value="'.$y.'"'.$selected_html.' >'.$y.'</option>';
|
||||
|
|
|
|||
Loading…
Reference in a new issue