Commit 0f10593e authored by catch's avatar catch
Browse files

Issue #3254986 by Matroskeen, paulocs, mikelutz, benjifisher, danflanagan8,... 

Issue #3254986 by Matroskeen, paulocs, mikelutz, benjifisher, danflanagan8, quietone: Remove requirements array from RequirementsException message

(cherry picked from commit 4c0f5077)
parent 58397499

core/modules/comment/tests/src/Kernel/Migrate/d6/MigrateCommentTypeTest.php

+3 −3
Original line number Diff line number Diff line
@@ -119,7 +119,7 @@ public function providerTestNoCommentTypeMigration() { 
        'Disabled source modules' => ['node'],

        'Expected messages' => [

          'error' => [

            'Migration d6_comment_type did not meet the requirements. The node module is not enabled in the source site. source_module_additional: node.',

            'Migration d6_comment_type did not meet the requirements. The node module is not enabled in the source site.',

          ],

        ],

      ],
@@ -127,7 +127,7 @@ public function providerTestNoCommentTypeMigration() { 
        'Disabled source modules' => ['comment'],

        'Expected messages' => [

          'error' => [

            'Migration d6_comment_type did not meet the requirements. The module comment is not enabled in the source site. source_module: comment.',

            'Migration d6_comment_type did not meet the requirements. The module comment is not enabled in the source site.',

          ],

        ],

      ],
@@ -135,7 +135,7 @@ public function providerTestNoCommentTypeMigration() { 
        'Disabled source modules' => ['comment', 'node'],

        'Expected messages' => [

          'error' => [

            'Migration d6_comment_type did not meet the requirements. The module comment is not enabled in the source site. source_module: comment.',

            'Migration d6_comment_type did not meet the requirements. The module comment is not enabled in the source site.',

          ],

        ],

      ],

core/modules/comment/tests/src/Kernel/Migrate/d7/MigrateCommentTypeTest.php

+3 −3
Original line number Diff line number Diff line
@@ -125,7 +125,7 @@ public function providerTestNoCommentTypeMigration() { 
        'Disabled source modules' => ['node'],

        'Expected messages' => [

          'error' => [

            'Migration d7_comment_type did not meet the requirements. The node module is not enabled in the source site. source_module_additional: node.',

            'Migration d7_comment_type did not meet the requirements. The node module is not enabled in the source site.',

          ],

        ],

      ],
@@ -133,7 +133,7 @@ public function providerTestNoCommentTypeMigration() { 
        'Disabled source modules' => ['comment'],

        'Expected messages' => [

          'error' => [

            'Migration d7_comment_type did not meet the requirements. The module comment is not enabled in the source site. source_module: comment.',

            'Migration d7_comment_type did not meet the requirements. The module comment is not enabled in the source site.',

          ],

        ],

      ],
@@ -141,7 +141,7 @@ public function providerTestNoCommentTypeMigration() { 
        'Disabled source modules' => ['comment', 'node'],

        'Expected messages' => [

          'error' => [

            'Migration d7_comment_type did not meet the requirements. The module comment is not enabled in the source site. source_module: comment.',

            'Migration d7_comment_type did not meet the requirements. The module comment is not enabled in the source site.',

          ],

        ],

      ],

core/modules/migrate/src/MigrateExecutable.php

+1 −2
Original line number Diff line number Diff line
@@ -164,11 +164,10 @@ public function import() { 
    catch (RequirementsException $e) {

      $this->message->display(

        $this->t(

          'Migration @id did not meet the requirements. @message @requirements',

          'Migration @id did not meet the requirements. @message',

          [

            '@id' => $this->migration->id(),

            '@message' => $e->getMessage(),

            '@requirements' => $e->getRequirementsString(),

          ]

        ),

        'error'