Skip to content

Commit a50c313

Browse files
authored
Merge pull request #62 from anthony-o/fix-for-TRANSREL-134
Fixing TRANSREL-134 without breaking functional tests and explaining …
2 parents aa10af7 + 76c4206 commit a50c313

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

grails-app/conf/UrlMappings.groovy

Lines changed: 19 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,21 @@
11
class UrlMappings {
2-
/* This empty mappings file seems to be required when inlining core-db
3-
* plugins. Otherwise we get:
4-
* Class not found loading Grails application: UrlMappings (https://travis-ci.org/transmart/SmartR/builds/163942859)
5-
* Inspired by https://github.com/transmart/transmart-rest-api/blob/v1.2.4/grails-app/conf/UrlMappings.groovy */
6-
static mappings = { }
2+
/**
3+
* This mappings file seems to be required when inlining core-db plugins
4+
* (as stated here https://github.com/transmart/transmart-rest-api/blob/v1.2.4/grails-app/conf/UrlMappings.groovy).
5+
* Without it, we get "Class not found loading Grails application: UrlMappings"
6+
* (https://travis-ci.org/transmart/SmartR/builds/163942859)
7+
*/
8+
9+
static mappings = {
10+
// Following block seems to be required by functional tests (BaseAPITestCase.groovy)
11+
"/$controller/$action?/$id?(.${format})?"{
12+
constraints {
13+
// apply constraints here
14+
}
15+
}
16+
17+
// Removed the two following lines in order to not interfere with transmartApp url mappings - TRANSREL-134
18+
//"/"(view:"/index")
19+
//"500"(view:'/error')
20+
}
721
}

0 commit comments

Comments
 (0)