Feeling Lucky
Feature
PR-5.2-dev
[#43361] - [5.2] Refactor all instances of File::getExt() & stripExt() to use framework
- Fixed in Code Base
- 15 Jul 2024
- Medium
- Build: 5.2-dev
- # 43361
- Diff
- Hackwar:5.2-filesystem-getext
- Pending continuous-integration/drone/pr Build is pending Details
User tests: Successful: Unsuccessful:
Summary of Changes
We are in the process of migrating away from the CMS filesystem package to use the framework version of it. This PR converts all instances where File::getExt() & stripExt() is used to use the framework version.
Testing Instructions
Codereview.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_media com_templates Libraries Front End Plugins |
| Labels |
Added:
PR-5.2-dev
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-07-15 09:27:49 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
Feature
|
||
HLeithner
- comment
- 15 Jul 2024
thanks, merged by code review, both functions are identical between framework and cms version of filesystem\File and no other usages found in the changed files for this classes.
No testing instructions, touches several places of the test is untestable, better make prs for the same area and not per type.