You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by "Christophe Lombart (JIRA)" <je...@jakarta.apache.org> on 2005/01/14 00:23:17 UTC

[jira] Created: (JS2-191) Rss Application - Use the Rome Framework

Rss Application - Use the Rome Framework
----------------------------------------

         Key: JS2-191
         URL: http://issues.apache.org/jira/browse/JS2-191
     Project: Jetspeed 2
        Type: Improvement
  Components: Demo Portlets  
    Versions: 2.0-M1    
    Reporter: Christophe Lombart
 Attachments: patch-rss.txt

In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Commented: (JS2-191) Rss Application - Use the Rome Framework

Posted by "David Sean Taylor (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=comments#action_57614 ]
     
David Sean Taylor commented on JS2-191:
---------------------------------------

I can't accept this patch.

You deleted the RSS XSLT portlet, and all the transformation cache code that worked very hard on.
My goal was to have two RSS portlets:

1. XSLT based
2. Rome based

Why did you remove the XSLT version and transform cache?

If you want to leave the XSLT (RSS) portlet as is, and put back the cache, then I'll be glad to check it in as the Rome implementation looks great.

Thanks

> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>  Attachments: patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Updated: (JS2-191) Rss Application - Use the Rome Framework

Posted by "Christophe Lombart (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=history ]

Christophe Lombart updated JS2-191:
-----------------------------------

    Attachment: patch-rss.txt

Here is a new patch. Both portlets should be there.

> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>     Assignee: David Sean Taylor
>  Attachments: patch-rss.txt, patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Resolved: (JS2-191) Rss Application - Use the Rome Framework

Posted by "David Sean Taylor (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=history ]
     
David Sean Taylor resolved JS2-191:
-----------------------------------

    Resolution: Fixed

patch applied and tested.

> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>     Assignee: David Sean Taylor
>  Attachments: patch-rss.txt, patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Assigned: (JS2-191) Rss Application - Use the Rome Framework

Posted by "David Sean Taylor (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=history ]

David Sean Taylor reassigned JS2-191:
-------------------------------------

    Assign To: David Sean Taylor

> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>     Assignee: David Sean Taylor
>  Attachments: patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Commented: (JS2-191) Rss Application - Use the Rome Framework

Posted by "Christophe Lombart (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=comments#action_57618 ]
     
Christophe Lombart commented on JS2-191:
----------------------------------------

No problem. I will send you a new patch this evening. 

Why do you want to have both implementation to have the same result ? As you suggested, Rome+Velocity is so simple.





> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>     Assignee: David Sean Taylor
>  Attachments: patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Commented: (JS2-191) Rss Application - Use the Rome Framework

Posted by "David Sean Taylor (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=comments#action_57635 ]
     
David Sean Taylor commented on JS2-191:
---------------------------------------

"Why do you want to have both implementation to have the same result ? As you suggested, Rome+Velocity is so simple."

Because its an open source project an lots of people work with XSLT, no matter how much I dislike it :-)

There are lots of existing transforms for RSS that can be easily integrated.

> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>     Assignee: David Sean Taylor
>  Attachments: patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


[jira] Updated: (JS2-191) Rss Application - Use the Rome Framework

Posted by "Christophe Lombart (JIRA)" <je...@jakarta.apache.org>.
     [ http://issues.apache.org/jira/browse/JS2-191?page=history ]

Christophe Lombart updated JS2-191:
-----------------------------------

    Attachment: patch-rss.txt

Here is the patch. Let me know if something is wrong. This patch doesn't contains some caching features.

> Rss Application - Use the Rome Framework
> ----------------------------------------
>
>          Key: JS2-191
>          URL: http://issues.apache.org/jira/browse/JS2-191
>      Project: Jetspeed 2
>         Type: Improvement
>   Components: Demo Portlets
>     Versions: 2.0-M1
>     Reporter: Christophe Lombart
>  Attachments: patch-rss.txt
>
> In order be to more generic, The RSS application can used the Rome framework. By this way, XSL is not necessary.

-- 
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
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org