[#33283] - [4] Use CMS library which is FTP Layer aware
- Fixed in Code Base
- 24 Apr 2021
- Medium
- Build: 4.0-dev
- # 33283
- Diff
- PhilETaylor:patch-136
User tests: Successful: Unsuccessful:
Summary of Changes
Logrotation currently uses the Joomla framework Filesystem package which is not FTP Layer aware.
This leads to issues when log rotation attempts to move a file when it has no permissions to do so, when FTP is needed.
This change starts using the Folder and File packages from the CMS and not the framework, although Folder doesn't yet need to be FTP aware, Im changing that so that if we ever did make Folder FTP Aware (hint: probably soon coming!) then it will "just work" in future.
Testing Instructions
Code review, unless you have a spare hour to follow along the whole FTP Layer set up.
Actual result BEFORE applying this Pull Request
After installation, and clocking on the Open Site button on the last page of the installation wizard
Expected result AFTER applying this Pull Request
Use the CMS Filesystem lib, which is FTP aware, and can move the file with FTP.
Documentation Changes Required
none
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-24 21:59:14 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
Thanks!