Skip to content

[installation] Improve ajax error treatment#11930

Closed
andrepereiradasilva wants to merge 3 commits intojoomla:stagingfrom
andrepereiradasilva:installation-error-catch
Closed

[installation] Improve ajax error treatment#11930
andrepereiradasilva wants to merge 3 commits intojoomla:stagingfrom
andrepereiradasilva:installation-error-catch

Conversation

@andrepereiradasilva
Copy link
Copy Markdown
Contributor

@andrepereiradasilva andrepereiradasilva commented Sep 4, 2016

Summary of Changes

There are some parts of the installation process that don't check for ajax request issues (timeout, syntax error, server error codes, etc) leaving the user with no clue on what is wrong.

This PR corrects that and also improves the error messages by replacing javascript alert messages with joomla message errors.

Didn't change the ftp part because don't have way to test it

Testing Instructions

  1. First apply this patch
  2. Delete configuration.php and reload any site page you'll be redirect to installation
  3. When in installation view: Clear all browser cache
  4. Test no regresssions: normal installations and check everything is fine
  5. Repeat step 2.
  6. Test forcing errors, for checking you always get some error message:
    • Add an sql error in joomla.sql file. Try to install, you should get an error
    • Add some non php text (anything that generates a php syntax error) inside /installation/model/configuration.php. Try to install, you should get an error
    • Rename /installation/controller/site.php to anything. Try to install, you should get an error
    • Throw some exception (example: throw new Exception('some error', 404);) here https://github.com/joomla/joomla-cms/blob/staging/installation/controller/site.php#L34 . Try to install, you should get an error
    • Test forcing any other error you can remember in the install process
  7. Code review. Use this to ignore whitespaces changes https://github.com/joomla/joomla-cms/pull/11930/files?w=1

Documentation Changes Required

None.

@piotr-cz
Copy link
Copy Markdown
Contributor

piotr-cz commented Sep 5, 2016

It's really hard to see scope of changes when you've changed whitespace formatting.

@andrepereiradasilva
Copy link
Copy Markdown
Contributor Author

andrepereiradasilva commented Sep 5, 2016

That's why i said (the problem is this file didn't use tabs for indent as all other joomla files. i corrected that also)

Code review. Use this to ignore whitespaces changes https://github.com/joomla/joomla-cms/pull/11930/files?w=1

@andrepereiradasilva
Copy link
Copy Markdown
Contributor Author

closing because lack of interrest

@andrepereiradasilva andrepereiradasilva deleted the installation-error-catch branch January 2, 2017 15:45
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.

3 participants