Merge pull request #159 from CLIxIndia-Dev/hotfix/cs/handle-ids-with-…

…trailing-slash

better regex for ids with trailing slash in URL

assessment/assessment.py

@@ -103,29 +103,29 @@
     "/banks/(.*)/assessmentstaken/(.*)/questions/(.*)/qti/?", "AssessmentTakenQuestionQTIDetails",
     "/banks/(.*)/assessmentstaken/(.*)/questions/(.*)/status/?", "AssessmentTakenQuestionStatus",
     "/banks/(.*)/assessmentstaken/(.*)/questions/(.*)/submit/?", "AssessmentTakenQuestionSubmit",
-    "/banks/(.*)/assessmentstaken/(.*)/questions/(.*)/?", "AssessmentTakenQuestionDetails",
+    "/banks/(.*)/assessmentstaken/(.*)/questions/(.*[^/])/?", "AssessmentTakenQuestionDetails",
     "/banks/(.*)/assessmentstaken/(.*)/questions/?", "AssessmentTakenQuestions",
     "/banks/(.*)/assessmentstaken/(.*)/finish/?", "FinishAssessmentTaken",
-    "/banks/(.*)/assessmentstaken/(.*)/?", "AssessmentTakenDetails",
+    "/banks/(.*)/assessmentstaken/(.*[^/])/?", "AssessmentTakenDetails",
     "/banks/(.*)/assessments/(.*)/assessmentsoffered/?", "AssessmentsOffered",
     "/banks/(.*)/assessmentsoffered/?", "AssessmentsOffered",
-    "/banks/(.*)/assessmentsoffered/(.*)/?", "AssessmentOfferedDetails",
-    "/banks/(.*)/assessments/(.*)/assignedbankids/(.*)/?", "AssessmentRemoveAssignedBankIds",
+    "/banks/(.*)/assessmentsoffered/(.*[^/])/?", "AssessmentOfferedDetails",
+    "/banks/(.*)/assessments/(.*)/assignedbankids/(.*[^/])/?", "AssessmentRemoveAssignedBankIds",
     "/banks/(.*)/assessments/(.*)/assignedbankids/?", "AssessmentAssignedBankIds",
-    "/banks/(.*)/assessments/(.*)/items/(.*)/?", "AssessmentItemDetails",
+    "/banks/(.*)/assessments/(.*)/items/(.*[^/])/?", "AssessmentItemDetails",
     "/banks/(.*)/assessments/(.*)/items/?", "AssessmentItemsList",
-    "/banks/(.*)/assessments/(.*)/?", "AssessmentDetails",
+    "/banks/(.*)/assessments/(.*[^/])/?", "AssessmentDetails",
     "/banks/(.*)/assessments/?", "AssessmentsList",
     "/banks/(.*)/items/(.*)/videoreplacement/?", "ItemVideoTagReplacement",
     "/banks/(.*)/items/(.*)/qti/?", "ItemQTIDetails",
-    "/banks/(.*)/items/(.*)/?", "ItemDetails",
+    "/banks/(.*)/items/(.*[^/])/?", "ItemDetails",
     "/banks/(.*)/items/?", "ItemsList",
     "/banks/(.*)/?", "AssessmentBankDetails",
     "/banks/?", "AssessmentBanksList",
-    "/hierarchies/roots/(.*)/?", "AssessmentHierarchiesRootDetails",
+    "/hierarchies/roots/(.*[^/])/?", "AssessmentHierarchiesRootDetails",
     "/hierarchies/roots/?", "AssessmentHierarchiesRootsList",
     "/hierarchies/nodes/(.*)/children/?", "AssessmentHierarchiesNodeChildrenList",
-    "/hierarchies/nodes/(.*)/?", "AssessmentHierarchiesNodeDetails"
+    "/hierarchies/nodes/(.*[^/])/?", "AssessmentHierarchiesNodeDetails"
 )
 
 

logging_/logging_.py

@@ -18,9 +18,9 @@
 
 urls = (
     "/logs/?", "LogsList",
-    "/logs/(.*)/logentries/(.*)/?", "LogEntryDetails",
+    "/logs/(.*)/logentries/(.*[^/])/?", "LogEntryDetails",
     "/logs/(.*)/logentries/?", "LogEntriesList",
-    "/logs/(.*)/?", "LogDetails",
+    "/logs/(.*[^/])/?", "LogDetails",
     "/genericlog/?", "GenericLogEntries"
 )
 

repository/repository.py

@@ -15,11 +15,11 @@
 
 urls = (
     "/repositories/(.*)/assets/(.*)/contents/(.*)/stream/?", "AssetContentStream",
-    "/repositories/(.*)/assets/(.*)/contents/(.*)/?", "AssetContentDetails",
+    "/repositories/(.*)/assets/(.*)/contents/(.*[^/])/?", "AssetContentDetails",
     "/repositories/(.*)/assets/(.*)/contents/?", "AssetContentsList",
-    "/repositories/(.*)/assets/(.*)/?", "AssetDetails",
+    "/repositories/(.*)/assets/(.*[^/])/?", "AssetDetails",
     "/repositories/(.*)/assets/?", "AssetsList",
-    "/repositories/(.*)/?", "RepositoryDetails",
+    "/repositories/(.*[^/])/?", "RepositoryDetails",
     "/repositories/?", "RepositoriesList"
 )