[xwiki-devs] [VOTE] Change default published date for XWikiDocument#getDate() instead of instead of #getCreationDate()
17 Mar
2011
17 Mar
'11
7:31 p.m.
Hi devs,
I was looking at http://jira.xwiki.org/jira/browse/XWIKI-5336 and
basically I'm like Jens, I don't understand why we are using creation
date in feed plugin.
So I'm proposing to change it for date (last version date).
FYI I'm planning to change
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getCreationDate();
}
to
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getDate();
}
in com.xpn.xwiki.plugin.feed.SyndEntryDocumentSource
WDYT ?
Here is my +1.
--
Thomas Mortagne
17 Mar
17 Mar
7:33 p.m.
+1
On Thu, Mar 17, 2011 at 7:31 PM, Thomas Mortagne
<thomas.mortagne(a)xwiki.com> wrote:
Hi devs,
I was looking at http://jira.xwiki.org/jira/browse/XWIKI-5336 and
basically I'm like Jens, I don't understand why we are using creation
date in feed plugin.
So I'm proposing to change it for date (last version date).
FYI I'm planning to change
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getCreationDate();
}
to
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getDate();
}
in com.xpn.xwiki.plugin.feed.SyndEntryDocumentSource
WDYT ?
Here is my +1.
--
Thomas Mortagne
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
18 Mar
18 Mar
6:07 p.m.
+1
On Thu, Mar 17, 2011 at 7:31 PM, Thomas Mortagne
<thomas.mortagne(a)xwiki.com> wrote:
Hi devs,
I was looking at http://jira.xwiki.org/jira/browse/XWIKI-5336 and
basically I'm like Jens, I don't understand why we are using creation
date in feed plugin.
So I'm proposing to change it for date (last version date).
FYI I'm planning to change
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getCreationDate();
}
to
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getDate();
}
in com.xpn.xwiki.plugin.feed.SyndEntryDocumentSource
WDYT ?
Here is my +1.
--
Thomas Mortagne
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
6:10 p.m.
On 03/17/2011 07:31 PM, Thomas Mortagne wrote:
Hi devs,
I was looking at http://jira.xwiki.org/jira/browse/XWIKI-5336 and
basically I'm like Jens, I don't understand why we are using creation
date in feed plugin.
So I'm proposing to change it for date (last version date).
FYI I'm planning to change
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getCreationDate();
}
to
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getDate();
}
in com.xpn.xwiki.plugin.feed.SyndEntryDocumentSource
WDYT ?
It depends on what the feed is supposed to show.
If it's a feed of "new items", it should (I'm 60% for of this) use the
creation date. For example the feed of new blog posts, the feed of new
users, etc.
If it's a feed of changes in the wiki, then it should use the
modification date.
So I'm +0 for changing the default date, I'd rather make this
configurable, as a parameter to the feed generation method.
+ since most feeds exposed by XWiki are about changes, and the Blog
currently doesn't use the feed plugin, so it won't be affected.
--
Sergiu Dumitriu
http://purl.org/net/sergiu/
22 Mar
22 Mar
8:58 a.m.
+0 like Sergiu
Denis
On Fri, Mar 18, 2011 at 18:10, Sergiu Dumitriu <sergiu(a)xwiki.com> wrote:
On 03/17/2011 07:31 PM, Thomas Mortagne wrote:
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
Hi devs,
I was looking at http://jira.xwiki.org/jira/browse/XWIKI-5336 and
basically I'm like Jens, I don't understand why we are using creation
date in feed plugin.
So I'm proposing to change it for date (last version date).
FYI I'm planning to change
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getCreationDate();
}
to
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getDate();
}
in com.xpn.xwiki.plugin.feed.SyndEntryDocumentSource
WDYT ?
It depends on what the feed is supposed to show.
If it's a feed of "new items", it should (I'm 60% for of this) use the
creation date. For example the feed of new blog posts, the feed of new
users, etc.
If it's a feed of changes in the wiki, then it should use the
modification date.
So I'm +0 for changing the default date, I'd rather make this
configurable, as a parameter to the feed generation method.
+ since most feeds exposed by XWiki are about changes, and the Blog
currently doesn't use the feed plugin, so it won't be affected.
--
Sergiu Dumitriu
http://purl.org/net/sergiu/
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
21 Mar
21 Mar
11:35 a.m.
+0
Thanks,
Marius
On 03/17/2011 08:31 PM, Thomas Mortagne wrote:
Hi devs,
I was looking at http://jira.xwiki.org/jira/browse/XWIKI-5336 and
basically I'm like Jens, I don't understand why we are using creation
date in feed plugin.
So I'm proposing to change it for date (last version date).
FYI I'm planning to change
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getCreationDate();
}
to
protected Date getDefaultPublishedDate(Document doc, Map<String,
Object> params, XWikiContext context)
{
return doc.getDate();
}
in com.xpn.xwiki.plugin.feed.SyndEntryDocumentSource
WDYT ?
Here is my +1.
5024
days inactive
5029
days old
5 comments
6 participants
participants (6)
-
Denis Gervalle -
Jerome Velociter -
Marius Dumitru Florea -
Marta Girdea -
Sergiu Dumitriu -
Thomas Mortagne