#91: Use definition object to access definition.

src/UiPatternsManager.php

@@ -8,6 +8,7 @@ use Drupal\Core\Cache\CacheBackendInterface;
 use Drupal\Core\Extension\ModuleHandlerInterface;
 use Drupal\Core\Extension\ThemeHandlerInterface;
 use Drupal\Core\Plugin\DefaultPluginManager;
+use Drupal\ui_patterns\Definition\PatternDefinition;
 
 /**
  * Provides the default ui_patterns manager.
@@ -60,7 +61,7 @@ class UiPatternsManager extends DefaultPluginManager implements PluginManagerInt
     // @todo should we statically cache this?
     $patterns = [];
     foreach ($this->getDefinitions() as $definition) {
-      $patterns[] = $this->getPattern($definition['id']);
+      $patterns[] = $this->getPattern($definition->id());
     }
     return $patterns;
   }
@@ -99,8 +100,8 @@ class UiPatternsManager extends DefaultPluginManager implements PluginManagerInt
    * {@inheritdoc}
    */
   public function isPatternHook($hook) {
-    $definitions = array_filter($this->getDefinitions(), function ($definition) use ($hook) {
-      return $definition['theme hook'] == $hook;
+    $definitions = array_filter($this->getDefinitions(), function (PatternDefinition $definition) use ($hook) {
+      return $definition->getThemeHook() == $hook;
     });
     return !empty($definitions);
   }

tests/src/Kernel/UiPatternsManagerTest.php

@@ -19,8 +19,11 @@ class UiPatternsManagerTest extends AbstractUiPatternsTest {
    * @covers ::getPatternDefinition
    */
   public function testGetPattern() {
-    foreach (UiPatterns::getPatternDefinitions() as $definition) {
-      assert($definition->id(), equals($definition['id']));
+    $manager = UiPatterns::getManager();
+
+    foreach ($manager->getDefinitions() as $definition) {
+      $pattern = $manager->getPattern($definition->id());
+      assert($pattern->getBaseId(), equals($definition->id()));
     }
   }