| Conditions | 40 |
| Paths | > 20000 |
| Total Lines | 374 |
| Code Lines | 245 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 2 | ||
| Bugs | 0 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 388 | public function __construct() |
||
| 389 | { |
||
| 390 | /* mandatory requirements follow */ |
||
| 391 | |||
| 392 | $installedPhpVersion = phpversion(); |
||
| 393 | $requiredPhpVersion = $this->getPhpRequiredVersion(); |
||
| 394 | |||
| 395 | $this->addRecommendation( |
||
| 396 | $requiredPhpVersion, |
||
| 397 | 'Vendors should be installed in order to check all requirements.', |
||
| 398 | 'Run the <code>composer install</code> command.', |
||
| 399 | 'Run the "composer install" command.' |
||
| 400 | ); |
||
| 401 | |||
| 402 | if (false !== $requiredPhpVersion) { |
||
| 403 | $this->addRequirement( |
||
| 404 | version_compare($installedPhpVersion, $requiredPhpVersion, '>='), |
||
| 405 | sprintf('PHP version must be at least %s (%s installed)', $requiredPhpVersion, $installedPhpVersion), |
||
| 406 | sprintf('You are running PHP version "<strong>%s</strong>", but Symfony needs at least PHP "<strong>%s</strong>" to run. |
||
| 407 | Before using Symfony, upgrade your PHP installation, preferably to the latest version.', |
||
| 408 | $installedPhpVersion, $requiredPhpVersion), |
||
| 409 | sprintf('Install PHP %s or newer (installed version is %s)', $requiredPhpVersion, $installedPhpVersion) |
||
| 410 | ); |
||
| 411 | } |
||
| 412 | |||
| 413 | $this->addRequirement( |
||
| 414 | version_compare($installedPhpVersion, '5.3.16', '!='), |
||
| 415 | 'PHP version must not be 5.3.16 as Symfony won\'t work properly with it', |
||
| 416 | 'Install PHP 5.3.17 or newer (or downgrade to an earlier PHP version)' |
||
| 417 | ); |
||
| 418 | |||
| 419 | $this->addRequirement( |
||
| 420 | is_dir(__DIR__.'/../vendor/composer'), |
||
| 421 | 'Vendor libraries must be installed', |
||
| 422 | 'Vendor libraries are missing. Install composer following instructions from <a href="http://getcomposer.org/">http://getcomposer.org/</a>. '. |
||
| 423 | 'Then run "<strong>php composer.phar install</strong>" to install them.' |
||
| 424 | ); |
||
| 425 | |||
| 426 | $cacheDir = is_dir(__DIR__.'/../var/cache') ? __DIR__.'/../var/cache' : __DIR__.'/cache'; |
||
| 427 | |||
| 428 | $this->addRequirement( |
||
| 429 | is_writable($cacheDir), |
||
| 430 | 'app/cache/ or var/cache/ directory must be writable', |
||
| 431 | 'Change the permissions of either "<strong>app/cache/</strong>" or "<strong>var/cache/</strong>" directory so that the web server can write into it.' |
||
| 432 | ); |
||
| 433 | |||
| 434 | $logsDir = is_dir(__DIR__.'/../var/logs') ? __DIR__.'/../var/logs' : __DIR__.'/logs'; |
||
| 435 | |||
| 436 | $this->addRequirement( |
||
| 437 | is_writable($logsDir), |
||
| 438 | 'app/logs/ or var/logs/ directory must be writable', |
||
| 439 | 'Change the permissions of either "<strong>app/logs/</strong>" or "<strong>var/logs/</strong>" directory so that the web server can write into it.' |
||
| 440 | ); |
||
| 441 | |||
| 442 | if (version_compare($installedPhpVersion, '7.0.0', '<')) { |
||
| 443 | $this->addPhpIniRequirement( |
||
| 444 | 'date.timezone', true, false, |
||
| 445 | 'date.timezone setting must be set', |
||
| 446 | 'Set the "<strong>date.timezone</strong>" setting in php.ini<a href="#phpini">*</a> (like Europe/Paris).' |
||
| 447 | ); |
||
| 448 | } |
||
| 449 | |||
| 450 | if (false !== $requiredPhpVersion && version_compare($installedPhpVersion, $requiredPhpVersion, '>=')) { |
||
| 451 | $timezones = array(); |
||
| 452 | foreach (DateTimeZone::listAbbreviations() as $abbreviations) { |
||
| 453 | foreach ($abbreviations as $abbreviation) { |
||
| 454 | $timezones[$abbreviation['timezone_id']] = true; |
||
| 455 | } |
||
| 456 | } |
||
| 457 | |||
| 458 | $this->addRequirement( |
||
| 459 | isset($timezones[@date_default_timezone_get()]), |
||
| 460 | sprintf('Configured default timezone "%s" must be supported by your installation of PHP', @date_default_timezone_get()), |
||
| 461 | 'Your default timezone is not supported by PHP. Check for typos in your <strong>php.ini</strong> file and have a look at the list of deprecated timezones at <a href="http://php.net/manual/en/timezones.others.php">http://php.net/manual/en/timezones.others.php</a>.' |
||
| 462 | ); |
||
| 463 | } |
||
| 464 | |||
| 465 | $this->addRequirement( |
||
| 466 | function_exists('iconv'), |
||
| 467 | 'iconv() must be available', |
||
| 468 | 'Install and enable the <strong>iconv</strong> extension.' |
||
| 469 | ); |
||
| 470 | |||
| 471 | $this->addRequirement( |
||
| 472 | function_exists('json_encode'), |
||
| 473 | 'json_encode() must be available', |
||
| 474 | 'Install and enable the <strong>JSON</strong> extension.' |
||
| 475 | ); |
||
| 476 | |||
| 477 | $this->addRequirement( |
||
| 478 | function_exists('session_start'), |
||
| 479 | 'session_start() must be available', |
||
| 480 | 'Install and enable the <strong>session</strong> extension.' |
||
| 481 | ); |
||
| 482 | |||
| 483 | $this->addRequirement( |
||
| 484 | function_exists('ctype_alpha'), |
||
| 485 | 'ctype_alpha() must be available', |
||
| 486 | 'Install and enable the <strong>ctype</strong> extension.' |
||
| 487 | ); |
||
| 488 | |||
| 489 | $this->addRequirement( |
||
| 490 | function_exists('token_get_all'), |
||
| 491 | 'token_get_all() must be available', |
||
| 492 | 'Install and enable the <strong>Tokenizer</strong> extension.' |
||
| 493 | ); |
||
| 494 | |||
| 495 | $this->addRequirement( |
||
| 496 | function_exists('simplexml_import_dom'), |
||
| 497 | 'simplexml_import_dom() must be available', |
||
| 498 | 'Install and enable the <strong>SimpleXML</strong> extension.' |
||
| 499 | ); |
||
| 500 | |||
| 501 | if (function_exists('apc_store') && ini_get('apc.enabled')) { |
||
| 502 | if (version_compare($installedPhpVersion, '5.4.0', '>=')) { |
||
| 503 | $this->addRequirement( |
||
| 504 | version_compare(phpversion('apc'), '3.1.13', '>='), |
||
| 505 | 'APC version must be at least 3.1.13 when using PHP 5.4', |
||
| 506 | 'Upgrade your <strong>APC</strong> extension (3.1.13+).' |
||
| 507 | ); |
||
| 508 | } else { |
||
| 509 | $this->addRequirement( |
||
| 510 | version_compare(phpversion('apc'), '3.0.17', '>='), |
||
| 511 | 'APC version must be at least 3.0.17', |
||
| 512 | 'Upgrade your <strong>APC</strong> extension (3.0.17+).' |
||
| 513 | ); |
||
| 514 | } |
||
| 515 | } |
||
| 516 | |||
| 517 | $this->addPhpIniRequirement('detect_unicode', false); |
||
| 518 | |||
| 519 | if (extension_loaded('suhosin')) { |
||
| 520 | $this->addPhpIniRequirement( |
||
| 521 | 'suhosin.executor.include.whitelist', |
||
| 522 | create_function('$cfgValue', 'return false !== stripos($cfgValue, "phar");'), |
||
| 523 | false, |
||
| 524 | 'suhosin.executor.include.whitelist must be configured correctly in php.ini', |
||
| 525 | 'Add "<strong>phar</strong>" to <strong>suhosin.executor.include.whitelist</strong> in php.ini<a href="#phpini">*</a>.' |
||
| 526 | ); |
||
| 527 | } |
||
| 528 | |||
| 529 | if (extension_loaded('xdebug')) { |
||
| 530 | $this->addPhpIniRequirement( |
||
| 531 | 'xdebug.show_exception_trace', false, true |
||
| 532 | ); |
||
| 533 | |||
| 534 | $this->addPhpIniRequirement( |
||
| 535 | 'xdebug.scream', false, true |
||
| 536 | ); |
||
| 537 | |||
| 538 | $this->addPhpIniRecommendation( |
||
| 539 | 'xdebug.max_nesting_level', |
||
| 540 | create_function('$cfgValue', 'return $cfgValue > 100;'), |
||
| 541 | true, |
||
| 542 | 'xdebug.max_nesting_level should be above 100 in php.ini', |
||
| 543 | 'Set "<strong>xdebug.max_nesting_level</strong>" to e.g. "<strong>250</strong>" in php.ini<a href="#phpini">*</a> to stop Xdebug\'s infinite recursion protection erroneously throwing a fatal error in your project.' |
||
| 544 | ); |
||
| 545 | } |
||
| 546 | |||
| 547 | $pcreVersion = defined('PCRE_VERSION') ? (float) PCRE_VERSION : null; |
||
| 548 | |||
| 549 | $this->addRequirement( |
||
| 550 | null !== $pcreVersion, |
||
| 551 | 'PCRE extension must be available', |
||
| 552 | 'Install the <strong>PCRE</strong> extension (version 8.0+).' |
||
| 553 | ); |
||
| 554 | |||
| 555 | if (extension_loaded('mbstring')) { |
||
| 556 | $this->addPhpIniRequirement( |
||
| 557 | 'mbstring.func_overload', |
||
| 558 | create_function('$cfgValue', 'return (int) $cfgValue === 0;'), |
||
| 559 | true, |
||
| 560 | 'string functions should not be overloaded', |
||
| 561 | 'Set "<strong>mbstring.func_overload</strong>" to <strong>0</strong> in php.ini<a href="#phpini">*</a> to disable function overloading by the mbstring extension.' |
||
| 562 | ); |
||
| 563 | } |
||
| 564 | |||
| 565 | /* optional recommendations follow */ |
||
| 566 | |||
| 567 | if (file_exists(__DIR__.'/../vendor/composer')) { |
||
| 568 | require_once __DIR__.'/../vendor/autoload.php'; |
||
| 569 | |||
| 570 | try { |
||
| 571 | $r = new ReflectionClass('Sensio\Bundle\DistributionBundle\SensioDistributionBundle'); |
||
| 572 | |||
| 573 | $contents = file_get_contents(dirname($r->getFileName()).'/Resources/skeleton/app/SymfonyRequirements.php'); |
||
| 574 | } catch (ReflectionException $e) { |
||
| 575 | $contents = ''; |
||
| 576 | } |
||
| 577 | $this->addRecommendation( |
||
| 578 | file_get_contents(__FILE__) === $contents, |
||
| 579 | 'Requirements file should be up-to-date', |
||
| 580 | 'Your requirements file is outdated. Run composer install and re-check your configuration.' |
||
| 581 | ); |
||
| 582 | } |
||
| 583 | |||
| 584 | $this->addRecommendation( |
||
| 585 | version_compare($installedPhpVersion, '5.3.4', '>='), |
||
| 586 | 'You should use at least PHP 5.3.4 due to PHP bug #52083 in earlier versions', |
||
| 587 | 'Your project might malfunction randomly due to PHP bug #52083 ("Notice: Trying to get property of non-object"). Install PHP 5.3.4 or newer.' |
||
| 588 | ); |
||
| 589 | |||
| 590 | $this->addRecommendation( |
||
| 591 | version_compare($installedPhpVersion, '5.3.8', '>='), |
||
| 592 | 'When using annotations you should have at least PHP 5.3.8 due to PHP bug #55156', |
||
| 593 | 'Install PHP 5.3.8 or newer if your project uses annotations.' |
||
| 594 | ); |
||
| 595 | |||
| 596 | $this->addRecommendation( |
||
| 597 | version_compare($installedPhpVersion, '5.4.0', '!='), |
||
| 598 | 'You should not use PHP 5.4.0 due to the PHP bug #61453', |
||
| 599 | 'Your project might not work properly due to the PHP bug #61453 ("Cannot dump definitions which have method calls"). Install PHP 5.4.1 or newer.' |
||
| 600 | ); |
||
| 601 | |||
| 602 | $this->addRecommendation( |
||
| 603 | version_compare($installedPhpVersion, '5.4.11', '>='), |
||
| 604 | 'When using the logout handler from the Symfony Security Component, you should have at least PHP 5.4.11 due to PHP bug #63379 (as a workaround, you can also set invalidate_session to false in the security logout handler configuration)', |
||
| 605 | 'Install PHP 5.4.11 or newer if your project uses the logout handler from the Symfony Security Component.' |
||
| 606 | ); |
||
| 607 | |||
| 608 | $this->addRecommendation( |
||
| 609 | (version_compare($installedPhpVersion, '5.3.18', '>=') && version_compare($installedPhpVersion, '5.4.0', '<')) |
||
| 610 | || |
||
| 611 | version_compare($installedPhpVersion, '5.4.8', '>='), |
||
| 612 | 'You should use PHP 5.3.18+ or PHP 5.4.8+ to always get nice error messages for fatal errors in the development environment due to PHP bug #61767/#60909', |
||
| 613 | 'Install PHP 5.3.18+ or PHP 5.4.8+ if you want nice error messages for all fatal errors in the development environment.' |
||
| 614 | ); |
||
| 615 | |||
| 616 | if (null !== $pcreVersion) { |
||
| 617 | $this->addRecommendation( |
||
| 618 | $pcreVersion >= 8.0, |
||
| 619 | sprintf('PCRE extension should be at least version 8.0 (%s installed)', $pcreVersion), |
||
| 620 | '<strong>PCRE 8.0+</strong> is preconfigured in PHP since 5.3.2 but you are using an outdated version of it. Symfony probably works anyway but it is recommended to upgrade your PCRE extension.' |
||
| 621 | ); |
||
| 622 | } |
||
| 623 | |||
| 624 | $this->addRecommendation( |
||
| 625 | class_exists('DomDocument'), |
||
| 626 | 'PHP-DOM and PHP-XML modules should be installed', |
||
| 627 | 'Install and enable the <strong>PHP-DOM</strong> and the <strong>PHP-XML</strong> modules.' |
||
| 628 | ); |
||
| 629 | |||
| 630 | $this->addRecommendation( |
||
| 631 | function_exists('mb_strlen'), |
||
| 632 | 'mb_strlen() should be available', |
||
| 633 | 'Install and enable the <strong>mbstring</strong> extension.' |
||
| 634 | ); |
||
| 635 | |||
| 636 | $this->addRecommendation( |
||
| 637 | function_exists('utf8_decode'), |
||
| 638 | 'utf8_decode() should be available', |
||
| 639 | 'Install and enable the <strong>XML</strong> extension.' |
||
| 640 | ); |
||
| 641 | |||
| 642 | $this->addRecommendation( |
||
| 643 | function_exists('filter_var'), |
||
| 644 | 'filter_var() should be available', |
||
| 645 | 'Install and enable the <strong>filter</strong> extension.' |
||
| 646 | ); |
||
| 647 | |||
| 648 | if (!defined('PHP_WINDOWS_VERSION_BUILD')) { |
||
| 649 | $this->addRecommendation( |
||
| 650 | function_exists('posix_isatty'), |
||
| 651 | 'posix_isatty() should be available', |
||
| 652 | 'Install and enable the <strong>php_posix</strong> extension (used to colorize the CLI output).' |
||
| 653 | ); |
||
| 654 | } |
||
| 655 | |||
| 656 | $this->addRecommendation( |
||
| 657 | extension_loaded('intl'), |
||
| 658 | 'intl extension should be available', |
||
| 659 | 'Install and enable the <strong>intl</strong> extension (used for validators).' |
||
| 660 | ); |
||
| 661 | |||
| 662 | if (extension_loaded('intl')) { |
||
| 663 | // in some WAMP server installations, new Collator() returns null |
||
| 664 | $this->addRecommendation( |
||
| 665 | null !== new Collator('fr_FR'), |
||
| 666 | 'intl extension should be correctly configured', |
||
| 667 | 'The intl extension does not behave properly. This problem is typical on PHP 5.3.X x64 WIN builds.' |
||
| 668 | ); |
||
| 669 | |||
| 670 | // check for compatible ICU versions (only done when you have the intl extension) |
||
| 671 | if (defined('INTL_ICU_VERSION')) { |
||
| 672 | $version = INTL_ICU_VERSION; |
||
| 673 | } else { |
||
| 674 | $reflector = new ReflectionExtension('intl'); |
||
| 675 | |||
| 676 | ob_start(); |
||
| 677 | $reflector->info(); |
||
| 678 | $output = strip_tags(ob_get_clean()); |
||
| 679 | |||
| 680 | preg_match('/^ICU version +(?:=> )?(.*)$/m', $output, $matches); |
||
| 681 | $version = $matches[1]; |
||
| 682 | } |
||
| 683 | |||
| 684 | $this->addRecommendation( |
||
| 685 | version_compare($version, '4.0', '>='), |
||
| 686 | 'intl ICU version should be at least 4+', |
||
| 687 | 'Upgrade your <strong>intl</strong> extension with a newer ICU version (4+).' |
||
| 688 | ); |
||
| 689 | |||
| 690 | if (class_exists('Symfony\Component\Intl\Intl')) { |
||
| 691 | $this->addRecommendation( |
||
| 692 | \Symfony\Component\Intl\Intl::getIcuDataVersion() <= \Symfony\Component\Intl\Intl::getIcuVersion(), |
||
| 693 | sprintf('intl ICU version installed on your system is outdated (%s) and does not match the ICU data bundled with Symfony (%s)', \Symfony\Component\Intl\Intl::getIcuVersion(), \Symfony\Component\Intl\Intl::getIcuDataVersion()), |
||
| 694 | 'To get the latest internationalization data upgrade the ICU system package and the intl PHP extension.' |
||
| 695 | ); |
||
| 696 | if (\Symfony\Component\Intl\Intl::getIcuDataVersion() <= \Symfony\Component\Intl\Intl::getIcuVersion()) { |
||
| 697 | $this->addRecommendation( |
||
| 698 | \Symfony\Component\Intl\Intl::getIcuDataVersion() === \Symfony\Component\Intl\Intl::getIcuVersion(), |
||
| 699 | sprintf('intl ICU version installed on your system (%s) does not match the ICU data bundled with Symfony (%s)', \Symfony\Component\Intl\Intl::getIcuVersion(), \Symfony\Component\Intl\Intl::getIcuDataVersion()), |
||
| 700 | 'To avoid internationalization data inconsistencies upgrade the symfony/intl component.' |
||
| 701 | ); |
||
| 702 | } |
||
| 703 | } |
||
| 704 | |||
| 705 | $this->addPhpIniRecommendation( |
||
| 706 | 'intl.error_level', |
||
| 707 | create_function('$cfgValue', 'return (int) $cfgValue === 0;'), |
||
| 708 | true, |
||
| 709 | 'intl.error_level should be 0 in php.ini', |
||
| 710 | 'Set "<strong>intl.error_level</strong>" to "<strong>0</strong>" in php.ini<a href="#phpini">*</a> to inhibit the messages when an error occurs in ICU functions.' |
||
| 711 | ); |
||
| 712 | } |
||
| 713 | |||
| 714 | $accelerator = |
||
| 715 | (extension_loaded('eaccelerator') && ini_get('eaccelerator.enable')) |
||
| 716 | || |
||
| 717 | (extension_loaded('apc') && ini_get('apc.enabled')) |
||
| 718 | || |
||
| 719 | (extension_loaded('Zend Optimizer+') && ini_get('zend_optimizerplus.enable')) |
||
| 720 | || |
||
| 721 | (extension_loaded('Zend OPcache') && ini_get('opcache.enable')) |
||
| 722 | || |
||
| 723 | (extension_loaded('xcache') && ini_get('xcache.cacher')) |
||
| 724 | || |
||
| 725 | (extension_loaded('wincache') && ini_get('wincache.ocenabled')) |
||
| 726 | ; |
||
| 727 | |||
| 728 | $this->addRecommendation( |
||
| 729 | $accelerator, |
||
| 730 | 'a PHP accelerator should be installed', |
||
| 731 | 'Install and/or enable a <strong>PHP accelerator</strong> (highly recommended).' |
||
| 732 | ); |
||
| 733 | |||
| 734 | if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') { |
||
| 735 | $this->addRecommendation( |
||
| 736 | $this->getRealpathCacheSize() >= 5 * 1024 * 1024, |
||
| 737 | 'realpath_cache_size should be at least 5M in php.ini', |
||
| 738 | 'Setting "<strong>realpath_cache_size</strong>" to e.g. "<strong>5242880</strong>" or "<strong>5M</strong>" in php.ini<a href="#phpini">*</a> may improve performance on Windows significantly in some cases.' |
||
| 739 | ); |
||
| 740 | } |
||
| 741 | |||
| 742 | $this->addPhpIniRecommendation('short_open_tag', false); |
||
| 743 | |||
| 744 | $this->addPhpIniRecommendation('magic_quotes_gpc', false, true); |
||
| 745 | |||
| 746 | $this->addPhpIniRecommendation('register_globals', false, true); |
||
| 747 | |||
| 748 | $this->addPhpIniRecommendation('session.auto_start', false); |
||
| 749 | |||
| 750 | $this->addRecommendation( |
||
| 751 | class_exists('PDO'), |
||
| 752 | 'PDO should be installed', |
||
| 753 | 'Install <strong>PDO</strong> (mandatory for Doctrine).' |
||
| 754 | ); |
||
| 755 | |||
| 756 | if (class_exists('PDO')) { |
||
| 757 | $drivers = PDO::getAvailableDrivers(); |
||
| 758 | $this->addRecommendation( |
||
| 759 | count($drivers) > 0, |
||
| 760 | sprintf('PDO should have some drivers installed (currently available: %s)', count($drivers) ? implode(', ', $drivers) : 'none'), |
||
| 761 | 'Install <strong>PDO drivers</strong> (mandatory for Doctrine).' |
||
| 762 | ); |
||
| 818 |