Merge pull request #175 from eadwinCode/operation_id_fix

fix: Operational ID generation starting with a number

Author: eadwinCode

ninja_extra/controllers/base.py

@@ -450,7 +450,10 @@ def __str__(self) -> str:  # pragma: no cover
     def _add_operation_from_route_function(self, route_function: RouteFunction) -> None:
         # converts route functions to Operation model
         if route_function.route.route_params.operation_id is None:
-            route_function.route.route_params.operation_id = f"{str(uuid.uuid4())[:8]}_controller_{route_function.route.view_func.__name__}"
+            controller_name = (
+                str(self.controller_class.__name__).lower().replace("controller", "")
+            )
+            route_function.route.route_params.operation_id = f"{controller_name}_{route_function.route.view_func.__name__}_{str(uuid.uuid4())[:8]}"
 
         if (
             self.auth

ninja_extra/controllers/model/builder.py

@@ -68,7 +68,7 @@ def _register_create_endpoint(self) -> None:
 
     def _register_update_endpoint(self) -> None:
         _path = "/{%s:%s}" % (
-            self._pk_type.__name__,
+            self._pk_type.__name__.lower(),
             self._model_pk_name,
         )
         kw = {
@@ -91,7 +91,7 @@ def _register_update_endpoint(self) -> None:
     def _register_patch_endpoint(self) -> None:
         _pk_type = self._pk_type
         _path = "/{%s:%s}" % (
-            _pk_type.__name__,
+            _pk_type.__name__.lower(),
             self._model_pk_name,
         )
 
@@ -114,7 +114,7 @@ def _register_patch_endpoint(self) -> None:
 
     def _register_find_one_endpoint(self) -> None:
         _path = "/{%s:%s}" % (
-            self._pk_type.__name__,
+            self._pk_type.__name__.lower(),
             self._model_pk_name,
         )
         kw = {
@@ -163,7 +163,7 @@ def _register_list_endpoint(self) -> None:
 
     def _register_delete_endpoint(self) -> None:
         _path = "/{%s:%s}" % (
-            self._pk_type.__name__,
+            self._pk_type.__name__.lower(),
             self._model_pk_name,
         )
         kw = {