Bookmark saving robustness fixes

First, if the tags box is empty, don't save an empty tag for the notice.

Second, if URL shortening fails, just use the regular URL.

remove boilerplate from NewMenuPlugin

wrap account actions in a section

Erroneous code ensuring Webfinger accounts

Ostatus_profile::ensureProfileURI() was accidentally falling through to
the default switch case, and was also calling common_log() incorrectly.

Merge remote branch 'gitorious/testing' into testing

Merge branch 'newmenuplugin' into testing

remove help link; Everyone => Public; Logo goes home

Fix error handling for missing group in apigroupmembership -- was trying to call methods on the variable before we checked it, which triggers PHP fatal error

Ticket #2166: accept aliases for local group names in API

Also simplifies the code by using User_group::getForNickname instead of duplicating half of it :D

Show some menu fixup for settings for some themes

change 'Everyone' to 'Public'

combine account and connect menus

Fix warning in subscribers/subscriptions list pages where we attempted to call free() an ArrayWrapper after it was used up, thus trying to forward the call to a nonexistent object.

Removed the free calls (unneeded since destructors now work), and added an error check w/ logging & an exception for future attempts to forward calls to nonexistent object.

Fix ticket #2392: sending invitation email fails when site name contains double quotes

Gotta escape quotes!

Ticket #1968: fix favoriting, reply when using InfiniteScroll

Fix regression in last year's update of InfiniteScroll -- having debug off caused breakage due to bad code interpreting every variable as a selector, and jQuery then failing when passed 'false'.

Note that the current version of the infinitescroll jquery plugin fixes this, but I'm not updating to it because the code's been altered from the upstream version, apparently to stop it from actually working as infinite scroll. WTF? :)

Ticket #2732: add some regression tests for groups to OStatus remote-tests.php

Note that these tests won't pass on master branch yet as the join/leave don't work, and there's a bug in Activity parsing which prevents interop between new feeds and old remote subscribers (both fixed in this branch).

Fix for atom/activity streams parsing: feed's <activity:subject> was being taken at a higher priority than entry's <atom:author>, which broke OStatus group posting since we retired <activity:actor>.
Added test case to ActivityParseTests.

allow group join/leave commands in api posting, at least for the moment (no other way to do remote subscribe without a preexisting local id number via api)

Fix for PHP notice spew in group creation via API: set default 'mainpage' in User_group::register() rather than forcing all callers to do it manually.

Merge branch 'testing-sig' of gitorious.org:~brion/statusnet/brion-fixes into testing

Salmon signature checks on incoming slaps now check both old and new signature formats.

Outgoing Salmon slaps now use the corrected signature format; if the first hit is rejected with an HTTP error, we try again with the old format.

(This is not 100% ideal; possibly should try to distinguish between server errors and rejections, etc.)

Switch autoloader from '__autoload' magic function name to registering our function with spl_autoload_register(); fixes compat problem with PHPUnit 3.5+ which seems to break the old __autoload

Add test cases for internal change in Salmon signing; fix for the new code.
Updated sig passes Tuomas's verifier, which is a good sign

Salmon slap / magicsig test script

Given a notice in the local system, we package it up as an Atom entry and MagicSig it up.
We run the magicenv verification on it locally to make sure our own functions can decode it.
Optionally with --verify we can send to Tuomas Koski's verification test service (not sure if this is working 100%)
If given --slap= with a target Salmon endpoint, we'll sent it on and see if it liked it. (Note that StatusNet will reject if there's not a relevant mention, but will report acceptance for dupes so you can use a message that's already been delivered as a test.)

Added doc comments on Salmon magicsig-related stuff to help in figuring out what's going on

move profile to subgroupnav

Revert "Revert "0.9.7alpha1""

This reverts commit fd205546512b75016b1e055aec24eab586ae818b.

Conflicts:

lib/common.php

Merge branch '0.9.x' into testing

Remove json2 author's "don't hotlink my script pls" alert line from our local copy of json2.js :D
Added json2.min.js to the makefile so it can be updated easily when json2.js changes.

new menu layout plugin

change bookmark-info from a <p> to a <div>

Fix ticket #2181: Can't save #000000 (black) in color fields on design page

It seems to have actually been saving correctly, but the update of the colors on the form success page wasn't working properly.
When a design object is pulled out of the database, the numeric fields are read in as strings, so black comes back as "0".
But, when we populate the new object and then stick it live, we've populated it with actual integers; with memcache on these might live for a while in the cache...

The fallback code in Design::toWebColor() did a check ($color == null) which would be false for the string "0", but counts as true for the *integer* 0.
Thus, the display code would initially interpret the correctly-saved black color as "use default".

Changing the check to === against null and "" empty string avoids the false positive on integers, and lets us see our nice black text immediately after save.

Merge branch 'master' of gitorious.org:statusnet/mainline into 0.9.x

Ticket #2649: fix for undefined $base_url in lighttpd code path

The $base_url var is never defined, and the code has actually always been wrong since it was added.
Just removing it seems to work fine :)

Typo fix: http://status.net/open-source/issues/2976

Merge branch 'revertversion' into 0.9.x

Revert "0.9.7alpha1"

This reverts commit 3d62c1cf5114760c0a7b97cd69ba363ac059e215.

Configuration options for using an HTTP proxy

We can make a lot of HTTP requests from the server side. This change
adds some configuration options for using an HTTP proxy, which can
cache hits from multiple sites (good for status.net-like services, for example).

