Some phpcs cleanup
This commit is contained in:
parent
45a2253649
commit
62ba25f53a
|
@ -65,7 +65,8 @@ class FacebookPlugin extends Plugin
|
||||||
|
|
||||||
$m->connect('facebook/app', array('action' => 'facebookhome'));
|
$m->connect('facebook/app', array('action' => 'facebookhome'));
|
||||||
$m->connect('facebook/app/index.php', array('action' => 'facebookhome'));
|
$m->connect('facebook/app/index.php', array('action' => 'facebookhome'));
|
||||||
$m->connect('facebook/app/settings.php', array('action' => 'facebooksettings'));
|
$m->connect('facebook/app/settings.php',
|
||||||
|
array('action' => 'facebooksettings'));
|
||||||
$m->connect('facebook/app/invite.php', array('action' => 'facebookinvite'));
|
$m->connect('facebook/app/invite.php', array('action' => 'facebookinvite'));
|
||||||
$m->connect('facebook/app/remove', array('action' => 'facebookremove'));
|
$m->connect('facebook/app/remove', array('action' => 'facebookremove'));
|
||||||
|
|
||||||
|
@ -87,6 +88,7 @@ class FacebookPlugin extends Plugin
|
||||||
* @return boolean hook return
|
* @return boolean hook return
|
||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function onAutoload($cls)
|
function onAutoload($cls)
|
||||||
{
|
{
|
||||||
switch ($cls) {
|
switch ($cls) {
|
||||||
|
@ -116,7 +118,15 @@ class FacebookPlugin extends Plugin
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Add in xmlns:fb
|
/**
|
||||||
|
* Override normal HTML output to force the content type to
|
||||||
|
* text/html and add in xmlns:fb
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*/
|
||||||
|
|
||||||
function onStartShowHTML($action)
|
function onStartShowHTML($action)
|
||||||
{
|
{
|
||||||
|
|
||||||
|
@ -128,6 +138,7 @@ class FacebookPlugin extends Plugin
|
||||||
// text/html even though Facebook Connect uses XHTML. This is
|
// text/html even though Facebook Connect uses XHTML. This is
|
||||||
// A bug in Facebook Connect, and this is a temporary solution
|
// A bug in Facebook Connect, and this is a temporary solution
|
||||||
// until they fix their JavaScript libs.
|
// until they fix their JavaScript libs.
|
||||||
|
|
||||||
header('Content-Type: text/html');
|
header('Content-Type: text/html');
|
||||||
|
|
||||||
$action->extraHeaders();
|
$action->extraHeaders();
|
||||||
|
@ -150,7 +161,16 @@ class FacebookPlugin extends Plugin
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Note: this script needs to appear in the <body>
|
/**
|
||||||
|
* Add in the Facebook Connect JavaScript stuff
|
||||||
|
*
|
||||||
|
* Note: this script needs to appear in the <body>
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
function onEndShowScripts($action)
|
function onEndShowScripts($action)
|
||||||
{
|
{
|
||||||
|
@ -204,6 +224,7 @@ class FacebookPlugin extends Plugin
|
||||||
$js = sprintf($js, $apikey, $login_url, $logout_url);
|
$js = sprintf($js, $apikey, $login_url, $logout_url);
|
||||||
|
|
||||||
// Compress the bugger down a bit
|
// Compress the bugger down a bit
|
||||||
|
|
||||||
$js = str_replace(' ', '', $js);
|
$js = str_replace(' ', '', $js);
|
||||||
|
|
||||||
$action->raw(" $js"); // leading two spaces to make it line up
|
$action->raw(" $js"); // leading two spaces to make it line up
|
||||||
|
@ -211,15 +232,33 @@ class FacebookPlugin extends Plugin
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Note: this script needs to appear as close as possible to </body>
|
/**
|
||||||
|
* Add in an additional Facebook Connect script that's supposed to
|
||||||
|
* appear as close as possible to </body>
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
function onEndShowFooter($action)
|
function onEndShowFooter($action)
|
||||||
{
|
{
|
||||||
if ($this->reqFbScripts($action)) {
|
if ($this->reqFbScripts($action)) {
|
||||||
$action->script('http://static.ak.connect.facebook.com/js/api_lib/v0.4/FeatureLoader.js.php');
|
$action->script('http://static.ak.connect.facebook.com' .
|
||||||
|
'/js/api_lib/v0.4/FeatureLoader.js.php');
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Output Facebook Connect specific CSS link
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
function onEndShowStatusNetStyles($action)
|
function onEndShowStatusNetStyles($action)
|
||||||
{
|
{
|
||||||
if ($this->reqFbScripts($action)) {
|
if ($this->reqFbScripts($action)) {
|
||||||
|
@ -232,12 +271,13 @@ class FacebookPlugin extends Plugin
|
||||||
* want to output FB namespace, scripts, CSS, etc. on the pages that
|
* want to output FB namespace, scripts, CSS, etc. on the pages that
|
||||||
* really need them.
|
* really need them.
|
||||||
*
|
*
|
||||||
* @param Action the action in question
|
* @param Action $action the current action
|
||||||
*
|
*
|
||||||
* @return boolean true
|
* @return boolean true
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function reqFbScripts($action) {
|
function reqFbScripts($action)
|
||||||
|
{
|
||||||
|
|
||||||
// If you're logged in w/FB Connect, you always need the FB stuff
|
// If you're logged in w/FB Connect, you always need the FB stuff
|
||||||
|
|
||||||
|
@ -299,10 +339,19 @@ class FacebookPlugin extends Plugin
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Add in a Facebook Connect avatar to the primary nav menu
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
function onStartPrimaryNav($action)
|
function onStartPrimaryNav($action)
|
||||||
{
|
{
|
||||||
|
|
||||||
$user = common_current_user();
|
$user = common_current_user();
|
||||||
|
|
||||||
$connect = 'FBConnectSettings';
|
$connect = 'FBConnectSettings';
|
||||||
if (common_config('xmpp', 'enabled')) {
|
if (common_config('xmpp', 'enabled')) {
|
||||||
$connect = 'imsettings';
|
$connect = 'imsettings';
|
||||||
|
@ -345,6 +394,16 @@ class FacebookPlugin extends Plugin
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Alter the local nav menu to have a Facebook Connect login and
|
||||||
|
* settings pages
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
function onStartShowLocalNavBlock($action)
|
function onStartShowLocalNavBlock($action)
|
||||||
{
|
{
|
||||||
$action_name = get_class($action);
|
$action_name = get_class($action);
|
||||||
|
@ -370,6 +429,14 @@ class FacebookPlugin extends Plugin
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Have the logout process do some Facebook Connect cookie cleanup
|
||||||
|
*
|
||||||
|
* @param Action $action the current action
|
||||||
|
*
|
||||||
|
* @return void
|
||||||
|
*/
|
||||||
|
|
||||||
function onStartLogout($action)
|
function onStartLogout($action)
|
||||||
{
|
{
|
||||||
$action->logout();
|
$action->logout();
|
||||||
|
@ -389,9 +456,16 @@ class FacebookPlugin extends Plugin
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Get the URL of the user's Facebook avatar
|
||||||
|
*
|
||||||
|
* @param int $fbuid the Facebook user ID
|
||||||
|
*
|
||||||
|
* @return string $url the url for the user's Facebook avatar
|
||||||
|
*/
|
||||||
|
|
||||||
function getProfilePicURL($fbuid)
|
function getProfilePicURL($fbuid)
|
||||||
{
|
{
|
||||||
|
|
||||||
$facebook = getFacebook();
|
$facebook = getFacebook();
|
||||||
$url = null;
|
$url = null;
|
||||||
|
|
||||||
|
@ -411,7 +485,6 @@ class FacebookPlugin extends Plugin
|
||||||
}
|
}
|
||||||
|
|
||||||
return $url;
|
return $url;
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -422,6 +495,7 @@ class FacebookPlugin extends Plugin
|
||||||
*
|
*
|
||||||
* @return boolean hook return
|
* @return boolean hook return
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function onStartEnqueueNotice($notice, &$transports)
|
function onStartEnqueueNotice($notice, &$transports)
|
||||||
{
|
{
|
||||||
array_push($transports, 'facebook');
|
array_push($transports, 'facebook');
|
||||||
|
@ -436,6 +510,7 @@ class FacebookPlugin extends Plugin
|
||||||
*
|
*
|
||||||
* @return boolean hook return
|
* @return boolean hook return
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function onUnqueueHandleNotice(&$notice, $queue)
|
function onUnqueueHandleNotice(&$notice, $queue)
|
||||||
{
|
{
|
||||||
if (($queue == 'facebook') && ($this->_isLocal($notice))) {
|
if (($queue == 'facebook') && ($this->_isLocal($notice))) {
|
||||||
|
@ -448,10 +523,11 @@ class FacebookPlugin extends Plugin
|
||||||
/**
|
/**
|
||||||
* Determine whether the notice was locally created
|
* Determine whether the notice was locally created
|
||||||
*
|
*
|
||||||
* @param Notice $notice
|
* @param Notice $notice the notice
|
||||||
*
|
*
|
||||||
* @return boolean locality
|
* @return boolean locality
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function _isLocal($notice)
|
function _isLocal($notice)
|
||||||
{
|
{
|
||||||
return ($notice->is_local == Notice::LOCAL_PUBLIC ||
|
return ($notice->is_local == Notice::LOCAL_PUBLIC ||
|
||||||
|
@ -466,6 +542,7 @@ class FacebookPlugin extends Plugin
|
||||||
* @return boolean hook return
|
* @return boolean hook return
|
||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function onGetValidDaemons($daemons)
|
function onGetValidDaemons($daemons)
|
||||||
{
|
{
|
||||||
array_push($daemons, INSTALLDIR .
|
array_push($daemons, INSTALLDIR .
|
||||||
|
|
Loading…
Reference in New Issue
Block a user