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

D8 improvement request: make the message "There were errors validating the config synchronization." more helpful #807

Closed
alberto56 opened this Issue Sep 10, 2014 · 2 comments

Comments

Projects
None yet
3 participants
@alberto56

Let's say you have two sites with different UUID values in the system.site.uuid configuration.

If you drush config-export from one of the sites and try to drush config-import to another, it runs into an error because only sites with the same system.site.uuid can be synced.

If you use Drupal's GUI, you see the error "The staged configuration cannot be imported, because it originates from a different site than this site. You can only synchronize configuration between cloned instances of this site.", which is helpful.

However, if you run drush config-import on the command line, you get the error "There were errors validating the config synchronization.".

It would be ideal to have the same error message on the command line as in the GUI.

@alberto56 alberto56 changed the title from D8 improvement request: make "There were errors validating the config synchronization." more helpful to D8 improvement request: make the message "There were errors validating the config synchronization." more helpful Sep 10, 2014

@weitzman

This comment has been minimized.

Show comment
Hide comment
@weitzman

weitzman Oct 19, 2014

Member

That message comes from a drupal_set_message() \Drupal\config\Form\ConfigSync::buildForm which is a really odd place to do validation. I think this needs fixing in drupal.

Member

weitzman commented Oct 19, 2014

That message comes from a drupal_set_message() \Drupal\config\Form\ConfigSync::buildForm which is a really odd place to do validation. I think this needs fixing in drupal.

@weitzman weitzman closed this Oct 19, 2014

@geerlingguy

This comment has been minimized.

Show comment
Hide comment
@geerlingguy

geerlingguy Feb 28, 2015

Link to corresponding core issue on Drupal.org: https://www.drupal.org/node/2332623

Link to corresponding core issue on Drupal.org: https://www.drupal.org/node/2332623

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment