feat: allow formAttributes method to work without accepting a method

Author: thetutlage

providers/edge_provider.ts

@@ -167,38 +167,44 @@ export default class EdgeServiceProvider {
      */
     edge.global('qs', qs)
 
-    edge.global(
-      'formAttributes',
-      function (route: string, method: string, params: any, options: URLOptions) {
-        /**
-         * Normalize method and keep a reference to the original method
-         */
-        options = options ?? {}
-        method = method.toUpperCase()
-        const original = method
-
-        /**
-         * If method if not GET and POST, then use the querystring _method
-         * to and force update the method to "POST"
-         */
-        if (method !== 'GET' && method !== 'POST') {
-          method = 'POST'
-          options = { ...options, qs: { _method: original, ...options.qs } }
-        }
-
-        const { action } = (router.urlBuilder.urlFor.method as any)(
-          original,
-          route,
-          params,
-          options
-        ).form
-
-        return {
-          action,
-          method,
-        }
+    edge.global('formAttributes', function (route: string, params: any, options: URLOptions) {
+      const matchingRoute = router.findOrFail(route)
+
+      /**
+       * Normalize method and keep a reference to the original method
+       */
+      options = options ?? {}
+      let method = matchingRoute.methods[0].toUpperCase()
+      const original = method
+
+      /**
+       * In case of HEAD, we must use the GET method
+       */
+      if (method === 'HEAD') {
+        method = 'GET'
       }
-    )
+
+      /**
+       * If method if not GET and POST, then use the querystring _method
+       * to and force update the method to "POST"
+       */
+      if (method !== 'GET' && method !== 'POST') {
+        method = 'POST'
+        options = { ...options, qs: { _method: original, ...options.qs } }
+      }
+
+      const { action } = (router.urlBuilder.urlFor.method as any)(
+        original,
+        route,
+        params,
+        options
+      ).form
+
+      return {
+        action,
+        method,
+      }
+    })
 
     /**
      * Creating a isolated instance of edge renderer

tests/bindings/edge.spec.ts

@@ -102,7 +102,7 @@ test.group('Bindings | Edge', () => {
     router.get('/users/:id', () => {}).as('users.show')
     router.commit()
 
-    assert.deepEqual(edge.globals.formAttributes('users.show', 'get', { id: 1 }), {
+    assert.deepEqual(edge.globals.formAttributes('users.show', { id: 1 }), {
       action: '/users/1',
       method: 'GET',
     })
@@ -129,7 +129,7 @@ test.group('Bindings | Edge', () => {
     router.put('/users/:id', () => {}).as('users.update')
     router.commit()
 
-    assert.deepEqual(edge.globals.formAttributes('users.update', 'put', { id: 1 }), {
+    assert.deepEqual(edge.globals.formAttributes('users.update', { id: 1 }), {
       action: '/users/1?_method=PUT',
       method: 'POST',
     })
@@ -163,7 +163,7 @@ test.group('Bindings | Edge', () => {
       },
     }
 
-    assert.deepEqual(edge.globals.formAttributes('users.update', 'put', { id: 1 }, options), {
+    assert.deepEqual(edge.globals.formAttributes('users.update', { id: 1 }, options), {
       action: '/users/1?_method=PUT&view=card',
       method: 'POST',
     })