Code review comment for lp:~leonardr/lazr.restful/web-link

Revision history for this message
Tim Penhey (thumper) wrote :

> if self.adapter_utility.publish_web_link:
> # Objects in the web service correspond to pages on some website.
> # Provide the link to the corresponding page on the website.
> browser_request = IWebBrowserOriginatingRequest(self.request)

Can we be sure here that there is an adapter for request to a
IWebBrowserOriginatingRequest?

I'm pretty sure that you can rewrite:
  queryAdapter(web_service_request, IWebBrowserOriginatingRequest)
as
  IWebBrowserOriginatingRequest(web_service_request, None)

Not sure if you think this is better or not.

review: Needs Information

« Back to merge proposal