? Success
Related to # 4703

User tests: Successful: Unsuccessful:

avatar donald-pg
donald-pg
8 Nov 2014

Thanks to the help of Chris D, we determined the real issue is that some redundant code is in this file for FTP but that screen has been removed during the installation.

edit (by @zero-24)

Testinstructions

Install Joomla on two webservers, one requiring FTP to be used, the other not requiring FTP to be used.
Upon completion in both situations:

  • the installation folder must be removed without problem.
  • Verify content of robots.txt, unchanged if existing before installation, otherwise equal to content of robots.txt.dist.
  • no joomla.xml may exist in root folder.
avatar donald-pg donald-pg - open - 8 Nov 2014
avatar jissues-bot jissues-bot - change - 8 Nov 2014
Labels Added: ?
avatar donald-pg
donald-pg - comment - 8 Nov 2014

See #4703

avatar roland-d roland-d - change - 8 Nov 2014
Rel_Number 4703
Relation Type Related to
avatar zero-24
zero-24 - comment - 9 Nov 2014

@donald-pg can you add testinstructions? I'm not sure how i can test this. Thanks

avatar zero-24 zero-24 - change - 9 Nov 2014
Category Installation
avatar sovainfo
sovainfo - comment - 9 Nov 2014

Testinstructions:
Install Joomla on two webservers, one requiring FTP to be used, the other not requiring FTP to be used.
Upon completion in both situations:

  • the installation folder must be removed without problem.
  • Verify content of robots.txt, unchanged if existing before installation, otherwise equal to content of robots.txt.dist.
    • no joomla.xml may exist in root folder.
avatar zero-24 zero-24 - change - 9 Nov 2014
The description was changed
avatar zero-24 zero-24 - change - 9 Nov 2014
The description was changed
avatar donald-pg
donald-pg - comment - 10 Nov 2014

Since configuring FTP during the installation process has been removed so there is no way to get to the remove installation folder part with FTP information loaded. I'm assuming we need to modify the text string to let people know they can't remove the installation folder in certain conditions (where you need FTP) as removing the installation folder button when needing FTP has not worked since the FTP part has been removed during the initial setup.

avatar donald-pg
donald-pg - comment - 10 Nov 2014

That being said...try the Remove Installation Folder button. It should be exactly the same as before as I just removed code that is no longer being executed or used.

avatar zero-24
zero-24 - comment - 10 Nov 2014

@donald-pg

Since configuring FTP during the installation process has been removed so there is no way to get to the remove installation folder part with FTP information loaded.

It is only hidden if joomla detect a server that FTP require it will disply it. See: #1980

avatar donald-pg
donald-pg - comment - 10 Nov 2014

Thanks. I'll revise the patch then. I was never able to see the FTP options and several others at the bug squashing session couldn't see the FTP screen so we all assumed it had been removed. I'll have to redo my patch to load the options as right now, the version that is out now, it will never drop into the FTP code because the options variable is never setup in that function.

It may take me a little bit to reconfigure my local environment to get the FTP options to show so I can reproduce the original problem of #4703 then fix it.

Thanks again for the info.

avatar Hackwar
Hackwar - comment - 14 Nov 2014

I have to say that I don't understand why all that code is necessary to delete that stuff... JFolder/JFile should be aware of FTP if it is set. So instead of having dozens of checks, calling JFolder/JFile directly should be enough, as @donald-pg has done. The only thing that we might have to do first, is to set the global config data regarding the FTP in case JFolder/JFile does not have access to that in the installation application automatically. In that case however, we should rather fix that than forcing so much code in there...

avatar brianteeman
brianteeman - comment - 27 Nov 2014

Any update @donald-pg ?

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5025.

avatar brianteeman brianteeman - change - 30 Nov 2014
Status Pending Information Required
avatar brianteeman
brianteeman - comment - 1 Jan 2015

After considerable time without update I am closing this. If you feel this is still an issue please provide further information and it can always be reopened


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5025.
avatar brianteeman brianteeman - change - 1 Jan 2015
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2015-01-01 16:57:29
Closed_By brianteeman
avatar brianteeman brianteeman - close - 1 Jan 2015
avatar Hackwar
Hackwar - comment - 1 Jan 2015

Hi @brianteeman, the issue is still valid and should be tested. As far as I can see, this code is still up-to-date and should be merged if possible. Simply needs testers...

avatar brianteeman
brianteeman - comment - 1 Jan 2015

Donald said he would revise the patch and hasn't

avatar Hackwar
Hackwar - comment - 1 Jan 2015

I can offer to work on this some time in this month and provide a new PR, but that would require that I remember this PR and that it be left open. :wink: Would that be acceptable?

avatar brianteeman
brianteeman - comment - 1 Jan 2015

Let's not do silly things like that. We can't and shouldnt keep stale
issues open for ever. If you want to create a new PR then make a note in
your calendar.

avatar Hackwar
Hackwar - comment - 1 Jan 2015

ok

Add a Comment

Login with GitHub to post a comment