Ticket #67 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

add author information to feed items

Reported by: mdodoo Owned by: mbonetti
Priority: normal Milestone: Gregarius 0.4.5
Component: FEATURES Version:
Severity: normal Keywords: items authors
Cc:

Description

Since some blogs have more than one author (i.e. Daily Kos, Slashdot, Boing Boing, etc.), adding information about authors to the items table would allow Gregarius users to distinguish between these different posts. It might also be possible to allow the author information be a hyperlink leading to a page displaying all items from that author, regardless of feed.

Attachments

add_author.patch (6.5 kB) - added by julien 3 years ago.
Patch versus revision 649
new_add_author.patch (7.0 kB) - added by julien 3 years ago.
new version of my patch, which includes correction to bug #68

Change History

Changed 3 years ago by mbonetti

  • component changed from BUGS to FEATURES
  • milestone set to Gregarius 0.4.5

IIRC Julien has a patch that does just this.

Changed 3 years ago by julien

Yes, I have such a patch... but as I haven't updated my gregarius installation for the past three weeks, it certainly is outdated. I'll have a look later today to see if I can adapt it to the new OO architecture and post it here.

Changed 3 years ago by julien

Patch versus revision 649

Changed 3 years ago by julien

Here's my patch against revision 649. Be warned : it modifies the default template (so as to display the author name) and the database schema (adds a column "author" in the table "item" to store the author name). Works fine on my installation, but I didn't run tests against other installations.

Changed 3 years ago by julien

Warning: my patch might have some trouble with the rss.config.datedesc config option. I'm looking into it.

Changed 3 years ago by julien

new version of my patch, which includes correction to bug #68

Changed 3 years ago by julien

The patch I just attached obsoletes the preceding one. This new version corrects the bug I found with rss.config.datedesc just above and also includes my patch for ticket #68 (as both problems are in fact the same).

Changed 3 years ago by julien

My final note on this bug for today: my patch just adds the displaying of the author, it doesn't include an hyperlink for a page with all items by the same author.

Changed 3 years ago by julien

One small remark regarding changeset [651]: there's a small "naming convention" problem in cls/items.php on lines 309 and 312. The $iauthor variable is the only one that doesn't end in "_" in this part of the code.

Changed 3 years ago by mbonetti

  • status changed from new to closed
  • resolution set to fixed

Fixed in [655] (I hope this is correct)

Note: See TracTickets for help on using tickets.