Ajax-based component class discovery

Fixes #8

Author: brandonkelly

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Release Notes for Webhooks for Craft CMS
 
+## Unreleased
+
+### Fixed
+- Fixed a bug where it wasn’t possible to create or edit webhooks if a plugin contained an invalid class. ([#8](https://github.com/craftcms/webhooks/issues/8))
+
 ## 2.0.0 - 2019-03-19
 
 ### Added

src/WebhookHelper.php

@@ -35,7 +35,7 @@ class WebhookHelper
      */
     public static function classSuggestions(): array
     {
-        $data = Craft::$app->getCache()->getOrSet('webhooks.classSuggestions', function() {
+        return Craft::$app->getCache()->getOrSet('webhooks.classSuggestions', function() {
             $classes = [];
             foreach (self::_findClasses() as $class) {
                 $classes[] = [
@@ -47,12 +47,6 @@ public static function classSuggestions(): array
         }, null, new FileDependency([
             'fileName' => Craft::$app->getPath()->getVendorPath() . '/composer/autoload_real.php',
         ]));
-
-        return [
-            [
-                'data' => $data,
-            ],
-        ];
     }
 
     /**

src/controllers/WebhooksController.php

@@ -79,17 +79,13 @@ public function actionEdit(int $id = null, int $groupId = null, Webhook $webhook
             }
         }
 
-        // Autosuggest classes
-        $classSuggestions = WebhookHelper::classSuggestions();
-
         Craft::$app->getView()->registerAssetBundle(EditAsset::class);
 
         return $this->renderTemplate('webhooks/_manage/edit', compact(
             'groupOptions',
             'webhook',
             'title',
-            'crumbs',
-            'classSuggestions'
+            'crumbs'
         ));
     }
 
@@ -169,6 +165,16 @@ public function actionDelete(): Response
         return $this->redirectToPostedUrl();
     }
 
+    /**
+     * Returns the available sender classes.
+     */
+    public function actionClassSuggestions(): Response
+    {
+        return $this->asJson([
+            'classes' => WebhookHelper::classSuggestions(),
+        ]);
+    }
+
     /**
      * Returns the available events for a component class.
      */

src/templates/_manage/class-autosuggest.html

@@ -0,0 +1,24 @@
+{% extends '_includes/forms/autosuggest' %}
+
+{% block data %}
+    {{ parent() }}
+    this.updateSuggestions()
+    {#data.inputProps.onInputChange = this.myOnInputChange;#}
+
+{% endblock %}
+
+
+{% block methods %}
+    {{ parent() }}
+    updateSuggestions() {
+        Craft.postActionRequest('webhooks/webhooks/class-suggestions', {}, $.proxy(function(response, textStatus) {
+            if (textStatus === 'success') {
+                this.suggestions = [
+                    {
+                        data: response.classes
+                    }
+                ];
+            }
+        }, this));
+    },
+{% endblock %}

src/templates/_manage/edit.html

@@ -41,18 +41,20 @@
         errors: webhook.getErrors('name')
     }) }}
 
-    {{ forms.autosuggestField({
-        label: 'Sender Class'|t('webhooks'),
-        required: true,
-        instructions: 'The class name the sender must be an instance of.'|t('webhooks'),
+    {% set classInput = include('webhooks/_manage/class-autosuggest', {
         id: 'class',
         name: 'class',
         class: 'code ltr',
-        suggestions: classSuggestions,
         placeholder: 'craft\\elements\\Entry',
         value: webhook.class,
         errors: webhook.getErrors('class')
-    }) }}
+    }, with_context = false) %}
+    {{ forms.field({
+        label: 'Sender Class'|t('webhooks'),
+        required: true,
+        instructions: 'The class name the sender must be an instance of.'|t('webhooks'),
+        errors: webhook.getErrors('class')
+    }, classInput) }}
 
     {% set eventInput = include('webhooks/_manage/event-autosuggest', {
         id: 'event',