New subject: [xwiki-devs] [Issue] Issue Comment Edited: (XWIKI-2882) Implement RSS macro for new rendering

29 Jan 2009

Thomas Mortagne (JIRA) wrote:
...
      [
http://jira.xwiki.org/jira/browse/XWIKI-2882?page=com.atlassian.jira.plugin…
] 

 Thomas Mortagne edited comment on XWIKI-2882 at 1/28/09 3:40 PM:
 -----------------------------------------------------------------

 *  this macro should not depends on radeox, expecially if it's just to use
org.radeox.util.Encoder. You should directly generate the List<Block> and not
generate a html string and call html macro to do it in RssMacro.renderSearch.

 * in RssMacro.renderEntries() you should use ParserUtils.parseInlineNonWiki to parse the
description, WordBlock is supposed to contains only one work and not an entire paragraph.

 * never use wiki syntax in macros
   -- macro is supposed to not be syntax dependent
   -- a macro is supposed to work on XDOM level
   -- create directly the right block is way faster
 If the problem is that you can't pass List<Block> for the title to
BoxMacroParameters, do it another way:
  -- you could define a protected List<Box> getTitle() methods in AbstractBoxMacro
like you have getClassProperty() and extend it with a RSSEntryMacro class
  -- you could also add a set/getBlockTitle in BoxMacroParameters based on
List<Block> which can be used from java (we could even imagine to be able to use it
from wiki syntax with the tight BeanUtil converter)
  -- ... Regarding the last issue, about whether to  use or not  wiki syntax for 
rendering the title I must say I'm a bit puzzled. I thought that the 
point of allowing wiki syntax to be specified in the title for the 
boxMacro was right this, to properly render the titles of the rss 
entries in the RssMacro.
................................

[15:42:26] Vincent Massol: now re the links I'm not sure yet
[15:42:37] … there are different ways of doing that I guess
[15:43:29] … one idea (brainstorming): we could allow inline wiki syntax 
to be used the title parameter maybe
[15:44:14] Dan Miron: yes, it could be a way
[15:44:17] Thomas Mortagne: I like that yes
[15:46:45] Vincent Massol: hmm I wonder if this is something we'd want 
for all macros
[15:47:09] … not sure
[15:47:17] Thomas Mortagne: at wors we can add a isTitleWiki
[15:47:19] Vincent Massol: even for this use case I'm stil not fully sure
[15:47:51] … it sounds good but I wonder if we'd get side effects
[15:48:09] … actually we wan't do that in a generic way for all macros
[15:48:20] … since it's not going to generate text
[15:48:25] … but Blocks
[15:48:32] … so the macro needs to handle that
[15:48:50] … s/Wan't/can't/
[15:49:33] … ok I think I'm +1 for making the image and title params 
allow inline wiki syntax here for this macro
[15:49:45] … (it's a good experiment, we'll see how it goes)
[15:49:49] … wdyt?
[15:49:53] Thomas Mortagne: +1
[15:50:03] Jean-Vincent Drean: +1

.................................. 

Anyway, if we need to quit using wiki syntax in the title, i'd prefer 
the second choice, "add a set/getBlockTitle in BoxMacroParameters based 
on List<Block> which can be used from java". Should I go on with this?

Tnx,
Dan

Re: [xwiki-devs] [Issue] Issue Comment Edited: (XWIKI-2882) Implement RSS macro for new rendering