git.gnu.io has moved to IP address 209.51.188.249 -- please double check where you are logging in.

Commit 56e2bc10 authored by Brion Vibber's avatar Brion Vibber

Cleanup stray PHP 4-style references in hook calls for navigation bars. We...

Cleanup stray PHP 4-style references in hook calls for navigation bars. We can't replace the live action from here, and don't need a reference to keep the object mutable. Dumping the references helps ensure we don't end up getting errors when things calling the hooks might forget to use the reference and the PHP error reporting settings expose this fact at us.
parent d0d1257d
......@@ -96,7 +96,7 @@ class AccountSettingsNav extends Widget
$action_name = $this->action->trimmed('action');
$this->action->elementStart('ul', array('class' => 'nav'));
if (Event::handle('StartAccountSettingsNav', array(&$this->action))) {
if (Event::handle('StartAccountSettingsNav', array($this->action))) {
$user = common_current_user();
if(Event::handle('StartAccountSettingsProfileMenuItem', array($this, &$menu))){
......@@ -142,7 +142,7 @@ class AccountSettingsNav extends Widget
Event::handle('EndAccountSettingsOtherMenuItem', array($this, &$menu));
}
Event::handle('EndAccountSettingsNav', array(&$this->action));
Event::handle('EndAccountSettingsNav', array($this->action));
}
$this->action->elementEnd('ul');
......
......@@ -96,7 +96,7 @@ class ConnectSettingsNav extends Widget
$action_name = $this->action->trimmed('action');
$this->action->elementStart('ul', array('class' => 'nav'));
if (Event::handle('StartConnectSettingsNav', array(&$this->action))) {
if (Event::handle('StartConnectSettingsNav', array($this->action))) {
# action => array('prompt', 'title')
$menu = array();
......@@ -129,7 +129,7 @@ class ConnectSettingsNav extends Widget
$action_name === $menuaction);
}
Event::handle('EndConnectSettingsNav', array(&$this->action));
Event::handle('EndConnectSettingsNav', array($this->action));
}
$this->action->elementEnd('ul');
......
......@@ -73,7 +73,7 @@ class LoginGroupNav extends Widget
$this->action->elementStart('ul', array('class' => 'nav'));
if (Event::handle('StartLoginGroupNav', array(&$this->action))) {
if (Event::handle('StartLoginGroupNav', array($this->action))) {
$this->action->menuItem(common_local_url('login'),
_('Login'),
......@@ -87,7 +87,7 @@ class LoginGroupNav extends Widget
$action_name === 'register');
}
Event::handle('EndLoginGroupNav', array(&$this->action));
Event::handle('EndLoginGroupNav', array($this->action));
}
$this->action->elementEnd('ul');
......
......@@ -242,7 +242,7 @@ class UserProfile extends Widget
$this->out->elementEnd('div');
return;
}
if (Event::handle('StartProfilePageActionsSection', array(&$this->out, $this->profile))) {
if (Event::handle('StartProfilePageActionsSection', array($this->out, $this->profile))) {
$cur = common_current_user();
......@@ -250,13 +250,13 @@ class UserProfile extends Widget
$this->out->element('h2', null, _('User actions'));
$this->out->elementStart('ul');
if (Event::handle('StartProfilePageActionsElements', array(&$this->out, $this->profile))) {
if (Event::handle('StartProfilePageActionsElements', array($this->out, $this->profile))) {
if (empty($cur)) { // not logged in
if (Event::handle('StartProfileRemoteSubscribe', array(&$this->out, $this->profile))) {
if (Event::handle('StartProfileRemoteSubscribe', array($this->out, $this->profile))) {
$this->out->elementStart('li', 'entity_subscribe');
$this->showRemoteSubscribeLink();
$this->out->elementEnd('li');
Event::handle('EndProfileRemoteSubscribe', array(&$this->out, $this->profile));
Event::handle('EndProfileRemoteSubscribe', array($this->out, $this->profile));
}
} else {
if ($cur->id == $this->profile->id) { // your own page
......@@ -376,13 +376,13 @@ class UserProfile extends Widget
}
}
Event::handle('EndProfilePageActionsElements', array(&$this->out, $this->profile));
Event::handle('EndProfilePageActionsElements', array($this->out, $this->profile));
}
$this->out->elementEnd('ul');
$this->out->elementEnd('div');
Event::handle('EndProfilePageActionsSection', array(&$this->out, $this->profile));
Event::handle('EndProfilePageActionsSection', array($this->out, $this->profile));
}
}
......
......@@ -74,7 +74,7 @@ class CasAuthenticationPlugin extends AuthenticationPlugin
return true;
}
function onEndLoginGroupNav(&$action)
function onEndLoginGroupNav($action)
{
$action_name = $action->trimmed('action');
......
......@@ -446,11 +446,11 @@ class FacebookPlugin extends Plugin
/*
* Add a login tab for Facebook Connect
*
* @param Action &action the current action
* @param Action $action the current action
*
* @return void
*/
function onEndLoginGroupNav(&$action)
function onEndLoginGroupNav($action)
{
if (self::hasKeys()) {
$action_name = $action->trimmed('action');
......@@ -470,11 +470,11 @@ class FacebookPlugin extends Plugin
/*
* Add a tab for managing Facebook Connect settings
*
* @param Action &action the current action
* @param Action $action the current action
*
* @return void
*/
function onEndConnectSettingsNav(&$action)
function onEndConnectSettingsNav($action)
{
if (self::hasKeys()) {
$action_name = $action->trimmed('action');
......
......@@ -186,11 +186,11 @@ class FacebookBridgePlugin extends Plugin
* Add a login tab for Facebook, but only if there's a Facebook
* application defined for the plugin to use.
*
* @param Action &action the current action
* @param Action $action the current action
*
* @return void
*/
function onEndLoginGroupNav(&$action)
function onEndLoginGroupNav($action)
{
$action_name = $action->trimmed('action');
......@@ -252,11 +252,11 @@ class FacebookBridgePlugin extends Plugin
* Add a tab for user-level Facebook settings if the user
* has a link to Facebook
*
* @param Action &action the current action
* @param Action $action the current action
*
* @return void
*/
function onEndConnectSettingsNav(&$action)
function onEndConnectSettingsNav($action)
{
if ($this->hasApplication()) {
$action_name = $action->trimmed('action');
......
......@@ -279,19 +279,19 @@ class NewMenuPlugin extends Plugin
return true;
}
function onStartAccountSettingsNav(&$action)
function onStartAccountSettingsNav($action)
{
$this->_settingsMenu($action);
return false;
}
function onStartConnectSettingsNav(&$action)
function onStartConnectSettingsNav($action)
{
$this->_settingsMenu($action);
return false;
}
private function _settingsMenu(&$action)
private function _settingsMenu($action)
{
$actionName = $action->trimmed('action');
......@@ -325,7 +325,7 @@ class NewMenuPlugin extends Plugin
_('Other options'),
$actionName == 'othersettings');
Event::handle('EndAccountSettingsNav', array(&$action));
Event::handle('EndAccountSettingsNav', array($action));
if (common_config('xmpp', 'enabled')) {
$action->menuItem(common_local_url('imsettings'),
......@@ -346,7 +346,7 @@ class NewMenuPlugin extends Plugin
_('Authorized connected applications'),
$actionName == 'oauthconnectionsettings');
Event::handle('EndConnectSettingsNav', array(&$action));
Event::handle('EndConnectSettingsNav', array($action));
}
function onEndShowStyles($action)
......
......@@ -248,11 +248,11 @@ class OpenIDPlugin extends Plugin
*
* If we're in openidOnly mode, we disable the menu for all other login.
*
* @param Action &$action Action being executed
* @param Action $action Action being executed
*
* @return boolean hook return
*/
function onStartLoginGroupNav(&$action)
function onStartLoginGroupNav($action)
{
if (common_config('site', 'openidonly')) {
$this->showOpenIDLoginTab($action);
......@@ -268,11 +268,11 @@ class OpenIDPlugin extends Plugin
/**
* Menu item for login
*
* @param Action &$action Action being executed
* @param Action $action Action being executed
*
* @return boolean hook return
*/
function onEndLoginGroupNav(&$action)
function onEndLoginGroupNav($action)
{
$this->showOpenIDLoginTab($action);
......@@ -318,11 +318,11 @@ class OpenIDPlugin extends Plugin
/**
* Menu item for OpenID settings
*
* @param Action &$action Action being executed
* @param Action $action Action being executed
*
* @return boolean hook return
*/
function onEndAccountSettingsNav(&$action)
function onEndAccountSettingsNav($action)
{
$action_name = $action->trimmed('action');
......
......@@ -137,11 +137,11 @@ class TwitterBridgePlugin extends Plugin
/*
* Add a login tab for 'Sign in with Twitter'
*
* @param Action &action the current action
* @param Action $action the current action
*
* @return void
*/
function onEndLoginGroupNav(&$action)
function onEndLoginGroupNav($action)
{
$action_name = $action->trimmed('action');
......@@ -160,11 +160,11 @@ class TwitterBridgePlugin extends Plugin
/**
* Add the Twitter Settings page to the Connect Settings menu
*
* @param Action &$action The calling page
* @param Action $action The calling page
*
* @return boolean hook return
*/
function onEndConnectSettingsNav(&$action)
function onEndConnectSettingsNav($action)
{
if (self::hasKeys()) {
$action_name = $action->trimmed('action');
......
......@@ -121,12 +121,12 @@ class UserFlagPlugin extends Plugin
/**
* Add a 'flag' button to profile page
*
* @param Action &$action The action being called
* @param Action $action The action being called
* @param Profile $profile Profile being shown
*
* @return boolean hook result
*/
function onEndProfilePageActionsElements(&$action, $profile)
function onEndProfilePageActionsElements($action, $profile)
{
$this->showFlagButton($action, $profile,
array('action' => 'showstream',
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment