Bug #82289

Check for LocalConfiguration.php and not if typo3conf is writable

Added by Benni Mack about 1 year ago. Updated about 2 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
System/Bootstrap/Configuration
Target version:
Start date:
2017-09-04
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Related issues

Related to TYPO3 Core - Bug #70263: LocalConfiguration.php not seen was writable if typo3conf directory is not writable Closed 2015-10-01

Associated revisions

Revision 7d6354d9 (diff)
Added by Benni Mack about 1 year ago

[BUGFIX] Check if LocalConfiguration.php is writable

In TYPO3 v8, the ConfigurationManager only checks if LocalConfiguration.php
is writaeble and not if typo3conf itself is writable.

Optimizing this check removes a warning in an environment where typo3conf/
is hardened to only have LocalConfiguration.php be modified and nothing else.
This change helps running TYPO3 v7 on platform.sh with fewer error messages.

You can compare this line of code with the same line in TYPO3 v8/master.

Resolves: #82289
Releases: 7.6
Change-Id: I7adca89eaa0d9f5b28e3f034f9452d97591a0d80
Reviewed-on: https://review.typo3.org/53878
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Tested-by: TYPO3com <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>

History

#1 Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

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

#2 Updated by Gerrit Code Review about 1 year ago

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

#3 Updated by Benni Mack about 1 year ago

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

#4 Updated by Riccardo De Contardi about 1 year ago

  • Related to Bug #70263: LocalConfiguration.php not seen was writable if typo3conf directory is not writable added

#5 Updated by Benni Mack about 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF