Project

General

Profile

Actions
Copy link

Feature #11771

closed

Check for write-permissions before installation

Added by Steffen Gebert over 13 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Could have
Assignee:
Alexander Opitz
Category:
Install Tool
Target version:
-
Start date:
2010-12-29
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

What do you think of checking for write-permissions on typo3conf/ and typo3temp/ at the very first place?

This also enables to use the FIRST_INSTALL file and doesn't show the missing-ENABLE_INSTALL_TOOL warning (and then the "I want to write to typo3temp" and then the "I want to write to typo3conf/localconf.php").

So this would require a change of the core behavior, but I would call it way more user-friendly instead of interpreting the meaning of error messages one by one.

Files

Download all files
11771.diff (4.12 KB) 11771.diff Steffen Gebert, 2011-06-02 23:24
forge_11771.png (24.1 KB) forge_11771.png Oliver Hader, 2011-12-19 19:45

Related issues 7 (0 open — 7 closed)

Related to TYPO3 Core - Bug #31255: CF: Could not create directoryClosed2011-10-25

Actions
Related to TYPO3 Core - Bug #46602: If typo3temp is not create-/writeable we get a uncaught exception 'RuntimeException'Closed2013-03-23

Actions
Related to TYPO3 Core - Bug #47066: If typo3conf is not writeable we get a uncaught exception 'RuntimeException'Closed2013-04-09

Actions
Related to TYPO3 Core - Feature #43477: Better error handling on dummy without permissions (instead of "RuntimeException" Fatal Error)Closed2012-11-30

Actions
Related to TYPO3 Core - Bug #42907: Installation Exceptions should look niceClosed2012-11-13

Actions
Related to TYPO3 Core - Bug #53464: Migrate all file relations hangs after wrong permissionsClosed2013-11-08

Actions
Has duplicate TYPO3 Core - Feature #12872: Initial check screen for writable pathsClosedSteffen Gebert2011-02-08

Actions
Actions
Copy link
 #1

Updated by Ben van 't Ende over 13 years ago

  • Category set to 602
  • Status changed from New to Under Review
  • Priority changed from Should have to Could have
  • Target version set to 784
Actions
Copy link
 #2

Updated by Steffen Gebert over 13 years ago

Working patch attached. I only have to think of the situation, when the fileadmin/ directory has been renamed..

Actions
Copy link
 #3

Updated by Steffen Gebert over 13 years ago

  • Tracker changed from Bug to Feature
  • Project changed from 978 to TYPO3 Core
  • Category deleted (602)
  • Assignee set to Steffen Gebert
  • Target version deleted (784)
Actions
Copy link
 #4

Updated by Steffen Gebert over 13 years ago

  • Subject changed from Check for write-permissions first to Check for write-permissions before installation
  • Category set to Install Tool
  • TYPO3 Version set to 4.6
Actions
Copy link
 #5

Updated by Steffen Ritter almost 13 years ago

  • Target version set to 4.7.0-alpha2
Actions
Copy link
 #6

Updated by Steffen Ritter over 12 years ago

Mind to push that to Gerrit? Or Should I?

Actions
Copy link
 #7

Updated by Gerrit Code Review over 12 years ago

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

Actions
Copy link
 #8

Updated by Oliver Hader over 12 years ago

Looks like this now:

Actions
Copy link
 #9

Updated by Steffen Gebert over 12 years ago

#31255 is a new point, where the installation breaks because of insufficient rights. This check should be somewhere before the Caching Framework dies. See #31255 for a strack trace.

Actions
Copy link
 #10

Updated by Oliver Hader over 12 years ago

  • Target version changed from 4.7.0-alpha2 to 4.7.0-alpha3
Actions
Copy link
 #11

Updated by Steffen Ritter over 12 years ago

  • Target version changed from 4.7.0-alpha3 to 4.7.0-beta1
Actions
Copy link
 #12

Updated by Steffen Ritter over 12 years ago

  • Target version deleted (4.7.0-beta1)
Actions
Copy link
 #13

Updated by Gerrit Code Review over 12 years ago

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

Actions
Copy link
 #14

Updated by Gerrit Code Review over 11 years ago

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

Actions
Copy link
 #15

Updated by Gerrit Code Review over 11 years ago

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

Actions
Copy link
 #16

Updated by Alexander Opitz over 11 years ago

  • TYPO3 Version changed from 4.6 to 6.1
Actions
Copy link
 #17

Updated by Kay Strobach over 11 years ago

why chmod 777 ??? -> isn't it better to chgrp www-data ? and chmod 770?

Thanks

Actions
Copy link
 #18

Updated by Kay Strobach over 11 years ago

why chmod 777 ??? -> isn't it better to chgrp www-data ? and chmod 770?

Thanks

Actions
Copy link
 #19

Updated by Alexander Opitz over 11 years ago

Hi Kay,

here is nothing about chmod 777, so did you post to false issue?

Actions
Copy link
 #20

Updated by Gerrit Code Review over 11 years ago

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

Actions
Copy link
 #21

Updated by Alexander Opitz over 11 years ago

  • Assignee changed from Steffen Gebert to Alexander Opitz
  • Target version set to 6.2.0
Actions
Copy link
 #22

Updated by Gerrit Code Review over 11 years ago

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

Actions
Copy link
 #23

Updated by Gerrit Code Review over 11 years ago

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

Actions
Copy link
 #24

Updated by Gerrit Code Review over 11 years ago

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

Actions
Copy link
 #25

Updated by Gerrit Code Review about 11 years ago

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

Actions
Copy link
 #26

Updated by Gerrit Code Review about 11 years ago

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

Actions
Copy link
 #27

Updated by Wouter Wolters almost 11 years ago

  • Target version deleted (6.2.0)
Actions
Copy link
 #28

Updated by Gerrit Code Review over 10 years ago

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

Actions
Copy link
 #29

Updated by Gerrit Code Review over 10 years ago

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

Actions
Copy link
 #30

Updated by Gerrit Code Review over 10 years ago

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

Actions
Copy link
 #31

Updated by Alexander Opitz over 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #32

Updated by Benni Mack almost 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF