Skip to content

[FEATURE] Allow SiteSets for wizard#15

Merged
foppelfb merged 6 commits into
mainfrom
smaller-fixes
Jun 3, 2026
Merged

[FEATURE] Allow SiteSets for wizard#15
foppelfb merged 6 commits into
mainfrom
smaller-fixes

Conversation

@calien666

Copy link
Copy Markdown
Member

With TYPO3 13, SiteSets were introduced in favour of shipping presets
for usage in site configuration.

In favour of this new functionality, the wizard has to respect the site
sets and should ship them within the creation process.

Added the TCA for the table creator matching the TCA functionality from
SiteConfiguration and placed the default handling of the SiteSets
right before the BeforeSiteConfigWriteEvent to allow manual
adjustments by an EventListener in case of this is needed.

The CreateProcess used `Environment::getProjectPath()` for writing and
detecting the site config path. This works fine in composer mode
instances but lacks path configuration in classic installations.

Use the `Environment::getConfigPath()` instead to always detect the
correct path.

As a necessary change, installed the site-based test trait extension
from sbuerk to ensure correct behaviour during functional test setup.

Used command:
```
Build/Scripts/runTests.sh -t 13 -p 8.2 -s composer -- require --dev \
    sbuerk/typo3-site-based-test-trait:"^2.0 || ^3.0"
```
@calien666 calien666 force-pushed the smaller-fixes branch 2 times, most recently from d65d246 to 5f5f606 Compare June 3, 2026 09:54
@calien666 calien666 requested a review from foppelfb June 3, 2026 10:25
calien666 added 3 commits June 3, 2026 12:55
With TYPO3 13, SiteSets were introduced in favour of shipping presets
for usage in site configuration.

In favour of this new functionality, the wizard has to respect the site
sets and should ship them within the creation process.

Added the TCA for the table creator matching the TCA functionality from
SiteConfiguration and placed the default handling of the SiteSets
right before the `BeforeSiteConfigWriteEvent` to allow manual
adjustments by an EventListener in case of this is needed.

Cleaned up the ext_tables.sql and removed the table for creator
completely, as TYPO3 is managing the required fields by TCA definition.

@foppelfb foppelfb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No further comments.
thank @calien666

@foppelfb foppelfb merged commit b2c2cf9 into main Jun 3, 2026
3 checks passed
@calien666 calien666 deleted the smaller-fixes branch June 24, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants