fix: avoid details route conflicting with count when overriding detai…

…ls route (#211)
ForestAdmin · Aug 3, 2021 · 0060bfd · 0060bfd
1 parent d31bcd7
commit 0060bfd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/services/dumper/templates/app/routes/route.hbs b/src/services/dumper/templates/app/routes/route.hbs
@@ -40,7 +40,7 @@ router.get('/{{ modelName }}/count', permissionMiddlewareCreator.list(), (reques
 });
 
 // Get a {{ modelNameReadableSingular }}
-router.get('/{{ modelName }}/:recordId(?!count)', permissionMiddlewareCreator.details(), (request, response, next) => {
+router.get('/{{ modelName }}/\\b(?!count\\b):recordId', permissionMiddlewareCreator.details(), (request, response, next) => {
   // Learn what this route does here: https://docs.forestadmin.com/documentation/v/v6/reference-guide/routes/default-routes#get-a-record
   next();
 });

diff --git a/test/services/analyzer/expected/sequelize/dumper-output/export.expected.route.js b/test/services/analyzer/expected/sequelize/dumper-output/export.expected.route.js
@@ -40,7 +40,7 @@ router.get('/modelExport/count', permissionMiddlewareCreator.list(), (request, r
 });
 
 // Get a Export
-router.get('/modelExport/:recordId(?!count)', permissionMiddlewareCreator.details(), (request, response, next) => {
+router.get('/modelExport/\\b(?!count\\b):recordId', permissionMiddlewareCreator.details(), (request, response, next) => {
   // Learn what this route does here: https://docs.forestadmin.com/documentation/v/v6/reference-guide/routes/default-routes#get-a-record
   next();
 });