Bug #57152

Task #52668: Install Tool: Remove permission checking and fixing code from "folder structure"

First Install lists directory errors: confusing

Added by Ernesto Baschny over 5 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Must have
Category:
Install Tool
Target version:
Start date:
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

First installation has no directories yet. Even if they could be created there are errors shown to the user. This is ugly!


Related issues

Related to TYPO3 Core - Bug #57159: Tests: Failing unit test Closed 2014-03-21

Associated revisions

Revision c23414d5 (diff)
Added by Ernesto Baschny over 5 years ago

[BUGFIX] First install lists directory errors

For a first installer the directory structure is empty and the
first step will try to create it. We do not need to show these
as errors to the user, only if the root directory cannot be
written.

To achieve this we raise "Warnings" instead (which are
not shown in first step). To keep the Folder Structure menu
item on the left menu in the Install Tool showing a red badge
for non existing directories, we change this to also include
Warnings.

Resolves: #57152
Releases: 6.2
Change-Id: Idd628ecdf946f9befdb7fefd7a9bb7bccc175c2b
Reviewed-on: https://review.typo3.org/28605
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe

Revision fde89fdd (diff)
Added by Christian Kuhn over 5 years ago

[BUGFIX] Test follow-up to #57152

Fix a failing unit test after change c23414d5a73

Change-Id: I427663a1a528b24a0e63d2fc9ed2a6371902a4f4
Resolves: #57159
Related: #57152
Releases: 6.2
Reviewed-on: https://review.typo3.org/28611
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe

History

#1 Updated by Gerrit Code Review over 5 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28605

#2 Updated by Gerrit Code Review over 5 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28605

#3 Updated by Gerrit Code Review over 5 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28605

#4 Updated by Ernesto Baschny over 5 years ago

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

#5 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF