FeedList widget had mismatching show() definition

This commit is contained in:
Mikael Nordfeldth 2015-06-23 12:26:44 +02:00
parent 1864a9de38
commit e6507a0f18
2 changed files with 21 additions and 24 deletions

View File

@ -534,17 +534,13 @@ class Action extends HTMLOutputter // lawsuit
*/ */
function showFeeds() function showFeeds()
{ {
$feeds = $this->getFeeds(); foreach ($this->getFeeds() as $feed) {
if ($feeds) {
foreach ($feeds as $feed) {
$this->element('link', array('rel' => $feed->rel(), $this->element('link', array('rel' => $feed->rel(),
'href' => $feed->url, 'href' => $feed->url,
'type' => $feed->mimeType(), 'type' => $feed->mimeType(),
'title' => $feed->title)); 'title' => $feed->title));
} }
} }
}
/** /**
* Show description. * Show description.
@ -1035,9 +1031,9 @@ class Action extends HTMLOutputter // lawsuit
function showExportData() function showExportData()
{ {
$feeds = $this->getFeeds(); $feeds = $this->getFeeds();
if ($feeds) { if (!empty($feeds)) {
$fl = new FeedList($this); $fl = new FeedList($this, $feeds);
$fl->show($feeds); $fl->show();
} }
} }
@ -1658,7 +1654,7 @@ class Action extends HTMLOutputter // lawsuit
*/ */
function getFeeds() function getFeeds()
{ {
return null; return array();
} }
/** /**

View File

@ -28,9 +28,7 @@
* @link http://status.net/ * @link http://status.net/
*/ */
if (!defined('STATUSNET') && !defined('LACONICA')) { if (!defined('GNUSOCIAL')) { exit(1); }
exit(1);
}
/** /**
* Widget for showing a list of feeds * Widget for showing a list of feeds
@ -50,30 +48,33 @@ class FeedList extends Widget
{ {
var $action = null; var $action = null;
function __construct($action=null) protected $feeds = null;
public function __construct(Action $action=null, array $feeds=array())
{ {
parent::__construct($action); parent::__construct($action);
$this->action = $action; $this->action = $action;
$this->feeds = $feeds;
} }
function show($feeds) public function show()
{ {
if (Event::handle('StartShowFeedLinkList', array($this->action, &$feeds))) { if (Event::handle('StartShowFeedLinkList', array($this->action, &$this->feeds))) {
if (!empty($feeds)) { if (!empty($this->feeds)) {
$this->out->elementStart('div', array('id' => 'export_data', $this->out->elementStart('div', array('id' => 'export_data',
'class' => 'section')); 'class' => 'section'));
// TRANS: Header for feed links (h2). // TRANS: Header for feed links (h2).
$this->out->element('h2', null, _('Feeds')); $this->out->element('h2', null, _('Feeds'));
$this->out->elementStart('ul', array('class' => 'xoxo')); $this->out->elementStart('ul', array('class' => 'xoxo'));
foreach ($feeds as $feed) { foreach ($this->feeds as $feed) {
$this->feedItem($feed); $this->feedItem($feed);
} }
$this->out->elementEnd('ul'); $this->out->elementEnd('ul');
$this->out->elementEnd('div'); $this->out->elementEnd('div');
} }
Event::handle('EndShowFeedLinkList', array($this->action, &$feeds)); Event::handle('EndShowFeedLinkList', array($this->action, &$this->feeds));
} }
} }