[ 
https://issues.apache.org/jira/browse/APA-32?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12849666#action_12849666
 ] 

Woonsan Ko commented on APA-32:
-------------------------------

Hi Gonzalo,

It seems very nice and I think it's a good option!
However, I think we need to keep the default preference value as it used to be.
I mean, you added the following, but it could make some problems to the 
existing rss portlets which have assumed proper escaping.

+               <preference>
+                       <name>disableescape</name>
+                       <value>true</value>             
+               </preference>

Therefore, I think we'd better the above default preference should be 'false' 
as it worked before.
Is it okay to change the default value to false?

Kind regards,

Woonsan

> Apa-RSS Include an option to disable html escape
> ------------------------------------------------
>
>                 Key: APA-32
>                 URL: https://issues.apache.org/jira/browse/APA-32
>             Project: Portals Apps
>          Issue Type: Improvement
>          Components: apa-rss
>    Affects Versions: apa-rss-1.1
>            Reporter: Gonzalo Aguilar
>            Assignee: David Sean Taylor
>             Fix For: apa-rss-1.1
>
>         Attachments: RSS-escape-option-apa-rss-APA-32.patch
>
>
> This issue is created to include support for full html support inside a RSS 
> Feed. 
> Now if a feed contains html markup it will be escaped and the html code will 
> be shown instead of showing the correct markup.

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

Reply via email to