Bug #82289

Check for LocalConfiguration.php and not if typo3conf is writable

Added by Benni Mack 6 months ago. Updated 6 months ago.

Status:
Resolved
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 6 months 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 6 months 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 6 months 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 6 months ago

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

#4 Updated by Riccardo De Contardi 6 months ago

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

Also available in: Atom PDF