Skip to content

Command app:config:dump add unnecessary spacing every time it's run #11664

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

Closed
Linek opened this issue Oct 23, 2017 · 1 comment
Closed

Command app:config:dump add unnecessary spacing every time it's run #11664

Linek opened this issue Oct 23, 2017 · 1 comment
Labels
Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed

Comments

@Linek
Copy link
Contributor

Linek commented Oct 23, 2017

Preconditions

  1. PHP 7.0.24
  2. nginx/1.11.13
  3. Percona Ver 14.14 Distrib 5.7.19-17
  4. Magento 2.2.0 installed with sample data by composer

Steps to reproduce

  1. Run "bin/magento app:config:dump" to dump configuration to app/etc/config.php file
  2. Save your config.php file somewhere to compare
  3. Run "bin/magento app:config:dump" again
  4. Compare config.php after second dump with the one after first dump

Expected result

File config.php should not change when there is no modification in configuration

Actual result

Every time app:config:dump is run few fields is generated with additional spaces

image

List of fields:
default_custom_instructions
minify_exclude
address_templates

@magento-engcom-team magento-engcom-team added the Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed label Oct 23, 2017
@davidalger
Copy link
Member

Closing as duplicate of #11328; fixed in 2.1-develop, there are PRs in progress for 2.2 and 2.3 to apply the same fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed
Projects
None yet
Development

No branches or pull requests

3 participants