[#33238] - [4][FTP Layer] use of is_writable when FTP Layer needed causes false error messages
- Closed
- 30 Apr 2021
- Medium
- Build: staging
- # 33238
Steps to reproduce the issue
Install Joomla 4 on a server that requires the use of the FTP Layer (see #31857)
Go to Joomla Global Config and save
Expected result
When checking if folder paths are writable, consideration should be given to the FTP Layer status.
Global configuration is saved
Green feedback message
Actual result
ApplicationModel.php uses is_writable to check paths, which returns false (as FTP is needed to write to folders!)
System information (as much as possible)
Additional comments
joomla-cms-bot
- | Labels |
Added:
?
|
||
56 occurrences of is_writable to manually review... to ensure each use is FTP Aware...
Dont waste your time
Im documenting the issues ... that's all... then no one can say it was not tested...
Easier to make a decision based on facts then... although good decisions are slow it seems...
@PhilETaylor before you go any further can you see if any of this is possible in J3. I don't believe it is as it failed when I tried. If that is correct then its even more of a waste of time to implement something in J4 if no one is currently using it
that was my plan after food later :)
I've put the FTP layer on the agenda for our JBS meeting tomorrow, and maybe our team lead can carry that to the PLT to discuss it there again.
So another month delay then :(
I want the opposite, I want to speed up decision and not cause another delay.
I.e. nobody should wait for that. But we should soon re-evaluate if we really need to keep the FTP layer and drop it if we can.
We have to wait. If the decision is to remove it then why waste time fixing something no one uses
and its been on the release blocker list for 3 years
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-04-30 10:37:09 |
| Closed_By | ⇒ | PhilETaylor |
Following the projects decision to remove the FTP Layer from Joomla 4 - this can now be closed.
This also effects the Template manager because in TemplateModel.php there is another
is_writablejoomla-cms/administrator/components/com_templates/src/Model/TemplateModel.php
Line 395 in 7752207