J4 Issue ?
avatar infograf768
infograf768
11 Aug 2019

Steps to reproduce the issue

Install a clean build.
Install or not some languages, then skip

Expected result

If there is some sample data to install, get the details and install.
If none is available, then go back to complete installation.

Actual result

As there is no more sample data to install we get a screen like below where the ini lang strings do not exist anymore as they have been recently deleted.

Screen Shot 2019-08-11 at 09 42 17

Code concerned is in
/installation/tmpl/remove/default.php
and
/installation/src/Form/Field/Installation/SampleField.php

avatar infograf768 infograf768 - open - 11 Aug 2019
avatar joomla-cms-bot joomla-cms-bot - change - 11 Aug 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 11 Aug 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Aug 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 11 Aug 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Aug 2019
Status New Discussion
avatar Quy
Quy - comment - 12 Aug 2019

Related #20262

avatar brianteeman
brianteeman - comment - 12 Aug 2019

Removing this step is easy BUT @wilsonge do you still want 3pd to be able to include their own sql or not? If you do then I guess we can have a check for the presence of a custom.sql and only display this step if it exists. Please advise - its an easy fix

avatar wilsonge
wilsonge - comment - 13 Aug 2019

As part of #25692 we auto-install custom.sql if it exists.

https://github.com/joomla/joomla-cms/blob/4.0-dev/installation/src/Model/DatabaseModel.php#L713-L722

Basically I'm taking the opinion that as most template providers already offer downloads of file or sample data the 90% use case is the users downloaded it to install sample data. If the template dev's want more flexibility than that then they'll have to go down the plugin approach.

TL/DR It's fine to just remove the step.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Aug 2019

Closed as having Pull Request #25828

avatar franz-wohlkoenig franz-wohlkoenig - close - 13 Aug 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 13 Aug 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-08-13 06:16:33
Closed_By franz-wohlkoenig

Add a Comment

Login with GitHub to post a comment