You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-dev@portals.apache.org by "Daniel Krieg (JIRA)" <ji...@apache.org> on 2006/12/12 06:56:21 UTC

[jira] Created: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Do not force cross servlet contex when portlet window context path is same as servlet context path
--------------------------------------------------------------------------------------------------

                 Key: PLUTO-276
                 URL: http://issues.apache.org/jira/browse/PLUTO-276
             Project: Pluto
          Issue Type: Improvement
          Components: portlet container
    Affects Versions: Current CVS
            Reporter: Daniel Krieg


Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "Daniel Krieg (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/PLUTO-276?page=all ]

Daniel Krieg updated PLUTO-276:
-------------------------------

    Attachment: pluto-1.1.0-crossContextFree.diff

This patch corrects pluto-container components by comparing the portletWindow contextPath with the servletContext contextPath and only if they are different does it issue servletContext.getContext(contextPath).  I have testing this patch on Tomcat 5.5.9 and I have successfully utilized pluto-portal as a 'starter' webapp and have been able to develop and test new portlets with this webapp.  One issue that I encountered for which I am creating another ticket is that the Castor mapping is unable to unmarshall the portlet.xml file unless the root element excludes the namespace attributes as such:
<portlet-app> 
...instead of ...
<portlet-app
    xmlns="http://java.sun.com/xml/ns/portlet/portlet-app_1_0.xsd"
    version="1.0"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://java.sun.com/xml/ns/portlet/portlet-app_1_0.xsd
                        http://java.sun.com/xml/ns/portlet/portlet-app_1_0.xsd">


> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>         Attachments: pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "Daniel Krieg (JIRA)" <ji...@apache.org>.
    [ http://issues.apache.org/jira/browse/PLUTO-276?page=comments#action_12458490 ] 
            
Daniel Krieg commented on PLUTO-276:
------------------------------------

sorry, should be:

patch -p6 < pluto-1.1.0-crossContextFree.diff

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>             Fix For: 1.1.1
>
>         Attachments: pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "Daniel Krieg (JIRA)" <ji...@apache.org>.
    [ http://issues.apache.org/jira/browse/PLUTO-276?page=comments#action_12458479 ] 
            
Daniel Krieg commented on PLUTO-276:
------------------------------------

I wil happily resubmit the path but applying the patch can be done as follows:

patch -p7 < pluto-1.1.0-crossContextFree.diff

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>             Fix For: 1.1.1
>
>         Attachments: pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "Daniel Krieg (JIRA)" <ji...@apache.org>.
    [ http://issues.apache.org/jira/browse/PLUTO-276?page=comments#action_12457594 ] 
            
Daniel Krieg commented on PLUTO-276:
------------------------------------

The afforementioned tick is PLUTO-277

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>         Attachments: pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "David DeWolf (JIRA)" <ji...@apache.org>.
    [ http://issues.apache.org/jira/browse/PLUTO-276?page=comments#action_12458338 ] 
            
David DeWolf commented on PLUTO-276:
------------------------------------

at first blush, I like this patch. Do you mind recreating it so that it doesn't include absolute paths (e.g. c://. . .).  Simply doing:

svn di > context.patch

from the root pluto src directory would be the best way to create it.

Thanks!

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>         Attachments: pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Resolved: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "David DeWolf (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/PLUTO-276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David DeWolf resolved PLUTO-276.
--------------------------------

    Resolution: Fixed
      Assignee: David DeWolf

Fixed; Not with patch, but a little different.

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: https://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>         Assigned To: David DeWolf
>             Fix For: 1.1.1
>
>         Attachments: pluto-1.1.0-crossContextFree.diff, pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "Daniel Krieg (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/PLUTO-276?page=all ]

Daniel Krieg updated PLUTO-276:
-------------------------------

    Attachment: pluto-1.1.0-crossContextFree.diff

This is the revised patch.

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>             Fix For: 1.1.1
>
>         Attachments: pluto-1.1.0-crossContextFree.diff, pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Updated: (PLUTO-276) Do not force cross servlet contex when portlet window context path is same as servlet context path

Posted by "David DeWolf (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/PLUTO-276?page=all ]

David DeWolf updated PLUTO-276:
-------------------------------

    Fix Version/s: 1.1.1

> Do not force cross servlet contex when portlet window context path is same as servlet context path
> --------------------------------------------------------------------------------------------------
>
>                 Key: PLUTO-276
>                 URL: http://issues.apache.org/jira/browse/PLUTO-276
>             Project: Pluto
>          Issue Type: Improvement
>          Components: portlet container
>    Affects Versions: Current CVS
>            Reporter: Daniel Krieg
>             Fix For: 1.1.1
>
>         Attachments: pluto-1.1.0-crossContextFree.diff
>
>
> Several component issue servletContext.getContext(contextPath) thus forcing a 'crossContext' enabled deployment senario and preventing a truly embedded Pluto implementation.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira