[Seaside-dev] Issue 315 in seaside: WAResourceBaseUrlTest>>#'createHandlers' references WAApplication that is not defined

[codesite-noreply at google.com]

Status: Accepted
Owner: ----
Labels: Type-Defect Priority-Medium Version-Seaside2.9

New issue 315 by WeybridgeWay: WAResourceBaseUrlTest>>#'createHandlers'  
references WAApplication that is not defined
http://code.google.com/p/seaside/issues/detail?id=315

What steps will reproduce the problem?
     WAResourceBaseUrlTest suite run.

Comment from jfitzell in #311:

The problem here is WARenderContext>>absoluteUrlForResource: which is in  
Core but calls #application. There are 6 senders of
that, two of which are also in Core, the other 4 in Canvas. The 3 methods  
in Core could be made class extensions from the Session
package. The 4 methods in Canvas are a problem: we don't hit a package  
which depends on Application *and* Canvas until
RenderLoop and I still want to break that single dependency somehow. So to  
deal with these 4 methods as well as their tests, we
would really need to either:
   a) move them up to Environment;
   b) create a new package that provides added functionality when using  
Application and Canvas together; or
   c) create a Session-specific subclass of RenderContext.

I suggest either doing nothing or doing (a) for the moment and (in either  
case) filing a separate issue for this one so we can see if
(b) or (c) makes sense in the long run.



--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings