Adds ability to use list for route params (#552)

* Adds ability to use list for route params

* added new assertion

masonite/request.py

@@ -787,7 +787,11 @@ def compile_route_to_url(self, route, params={}):
                 # if the url contains a parameter variable like @id:int
                 if '@' in url:
                     url = url.replace('@', '').split(':')[0]
-                    compiled_url += str(params[url]) + '/'
+                    if isinstance(params, dict):
+                        compiled_url += str(params[url]) + '/'
+                    elif isinstance(params, list):
+                        compiled_url += str(params[0]) + '/'
+                        del params[0]
                 else:
                     compiled_url += url + '/'
 

tests/test_requests.py

@@ -263,6 +263,7 @@ def test_request_route_returns_url(self):
 
         assert request.route('test.url') == '/test/url'
         assert request.route('test.id', {'id': 1}) == '/test/url/1'
+        assert request.route('test.id', [1]) == '/test/url/1'
 
     def test_request_redirection(self):
         app = App()
@@ -327,6 +328,21 @@ def test_redirect_compiles_url_with_parameters(self):
 
         assert request.compile_route_to_url(route, params) == '/test/1'
 
+    def test_redirect_compiles_url_with_list_parameters(self):
+        app = App()
+        app.bind('Request', self.request)
+        request = app.make('Request').load_app(app)
+
+        route = 'test/@id'
+        params = ['1']
+
+        assert request.compile_route_to_url(route, params) == '/test/1'
+
+        route = 'test/@id/@user/test/@slug'
+        params = ['1', '2', '3']
+
+        assert request.compile_route_to_url(route, params) == '/test/1/2/test/3'
+
     def test_redirect_compiles_url_with_multiple_parameters(self):
         app = App()
         app.bind('Request', self.request)