Skip to content

Commit 8de6188

Browse files
committed
Issue #3000677 by catch, Shane Birley, featherbelly, alexpott, larowlan: Fatal error after upgrade to 8.6x [due to regression in extension system]
1 parent 4bcd11e commit 8de6188

File tree

2 files changed

+30
-1
lines changed

2 files changed

+30
-1
lines changed

lib/Drupal/Core/Extension/ModuleHandler.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -777,7 +777,12 @@ public function getModuleDirectories() {
777777
* {@inheritdoc}
778778
*/
779779
public function getName($module) {
780-
return \Drupal::service('extension.list.module')->getName($module);
780+
try {
781+
return \Drupal::service('extension.list.module')->getName($module);
782+
}
783+
catch (UnknownExtensionException $e) {
784+
return $module;
785+
}
781786
}
782787

783788
}
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
<?php
2+
3+
namespace Drupal\KernelTests\Core\Extension;
4+
5+
use Drupal\KernelTests\KernelTestBase;
6+
7+
/**
8+
* @coversDefaultClass \Drupal\Core\Extension\ModuleHandler
9+
*
10+
* @group Extension
11+
*/
12+
class ModuleHandlerTest extends KernelTestBase {
13+
14+
/**
15+
* Tests requesting the name of an invalid module.
16+
*
17+
* @covers ::getName
18+
*/
19+
public function testInvalidGetName() {
20+
$module_handler = $this->container->get('module_handler');
21+
$this->assertSame('module_nonsense', $module_handler->getName('module_nonsense'));
22+
}
23+
24+
}

0 commit comments

Comments
 (0)