Skip to content

Commit 1ca470f

Browse files
committed
refactor!: remove deprecated Logger::cleanFilenames() and TestLogger::cleanup()
1 parent 6316c15 commit 1ca470f

File tree

3 files changed

+3
-29
lines changed

3 files changed

+3
-29
lines changed

system/Log/Logger.php

-15
Original file line numberDiff line numberDiff line change
@@ -410,19 +410,4 @@ public function determineFile(): array
410410
'unknown',
411411
];
412412
}
413-
414-
/**
415-
* Cleans the paths of filenames by replacing APPPATH, SYSTEMPATH, FCPATH
416-
* with the actual var. i.e.
417-
*
418-
* /var/www/site/app/Controllers/Home.php
419-
* becomes:
420-
* APPPATH/Controllers/Home.php
421-
*
422-
* @deprecated Use dedicated `clean_path()` function.
423-
*/
424-
protected function cleanFileNames(string $file): string
425-
{
426-
return clean_path($file);
427-
}
428413
}

system/Test/TestLogger.php

-14
Original file line numberDiff line numberDiff line change
@@ -92,18 +92,4 @@ public static function didLog(string $level, $message, bool $useExactComparison
9292

9393
return false;
9494
}
95-
96-
/**
97-
* Expose filenames.
98-
*
99-
* @param string $file
100-
*
101-
* @return string
102-
*
103-
* @deprecated No longer needed as underlying protected method is also deprecated.
104-
*/
105-
public function cleanup($file)
106-
{
107-
return clean_path($file);
108-
}
10995
}

user_guide_src/source/changelogs/v4.6.0.rst

+3
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,9 @@ Removed Deprecated Items
3838
has been removed. Use ``failValidationErrors()`` instead.
3939
- **HTTP:** The deprecated ``getReason()`` method in ``CodeIgniter\HTTP\Response``
4040
and ``ResponseInterface`` has been removed. Use ``getReasonPhrase()`` instead.
41+
- **Logger:** The deprecated ``CodeIgniter\Log\Logger::cleanFilenames()`` and
42+
``CodeIgniter\Test\TestLogger::cleanup()`` have been removed. Use the
43+
``clean_path()`` function instead.
4144

4245
************
4346
Enhancements

0 commit comments

Comments
 (0)