Issues (4868)

timesheet/index.php (2 issues)

1
<?php
2
/**
3
 * TimeSheet - index
4
 *
5
 * @link http://www.egroupware.org
6
 * @author Ralf Becker <RalfBecker-AT-outdoor-training.de>
7
 * @package timesheet
8
 * @copyright (c) 2005-16 by Ralf Becker <RalfBecker-AT-outdoor-training.de>
9
 * @license http://opensource.org/licenses/gpl-license.php GPL - GNU General Public License
10
 * @version $Id$
11
 */
12
13
use EGroupware\Api\Framework;
14
15
include_once('./setup/setup.inc.php');
16
$ts_version = $setup_info[TIMESHEET_APP]['version'];
17
unset($setup_info);
18
19
$GLOBALS['egw_info'] = array(
20
	'flags' => array(
21
		'currentapp'	=> TIMESHEET_APP,
22
		'noheader'		=> True,
23
		'nonavbar'		=> True
24
));
25
include('../header.inc.php');
26
27
if ($ts_version != $GLOBALS['egw_info']['apps'][TIMESHEET_APP]['version'])
28
{
29
	Framework::render('<p style="text-align: center; color:red; font-weight: bold;">'.
0 ignored issues
show
Bug Best Practice introduced by
The method EGroupware\Api\Framework::render() is not static, but was called statically. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

29
	Framework::/** @scrutinizer ignore-call */ 
30
            render('<p style="text-align: center; color:red; font-weight: bold;">'.
Loading history...
30
		lang('Your database is NOT up to date (%1 vs. %2), please run %3setup%4 to update your database.',
31
		$ts_version,$GLOBALS['egw_info']['apps'][TIMESHEET_APP]['version'],
0 ignored issues
show
The call to lang() has too many arguments starting with $ts_version. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

31
		/** @scrutinizer ignore-call */ 
32
  lang('Your database is NOT up to date (%1 vs. %2), please run %3setup%4 to update your database.',

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
32
		'<a href="../setup/">','</a>')."</p>\n", null, true);
33
	exit();
34
}
35
36
Framework::redirect_link('/index.php',array('menuaction'=>TIMESHEET_APP.'.timesheet_ui.index'));
37