Magento 2.3.0: After composer installation sampledata can't be installed from command line

[dverkade]

Preconditions

1. Magento 2.3.0
2. Install Magento with composer
3. Remove the version number from the composer.json file in the root of the project.

Steps to reproduce

1. Install Magento with composer using the following command composer create-project --repository=https://repo.magento.com/ magento/project-community-edition <install-directory-name>
2. After successful installation and setup run the following command to install the sample data: bin/magento sampledata:deploy

Expected result

1. Sample data deploys to the freshly installed Magento environment.

Actual result

1. Command throws the following error:

bin/magento sampledata:deploy
Git installations must deploy sample data from GitHub; see https://devdocs.magento.com/guides/v2.3/install-gde/install/sample-data-after-clone.html for more information.

[magento-engcom-team]

Hi @dverkade. Thank you for your report.
To help us process this issue please make sure that you provided the following information:

• Summary of the issue
• Information on your environment
• Steps to reproduce
• Expected and actual results

Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:

@magento-engcom-team give me $VERSION instance

where $VERSION is version tags (starting from 2.2.0+) or develop branches (for example: 2.3-develop).
For more details, please, review the Magento Contributor Assistant documentation.

@dverkade do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?

• yes
• no

[magento-engcom-team]

Hi @engcom-backlog-andrii. Thank you for working on this issue.
In order to make sure that issue has enough information and ready for development, please read and check the following instruction: 👇

• 1. Verify that issue has all the required information. (Preconditions, Steps to reproduce, Expected result, Actual result).

  Details

  If the issue has a valid description, the label Issue: Format is valid will be added to the issue automatically. Please, edit issue description if needed, until label Issue: Format is valid appears.

• 2. Verify that issue has a meaningful description and provides enough information to reproduce the issue. If the report is valid, add Issue: Clear Description label to the issue by yourself.

• 3. Add Component: XXXXX label(s) to the ticket, indicating the components it may be related to.

• 4. Verify that the issue is reproducible on 2.3-develop branch

  Details

  - Add the comment @magento-engcom-team give me 2.3-develop instance to deploy test instance on Magento infrastructure.
  - If the issue is reproducible on 2.3-develop branch, please, add the label Reproduced on 2.3.x.
  - If the issue is not reproducible, add your comment that issue is not reproducible and close the issue and stop verification process here!

• 5. Verify that the issue is reproducible on 2.2-develop branch.

  Details

  - Add the comment @magento-engcom-team give me 2.2-develop instance to deploy test instance on Magento infrastructure.
  - If the issue is reproducible on 2.2-develop branch, please add the label Reproduced on 2.2.x

• 6. Add label Issue: Confirmed once verification is complete.

• 7. Make sure that automatic system confirms that report has been added to the backlog.

[orlangur]

@dverkade what is not clear here? :) Such installation is really not a Composer-based but something similar to git clone.

You can use Composer to create new projects from an existing package. This is the equivalent of doing a git clone/svn checkout followed by a composer install of the vendors.

[dverkade]

@orlangur Look at the DevDocs here: https://devdocs.magento.com/guides/v2.3/install-gde/composer.html

It clearly states that you are installing with composer (it's in the title). Then when you install the sample data it says that it's installed with GIT. So totally not clear and sampledata should be able to install when you follow the steps from the DevDocs documentation, which was working correctly in Magento 2.2.X by the way.

The DevDocs documentation of installing the sample data referenced by the error message (https://devdocs.magento.com/guides/v2.3/install-gde/install/sample-data-after-clone.html ) clearly states that this only works when: "You cloned the Magento 2 repository." By following the steps described in this issue, this is clearly not the case it was not cloned, but installed with Composer.

[orlangur]

@dverkade sorry, you're right, I misread the installation command. magento/project-community-edition is a right package for Composer-based installation.

Hi @dverkade. Thanks for the report. I'm not able to reproduce following steps you described. Please see attachment of screen:

Also happens with "magento/product-enterprise-edition": "2.3.0" specified in my composer.json, using php 7.2 and composer 1.7.3.

Meta: What's with that mentality? Can't reproduce, can't be a bug -- so let's just close it?
I'll try to investigate when I've got some spare time, currently I'm low on time (due to issues like this).

EDIT: Formatting consistency

[dverkade]

I did some more digging into this issue and found out why it's not installing for me. My root composer.json does not have a version number in it. The command sampledata:deploy is checking if there is a version number present in the root composer.json file. When a version number is not found / present it will throw the error shown in this issue. This happens on the following line in the code:
https://github.com/magento/magento2/blob/2.3-develop/app/code/Magento/SampleData/Console/Command/SampleDataDeployCommand.php#L87

Please note that according to the Composer documentation, the version number is not required and should be omitted. More information here:
https://getcomposer.org/doc/04-schema.md#version
The documentation also states: Specifying the version yourself will most likely end up creating problems at some point due to human error.

So a better solution is the check if the file /app/code/Magento/SampleData/composer.json file exists in order to determine if the installation is installed with composer or cloned from Github.