So far we have 3 +1 (mine included), 1 +0, no -1 in favor of no limit by
default. I'll change the patch accordingly.
Also, see below,
Hi devs,
Before Jerome applies the patch, I need to make a small change regarding
the following question:
Do we include in the feed by default the entire
description/content/summary of a feed entry or only the first N
characters? There is a ContentLength length property that could have by
default the -1 value, meaning the entire content, or N value, meaning
the
first N characters. For instance, in the current XWiki.BlogRssCode, the
default length is 400 characters (so not the entire content).$
I am +0 with no limit as default setting.
Another thing is jtidy printing a lot of warnings in the console.
According to Marius, there is a way to make it quiet, using
TIDY_FEED_CONFIG.setProperty("quiet", "yes");
But maybe we do not want it quiet anycase, so do you think it would be
possible to catch that log, and push it in the feed plugin logger ?
JTidy allows me to register a message listener. Thus I'll add a utility
class that catches all these messages and writes them to a given log4j
logger. If anyone has something against it please reply ASAP.
[snip]
Thanks,
Marius