Bug #52367

Task #49162: Rewrite install tool

ext:install Better error handling in first folder step

Added by Christian Kuhn almost 2 years ago. Updated almost 2 years ago.

Status:Resolved Start date:2013-09-29
Priority:Should have Due date:
Assigned To:- % Done:

100%
Category:Install Tool Spent time: -
Target version:6.2.0
TYPO3 Version:6.2 Is Regression:No
PHP Version:5.4 Sprint Focus:
Complexity:medium

Associated revisions

Revision 2d9fb1a4
Added by Christian Kuhn almost 2 years ago

[BUGFIX] ext:install Better error handling in first folder step

During installation it may happen that the folder structure reports
errors during creation, but still creates the main folder structure.
This is the case for example if the document root or fileadmin
is a link and not a directory. In those cases the installer does not
create a LocalConfiguration and shows the first step again. It will
stop with a "ENABLE_INSTALL_TOOL" file missing message afterwards.

This is better sanitized with the patch: If the folder structure
reports errors but still successfully created a typo3conf folder, the
LocalConfiguration is also created and the next step is called after
redirect.

Furthermore, two possible warnings are caught in the folder structure
and adapted status messages are created.

Change-Id: I54d99f826336c513641028f29d389da845d7d26a
Resolves: #52367
Releases: 6.2
Reviewed-on: https://review.typo3.org/24135
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24135

#2 Updated by Gerrit Code Review almost 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24135

#3 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24135

#4 Updated by Gerrit Code Review almost 2 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24135

#5 Updated by Gerrit Code Review almost 2 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24135

#6 Updated by Christian Kuhn almost 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF