From 5311fac571d0661828013c364cc6074204fa2e74 Mon Sep 17 00:00:00 2001 From: Wilmer Arambula Date: Fri, 22 Sep 2023 18:19:59 -0300 Subject: [PATCH] Remove deprecated property to `FileTarget::class`. --- framework/log/FileTarget.php | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/framework/log/FileTarget.php b/framework/log/FileTarget.php index d816f73a8db..29e76d470c6 100644 --- a/framework/log/FileTarget.php +++ b/framework/log/FileTarget.php @@ -20,9 +20,6 @@ * files are moved backwards by one place, i.e., '.2' to '.3', '.1' to '.2', and so on. * The property [[maxLogFiles]] specifies how many history files to keep. * - * Since 2.0.46 rotation of the files is done only by copy and the - * `rotateByCopy` property is deprecated. - * * @author Qiang Xue * @since 2.0 */ @@ -61,22 +58,6 @@ class FileTarget extends Target * but read-only for other users. */ public $dirMode = 0775; - /** - * @var bool Whether to rotate log files by copy and truncate in contrast to rotation by - * renaming files. Defaults to `true` to be more compatible with log tailers and windows - * systems which do not play well with rename on open files. Rotation by renaming however is - * a bit faster. - * - * The problem with windows systems where the [rename()](https://www.php.net/manual/en/function.rename.php) - * function does not work with files that are opened by some process is described in a - * [comment by Martin Pelletier](https://www.php.net/manual/en/function.rename.php#102274) in - * the PHP documentation. By setting rotateByCopy to `true` you can work - * around this. - * @deprecated since 2.0.46 and setting it to false has no effect anymore - * since rotating is now always done by copy. - */ - public $rotateByCopy = true; - /** * Initializes the route.