User tests: Successful: Unsuccessful:
This removes the GUI parts of the FTP layer. This PR is the counterpart to #31857. The PLT needs to decide if it wants to either merge this or the other PR.
This PR is not entirely complete. There is still FTP code in the joomlaupdate component and the FTP layer has not been removed from the libraries folder.
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_admin com_config com_installer com_languages com_templates Language & Strings Installation |
Labels |
Added:
?
?
|
Stop picking hairs in everything. @Hackwar is ALSO quoted as saying 4 days ago that his other PR to FIX the FTP Layer was him volunteering
to execute decisions the PLT has made
So if the PLT made a decision (communicated only to Hackwar it appears) to FIX the FTP layer why then is this PR here from the same person to remove the FTP Layer - it makes no sense
It is a genuine question that requires answering.
The last PUBLIC decision made on the FTP Layer that I can find is now almost a year out of date, from @wilsonge, who said the FTP Layer was being kept. Unless that decision has changed, then there is no need for this PR. And if that decision has been changed IT MUST BE MADE PUBLIC and not just shared with a select few people.
I have now worked almost a week this month, exclusively, working on fixing issues in the FTP Layer, to remove release blocker for Joomla 4... it would be nice to actually have some leadership announcing their decisions made in private.
I would ask "the PLT Team" directly - however https://volunteers.joomla.org/departments/production-leadership-team seems to not display ANY members and the last Department Joomla 4 report there was from 2017... hahaha
@Hackwar "Hannes Papenberg" is listed here https://volunteers.joomla.org/departments/production as a Department Member, so one would hope that he actually knows what his leadership team decisions are?? Maybe not though. But here https://volunteers.joomla.org/departments/production/reports/1475-production-dept-meeting-minutes-march-23-2021 he is listed as a "Team Leader" so surely he is part of the team actually making the decisions?
I personally emailed Marco Dings many weeks ago - and did not receive a reply on this subject.
Looking back at my historic emails, I have never received an email reply from him over the years. Always ignored.
Im doing all I can to move Joomla 4 closer to being released, as the FTP Layer has been repeatedly listed as a Release Blocker...
@PhilETaylor The PLT decided some time ago to keep the FTP layer (I'd have to look through the meeting minutes to find the exact date) Some time after that, I volunteered to fix the installation via FTP and that is the PR that you already participated in. Thanks to your meta issue about all the issues in the FTP layer, I looked into those issues and decided that indeed the FTP layer is out of date and should probably be removed. I brought this up in the PLT almost a week ago (on the 23rd to be exact) and the few people who answered, followed mine (and your) opinion on this.
However, at this moment, there is no official decision by the PLT. As long as there is no official decision, the old one stands to keep the FTP layer. I have no idea when that decision will be made.
Whichever way the PLT decides, I want to not waste more time and started this PR. If we decide to keep the FTP layer, my old PR would be there. If we decide to drop it, this PR here could be merged. Yes, both aren't 100% yet, but I'm hoping that we can save at least some time with this work.
The next PLT meeting is Tuesday, May 4th.
Believe me, that I, too, am trying to push J4 closer to release.
Awesome thats.
followed mine (and your) opinion on this.
My opinion is just to get whatever needs doing done - but - that needs a new reiteration of the decision to keep or remove the FTP layer and not based on a year old decision - so that no one wastes weeks of time only to be told it is going to be removed. I guess my opinion is that is should be removed as a FTP layer, especially as on investigation its "mainly broken"
The next PLT meeting is Tuesday, May 4th.
I will officially suspend all my (published and yet unpublished) work on the FTP Layer pending the May 4th Meeting.
The only exception to that will be to retain and keep up to date the Summary Issue #33251 so that a clearer picture of the issues involved is kept in peoples minds (instead of believing like George did that "it mostly works" even though he admits he has never used or tested it).
Thank you for the clarity your post has given.
You're welcome. I've asked to come to a decision regarding this earlier than next meeting.
A motion regarding this is currently being voted on by mail in the PLT.
Thanks for the update. Will the votes cast, by who and which way, be published also?
And is the motion (as I would assume) to Keep the FTP layer or choose to remove it? or does it have some crazy motion text? Is it a binary choice being made?
I don't think that making the votes public would provide any benefit for any of us, but if you insist on it, I can bring this up in the next meeting.
The motion is open either way. At the end of the voting, we will have a clear answer to either keep or remove the FTP layer.
To make this clear: This would remove the FTP layer from the CMS. We would keep it in the Filesystem package for now in case someone would still use it, but it would not be available for installation of Joomla or any of the core components filesystem operations. In that regard, this PR would even be done, I think... I've kept my fingers from the com_joomlaupdate component...
Thanks.
There is not any FTP in the Filesystem FRAMEWORK package (eg: https://github.com/joomla-framework/filesystem/blob/master/src/File.php) but I guess you meant the package in the CMS...
Some optimisation can be done in the CMS Filesystem package if FTP is no longer going to be part of the CMS, for performance and logic reasons, mainly caching the FTPOptions from ClientHelper so that we dont have to keep going there, checking the config, checking the session, just to return enabled = 0 each time.
I've kept my fingers from the com_joomlaupdate component...
Its not so much the com_joomlaupdate, its the restore.php that's a large issue :) I think Im the only one brave enough in modern history to make large changes to Nic's code ;-)
The whole class AKPostprocFTP
could be removed eventually.. but boy will that be a painful day in the office, and getting people to test :)
I think I'm the only one brave enough in modern history to make large changes to Nic's code
You might not be the only one. I even tried to debug it to try to fix update using FTP on Windows , lol.
The whole class AKPostprocFTP could be removed eventually.. but boy will that be a painful day in the office, and getting people to test :)
Guess we could remove AKPostprocHybrid, AKPostprocSFTP, too. And test that is not hard. We just need to test and make sure update using Direct method works and it will be fine.
The votes have been cast and the decision is to drop the FTP layer from the system. This means that the CMS does not provide an FTP fallback during installation of the system itself or when writing files in Joomla core components. The code in the CMS Filesystem package will stay in case third party extensions are depending on this. We have not made a decision if that code should now be marked as deprecated or not. Since that would only be possible in Joomla 5.0 anyways, I don't think we have to decide on this now.
With that said, this PR would need to be tested and reviewed.
Please can this be announced officially in some place other than a comment in a GitHub issue buried with all the other issues and comments please...
Im sure I'll not be the first to say - PHEW.
Thanks.
@PhilETaylor it was a production motion "PROD2021/002 PASSED" I expect that this will be published soon.
@HLeithner Thanks - was it a close vote ? hahaaha
@PhilETaylor Not really only one vote against it.
@HLeithner was there also a decision for @ciar4n 's PR, the Atum repaint?
@HLeithner was there also a decision for @ciar4n 's PR, the Atum repaint?
That's a bit off topic here, at least we talked about it and some actions have been done, I would expect more information soon.
@PhilETaylor it was a production motion "PROD2021/002 PASSED" I expect that this will be published soon.
Is it off topic to ask if the registry of motions will be updated with all the motions from the last 12 months
@brianteeman of course it's off topic but I think you know this ;-) I posted your request to production and hope it will be updated soon. ( I can't do more)
@HLeithner thanks
I would ask "the PLT Team" directly - however https://volunteers.joomla.org/departments/production-leadership-team seems to not display ANY members and the last Department Joomla 4 report there was from 2017... hahaha
Just fwiw although Production is behind it's not that bad - not quite sure how the URLs work on the volunteer portal - but this has the more up-to-date reports https://volunteers.joomla.org/departments/production (I guess the other link is the old PLT stuff before Joomla did the restructure)
@brianteeman of course it's off topic but I think you know this ;-) I posted your request to production and hope it will be updated soon. ( I can't do more)
The registry is located here and updated by Production or Luca and Me, when we receive the information.
https://www.opensourcematters.org/organisation/team-membership/registry-of-motions/289-2020.html#production-department-motions-2020
Just send the list and it will be updated.
Labels |
Added:
?
|
@brianteeman done
@joomdonation Thank you.
Labels |
Added:
?
Removed: ? |
@joomdonation Agreed. I've reverted that.
If we can get 2 positive tests for this then, we can merge this.
I have tested this item
- Code review
All works as expected. So it is good.
I have tested this item
@richard67 This PR could be set to RTC now.
@richard67 This PR could be set to RTC now.
No need to extra ping me, I am checking every now and then if there are PR's which have 2 tests and would have found it sooner or later.
Drone failure is not related to this PR.
@richard67 OK, will remember that :).
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
?
?
Removed: ? |
Labels |
Added:
?
?
Removed: ? ? |
Labels |
Added:
?
?
Removed: ? ? |
I've updated the branch to latest 4.0-dev to get rid of the unrelated javascript-cs error in drone and restored the previous test results in the issue tracker.
Labels |
Added:
?
?
Removed: ? ? |
Labels |
Added:
?
?
Removed: ? ? |
thx
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-03 15:33:00 |
Closed_By | ⇒ | wilsonge | |
Labels |
Added:
?
?
Removed: ? ? |
Thanks!
Is this implementing a "decision" or is this a speculative PR?