Rss Application - Use the Rome Framework

• Key: JS2-191
• Type: Improvement
• Status: Resolved
• Resolution: Fixed
• Priority: Major
• Assignee: David Sean Taylor
• Reporter: Christophe Lombart
• Component/s: Demo Portlets
• Affects Version/s: 2.0-M1

Description

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

Comments

[Christophe Lombart]

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

[David Sean Taylor]

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

[Christophe Lombart]

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.

[David Sean Taylor]

"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.

[Christophe Lombart]

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

[David Sean Taylor]

patch applied and tested.