Skip to content

Conversation

@joomdonation
Copy link
Contributor

@joomdonation joomdonation commented Apr 24, 2021

Pull Request for Issue # .

Summary of Changes

During update process (using FTP or Hybrid installation method), Joomla! needs to write FTP credentials into restoration.php file so that these data can be used by restore.php on restoration process.

Due to a bug in getting FTP credentials (trying to get directly from input but these fields are not available in the input at all), update Joomla using these two methods never works (or at least it never works in the recent version). Also, the code in restore.php file does not like the path with window style, so I added some code to change it to Linux path style

Tested on my local Joomla 4 using Filezilla server and it worked. Not sure how to give testing instructions because now it is hard to find testers for FTP stuffs.

Testing Instructions

@brianteeman
Copy link
Contributor

Doesn't the same problem exist in Joomla 3? At least from your description of the reason it would appear so.

@joomdonation
Copy link
Contributor Author

Yes, same happens with Joomla 3. I just made the PR here quickly to prove that updates Joomla never works properly (at least for recent release, maybe from the time we added Upload and Update option back, I don't remember the time)

So the point here is if update is not working and no-one reports it, can we assume that no-one is using it? We can try to spend time to fix FTP issues, but if no one is using it, it will be a waste of time

@brianteeman
Copy link
Contributor

So the point here is if update is not working and no-one reports it, can we assume that no-one is using it? We can try to spend time to fix FTP issues, but if no one is using it, it will be a waste of time

That's what I've been trying to say but people keep sidetracking with conversations about code purity

@joomdonation
Copy link
Contributor Author

Hope this PR shows something. If update Joomla not working, if install extensions not working (as I saw somewhere from list of issues which Phil mentioned) and no-one complains, we should not have to spend time to fix it.

@PhilETaylor

This comment was marked as abuse.

@PhilETaylor

This comment was marked as abuse.

@joomdonation
Copy link
Contributor Author

Is this related to this Joomla 3 issue? #28199

Yes, I think so.

@PhilETaylor

This comment was marked as abuse.

@richard67
Copy link
Member

Drone resuscitated 😄 .

@joomdonation
Copy link
Contributor Author

Closing because FTP layer will be dropped.

@richard67
Copy link
Member

Closing because FTP layer will be dropped.

@joomdonation Who says that?

@PhilETaylor

This comment was marked as abuse.

@joomdonation
Copy link
Contributor Author

@richard67 See #33390 (comment)

@joomdonation joomdonation deleted the update_joomla_using_ftp_fix branch May 13, 2021 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants