Project

General

Profile

Actions

Bug #99744

closed

Take care of migrating AdditionalConfiguration

Added by Josef Glatz almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
System/Bootstrap/Configuration
Target version:
-
Start date:
2023-01-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Problem/Description

There are situations where an AdditionalConfiguration.php needs to be migrated too, even if a settings.php is in place:

One problematic scenario is DDEV: It's actually not really straight forward for newcomers or contributors to work on the core due to the AdditionalConfiguration.php generation. Since there are big changes in core's file structure, DDEV will wait for any modifications in the TYPO3 integration part until TYPO3 v12 lts is released. This was recently (jan 2023) discussed between @Benni Mack and @Simon Gilli.

Instead of adding infos in the documentation of the TYPO3 contribution guide, I walked through the code and believe that this fix should be right in the core.

ToDo(s)

  • Add a check if AdditionalConfiguration.php needs migrated even if settings.php is already in place
  • Add a "silent" migration like it is already in place for the "LocalConfiguration.php"

Acceptance Criteria

  • main-installation with DDEV migrates typo3conf/AdditionalConfiguration.php even if the recently introduced settings.php is already in place.

Scenarios for Testing

$properlyConfigured = TRUE
```
new:
    settings.php
legacy:
    -
```

$properlyConfigured = TRUE
```
new:
    settings.php
    additional.php
legacy:
    -
```

$properlyConfigured = FALSE
```
    new:
        settings.php
    legacy:
        AdditionalConfiguration.php
```

$properlyConfigured = TRUE
```
    new:
        settings.php
        additional.php
    legacy:
        AdditionalConfiguration.php
```

$properlyConfigured = FALSE
```
    new:

    legacy:
        LocalConfiguration.php
        AdditionalConfiguration.php
```

$properlyConfigured = FALSE
```
    new:

    legacy:
        LocalConfiguration.php
```


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Task #100013: Ensure minimal I/O for configuration migration checks during BootstrapClosedMarkus Klein2023-02-22

Actions
Related to TYPO3 Core - Task #105546: Extend migration lifetime of legacy configuration migrationResolvedOliver Bartsch2024-11-05

Actions
Actions

Also available in: Atom PDF