Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Remaining install sample data screen #25813

Closed
infograf768 opened this issue Aug 11, 2019 · 4 comments
Closed

[4.0] Remaining install sample data screen #25813

infograf768 opened this issue Aug 11, 2019 · 4 comments

Comments

@infograf768
Copy link
Member

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

@Quy
Copy link
Contributor

Quy commented Aug 12, 2019

Related #20262

@brianteeman
Copy link
Contributor

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

@wilsonge
Copy link
Contributor

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.

brianteeman added a commit to brianteeman/joomla-cms that referenced this issue Aug 13, 2019
@ghost
Copy link

ghost commented Aug 13, 2019

Closed as having Pull Request #25828

@ghost ghost closed this as completed Aug 13, 2019
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants