Grails

Support reverse lookup of REST URL mappings

Details

  • Type: Sub-task Sub-task
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 2.0.1
  • Fix Version/s: None
  • Component/s: TagLib, URL mappings
  • Labels:
    None

Description

Reverse lookup for a mapping like this:

"/api/books"(controller: "book") {
    action = [GET: "apiShow", POST: "apiNew"]
}

doesn't work if you provide an 'action' attribute to any of the link generation tags. It only works if you leave 'action' out. This is pretty confusing, so it would be good if createLink(controller: "book", action: "apiNew") generated "/api/books".

To demonstrate the issue, unpack the attached bug report, run the application, and open the home page. The two "API ..." links displayed on the page link to a different location than "/api/books".

Issue Links

is related to

Sub-task - The sub-task of the issue GRAILS-9326 default controller name not applied, custom mapping in URLMappings applied

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Sub-task - The sub-task of the issue GRAILS-2843 Form tag doesn't generate correct URL when REST-ful HTTP protocol is used

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Open - The issue is open and ready for the assignee to start work on it.

Bug - A problem which impairs or prevents the functions of the product. GRAILS-4588 reverse URL mapping breaks in controllers with alternate REST mappings

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.
relates to

Sub-task - The sub-task of the issue GRAILS-2843 Form tag doesn't generate correct URL when REST-ful HTTP protocol is used

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Open - The issue is open and ready for the assignee to start work on it.

Bug - A problem which impairs or prevents the functions of the product. GRAILS-5207 Reverse URL mappings / redirect to REST url mapping fails

  • Blocker - Blocks development and/or testing work, production could not run.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Rick Jensen added a comment -

GRAILS-2843 is a specific instance of this, or is at least closely related.

Show
Rick Jensen added a comment - GRAILS-2843 is a specific instance of this, or is at least closely related.
Hide
Permalink
Iván López added a comment -

I think the main problem here is because grails "enforces you" in some way to use createLink, redirect,... using controller/action values instead of using named URLs.
I think that the default use should be creating a named URL and use it in all the application. There are only advantages:

  • Easy refactors for controllers. It is only necessary to change the action in the URL mappings file and not in all controllers and GSPs.
  • Write less code. You only write createLink(mapping:'myMapping') instead of createLink (controller:'myController',action:'myAction')

As a final thought, in all our grails project we always use named URLs because we have leant from our previous errors in others projects.

Show
Iván López added a comment - I think the main problem here is because grails "enforces you" in some way to use createLink, redirect,... using controller/action values instead of using named URLs. I think that the default use should be creating a named URL and use it in all the application. There are only advantages: Easy refactors for controllers. It is only necessary to change the action in the URL mappings file and not in all controllers and GSPs. Write less code. You only write createLink(mapping:'myMapping') instead of createLink (controller:'myController',action:'myAction') As a final thought, in all our grails project we always use named URLs because we have leant from our previous errors in others projects.

People

  • Assignee:
    Unassigned
    Reporter:
    Peter Ledbrook
Vote (2)
Watch (2)

Dates

  • Created:
    Updated:
    Last Reviewed: