Merge branch 'master' of gitorious.org:statusnet/mainline
This commit is contained in:
commit
50bfa21814
@ -985,8 +985,6 @@ class Notice extends Memcached_DataObject
|
||||
$reply->profile_id = $user->id;
|
||||
|
||||
$id = $reply->insert();
|
||||
|
||||
self::blow('reply:stream:%d', $user->id);
|
||||
}
|
||||
}
|
||||
|
||||
@ -1052,6 +1050,7 @@ class Notice extends Memcached_DataObject
|
||||
throw new ServerException("Couldn't save reply for {$this->id}, {$mentioned->id}");
|
||||
} else {
|
||||
$replied[$mentioned->id] = 1;
|
||||
self::blow('reply:stream:%d', $mentioned->id);
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -1107,7 +1106,6 @@ class Notice extends Memcached_DataObject
|
||||
foreach ($recipientIds as $recipientId) {
|
||||
$user = User::staticGet('id', $recipientId);
|
||||
if (!empty($user)) {
|
||||
self::blow('reply:stream:%d', $recipientId);
|
||||
mail_notify_attn($user, $this);
|
||||
}
|
||||
}
|
||||
|
@ -22,6 +22,20 @@ class Reply extends Memcached_DataObject
|
||||
/* the code above is auto generated do not remove the tag below */
|
||||
###END_AUTOCODE
|
||||
|
||||
/**
|
||||
* Wrapper for record insertion to update related caches
|
||||
*/
|
||||
function insert()
|
||||
{
|
||||
$result = parent::insert();
|
||||
|
||||
if ($result) {
|
||||
self::blow('reply:stream:%d', $this->profile_id);
|
||||
}
|
||||
|
||||
return $result;
|
||||
}
|
||||
|
||||
function stream($user_id, $offset=0, $limit=NOTICES_PER_PAGE, $since_id=0, $max_id=0)
|
||||
{
|
||||
$ids = Notice::stream(array('Reply', '_streamDirect'),
|
||||
|
@ -83,6 +83,7 @@ class Activity
|
||||
const CREATOR = 'creator';
|
||||
|
||||
const CONTENTNS = 'http://purl.org/rss/1.0/modules/content/';
|
||||
const ENCODED = 'encoded';
|
||||
|
||||
public $actor; // an ActivityObject
|
||||
public $verb; // a string (the URL)
|
||||
@ -268,14 +269,21 @@ class Activity
|
||||
|
||||
$this->title = ActivityUtils::childContent($item, ActivityObject::TITLE, self::RSS);
|
||||
|
||||
$contentEl = ActivityUtils::child($item, ActivityUtils::CONTENT, self::CONTENTNS);
|
||||
$contentEl = ActivityUtils::child($item, self::ENCODED, self::CONTENTNS);
|
||||
|
||||
if (!empty($contentEl)) {
|
||||
$this->content = htmlspecialchars_decode($contentEl->textContent, ENT_QUOTES);
|
||||
// <content:encoded> XML node's text content is HTML; no further processing needed.
|
||||
$this->content = $contentEl->textContent;
|
||||
} else {
|
||||
$descriptionEl = ActivityUtils::child($item, self::DESCRIPTION, self::RSS);
|
||||
if (!empty($descriptionEl)) {
|
||||
$this->content = htmlspecialchars_decode($descriptionEl->textContent, ENT_QUOTES);
|
||||
// Per spec, <description> must be plaintext.
|
||||
// In practice, often there's HTML... but these days good
|
||||
// feeds are using <content:encoded> which is explicitly
|
||||
// real HTML.
|
||||
// We'll treat this following spec, and do HTML escaping
|
||||
// to convert from plaintext to HTML.
|
||||
$this->content = htmlspecialchars($descriptionEl->textContent);
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -213,11 +213,19 @@ class ActivityUtils
|
||||
// slavishly following http://atompub.org/rfc4287.html#rfc.section.4.1.3.3
|
||||
|
||||
if (empty($type) || $type == 'text') {
|
||||
return $el->textContent;
|
||||
// We have plaintext saved as the XML text content.
|
||||
// Since we want HTML, we need to escape any special chars.
|
||||
return htmlspecialchars($el->textContent);
|
||||
} else if ($type == 'html') {
|
||||
// We have HTML saved as the XML text content.
|
||||
// No additional processing required once we've got it.
|
||||
$text = $el->textContent;
|
||||
return htmlspecialchars_decode($text, ENT_QUOTES);
|
||||
return $text;
|
||||
} else if ($type == 'xhtml') {
|
||||
// Per spec, the <content type="xhtml"> contains a single
|
||||
// HTML <div> with XHTML namespace on it as a child node.
|
||||
// We need to pull all of that <div>'s child nodes and
|
||||
// serialize them back to an (X)HTML source fragment.
|
||||
$divEl = ActivityUtils::child($el, 'div', 'http://www.w3.org/1999/xhtml');
|
||||
if (empty($divEl)) {
|
||||
return null;
|
||||
|
@ -262,7 +262,7 @@ class BlacklistPlugin extends Plugin
|
||||
$patterns = $this->_getUrlPatterns();
|
||||
|
||||
foreach ($patterns as $pattern) {
|
||||
if (preg_match("/$pattern/", $url)) {
|
||||
if ($pattern != '' && preg_match("/$pattern/", $url)) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
@ -285,7 +285,7 @@ class BlacklistPlugin extends Plugin
|
||||
$patterns = $this->_getNicknamePatterns();
|
||||
|
||||
foreach ($patterns as $pattern) {
|
||||
if (preg_match("/$pattern/", $nickname)) {
|
||||
if ($pattern != '' && preg_match("/$pattern/", $nickname)) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
@ -94,7 +94,7 @@ class Homepage_blacklist extends Memcached_DataObject
|
||||
|
||||
function keys()
|
||||
{
|
||||
return array('pattern' => 'K');
|
||||
return array_keys($this->keyTypes());
|
||||
}
|
||||
|
||||
/**
|
||||
@ -108,7 +108,7 @@ class Homepage_blacklist extends Memcached_DataObject
|
||||
|
||||
function keyTypes()
|
||||
{
|
||||
return $this->keys();
|
||||
return array('pattern' => 'K');
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -88,7 +88,7 @@ class Nickname_blacklist extends Memcached_DataObject
|
||||
|
||||
function keys()
|
||||
{
|
||||
return array('pattern' => 'K');
|
||||
return array_keys($this->keyTypes());
|
||||
}
|
||||
|
||||
/**
|
||||
@ -99,7 +99,7 @@ class Nickname_blacklist extends Memcached_DataObject
|
||||
|
||||
function keyTypes()
|
||||
{
|
||||
return $this->keys();
|
||||
return array('pattern' => 'K');
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -88,28 +88,27 @@ class BlacklistadminpanelAction extends AdminPanelAction
|
||||
|
||||
function saveSettings()
|
||||
{
|
||||
$nickPatterns = array();
|
||||
|
||||
$rawNickPatterns = explode("\n", $this->trimmed('blacklist-nicknames'));
|
||||
|
||||
foreach ($rawNickPatterns as $raw) {
|
||||
$nickPatterns[] = trim($raw);
|
||||
}
|
||||
|
||||
$nickPatterns = $this->splitPatterns($this->trimmed('blacklist-nicknames'));
|
||||
Nickname_blacklist::saveNew($nickPatterns);
|
||||
|
||||
$rawUrlPatterns = explode("\n", $this->trimmed('blacklist-urls'));
|
||||
$urlPatterns = array();
|
||||
|
||||
foreach ($rawUrlPatterns as $raw) {
|
||||
$urlPatterns[] = trim($raw);
|
||||
}
|
||||
|
||||
$urlPatterns = $this->splitPatterns($this->trimmed('blacklist-urls'));
|
||||
Homepage_blacklist::saveNew($urlPatterns);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
protected function splitPatterns($text)
|
||||
{
|
||||
$patterns = array();
|
||||
foreach (explode("\n", $text) as $raw) {
|
||||
$trimmed = trim($raw);
|
||||
if ($trimmed != '') {
|
||||
$patterns[] = $trimmed;
|
||||
}
|
||||
}
|
||||
return $patterns;
|
||||
}
|
||||
|
||||
/**
|
||||
* Validate the values
|
||||
*
|
||||
|
147
plugins/OStatus/scripts/update-profile.php
Normal file
147
plugins/OStatus/scripts/update-profile.php
Normal file
@ -0,0 +1,147 @@
|
||||
#!/usr/bin/env php
|
||||
<?php
|
||||
/*
|
||||
* StatusNet - a distributed open-source microblogging tool
|
||||
* Copyright (C) 2010, StatusNet, Inc.
|
||||
*
|
||||
* This program is free software: you can redistribute it and/or modify
|
||||
* it under the terms of the GNU Affero General Public License as published by
|
||||
* the Free Software Foundation, either version 3 of the License, or
|
||||
* (at your option) any later version.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU Affero General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU Affero General Public License
|
||||
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||
*/
|
||||
|
||||
define('INSTALLDIR', realpath(dirname(__FILE__) . '/../../..'));
|
||||
|
||||
$helptext = <<<END_OF_HELP
|
||||
update-profile.php [options] http://example.com/profile/url
|
||||
|
||||
Rerun profile and feed info discovery for the given OStatus remote profile,
|
||||
and reinitialize its PuSH subscription for the given feed. This may help get
|
||||
things restarted if the hub or feed URLs have changed for the profile.
|
||||
|
||||
|
||||
END_OF_HELP;
|
||||
|
||||
require_once INSTALLDIR.'/scripts/commandline.inc';
|
||||
|
||||
if (empty($args[0]) || !Validate::uri($args[0])) {
|
||||
print "$helptext";
|
||||
exit(1);
|
||||
}
|
||||
|
||||
$uri = $args[0];
|
||||
|
||||
|
||||
$oprofile = Ostatus_profile::staticGet('uri', $uri);
|
||||
|
||||
if (!$oprofile) {
|
||||
print "No OStatus remote profile known for URI $uri\n";
|
||||
exit(1);
|
||||
}
|
||||
|
||||
print "Old profile state for $oprofile->uri\n";
|
||||
showProfile($oprofile);
|
||||
|
||||
print "\n";
|
||||
print "Re-running feed discovery for profile URL $oprofile->uri\n";
|
||||
// @fixme will bork where the URI isn't the profile URL for now
|
||||
$discover = new FeedDiscovery();
|
||||
$feedurl = $discover->discoverFromURL($oprofile->uri);
|
||||
$huburi = $discover->getAtomLink('hub');
|
||||
$salmonuri = $discover->getAtomLink(Salmon::NS_REPLIES);
|
||||
|
||||
print " Feed URL: $feedurl\n";
|
||||
print " Hub URL: $huburi\n";
|
||||
print " Salmon URL: $salmonuri\n";
|
||||
|
||||
if ($feedurl != $oprofile->feeduri || $salmonuri != $oprofile->salmonuri) {
|
||||
print "\n";
|
||||
print "Updating...\n";
|
||||
// @fixme update keys :P
|
||||
#$orig = clone($oprofile);
|
||||
#$oprofile->feeduri = $feedurl;
|
||||
#$oprofile->salmonuri = $salmonuri;
|
||||
#$ok = $oprofile->update($orig);
|
||||
$ok = $oprofile->query('UPDATE ostatus_profile SET ' .
|
||||
'feeduri=\'' . $oprofile->escape($feedurl) . '\',' .
|
||||
'salmonuri=\'' . $oprofile->escape($salmonuri) . '\' ' .
|
||||
'WHERE uri=\'' . $oprofile->escape($uri) . '\'');
|
||||
|
||||
if (!$ok) {
|
||||
print "Failed to update profile record...\n";
|
||||
exit(1);
|
||||
}
|
||||
|
||||
$oprofile->decache();
|
||||
} else {
|
||||
print "\n";
|
||||
print "Ok, ostatus_profile record unchanged.\n\n";
|
||||
}
|
||||
|
||||
$sub = FeedSub::ensureFeed($feedurl);
|
||||
|
||||
if ($huburi != $sub->huburi) {
|
||||
print "\n";
|
||||
print "Updating hub record for feed; was $sub->huburi\n";
|
||||
$orig = clone($sub);
|
||||
$sub->huburi = $huburi;
|
||||
$ok = $sub->update($orig);
|
||||
|
||||
if (!$ok) {
|
||||
print "Failed to update sub record...\n";
|
||||
exit(1);
|
||||
}
|
||||
} else {
|
||||
print "\n";
|
||||
print "Feed record ok, not changing.\n\n";
|
||||
}
|
||||
|
||||
print "\n";
|
||||
print "Pinging hub $sub->huburi with new subscription for $sub->uri\n";
|
||||
$ok = $sub->subscribe();
|
||||
|
||||
if ($ok) {
|
||||
print "ok\n";
|
||||
} else {
|
||||
print "Could not confirm.\n";
|
||||
}
|
||||
|
||||
$o2 = Ostatus_profile::staticGet('uri', $uri);
|
||||
|
||||
print "\n";
|
||||
print "New profile state:\n";
|
||||
showProfile($o2);
|
||||
|
||||
print "\n";
|
||||
print "New feed state:\n";
|
||||
$sub2 = FeedSub::ensureFeed($feedurl);
|
||||
showSub($sub2);
|
||||
|
||||
function showProfile($oprofile)
|
||||
{
|
||||
print " Feed URL: $oprofile->feeduri\n";
|
||||
print " Salmon URL: $oprofile->salmonuri\n";
|
||||
print " Avatar URL: $oprofile->avatar\n";
|
||||
print " Profile ID: $oprofile->profile_id\n";
|
||||
print " Group ID: $oprofile->group_id\n";
|
||||
print " Record created: $oprofile->created\n";
|
||||
print " Record modified: $oprofile->modified\n";
|
||||
}
|
||||
|
||||
function showSub($sub)
|
||||
{
|
||||
print " Subscription state: $sub->sub_state\n";
|
||||
print " Verify token: $sub->verify_token\n";
|
||||
print " Signature secret: $sub->secret\n";
|
||||
print " Sub start date: $sub->sub_start\n";
|
||||
print " Record created: $sub->created\n";
|
||||
print " Record modified: $sub->modified\n";
|
||||
}
|
@ -299,11 +299,21 @@ class AutosubmitAction extends Action
|
||||
|
||||
function title()
|
||||
{
|
||||
return _m('OpenID Auto-Submit');
|
||||
return _m('OpenID Login Submission');
|
||||
}
|
||||
|
||||
function showContent()
|
||||
{
|
||||
$this->raw('<p style="margin: 20px 80px">');
|
||||
// @fixme this would be better using standard CSS class, but the present theme's a bit scary.
|
||||
$this->element('img', array('src' => Theme::path('images/icons/icon_processing.gif', 'base'),
|
||||
// for some reason the base CSS sets <img>s as block display?!
|
||||
'style' => 'display: inline'));
|
||||
$this->text(_m('Requesting authorization from your login provider...'));
|
||||
$this->raw('</p>');
|
||||
$this->raw('<p style="margin-top: 60px; font-style: italic">');
|
||||
$this->text(_m('If you are not redirected to your login provider in a few seconds, try pushing the button below.'));
|
||||
$this->raw('</p>');
|
||||
$this->raw($this->form_html);
|
||||
}
|
||||
|
||||
@ -311,8 +321,6 @@ class AutosubmitAction extends Action
|
||||
{
|
||||
parent::showScripts();
|
||||
$this->element('script', null,
|
||||
'$(document).ready(function() { ' .
|
||||
' $(\'#'. $this->form_id .'\').submit(); '.
|
||||
'});');
|
||||
'document.getElementById(\'' . $this->form_id . '\').submit();');
|
||||
}
|
||||
}
|
||||
|
@ -32,6 +32,18 @@ class ActivityParseTests extends PHPUnit_Framework_TestCase
|
||||
$this->assertEquals('tag:versioncentral.example.org,2009:/change/1643245', $act->objects[0]->id);
|
||||
}
|
||||
|
||||
public function testExample2()
|
||||
{
|
||||
global $_example2;
|
||||
$dom = DOMDocument::loadXML($_example2);
|
||||
$act = new Activity($dom->documentElement);
|
||||
|
||||
$this->assertFalse(empty($act));
|
||||
// Did we handle <content type="html"> correctly with a typical payload?
|
||||
$this->assertEquals("<p>Geraldine posted a Photo on PhotoPanic</p>\n " .
|
||||
"<img src=\"/geraldine/photo1.jpg\">", trim($act->content));
|
||||
}
|
||||
|
||||
public function testExample3()
|
||||
{
|
||||
global $_example3;
|
||||
@ -305,6 +317,71 @@ class ActivityParseTests extends PHPUnit_Framework_TestCase
|
||||
|
||||
}
|
||||
|
||||
public function testAtomContent()
|
||||
{
|
||||
$tests = array(array("<content>Some regular plain text.</content>",
|
||||
"Some regular plain text."),
|
||||
array("<content><b>this is not HTML</b></content>",
|
||||
"<b>this is not HTML</b>"),
|
||||
array("<content type='html'>Some regular plain HTML.</content>",
|
||||
"Some regular plain HTML."),
|
||||
array("<content type='html'><b>this is too HTML</b></content>",
|
||||
"<b>this is too HTML</b>"),
|
||||
array("<content type='html'>&lt;b&gt;but this is not HTML!&lt;/b&gt;</content>",
|
||||
"<b>but this is not HTML!</b>"),
|
||||
array("<content type='xhtml'><div xmlns='http://www.w3.org/1999/xhtml'>Some regular plain XHTML.</div></content>",
|
||||
"Some regular plain XHTML."),
|
||||
array("<content type='xhtml'><div xmlns='http://www.w3.org/1999/xhtml'><b>This is some XHTML!</b></div></content>",
|
||||
"<b>This is some XHTML!</b>"),
|
||||
array("<content type='xhtml'><div xmlns='http://www.w3.org/1999/xhtml'><b>This is not some XHTML!</b></div></content>",
|
||||
"<b>This is not some XHTML!</b>"),
|
||||
array("<content type='xhtml'><div xmlns='http://www.w3.org/1999/xhtml'>&lt;b&gt;This is not some XHTML either!&lt;/b&gt;</div></content>",
|
||||
"&lt;b&gt;This is not some XHTML either!&lt;/b&gt;"));
|
||||
foreach ($tests as $data) {
|
||||
list($source, $output) = $data;
|
||||
$xml = "<entry xmlns='http://www.w3.org/2005/Atom'>" .
|
||||
"<id>http://example.com/fakeid</id>" .
|
||||
"<author><name>Test</name></author>" .
|
||||
"<title>Atom content tests</title>" .
|
||||
$source .
|
||||
"</entry>";
|
||||
$dom = DOMDocument::loadXML($xml);
|
||||
$act = new Activity($dom->documentElement);
|
||||
|
||||
$this->assertFalse(empty($act));
|
||||
$this->assertEquals($output, trim($act->content));
|
||||
}
|
||||
}
|
||||
|
||||
public function testRssContent()
|
||||
{
|
||||
$tests = array(array("<content:encoded>Some regular plain HTML.</content:encoded>",
|
||||
"Some regular plain HTML."),
|
||||
array("<content:encoded>Some <b>exciting bold HTML</b></content:encoded>",
|
||||
"Some <b>exciting bold HTML</b>"),
|
||||
array("<content:encoded>Some &lt;b&gt;escaped non-HTML.&lt;/b&gt;</content:encoded>",
|
||||
"Some <b>escaped non-HTML.</b>"),
|
||||
array("<description>Some plain text.</description>",
|
||||
"Some plain text."),
|
||||
array("<description>Some <b>non-HTML text</b></description>",
|
||||
"Some <b>non-HTML text</b>"),
|
||||
array("<description>Some &lt;b&gt;double-escaped text&lt;/b&gt;</description>",
|
||||
"Some &lt;b&gt;double-escaped text&lt;/b&gt;"));
|
||||
foreach ($tests as $data) {
|
||||
list($source, $output) = $data;
|
||||
$xml = "<item xmlns:content='http://purl.org/rss/1.0/modules/content/'>" .
|
||||
"<guid>http://example.com/fakeid</guid>" .
|
||||
"<title>RSS content tests</title>" .
|
||||
$source .
|
||||
"</item>";
|
||||
$dom = DOMDocument::loadXML($xml);
|
||||
$act = new Activity($dom->documentElement);
|
||||
|
||||
$this->assertFalse(empty($act));
|
||||
$this->assertEquals($output, trim($act->content));
|
||||
}
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
$_example1 = <<<EXAMPLE1
|
||||
|
Loading…
Reference in New Issue
Block a user