Bookmark plugin: tweak post-upload success message to distinguish between "already done" (UnQueueManager) and "started, should finish eventually" (other queue manager)

Bookmark plugin: fix for delicious import with queues enabled

We were passing DOM nodes directly into the queues for the final bookmark import stage; unfortunately these don't actually survive serialization.
Moved the extraction of properties from the HTML up to the first-stage handler, so now we don't have to worry about moving DOM nodes from one handler to the next. Instead passing an associative array of properties, which is fed into the Bookmark::saveNew by the per-bookmark handler.

Bookmark plugin: fixes for bad DOM element nesting in delicious import data

delicious bookmark exports use the godawful HTML bookmark file format that ancient versions of Netscape used (and has thus been the common import/export format for bookmarks since the dark ages of the web :)
This arranges bookmark entries as an HTML definition list, using a lot of implied close tags (leaving off the </dt> and </dd>).
DOMDocument->loadHTML() uses libxml2's HTML mode, which generally does ok with muddling through things but apparently is really, really bad about handling those implied close tags.

Sequences of adjacent <dt> elements (eg bookmark without a description, followed by another bookmark "<dt><dt>"), end up interpreted as nested ("<dt><dt></dt></dt>") instead of as siblings ("<dt></dt><dt></dt>").
The first round of code tried to resolve the nesting inline, but ended up a bit funky in places.
I've replaced this with a standalone run through the data to re-order the elements, based on our knowing that <dt> and <dd> cannot directly contain one another; once that's done, our main logic loop can be a bit cleaner. I'm not 100% sure it's doing nested sublists correctly, but these don't seem to show up in delicious export (and even if they do, with the way we flatten the input it shouldn't make a difference).

Also fixed a clearer edge case where some bookmarks didn't get imported when missing descriptions.

Bookmark plugin: graceful error out for failure to import a delicious bookmark due to it being already bookmarked

Fix up edge case in nickname processing: overlong display forms should be rejected before normalization (storage of display forms will also have fields with limited length)

Generated an extra class on bookmark notice <li>s

Events to hook for NoticeListElement <li> generation

switch bookmark CSS classes to use dash instead of underscore

Merge branch 'testing' of gitorious.org:statusnet/mainline into testing

don't show some bookmark elements if empty

Tweak the post-form return on bookmarklet if we're not in a popup that we can close

Make sure the UUIDs are unique too :)

Update ActivityCreationTests for our switch away from the obsolete <activity:actor>

Use UUIDs for Bookmark unique ID

I was trying to generate URIs for Bookmarks based on (profile, crc32(url), created).

I failed at that. CRC32s are unsigned ints, and our schema code didn't like that.

On top of that, my code to encode and restore created timestamps was problematic.

So, I switched back to using a meaningless unique ID for Bookmarks.

One way to do this would be to use an auto-incrementing integer ID. However, we've been
kind of crabbed out a few times for exposing auto-incrementing integer IDs as URIs, so
I thought maybe using a random UUID would be a better way to do it.

So, this patch sets random UUIDs for URIs of bookmarks.

UUID-generation tools

version 0.9.8dev

0.9.7alpha1

update version to 0.9.7

Show <activity:subject> in Atom group notice feed

Version 0.9.6 and below of StatusNet assume anything in <author> is a
Person. So, we include an <activity:subject> element, which will be
checked first by those versions of the code, only for group feeds.

At some point we'll take this out, but it's useful for now.

Merge branch '0.9.x' into noactor

Merge branch '0.9.x' of gitorious.org:statusnet/mainline into 0.9.x

don't distribute when restoring archived delicious bookmarks

phpcs showbookmark.php

phpcs noticebyurl.php

phpcs importdelicious.php

phpcs Bookmark.php

phpcs BookmarkPlugin.php

Add link to delicious bookmark importer to profile settings

add hooks for the account-management tools

Web UI for importing delicious backup files

Bad method call in File_to_post

fixup date handling in showbookmark for PHP 5.2

show count of other bookmarks with link to stream

method to count notices linking to an URL

Delete file links when Notice is deleted

Merge branch '0.9.x' into socialbookmark

Page with a list of notices that link to an URL

Fixes from testing File::stream()

Stream of notices linking to an URL

Merge branch 'master' into 0.9.x

Prevent group creation by silenced users.

* adds Right::CREATEGROUP
* logic in Profile::hasRight() checks for silencing
* NewgroupAction checks for the permission before letting you see or process the form in the UI
* User_group::register() logic does a low-level check on the specified initial group admin, and rejects creation if that user doesn't have the right; guaranteeing that API methods etc will also have this restriction applied sensibly.

If notice has been deleted before, don't store URI again

import bookmarks from backups

Don't double-insert a bookmark

Merge branch '0.9.x' into socialbookmark

document events for activity import

pass through $idField and $createdField in Notice queries

add events for restoring activities

return saved notice to AtomPub for Bookmarks

Merge branch '0.9.x' into socialbookmark

Documentation for change in atompub events

include saved notice in atompub events

Make AtomPub work for bookmarks

Merge branch '0.9.x' into socialbookmark

Add a user parameter to atom pub api events

Better UI for bookmark popup

better layout for single-bookmark page

send and receive bookmarks by Salmon

New event for Salmon including target

Better job making an ActivityObject from an <atom:author>