Skip to content
Snippets Groups Projects
Verified Commit 5b72d300 authored by Dave Long's avatar Dave Long
Browse files

Revert "Issue #3414993 by alexpott: Add ConfigImporter to...

Revert "Issue #3414993 by alexpott: Add ConfigImporter to \Drupal\Core\Config\Importer\MissingContentEvent"

This reverts commit eafa8564.
parent 05737686
No related branches found
No related tags found
30 merge requests!122353526426-warning-for-missing,!12212Issue #3445525 by alexpott, japerry, catch, mglaman, longwave: Add BC layer...,!11958Issue #3490507 by alexpott, smustgrave: Fix bogus mocking in...,!11769Issue #3517987: Add option to contextual filters to encode slashes in query parameter.,!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4,!10602Issue #3438769 by vinmayiswamy, antonnavi, michelle, amateescu: Sub workspace does not clear,!10301Issue #3469309 by mstrelan, smustgrave, moshe weitzman: Use one-time login...,!10187Issue #3487488 by dakwamine: ExtensionMimeTypeGuesser::guessMimeType must support file names with "0" (zero) like foo.0.zip,!9944Issue #3483353: Consider making the createCopy config action optionally fail...,!9929Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9787Resolve issue 3479427 - bootstrap barrio issue under Windows,!9742Issue #3463908 by catch, quietone: Split OptionsFieldUiTest into two,!9526Issue #3458177 by mondrake, catch, quietone, godotislate, longwave, larowlan,...,!8738Issue #3424162 by camilledavis, dineshkumarbollu, smustgrave: Claro...,!8704Make greek characters available in ckeditor5,!8597Draft: Issue #3442259 by catch, quietone, dww: Reduce time of Migrate Upgrade tests...,!8533Issue #3446962 by kim.pepper: Remove incorrectly added...,!8517Issue #3443748 by NexusNovaz, smustgrave: Testcase creates false positive,!8325Update file Sort.php,!8095Expose document root on install,!7930Resolve #3427374 "Taxonomytid viewsargumentdefault plugin",!7627Issue #3439440 by nicxvan, Binoli Lalani, longwave: Remove country support from DateFormatter,!7445Issue #3440169: When using drupalGet(), provide an associative array for $headers,!7401#3271894 Fix documented StreamWrapperInterface return types for realpath() and dirname(),!7384Add constraints to system.advisories,!7078Issue #3320569 by Spokje, mondrake, smustgrave, longwave, quietone, Lendude,...,!6622Issue #2559833 by piggito, mohit_aghera, larowlan, guptahemant, vakulrai,...,!6502Draft: Resolve #2938524 "Plach testing issue",!38582585169-10.1.x,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key
Pipeline #100759 passed with warnings
Pipeline: drupal

#100790

    Pipeline: drupal

    #100786

      Pipeline: drupal

      #100775

        +1
        ......@@ -701,7 +701,7 @@ protected function processMissingContent(&$context) {
        $missing_content = $sandbox['missing_content']['data'];
        }
        if (!empty($missing_content)) {
        $event = new MissingContentEvent($missing_content, $this);
        $event = new MissingContentEvent($missing_content);
        // Fire an event to allow listeners to create the missing content.
        $this->eventDispatcher->dispatch($event, ConfigEvents::IMPORT_MISSING_CONTENT);
        $sandbox['missing_content']['data'] = $event->getMissingContent();
        ......
        ......@@ -2,15 +2,14 @@
        namespace Drupal\Core\Config\Importer;
        use Drupal\Core\Config\ConfigImporter;
        use Drupal\Core\Config\ConfigImporterEvent;
        use Drupal\Component\EventDispatcher\Event;
        /**
        * Wraps a configuration event for event listeners.
        *
        * @see \Drupal\Core\Config\ConfigEvents::IMPORT_MISSING_CONTENT
        */
        class MissingContentEvent extends ConfigImporterEvent {
        class MissingContentEvent extends Event {
        /**
        * A list of missing content dependencies.
        ......@@ -24,11 +23,8 @@ class MissingContentEvent extends ConfigImporterEvent {
        *
        * @param array $missing_content
        * Missing content information.
        * @param \Drupal\Core\Config\ConfigImporter $config_importer
        * The config importer that triggered this event.
        */
        public function __construct(array $missing_content, ConfigImporter $config_importer) {
        parent::__construct($config_importer);
        public function __construct(array $missing_content) {
        $this->missingContent = $missing_content;
        }
        ......
        ......@@ -4,7 +4,6 @@
        use Drupal\Core\Config\ConfigCrudEvent;
        use Drupal\Core\Config\ConfigEvents;
        use Drupal\Core\Config\ConfigImporter;
        use Drupal\Core\Config\ConfigImporterEvent;
        use Drupal\Core\Config\Importer\MissingContentEvent;
        use Drupal\Core\State\StateInterface;
        ......@@ -73,9 +72,6 @@ public function onConfigImporterMissingContentOne(MissingContentEvent $event) {
        * The missing content event.
        */
        public function onConfigImporterMissingContentTwo(MissingContentEvent $event) {
        if (!$event->getConfigImporter() instanceof ConfigImporter) {
        throw new \LogicException('\Drupal\Core\Config\Importer\MissingContentEvent is missing the ConfigImporter');
        }
        if ($this->state->get('config_import_test.config_import_missing_content', FALSE) && $this->state->get('config_import_test.config_import_missing_content_two', FALSE) === FALSE) {
        $missing = $event->getMissingContent();
        $uuid = key($missing);
        ......
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Please register or to comment