Joshua Lucas wrote:
<snip>
> Cool! I still have an issue with the CocoonPortlet though.
>
> I don't believe we should be throwing a PortletException within getPI().
> The String style can be null which is why the additional getPI(Rundata
> rundata) method was created.
I moved your gePI(RunData rundata) logic into getPI() and included your
@author tag. It now checks in both locations and then throws a
PortletException if both aren't defined.
> I would say we take out the check for null as well as the PersistentDocument
> lookup. I believe this was in the patch I sent earlier but if not, I'll
> send you another one if you need it.
This would be bad. I don't want to fetch the content over HTTP all the
time. We would end up with extra TCP latency when going through the web
server.
The PersistentDocumentDaemon should make sure that this doesn't go out
of date. Also we should have an admin interface so that you can
manually refresh certain URLs.
Was there a reason you wanted this? You note said "it wasn't working".
I will see what is going on.
Kevin
--
Kevin A Burton
http://relativity.yi.org
Linux - The revolution will NOT be televised
--
--------------------------------------------------------------
Please read the FAQ! <http://java.apache.org/faq/>
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Archives and Other: <http://java.apache.org/main/mail.html>
Problems?: [EMAIL PROTECTED]
