diff --git a/plugins/Bookmark/BookmarkPlugin.php b/plugins/Bookmark/BookmarkPlugin.php index ae0f493abb..9b8addf632 100644 --- a/plugins/Bookmark/BookmarkPlugin.php +++ b/plugins/Bookmark/BookmarkPlugin.php @@ -290,6 +290,14 @@ class BookmarkPlugin extends Plugin return true; } + /** + * Add our two queue handlers to the queue manager + * + * @param QueueManager $qm current queue manager + * + * @return boolean hook value + */ + function onEndInitializeQueueManager($qm) { $qm->connect('dlcsback', 'DeliciousBackupImporter'); diff --git a/plugins/Bookmark/bookmarkpopup.php b/plugins/Bookmark/bookmarkpopup.php index 52a40de921..2e6d457a83 100644 --- a/plugins/Bookmark/bookmarkpopup.php +++ b/plugins/Bookmark/bookmarkpopup.php @@ -44,12 +44,11 @@ if (!defined('STATUSNET')) { */ class BookmarkpopupAction extends NewbookmarkAction { - function prepare($args) - { - $result = parent::prepare($args); - common_debug('Values: ' . $this->_title . ' ' . $this->_url); - return $result; - } + /** + * Show the title section of the window + * + * @return void + */ function showTitle() { @@ -60,6 +59,14 @@ class BookmarkpopupAction extends NewbookmarkAction common_config('site', 'name'))); } + /** + * Show the header section of the page + * + * Shows a stub page and the bookmark form. + * + * @return void + */ + function showHeader() { $this->elementStart('div', array('id' => 'header')); @@ -70,17 +77,29 @@ class BookmarkpopupAction extends NewbookmarkAction $this->elementEnd('address'); if (common_logged_in()) { $form = new BookmarkForm($this, - $this->_title, - $this->_url); + $this->title, + $this->url); $form->show(); } $this->elementEnd('div'); } + /** + * Hide the core section of the page + * + * @return void + */ + function showCore() { } + /** + * Hide the footer section of the page + * + * @return void + */ + function showFooter() { } diff --git a/plugins/Bookmark/newbookmark.php b/plugins/Bookmark/newbookmark.php index 7a11b08ad3..94d8f3f4f0 100644 --- a/plugins/Bookmark/newbookmark.php +++ b/plugins/Bookmark/newbookmark.php @@ -27,7 +27,6 @@ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html AGPL 3.0 * @link http://status.net/ */ - if (!defined('STATUSNET')) { // This check helps protect against security problems; // your code file can't be executed directly from the web. @@ -47,13 +46,13 @@ if (!defined('STATUSNET')) { class NewbookmarkAction extends Action { - protected $_user = null; - protected $_error = null; - protected $_complete = null; - protected $_title = null; - protected $_url = null; - protected $_tags = null; - protected $_description = null; + protected $user = null; + protected $error = null; + protected $complete = null; + protected $title = null; + protected $url = null; + protected $tags = null; + protected $description = null; /** * Returns the title of the action @@ -78,9 +77,9 @@ class NewbookmarkAction extends Action { parent::prepare($argarray); - $this->_user = common_current_user(); + $this->user = common_current_user(); - if (empty($this->_user)) { + if (empty($this->user)) { throw new ClientException(_("Must be logged in to post a bookmark."), 403); } @@ -89,10 +88,10 @@ class NewbookmarkAction extends Action $this->checkSessionToken(); } - $this->_title = $this->trimmed('title'); - $this->_url = $this->trimmed('url'); - $this->_tags = $this->trimmed('tags'); - $this->_description = $this->trimmed('description'); + $this->title = $this->trimmed('title'); + $this->url = $this->trimmed('url'); + $this->tags = $this->trimmed('tags'); + $this->description = $this->trimmed('description'); return true; } @@ -127,23 +126,23 @@ class NewbookmarkAction extends Action function newBookmark() { try { - if (empty($this->_title)) { + if (empty($this->title)) { throw new ClientException(_('Bookmark must have a title.')); } - if (empty($this->_url)) { + if (empty($this->url)) { throw new ClientException(_('Bookmark must have an URL.')); } - $saved = Notice_bookmark::saveNew($this->_user, - $this->_title, - $this->_url, - $this->_tags, - $this->_description); + $saved = Notice_bookmark::saveNew($this->user, + $this->title, + $this->url, + $this->tags, + $this->description); } catch (ClientException $ce) { - $this->_error = $ce->getMessage(); + $this->error = $ce->getMessage(); $this->showPage(); return; } @@ -159,15 +158,15 @@ class NewbookmarkAction extends Action function showContent() { - if (!empty($this->_error)) { - $this->element('p', 'error', $this->_error); + if (!empty($this->error)) { + $this->element('p', 'error', $this->error); } $form = new BookmarkForm($this, - $this->_title, - $this->_url, - $this->_tags, - $this->_description); + $this->title, + $this->url, + $this->tags, + $this->description); $form->show();