Some phpcs cleanup

This commit is contained in:
Zach Copley 2009-12-14 21:24:49 +00:00
parent 655dbcedb3
commit 3e6b80d3e9
4 changed files with 57 additions and 42 deletions

View File

@ -111,12 +111,13 @@ class LoggingAggregatorAction extends Action
}
header('Content-Type: text/xml');
echo '<notifyResult success=\'true\' msg=\'Thanks for the update.\' />' . "\n";
Echo "<notifyResult success='true' msg='Thanks for the update.' />\n";
}
$this->ip = $_SERVER['REMOTE_ADDR'];
common_log(LOG_INFO, 'RSSCloud Logging Aggregator - ' . $this->ip . ' claims the feed at ' .
common_log(LOG_INFO, 'RSSCloud Logging Aggregator - ' .
$this->ip . ' claims the feed at ' .
$this->url . ' has been updated.');
}

View File

@ -41,16 +41,16 @@ if (!defined('STATUSNET')) {
* @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
* @link http://status.net/
**/
class RSSCloudNotifier {
class RSSCloudNotifier
{
const MAX_FAILURES = 3;
/**
* Send an HTTP GET to the notification handler with a
* challenge string to see if it repsonds correctly.
*
* @param String $endpoint URL of the notification handler
* @param String $feed the feed being subscribed to
* @param string $endpoint URL of the notification handler
* @param string $feed the feed being subscribed to
*
* @return boolean success
*/
@ -64,7 +64,8 @@ class RSSCloudNotifier {
$client = new HTTPClient();
$response = $client->get($url);
} catch (HTTP_Request2_Exception $e) {
common_log(LOG_INFO, 'RSSCloud plugin - failure testing notify handler ' .
common_log(LOG_INFO,
'RSSCloud plugin - failure testing notify handler ' .
$endpoint . ' - ' . $e->getMessage());
return false;
}
@ -110,7 +111,8 @@ class RSSCloudNotifier {
*
* @return boolean success
*/
function postUpdate($endpoint, $feed) {
function postUpdate($endpoint, $feed)
{
$headers = array();
$postdata = array('url' => $feed);
@ -153,6 +155,7 @@ class RSSCloudNotifier {
$profile->nickname . '.rss';
$cloudSub = new RSSCloudSubscription();
$cloudSub->subscribed = $profile->id;
if ($cloudSub->find()) {
@ -186,7 +189,8 @@ class RSSCloudNotifier {
if ($failCnt == self::MAX_FAILURES) {
common_log(LOG_INFO,
'Deleting RSSCloud subcription (max failure count reached), profile: ' .
'Deleting RSSCloud subcription ' .
'(max failure count reached), profile: ' .
$cloudSub->subscribed .
' handler: ' .
$cloudSub->url);
@ -209,7 +213,8 @@ class RSSCloudNotifier {
} else {
common_debug('Updating failure count on RSSCloud subscription. ' . $failCnt);
common_debug('Updating failure count on RSSCloud subscription. ' .
$failCnt);
$failCnt = $cloudSub->failures + 1;
@ -224,7 +229,9 @@ class RSSCloudNotifier {
if (!$result) {
common_log_db_error($cloudsub, 'UPDATE', __FILE__);
common_log(LOG_ERR, 'Could not update failure count on RSSCloud subscription');
common_log(LOG_ERR,
'Could not update failure ' .
'count on RSSCloud subscription');
}
}
}

View File

@ -98,16 +98,20 @@ class RSSCloudPlugin extends Plugin
*
* Hook for RouterInitialized event.
*
* @param Mapper &$m URL parser and mapper
*
* @return boolean hook return
*/
function onRouterInitialized(&$m)
{
$m->connect('/main/rsscloud/request_notify', array('action' => 'RSSCloudRequestNotify'));
$m->connect('/main/rsscloud/request_notify',
array('action' => 'RSSCloudRequestNotify'));
// XXX: This is just for end-to-end testing. Uncomment if you need to pretend
// to be a cloud hub for some reason.
// $m->connect('/main/rsscloud/notify', array('action' => 'LoggingAggregator'));
//$m->connect('/main/rsscloud/notify',
// array('action' => 'LoggingAggregator'));
return true;
}
@ -127,14 +131,15 @@ class RSSCloudPlugin extends Plugin
switch ($cls)
{
case 'RSSCloudSubscription':
include_once(INSTALLDIR . '/plugins/RSSCloud/RSSCloudSubscription.php');
include_once INSTALLDIR . '/plugins/RSSCloud/RSSCloudSubscription.php';
return false;
case 'RSSCloudNotifier':
include_once(INSTALLDIR . '/plugins/RSSCloud/RSSCloudNotifier.php');
include_once INSTALLDIR . '/plugins/RSSCloud/RSSCloudNotifier.php';
return false;
case 'RSSCloudRequestNotifyAction':
case 'LoggingAggregatorAction':
include_once(INSTALLDIR . '/plugins/RSSCloud/' . mb_substr($cls, 0, -6) . '.php');
include_once INSTALLDIR . '/plugins/RSSCloud/' .
mb_substr($cls, 0, -6) . '.php';
return false;
default:
return true;
@ -145,7 +150,7 @@ class RSSCloudPlugin extends Plugin
* Add a <cloud> element to the RSS feed (after the rss <channel>
* element is started).
*
* @param Action $action
* @param Action $action the ApiAction
*
* @return void
*/
@ -215,7 +220,7 @@ class RSSCloudPlugin extends Plugin
/**
* Determine whether the notice was locally created
*
* @param Notice $notice
* @param Notice $notice the notice in question
*
* @return boolean locality
*/
@ -233,7 +238,8 @@ class RSSCloudPlugin extends Plugin
* @return boolean hook return
*/
function onCheckSchema() {
function onCheckSchema()
{
$schema = Schema::get();
$schema->ensureTable('rsscloud_subscription',
array(new ColumnDef('subscribed', 'integer',
@ -248,8 +254,7 @@ class RSSCloudPlugin extends Plugin
null, false, null,
'CURRENT_TIMESTAMP',
'on update CURRENT_TIMESTAMP')
)
);
));
return true;
}

View File

@ -124,7 +124,8 @@ class RSSCloudRequestNotifyAction extends Action
}
if (empty($this->feeds)) {
$msg = 'You must provide at least one valid profile feed url (url1, url2, url3 ... urlN).';
$msg = 'You must provide at least one valid profile feed url ' .
'(url1, url2, url3 ... urlN).';
$this->showResult(false, $msg);
return;
}
@ -195,7 +196,7 @@ class RSSCloudRequestNotifyAction extends Action
{
$feeds = array();
while (list($key, $feed) = each ($this->args)) {
while (list($key, $feed) = each($this->args)) {
if (preg_match('/^url\d*$/', $key)) {
$feeds[] = $feed;
}
@ -283,7 +284,7 @@ class RSSCloudRequestNotifyAction extends Action
/**
* Save an RSSCloud subscription
*
* @param $feed a valid profile feed
* @param string $feed a valid profile feed
*
* @return boolean success result
*/
@ -329,7 +330,8 @@ class RSSCloudRequestNotifyAction extends Action
function showResult($success, $msg)
{
$this->startXML();
$this->elementStart('notifyResult', array('success' => ($success) ? 'true' : 'false',
$this->elementStart('notifyResult',
array('success' => ($success) ? 'true' : 'false',
'msg' => $msg));
$this->endXML();
}