59% of this comment could be valid code. Did you maybe forget this after debugging?
Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have
checked you do not need it.
The code might also have been commented out for debugging purposes. In this case it is vital that
someone uncomments it again or your project may behave in very unexpected ways in production.
This check looks for comments that seem to be mostly valid code and reports them.
Loading history...
10
define('1',E_ERROR);
11
define('2',E_WARNING);
12
define('4',E_PARSE);
13
define('8',E_NOTICE);
14
define('16',E_CORE_ERROR);
15
define('32',E_CORE_WARNING);
16
define('64',E_COMPILE_ERROR);
17
define('128',E_COMPILE_WARNING);
18
define('256',E_USER_ERROR);
19
define('512',E_USER_WARNING);
20
define('1024',E_USER_NOTICE);
21
define('2048',E_STRICT);
22
define('4096',E_RECOVERABLE_ERROR);
23
define('8192',E_DEPRECATED);
24
define('16384',E_USER_DEPRECATED);
25
define('30719',E_ALL);
26
*/
27
/**
28
*/
29
class SapphireREPL extends Controller
30
{
31
32
private static $allowed_actions = array(
33
'index'
34
);
35
36
public function error_handler($errno, $errstr, $errfile, $errline, $errctx)
37
{
38
// Ignore unless important error
39
if (($errno & ~( 2048 | 8192 | 16384 )) == 0) {
40
return ;
41
}
42
// Otherwise throw exception to handle in REPL loop
43
throw new Exception(sprintf("%s:%d\r\n%s", $errfile, $errline, $errstr));
44
}
45
46
public function index()
47
{
48
if (!Director::is_cli()) {
49
return "The SilverStripe Interactive Command-line doesn't work in a web browser."
50
. " Use 'sake interactive' from the command-line to run.";
51
}
52
53
54
/* Try using PHP_Shell if it exists */
55
@include 'php-shell-cmd.php' ;
56
57
/* Fall back to our simpler interface */
58
if (empty($__shell)) {
59
set_error_handler(array($this, 'error_handler'));
60
61
echo "SilverStripe Interactive Command-line (REPL interface). Type help for hints.\n\n";
62
while (true) {
63
echo CLI::text("?> ", "cyan");
64
echo CLI::start_colour("yellow");
65
$command = trim(fgets(STDIN, 4096));
66
echo CLI::end_colour();
67
68
if ($command == 'help' || $command == '?') {
69
print "help or ? to exit\n" ;
70
print "quit or \q to exit\n" ;
71
print "install PHP_Shell for a more advanced interface with"
silverstripe /
silverstripe-framework
Sam Minnee
Damian Mooyman
Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.
The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.
This check looks for comments that seem to be mostly valid code and reports them.