Merge branch 'testing' of git@gitorious.org:statusnet/mainline into testing
Conflicts: plugins/OStatus/actions/salmon.php
This commit is contained in:
commit
d69f6dff6a
|
@ -249,7 +249,7 @@ class Action extends HTMLOutputter // lawsuit
|
|||
$this->script('jquery.min.js');
|
||||
$this->script('jquery.form.js');
|
||||
$this->script('jquery.cookie.js');
|
||||
$this->script('json2.js');
|
||||
$this->inlineScript('if (typeof window.JSON !== "object") { $.getScript("'.common_path('js/json2.js').'"); }');
|
||||
$this->script('jquery.joverlay.min.js');
|
||||
Event::handle('EndShowJQueryScripts', array($this));
|
||||
}
|
||||
|
@ -259,8 +259,7 @@ class Action extends HTMLOutputter // lawsuit
|
|||
$this->script('util.js');
|
||||
$this->script('geometa.js');
|
||||
// Frame-busting code to avoid clickjacking attacks.
|
||||
$this->element('script', array('type' => 'text/javascript'),
|
||||
'if (window.top !== window.self) { window.top.location.href = window.self.location.href; }');
|
||||
$this->inlineScript('if (window.top !== window.self) { window.top.location.href = window.self.location.href; }');
|
||||
Event::handle('EndShowStatusNetScripts', array($this));
|
||||
Event::handle('EndShowLaconicaScripts', array($this));
|
||||
}
|
||||
|
|
|
@ -253,17 +253,22 @@ class OStatusPlugin extends Plugin
|
|||
*/
|
||||
function onEndUnsubscribe($user, $other)
|
||||
{
|
||||
if ($user instanceof Profile) {
|
||||
$profile = $user;
|
||||
} else if ($user instanceof Profile) {
|
||||
$profile = $user->getProfile();
|
||||
}
|
||||
$oprofile = Ostatus_profile::staticGet('profile_id', $other->id);
|
||||
if ($oprofile) {
|
||||
// Notify the remote server of the unsub, if supported.
|
||||
$oprofile->notify($user->getProfile(), ActivityVerb::UNFOLLOW, $oprofile);
|
||||
$oprofile->notify($profile, ActivityVerb::UNFOLLOW, $oprofile);
|
||||
|
||||
// Drop the PuSH subscription if there are no other subscribers.
|
||||
$sub = new Subscription();
|
||||
$sub->subscribed = $other->id;
|
||||
$sub->limit(1);
|
||||
if (!$sub->find(true)) {
|
||||
common_log(LOG_INFO, "Unsubscribing from now-unused feed $oprofile->feeduri on hub $oprofile->huburi");
|
||||
common_log(LOG_INFO, "Unsubscribing from now-unused feed $oprofile->feeduri");
|
||||
$oprofile->unsubscribe();
|
||||
}
|
||||
}
|
||||
|
|
|
@ -68,14 +68,6 @@ class FeedSubSettingsAction extends ConnectSettingsAction
|
|||
|
||||
$profile = $user->getProfile();
|
||||
|
||||
$fuser = null;
|
||||
|
||||
$flink = Foreign_link::getByUserID($user->id, FEEDSUB_SERVICE);
|
||||
|
||||
if (!empty($flink)) {
|
||||
$fuser = $flink->getForeignUser();
|
||||
}
|
||||
|
||||
$this->elementStart('form', array('method' => 'post',
|
||||
'id' => 'form_settings_feedsub',
|
||||
'class' => 'form_settings',
|
||||
|
|
|
@ -69,7 +69,7 @@ class SalmonAction extends Action
|
|||
}
|
||||
|
||||
/**
|
||||
* @fixme probably call Ostatus_profile::processFeed
|
||||
* Check the posted activity type and break out to appropriate processing.
|
||||
*/
|
||||
|
||||
function handle($args)
|
||||
|
@ -94,13 +94,19 @@ class SalmonAction extends Action
|
|||
case ActivityVerb::FRIEND:
|
||||
$this->handleFollow();
|
||||
break;
|
||||
case ActivityVerb::UNFOLLOW:
|
||||
$this->handleUnfollow();
|
||||
break;
|
||||
}
|
||||
Event::handle('EndHandleSalmon', array($this->user, $this->activity));
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @fixme probably call Ostatus_profile::processFeed
|
||||
* We've gotten a post event on the Salmon backchannel, probably a reply.
|
||||
*
|
||||
* @todo validate if we need to handle this post, then call into
|
||||
* ostatus_profile's general incoming-post handling.
|
||||
*/
|
||||
function handlePost()
|
||||
{
|
||||
|
@ -137,38 +143,45 @@ class SalmonAction extends Action
|
|||
}
|
||||
|
||||
$profile = $this->ensureProfile();
|
||||
|
||||
// @fixme do something with the post
|
||||
}
|
||||
|
||||
/**
|
||||
* @fixme probably call Ostatus_profile::processFeed
|
||||
* We've gotten a follow/subscribe notification from a remote user.
|
||||
* Save a subscription relationship for them.
|
||||
*/
|
||||
|
||||
function handleFollow()
|
||||
{
|
||||
$object = $this->act->object;
|
||||
|
||||
if ($object->id != $this->user->uri) {
|
||||
throw new ClientException("Subscription notice not for this user.");
|
||||
$oprofile = $this->ensureProfile();
|
||||
if ($oprofile) {
|
||||
common_log(LOG_INFO, "Setting up subscription from remote {$oprofile->uri} to local {$this->user->nickname}");
|
||||
$oprofile->subscribeRemoteToLocal($this->user);
|
||||
} else {
|
||||
common_log(LOG_INFO, "Can't set up subscription from remote; missing profile.");
|
||||
}
|
||||
|
||||
$profile = $this->ensureProfile();
|
||||
|
||||
$sub = Subscription::pkeyGet(array('subscriber' => $profile->id,
|
||||
'subscribed' => $this->user->id));
|
||||
|
||||
if (!empty($sub)) {
|
||||
throw new ClientException("Already subscribed.");
|
||||
}
|
||||
|
||||
if ($this->user->hasBlocked($profile)) {
|
||||
throw new ClientException("Already subscribed.");
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
/**
|
||||
* @fixme probably call Ostatus_profile::processFeed
|
||||
* We've gotten an unfollow/unsubscribe notification from a remote user.
|
||||
* Check if we have a subscription relationship for them and kill it.
|
||||
*
|
||||
* @fixme probably catch exceptions on fail?
|
||||
*/
|
||||
function handleUnfollow()
|
||||
{
|
||||
$oprofile = $this->ensureProfile();
|
||||
if ($oprofile) {
|
||||
common_log(LOG_INFO, "Canceling subscription from remote {$oprofile->uri} to local {$this->user->nickname}");
|
||||
Subscription::cancel($oprofile->localProfile(), $this->user->getProfile());
|
||||
} else {
|
||||
common_log(LOG_ERR, "Can't cancel subscription from remote, didn't find the profile");
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Remote user likes one of our posts.
|
||||
* Confirm the post is ours, and save a local favorite event.
|
||||
*/
|
||||
|
||||
function handleFavorite()
|
||||
|
@ -221,26 +234,37 @@ class SalmonAction extends Action
|
|||
}
|
||||
|
||||
/**
|
||||
* @fixme probably call Ostatus_profile::processFeed
|
||||
* Remote user doesn't like one of our posts after all!
|
||||
* Confirm the post is ours, and save a local favorite event.
|
||||
*/
|
||||
function handleUnfavorite()
|
||||
{
|
||||
}
|
||||
|
||||
/**
|
||||
* Hmmmm
|
||||
*/
|
||||
function handleShare()
|
||||
{
|
||||
}
|
||||
|
||||
/**
|
||||
* @return Ostatus_profile
|
||||
*/
|
||||
function ensureProfile()
|
||||
{
|
||||
$actor = $this->act->actor;
|
||||
|
||||
common_log(LOG_DEBUG, "Received salmon bit: " . var_export($this->act, true));
|
||||
if (empty($actor->id)) {
|
||||
common_log(LOG_ERR, "broken actor: " . var_export($actor, true));
|
||||
throw new Exception("Received a salmon slap from unidentified actor.");
|
||||
}
|
||||
|
||||
$ostatusProfile = Ostatus_profile::ensureActorProfile($this->act);
|
||||
return $oprofile->localProfile();
|
||||
return Ostatus_profile::ensureActorProfile($this->act);
|
||||
}
|
||||
|
||||
/**
|
||||
* @fixme anything new in here probably should be merged into Ostatus_profile::ensureActorProfile and friends
|
||||
* @fixme merge into Ostatus_profile::ensureActorProfile and friends
|
||||
*/
|
||||
function createProfile()
|
||||
{
|
||||
|
|
|
@ -99,7 +99,7 @@ class FeedSub extends Memcached_DataObject
|
|||
'sub_state' => DB_DATAOBJECT_STR + DB_DATAOBJECT_NOTNULL,
|
||||
'sub_start' => DB_DATAOBJECT_STR + DB_DATAOBJECT_DATE + DB_DATAOBJECT_TIME,
|
||||
'sub_end' => DB_DATAOBJECT_STR + DB_DATAOBJECT_DATE + DB_DATAOBJECT_TIME,
|
||||
'last_update' => DB_DATAOBJECT_STR + DB_DATAOBJECT_DATE + DB_DATAOBJECT_TIME + DB_DATAOBJECT_NOTNULL,
|
||||
'last_update' => DB_DATAOBJECT_STR + DB_DATAOBJECT_DATE + DB_DATAOBJECT_TIME,
|
||||
'created' => DB_DATAOBJECT_STR + DB_DATAOBJECT_DATE + DB_DATAOBJECT_TIME + DB_DATAOBJECT_NOTNULL,
|
||||
'modified' => DB_DATAOBJECT_STR + DB_DATAOBJECT_DATE + DB_DATAOBJECT_TIME + DB_DATAOBJECT_NOTNULL);
|
||||
}
|
||||
|
|
|
@ -310,8 +310,15 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
* @param $verb eg Activity::SUBSCRIBE or Activity::JOIN
|
||||
* @param $object object of the action; if null, the remote entity itself is assumed
|
||||
*/
|
||||
public function notify(Profile $actor, $verb, $object=null)
|
||||
public function notify($actor, $verb, $object=null)
|
||||
{
|
||||
if (!($actor instanceof Profile)) {
|
||||
$type = gettype($actor);
|
||||
if ($type == 'object') {
|
||||
$type = get_class($actor);
|
||||
}
|
||||
throw new ServerException("Invalid actor passed to " . __METHOD__ . ": " . $type);
|
||||
}
|
||||
if ($object == null) {
|
||||
$object = $this;
|
||||
}
|
||||
|
@ -328,7 +335,7 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
$entry->element('id', null, $id);
|
||||
$entry->element('title', null, $text);
|
||||
$entry->element('summary', null, $text);
|
||||
$entry->element('published', null, common_date_w3dtf(time()));
|
||||
$entry->element('published', null, common_date_w3dtf(common_sql_now()));
|
||||
|
||||
$entry->element('activity:verb', null, $verb);
|
||||
$entry->raw($actor->asAtomAuthor());
|
||||
|
@ -340,7 +347,7 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
$feed->addEntry($entry);
|
||||
|
||||
$xml = $feed->getString();
|
||||
common_log(LOG_INFO, "Posting to Salmon endpoint $salmon: $xml");
|
||||
common_log(LOG_INFO, "Posting to Salmon endpoint $this->salmonuri: $xml");
|
||||
|
||||
$salmon = new Salmon(); // ?
|
||||
$salmon->post($this->salmonuri, $xml);
|
||||
|
@ -516,7 +523,7 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
throw new FeedSubException('empty feed');
|
||||
}
|
||||
$first = new Activity($entries->item(0), $discover->feed);
|
||||
return self::ensureActorProfile($first, $feeduri);
|
||||
return self::ensureActorProfile($first, $feeduri, $salmonuri);
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -531,9 +538,14 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
$temp_filename = tempnam(sys_get_temp_dir(), 'listener_avatar');
|
||||
copy($url, $temp_filename);
|
||||
|
||||
if ($this->isGroup()) {
|
||||
$id = $this->group_id;
|
||||
} else {
|
||||
$id = $this->profile_id;
|
||||
}
|
||||
// @fixme should we be using different ids?
|
||||
$imagefile = new ImageFile($this->id, $temp_filename);
|
||||
$filename = Avatar::filename($this->id,
|
||||
$imagefile = new ImageFile($id, $temp_filename);
|
||||
$filename = Avatar::filename($id,
|
||||
image_type_to_extension($imagefile->type),
|
||||
null,
|
||||
common_timestamp());
|
||||
|
@ -598,13 +610,14 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
*
|
||||
* @param Activity $activity
|
||||
* @param string $feeduri if we already know the canonical feed URI!
|
||||
* @param string $salmonuri if we already know the salmon return channel URI
|
||||
* @return Ostatus_profile
|
||||
*/
|
||||
public static function ensureActorProfile($activity, $feeduri=null)
|
||||
public static function ensureActorProfile($activity, $feeduri=null, $salmonuri=null)
|
||||
{
|
||||
$profile = self::getActorProfile($activity);
|
||||
if (!$profile) {
|
||||
$profile = self::createActorProfile($activity, $feeduri);
|
||||
$profile = self::createActorProfile($activity, $feeduri, $salmonuri);
|
||||
}
|
||||
return $profile;
|
||||
}
|
||||
|
@ -640,12 +653,12 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
/**
|
||||
* @fixme validate stuff somewhere
|
||||
*/
|
||||
protected static function createActorProfile($activity, $feeduri=null)
|
||||
protected static function createActorProfile($activity, $feeduri=null, $salmonuri=null)
|
||||
{
|
||||
$actor = $activity->actor;
|
||||
$homeuri = self::getActorProfileURI($activity);
|
||||
$nickname = self::getAuthorNick($activity);
|
||||
$avatar = self::getAvatar($actor, $feed);
|
||||
$avatar = self::getAvatar($actor, $activity->feed);
|
||||
|
||||
if (!$homeuri) {
|
||||
common_log(LOG_DEBUG, __METHOD__ . " empty actor profile URI: " . var_export($activity, true));
|
||||
|
@ -674,10 +687,17 @@ class Ostatus_profile extends Memcached_DataObject
|
|||
$oprofile = new Ostatus_profile();
|
||||
$oprofile->uri = $homeuri;
|
||||
if ($feeduri) {
|
||||
// If we don't have these, we can look them up later.
|
||||
$oprofile->feeduri = $feeduri;
|
||||
if ($salmonuri) {
|
||||
$oprofile->salmonuri = $salmonuri;
|
||||
}
|
||||
}
|
||||
$oprofile->profile_id = $profile->id;
|
||||
|
||||
$oprofile->created = common_sql_now();
|
||||
$oprofile->modified = common_sql_now();
|
||||
|
||||
$ok = $oprofile->insert();
|
||||
if ($ok) {
|
||||
$oprofile->updateAvatar($avatar);
|
||||
|
|
Loading…
Reference in New Issue
Block a user