When I reinstall the site I'm working I got the following error:

Configuration node_title_validation_config.node_title_validation_settings depends on the node_title_validation_config extension that will not be installed after import. [error]

There is no reference to this configuration in the module so I propose that this can just be removed. All configuration is currently being exported successfully to node_title_validation_config.node_title_validation_settings

CommentFileSizeAuthor
#7 2855289-7.patch3.06 KBgodotislate
#2 2855289-2.patch376 bytesnlisgo

Comments

nlisgo created an issue. See original summary.

nlisgo’s picture

nlisgo commented
Assigned: nlisgo » Unassigned
Status: Active » Needs review
StatusFileSize
new 2855289-2.patch376 bytes
dgilbert’s picture

dgilbert commented

The patch hard-codes the path as /config/install/. Our site uses a different path for config files so the patch won't actually do anything.

codesmith’s picture

codesmith
Location San Francisco
commented

Hmmm neither applying the patch nor removing the file worked for me. :(

Still getting this error and have to disable it to import configuration item.s

codesmith’s picture

codesmith
Location San Francisco
commented
Status: Needs review » Needs work
codesmith’s picture

codesmith
Location San Francisco
commented
Title: config/install/node_title_validation.node_title_validation_settings.yml is not needed » Unable to import configuration
godotislate’s picture

godotislate
he/him
commented
Status: Needs work » Needs review
StatusFileSize
new 2855289-7.patch3.06 KB

The prefix of the config item referenced in code is node_title_validation_config, but it should match the name of the module instead: node_title_validation. The config importer determines a module dependency from this prefix, so it's expecting the config to require the node_title_validation_config module. Here's a patch.

codesmith’s picture

codesmith
Location San Francisco
commented

I applied the patch in #7, ran the database update, and successfully imported configuration. Thanks godotislate!

j2r’s picture

j2r commented
Status: Needs review » Reviewed & tested by the community

Work as expected.

Note : Needs to run drush updb once the patch is applied.

drupalninja99’s picture

drupalninja99 commented

I can confirm also that Patch #7 worked for me!

  • jcisio committed 3db48c4 on 8.x-1.x authored by godotislate 
    Issue #2855289 by godotislate, codesmith, j2r, drupalninja99, jcisio:...
jcisio’s picture

jcisio commented
Status: Reviewed & tested by the community » Fixed

Tested several cases and all worked. Patch is also good, so I committed and pushed. Thanks all.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

-nrzr-’s picture

-nrzr- commented

I applied patch #7 and it didn't fix the error.
By inspecting the node_title_validation.install, found that
$new_config->set('node_title_validation_config', $old_config->get('node_title_validation_config'));
should be
$new_config->set('node_title_validation', $old_config->get('node_title_validation_config'));
Can you please confirm?

-nrzr-’s picture

-nrzr- commented

My bad. Patch #7 is working fine.

yob1224’s picture

yob1224 commented

Hello, how can I apply this patch? I tried applying the patch through composer but i can't see the patch working. Thanks!

       "patches": {
            "drupal/address": {
                "Unable to import configuration": "https://www.drupal.org/files/issues/2855289-7.patch"     
            }
        }
nlisgo’s picture

nlisgo commented

@yob1224, you also need to pull down the cweagans/composer-patches project:

composer require cweagans/composer-patches

givanov’s picture

givanov commented

I have recently used patch #7 and its latest version worked as expected. Thanks to the creator.