Fix scrutinizer warning
This commit is contained in:
parent
935327d911
commit
c2fc9ecf68
@ -40,7 +40,7 @@ $form = new Form($db);
|
|||||||
$title=$langs->trans("SystemToolsArea");
|
$title=$langs->trans("SystemToolsArea");
|
||||||
if (GETPOST('leftmenu') == 'admintools') $title=$langs->trans("ModulesSystemTools");
|
if (GETPOST('leftmenu') == 'admintools') $title=$langs->trans("ModulesSystemTools");
|
||||||
|
|
||||||
llxHeader(array(),$title);
|
llxHeader('', $title);
|
||||||
|
|
||||||
print load_fiche_titre($title,'','title_setup');
|
print load_fiche_titre($title,'','title_setup');
|
||||||
|
|
||||||
|
|||||||
@ -178,7 +178,7 @@ if (! $mesg)
|
|||||||
$px2->draw($filenameamount,$fileurlamount);
|
$px2->draw($filenameamount,$fileurlamount);
|
||||||
}
|
}
|
||||||
|
|
||||||
$data = $stats->getAverageByMonthWithPrevYear($endyear, $startyear, $filter);
|
$data = $stats->getAverageByMonthWithPrevYear($endyear, $startyear);
|
||||||
|
|
||||||
$fileurl_avg='';
|
$fileurl_avg='';
|
||||||
if (!$user->rights->societe->client->voir || $user->societe_id)
|
if (!$user->rights->societe->client->voir || $user->societe_id)
|
||||||
|
|||||||
@ -314,7 +314,7 @@ class PaiementFourn extends Paiement
|
|||||||
// Si c'est le cas, on refuse le delete
|
// Si c'est le cas, on refuse le delete
|
||||||
if ($bank_line_id)
|
if ($bank_line_id)
|
||||||
{
|
{
|
||||||
$accline = new AccountLine($this->db,$bank_line_id);
|
$accline = new AccountLine($this->db);
|
||||||
$accline->fetch($bank_line_id);
|
$accline->fetch($bank_line_id);
|
||||||
if ($accline->rappro)
|
if ($accline->rappro)
|
||||||
{
|
{
|
||||||
|
|||||||
@ -677,7 +677,7 @@ $cp = new Holiday($db);
|
|||||||
|
|
||||||
$listhalfday=array('morning'=>$langs->trans("Morning"),"afternoon"=>$langs->trans("Afternoon"));
|
$listhalfday=array('morning'=>$langs->trans("Morning"),"afternoon"=>$langs->trans("Afternoon"));
|
||||||
|
|
||||||
llxHeader(array(),$langs->trans('CPTitreMenu'));
|
llxHeader('', $langs->trans('CPTitreMenu'));
|
||||||
|
|
||||||
if (empty($id) || $action == 'add' || $action == 'request' || $action == 'create')
|
if (empty($id) || $action == 'add' || $action == 'request' || $action == 'create')
|
||||||
{
|
{
|
||||||
|
|||||||
@ -155,7 +155,7 @@ elseif($action == 'add_event')
|
|||||||
$form = new Form($db);
|
$form = new Form($db);
|
||||||
$userstatic=new User($db);
|
$userstatic=new User($db);
|
||||||
|
|
||||||
llxHeader(array(),$langs->trans('CPTitreMenu'));
|
llxHeader('', $langs->trans('CPTitreMenu'));
|
||||||
|
|
||||||
print load_fiche_titre($langs->trans('MenuConfCP'), '', 'title_hrm.png');
|
print load_fiche_titre($langs->trans('MenuConfCP'), '', 'title_hrm.png');
|
||||||
|
|
||||||
|
|||||||
@ -114,7 +114,7 @@ $max_year = 5;
|
|||||||
$min_year = 10;
|
$min_year = 10;
|
||||||
$filter='';
|
$filter='';
|
||||||
|
|
||||||
llxHeader(array(),$langs->trans('CPTitreMenu'));
|
llxHeader('', $langs->trans('CPTitreMenu'));
|
||||||
|
|
||||||
$order = $db->order($sortfield,$sortorder).$db->plimit($limit + 1, $offset);
|
$order = $db->order($sortfield,$sortorder).$db->plimit($limit + 1, $offset);
|
||||||
|
|
||||||
|
|||||||
@ -50,7 +50,7 @@ $langs->load('users');
|
|||||||
|
|
||||||
$cp = new Holiday($db);
|
$cp = new Holiday($db);
|
||||||
|
|
||||||
llxHeader(array(),$langs->trans('CPTitreMenu').' ('.$langs->trans("Year").' '.$year.')');
|
llxHeader('', $langs->trans('CPTitreMenu').' ('.$langs->trans("Year").' '.$year.')');
|
||||||
|
|
||||||
// Recent changes are more important than old changes
|
// Recent changes are more important than old changes
|
||||||
$log_holiday = $cp->fetchLog('ORDER BY cpl.rowid DESC', " AND date_action BETWEEN '".$db->idate(dol_get_first_day($year,1,1))."' AND '".$db->idate(dol_get_last_day($year,12,1))."'"); // Load $cp->logs
|
$log_holiday = $cp->fetchLog('ORDER BY cpl.rowid DESC', " AND date_action BETWEEN '".$db->idate(dol_get_first_day($year,1,1))."' AND '".$db->idate(dol_get_last_day($year,12,1))."'"); // Load $cp->logs
|
||||||
|
|||||||
@ -65,7 +65,7 @@ $holidaystatic=new Holiday($db);
|
|||||||
$childids = $user->getAllChildIds();
|
$childids = $user->getAllChildIds();
|
||||||
$childids[]=$user->id;
|
$childids[]=$user->id;
|
||||||
|
|
||||||
llxHeader(array(),$langs->trans('HRMArea'));
|
llxHeader('', $langs->trans('HRMArea'));
|
||||||
|
|
||||||
print load_fiche_titre($langs->trans("HRMArea"),'', 'title_hrm.png');
|
print load_fiche_titre($langs->trans("HRMArea"),'', 'title_hrm.png');
|
||||||
|
|
||||||
|
|||||||
@ -3540,7 +3540,7 @@ class Product extends CommonObject
|
|||||||
$handle=opendir($dir_osencoded);
|
$handle=opendir($dir_osencoded);
|
||||||
if (is_resource($handle))
|
if (is_resource($handle))
|
||||||
{
|
{
|
||||||
while (($file = readdir($handle)) != false)
|
while (($file = readdir($handle)) !== false)
|
||||||
{
|
{
|
||||||
if (! utf8_check($file)) $file=utf8_encode($file); // To be sure data is stored in UTF8 in memory
|
if (! utf8_check($file)) $file=utf8_encode($file); // To be sure data is stored in UTF8 in memory
|
||||||
if (dol_is_file($dir.$file)) return true;
|
if (dol_is_file($dir.$file)) return true;
|
||||||
@ -3598,7 +3598,7 @@ class Product extends CommonObject
|
|||||||
$handle=opendir($dir_osencoded);
|
$handle=opendir($dir_osencoded);
|
||||||
if (is_resource($handle))
|
if (is_resource($handle))
|
||||||
{
|
{
|
||||||
while (($file = readdir($handle)) != false)
|
while (($file = readdir($handle)) !== false)
|
||||||
{
|
{
|
||||||
$photo='';
|
$photo='';
|
||||||
|
|
||||||
@ -3744,7 +3744,7 @@ class Product extends CommonObject
|
|||||||
$handle=@opendir($dir_osencoded);
|
$handle=@opendir($dir_osencoded);
|
||||||
if (is_resource($handle))
|
if (is_resource($handle))
|
||||||
{
|
{
|
||||||
while (($file = readdir($handle)) != false)
|
while (($file = readdir($handle)) !== false)
|
||||||
{
|
{
|
||||||
if (! utf8_check($file)) $file=utf8_encode($file); // readdir returns ISO
|
if (! utf8_check($file)) $file=utf8_encode($file); // readdir returns ISO
|
||||||
if (dol_is_file($dir.$file) && preg_match('/('.$this->regeximgext.')$/i', $dir.$file))
|
if (dol_is_file($dir.$file) && preg_match('/('.$this->regeximgext.')$/i', $dir.$file))
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user