mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 14:05:37 +02:00
Compare commits
2 Commits
edf8a32884
...
5f5cf480d0
Author | SHA1 | Date | |
---|---|---|---|
|
5f5cf480d0 | ||
|
9cc00719c7 |
@ -36,9 +36,8 @@
|
|||||||
"psr/container": "^1.1 || ^2.0",
|
"psr/container": "^1.1 || ^2.0",
|
||||||
"psr/http-message": "^1",
|
"psr/http-message": "^1",
|
||||||
"sebastian/diff": "^4.0",
|
"sebastian/diff": "^4.0",
|
||||||
"silverstripe/config": "^2.2",
|
"silverstripe/config": "^2",
|
||||||
"silverstripe/assets": "^2.3",
|
"silverstripe/assets": "^2.3",
|
||||||
"silverstripe/supported-modules": "^1",
|
|
||||||
"silverstripe/vendor-plugin": "^2",
|
"silverstripe/vendor-plugin": "^2",
|
||||||
"sminnee/callbacklist": "^0.1.1",
|
"sminnee/callbacklist": "^0.1.1",
|
||||||
"symfony/cache": "^6.1",
|
"symfony/cache": "^6.1",
|
||||||
|
@ -19,6 +19,7 @@ use SilverStripe\View\TemplateGlobalProvider;
|
|||||||
*/
|
*/
|
||||||
class Controller extends RequestHandler implements TemplateGlobalProvider
|
class Controller extends RequestHandler implements TemplateGlobalProvider
|
||||||
{
|
{
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* An array of arguments extracted from the URL.
|
* An array of arguments extracted from the URL.
|
||||||
*
|
*
|
||||||
|
@ -3,13 +3,11 @@
|
|||||||
namespace SilverStripe\Dev;
|
namespace SilverStripe\Dev;
|
||||||
|
|
||||||
use BadMethodCallException;
|
use BadMethodCallException;
|
||||||
use RuntimeException;
|
|
||||||
use SilverStripe\Control\Director;
|
use SilverStripe\Control\Director;
|
||||||
use SilverStripe\Core\Environment;
|
use SilverStripe\Core\Environment;
|
||||||
use SilverStripe\Core\Injector\InjectionCreator;
|
use SilverStripe\Core\Injector\InjectionCreator;
|
||||||
use SilverStripe\Core\Injector\InjectorLoader;
|
use SilverStripe\Core\Injector\InjectorLoader;
|
||||||
use SilverStripe\Core\Manifest\Module;
|
use SilverStripe\Core\Manifest\Module;
|
||||||
use SilverStripe\Core\Path;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Handles raising an notice when accessing a deprecated method, class, configuration, or behaviour.
|
* Handles raising an notice when accessing a deprecated method, class, configuration, or behaviour.
|
||||||
@ -82,14 +80,7 @@ class Deprecation
|
|||||||
/**
|
/**
|
||||||
* @internal
|
* @internal
|
||||||
*/
|
*/
|
||||||
private static bool $showNoticesCalledFromSupportedCode = false;
|
private static bool $showCalledFromSupportedCodeNotices = false;
|
||||||
|
|
||||||
/**
|
|
||||||
* Cache of supported module directories, read from silverstripe/supported-modules repositories.json
|
|
||||||
*
|
|
||||||
* @internal
|
|
||||||
*/
|
|
||||||
private static array $supportedModuleDirectories = [];
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Enable throwing deprecation warnings. By default, this excludes warnings for
|
* Enable throwing deprecation warnings. By default, this excludes warnings for
|
||||||
@ -167,11 +158,11 @@ class Deprecation
|
|||||||
private static function get_called_from_trace(array $backtrace, int $level): array
|
private static function get_called_from_trace(array $backtrace, int $level): array
|
||||||
{
|
{
|
||||||
$newLevel = $level;
|
$newLevel = $level;
|
||||||
// handle closures inside withSuppressedNotice()
|
if (Deprecation::$insideNoticeSuppression) {
|
||||||
if (Deprecation::$insideNoticeSuppression
|
// handle closures inside withSuppressedNotice()
|
||||||
&& substr($backtrace[$newLevel]['function'], -strlen('{closure}')) === '{closure}'
|
if (substr($backtrace[$newLevel]['function'], -strlen('{closure}')) === '{closure}') {
|
||||||
) {
|
$newLevel = $newLevel + 2;
|
||||||
$newLevel = $newLevel + 2;
|
}
|
||||||
}
|
}
|
||||||
// handle call_user_func
|
// handle call_user_func
|
||||||
if ($level === 4 && strpos($backtrace[2]['function'] ?? '', 'call_user_func') !== false) {
|
if ($level === 4 && strpos($backtrace[2]['function'] ?? '', 'call_user_func') !== false) {
|
||||||
@ -196,46 +187,40 @@ class Deprecation
|
|||||||
return $called;
|
return $called;
|
||||||
}
|
}
|
||||||
|
|
||||||
private static function isCalledFromSupportedCode(array $backtrace): bool
|
private static function calledFromSupportedCode(array $backtrace): bool
|
||||||
{
|
{
|
||||||
$called = Deprecation::get_called_from_trace($backtrace, 1);
|
$called = Deprecation::get_called_from_trace($backtrace, 1);
|
||||||
$file = $called['file'] ?? '';
|
$file = $called['file'] ?? '';
|
||||||
if (!$file) {
|
if ($file) {
|
||||||
return false;
|
$isSupportedVendorFile = Deprecation::isSupportedVendorFile($file);
|
||||||
|
if (!$isSupportedVendorFile) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
return Deprecation::fileIsInSupportedModule($file);
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Check whether a file (path to file) is in a supported module
|
* Whether the given file is supported code
|
||||||
*/
|
*/
|
||||||
public static function fileIsInSupportedModule(string $file): bool
|
private static function isSupportedVendorFile(string $file): bool
|
||||||
{
|
{
|
||||||
// Cache the supported modules list
|
// This is a special case for silverstripe-framework when running in CI
|
||||||
if (count(Deprecation::$supportedModuleDirectories) === 0) {
|
if (str_contains($file, '/silverstripe-framework/')) {
|
||||||
// Manually load the supported modules list rather than use MetaData::getAllRepositoryMetaData()
|
return true;
|
||||||
// because we do not want to make a network request which could slow down a website
|
|
||||||
// While there is a small risk of the list being out of date, there is minimal downside to this
|
|
||||||
$path = Path::join(BASE_PATH, 'vendor/silverstripe/supported-modules/repositories.json');
|
|
||||||
if (!file_exists($path)) {
|
|
||||||
throw new RuntimeException('Could not find supported modules list');
|
|
||||||
}
|
|
||||||
$json = json_decode(file_get_contents($path), true);
|
|
||||||
if (is_null($json)) {
|
|
||||||
throw new RuntimeException('Could not parse supported modules list');
|
|
||||||
}
|
|
||||||
$dirs = array_map(fn($module) => "/vendor/{$module['packagist']}/", $json['supportedModules']);
|
|
||||||
// This is a special case for silverstripe-framework when running in CI
|
|
||||||
// Needed because module is run in the root folder rather than in the vendor folder
|
|
||||||
$dirs[] = '/silverstripe-framework/';
|
|
||||||
Deprecation::$supportedModuleDirectories = $dirs;
|
|
||||||
}
|
}
|
||||||
foreach (Deprecation::$supportedModuleDirectories as $dir) {
|
// Doing a fairly simple check to see if a file is in a supported vendor folder, rather than whether
|
||||||
if (str_contains($file, $dir)) {
|
// the module itself is actually supported
|
||||||
return true;
|
$vendors = implode('|', [
|
||||||
}
|
'bringyourownideas',
|
||||||
}
|
'colymba',
|
||||||
return false;
|
'cwp',
|
||||||
|
'dnadesign',
|
||||||
|
'silverstripe',
|
||||||
|
'symbiote',
|
||||||
|
'tractorcow',
|
||||||
|
]);
|
||||||
|
return (bool) preg_match("#/vendor/($vendors)/#", $file);
|
||||||
}
|
}
|
||||||
|
|
||||||
public static function isEnabled(): bool
|
public static function isEnabled(): bool
|
||||||
@ -319,17 +304,14 @@ class Deprecation
|
|||||||
/**
|
/**
|
||||||
* If true, deprecation warnings will be shown for deprecated code which is called by core Silverstripe modules.
|
* If true, deprecation warnings will be shown for deprecated code which is called by core Silverstripe modules.
|
||||||
*/
|
*/
|
||||||
public static function getShowNoticesCalledFromSupportedCode(): bool
|
public static function getShowCalledFromSupportedCodeNotices(): bool
|
||||||
{
|
{
|
||||||
return Deprecation::$showNoticesCalledFromSupportedCode;
|
return Deprecation::$showCalledFromSupportedCodeNotices;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
public static function setShowCalledFromSupportedCodeNotices(bool $value): void
|
||||||
* Set whether deprecation warnings will be shown for deprecated code which is called by core Silverstripe modules.
|
|
||||||
*/
|
|
||||||
public static function setShowNoticesCalledFromSupportedCode(bool $value): void
|
|
||||||
{
|
{
|
||||||
Deprecation::$showNoticesCalledFromSupportedCode = $value;
|
Deprecation::$showCalledFromSupportedCodeNotices = $value;
|
||||||
}
|
}
|
||||||
|
|
||||||
public static function outputNotices(): void
|
public static function outputNotices(): void
|
||||||
@ -345,11 +327,11 @@ class Deprecation
|
|||||||
$arr = array_shift(Deprecation::$userErrorMessageBuffer);
|
$arr = array_shift(Deprecation::$userErrorMessageBuffer);
|
||||||
$message = $arr['message'];
|
$message = $arr['message'];
|
||||||
$calledWithNoticeSuppression = $arr['calledWithNoticeSuppression'];
|
$calledWithNoticeSuppression = $arr['calledWithNoticeSuppression'];
|
||||||
$isCalledFromSupportedCode = $arr['isCalledFromSupportedCode'];
|
$calledFromSupportedCode = $arr['calledFromSupportedCode'];
|
||||||
if ($calledWithNoticeSuppression && !Deprecation::$showNoReplacementNotices) {
|
if ($calledWithNoticeSuppression && !Deprecation::$showNoReplacementNotices) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if ($isCalledFromSupportedCode && !Deprecation::$showNoticesCalledFromSupportedCode) {
|
if ($calledFromSupportedCode && !Deprecation::$showCalledFromSupportedCodeNotices) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
Deprecation::$isTriggeringError = true;
|
Deprecation::$isTriggeringError = true;
|
||||||
@ -385,10 +367,7 @@ class Deprecation
|
|||||||
$data = [
|
$data = [
|
||||||
'key' => sha1($string),
|
'key' => sha1($string),
|
||||||
'message' => $string,
|
'message' => $string,
|
||||||
// Setting to `false` as here as any SCOPE_CONFIG notices from supported modules have
|
'calledFromSupportedCode' => false,
|
||||||
// already been filtered out if needed if they came from a supported module in
|
|
||||||
// SilverStripe\Config\Transformer\YamlTransformer::checkForDeprecatedConfig()
|
|
||||||
'isCalledFromSupportedCode' => false,
|
|
||||||
'calledWithNoticeSuppression' => Deprecation::$insideNoticeSuppression
|
'calledWithNoticeSuppression' => Deprecation::$insideNoticeSuppression
|
||||||
];
|
];
|
||||||
} else {
|
} else {
|
||||||
@ -423,7 +402,7 @@ class Deprecation
|
|||||||
$data = [
|
$data = [
|
||||||
'key' => sha1($string),
|
'key' => sha1($string),
|
||||||
'message' => $string,
|
'message' => $string,
|
||||||
'isCalledFromSupportedCode' => Deprecation::isCalledFromSupportedCode($backtrace),
|
'calledFromSupportedCode' => Deprecation::calledFromSupportedCode($backtrace),
|
||||||
'calledWithNoticeSuppression' => Deprecation::$insideNoticeSuppression
|
'calledWithNoticeSuppression' => Deprecation::$insideNoticeSuppression
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
@ -457,7 +436,8 @@ class Deprecation
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Shorthand method to create a suppressed notice for something with no immediate replacement.
|
* Shorthand method to create a suppressed notice for something with no immediate replacement.
|
||||||
* If $string is empty, then a standardised message will be used
|
* If $string is empty, then a standardised message will be used, which is:
|
||||||
|
* Will be removed without equivalent functionality to replace it.
|
||||||
*/
|
*/
|
||||||
public static function noticeWithNoReplacment(
|
public static function noticeWithNoReplacment(
|
||||||
string $atVersion,
|
string $atVersion,
|
||||||
|
@ -33,7 +33,7 @@ class DeprecationTest extends SapphireTest
|
|||||||
// https://github.com/laminas/laminas-di/pull/30#issuecomment-927585210
|
// https://github.com/laminas/laminas-di/pull/30#issuecomment-927585210
|
||||||
parent::setup();
|
parent::setup();
|
||||||
$this->noticesWereEnabled = Deprecation::isEnabled();
|
$this->noticesWereEnabled = Deprecation::isEnabled();
|
||||||
$this->showSupportedNoticesWasEnabled = Deprecation::getShowNoticesCalledFromSupportedCode();
|
$this->showSupportedNoticesWasEnabled = Deprecation::getShowCalledFromSupportedCodeNotices();
|
||||||
$this->oldHandler = set_error_handler(function (int $errno, string $errstr, string $errfile, int $errline) {
|
$this->oldHandler = set_error_handler(function (int $errno, string $errstr, string $errfile, int $errline) {
|
||||||
if ($errno === E_USER_DEPRECATED) {
|
if ($errno === E_USER_DEPRECATED) {
|
||||||
if (str_contains($errstr, 'SilverStripe\\Dev\\Tests\\DeprecationTest')) {
|
if (str_contains($errstr, 'SilverStripe\\Dev\\Tests\\DeprecationTest')) {
|
||||||
@ -49,7 +49,9 @@ class DeprecationTest extends SapphireTest
|
|||||||
// Fallback to default PHP error handler
|
// Fallback to default PHP error handler
|
||||||
return false;
|
return false;
|
||||||
});
|
});
|
||||||
// This is required to clear out the notice from instantiating DeprecationTestObject in TableBuilder::buildTables().
|
// This is required to clear out the message
|
||||||
|
// 'SilverStripe\Dev\Tests\DeprecationTest\DeprecationTestObject is deprecated. Some class message. Called from
|
||||||
|
// SilverStripe\ORM\Connect\TableBuilder->buildTables."
|
||||||
Deprecation::outputNotices();
|
Deprecation::outputNotices();
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -60,7 +62,7 @@ class DeprecationTest extends SapphireTest
|
|||||||
} else {
|
} else {
|
||||||
Deprecation::disable();
|
Deprecation::disable();
|
||||||
}
|
}
|
||||||
Deprecation::setShowNoticesCalledFromSupportedCode($this->showSupportedNoticesWasEnabled);
|
Deprecation::setShowCalledFromSupportedCodeNotices($this->showSupportedNoticesWasEnabled);
|
||||||
restore_error_handler();
|
restore_error_handler();
|
||||||
$this->oldHandler = null;
|
$this->oldHandler = null;
|
||||||
parent::tearDown();
|
parent::tearDown();
|
||||||
@ -81,7 +83,7 @@ class DeprecationTest extends SapphireTest
|
|||||||
private function enableDeprecationNotices(bool $showNoReplacementNotices = false): void
|
private function enableDeprecationNotices(bool $showNoReplacementNotices = false): void
|
||||||
{
|
{
|
||||||
Deprecation::enable($showNoReplacementNotices);
|
Deprecation::enable($showNoReplacementNotices);
|
||||||
Deprecation::setShowNoticesCalledFromSupportedCode(true);
|
Deprecation::setShowCalledFromSupportedCodeNotices(true);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testNotice()
|
public function testNotice()
|
||||||
@ -258,11 +260,11 @@ class DeprecationTest extends SapphireTest
|
|||||||
Deprecation::outputNotices();
|
Deprecation::outputNotices();
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testshowNoticesCalledFromSupportedCode()
|
public function testShowCalledFromSupportedCodeNotices()
|
||||||
{
|
{
|
||||||
$this->expectNotToPerformAssertions();
|
$this->expectNotToPerformAssertions();
|
||||||
$this->enableDeprecationNotices(true);
|
$this->enableDeprecationNotices(true);
|
||||||
// showNoticesCalledFromSupportedCode is set to true by default for these unit tests
|
// showCalledFromSupportedCodeNotices is set to true by default for these unit tests
|
||||||
// as it is testing code within vendor/silverstripe
|
// as it is testing code within vendor/silverstripe
|
||||||
// This test is to ensure that the method works as expected when we disable this
|
// This test is to ensure that the method works as expected when we disable this
|
||||||
// and we should expect no exceptions to be thrown
|
// and we should expect no exceptions to be thrown
|
||||||
@ -272,7 +274,7 @@ class DeprecationTest extends SapphireTest
|
|||||||
// Deprecation::withSuppressedNotice(function () {
|
// Deprecation::withSuppressedNotice(function () {
|
||||||
// Deprecation::notice('123', 'My message.');
|
// Deprecation::notice('123', 'My message.');
|
||||||
// });
|
// });
|
||||||
Deprecation::setShowNoticesCalledFromSupportedCode(false);
|
Deprecation::setShowCalledFromSupportedCodeNotices(false);
|
||||||
// notice()
|
// notice()
|
||||||
$this->myDeprecatedMethod();
|
$this->myDeprecatedMethod();
|
||||||
// noticeNoReplacement()
|
// noticeNoReplacement()
|
||||||
|
Loading…
Reference in New Issue
Block a user