Fix incorrect web interface styling when using parent diffs and new files

Add a check for parent_diff before adding the 'newfile' table style class in
diff_file_fragment.html

Manual testing: following the steps described in the issue will no longer
produce the issue. Unit tests passed.

Fixes bug 2389.
Reviewed at http://reviews.reviewboard.org/r/3372/
Add links to issues shown on the dashboard

Adds links to issues shown in the dashboard table/datagrid.

This review request updates r/3275, addressing the recommendations in that code
review. Thanks to Tommy Chong for the original implementation.

Tested on local machine with custom and GitHub issue tracker urls. Also tested
with malformed issue tracker urls.

Fixes bug 2589
Reviewed at http://reviews.reviewboard.org/r/3367/
Minor change in getting-started documentation

source ~/envs/reviewboard now reads source ~/envs/reviewboard/bin/activate
Fix a bug with linkify js globbing parentheses

This further fixes the bug with regards to linkifying text. Now, URLs with
anchors (open and close parens, with or without text in between them) work
properly. This fix doesn't affect the behaviour observed with url wrapped in
parens.

Testing done:
Self tested on own environment with different types of urls
- blah blah (http://www.foo.com/)
- http://download.oracle.com/javase/1.4.2/docs/api/java/lang/Object.html#clone()
- http://download.oracle.com/javase/1.4.2/docs/api/java/lang/Object.html#equals(java.lang.Object)
- gwegdss http://www.foo.com/

Fixes bug 2002
Reviewed at http://reviews.reviewboard.org/r/3374/
Fix for javascript errors publishing reviews with screenshot comments.

Somewhere in here, we ended up trying to check the issue status for screenshot
comments. This works fine on 1.7 where screenshot comments with issues are a
thing, but broke on 1.6. This workaround was suggested in the bug.

Fixes issue 2707
Merge branch 'release-1.6.x'
Increase hit target for expand/collapse button

Increase the size of the expand and collapse button images to make it a bigger
hit target. This reduces the chances of someone accidentally clicking the user's
name while trying to expand/collapse a review.

Tested it locally on Chrome Version 22.0.1229 (Mac)

Fixes bug 2358
Reviewed at http://reviews.reviewboard.org/r/3365/
Fix weird avatar alignment in header bar.

I noticed that the avatar in the header bar was overflowing the bar by a few
pixels in Chromium. Strangely, it underflowed in Firefox.

The solution seemed to be to set an explicit font-size for the text input and
then align the avatar to top instead of middle.
Add additional validation to bug tracker URLs

When specifying a custom bug tracker URL, the URL is passed through additional
validation to ensure that there is a '%s' string formatting element.

Tested on local development server against custom urls and a new GitHub
repository's issue tracker. Added unit test for validator.

Fixes issue 1996
Reviewed at http://reviews.reviewboard.org/r/3378/
Fix svn keywords not being correctly collapsed

Diffs were being created for files where the area surrounding keywords had
changed, and the keywords were not being collapsed.  This was happening for two
reasons:
* It couldn't understand paths with two leading slashes
* `svn propget svn:keywords` can return strings with non-space delimiters

Using reviewboard 1.5.5 on a Windows 7 server using Apache+mod_wsgi running on
python 2.7 with subversion 1.6.15 (if that matters)

Fixes issue 2134
Reviewed at http://reviews.reviewboard.org/r/2441/
Add  filter in FileDiffCommentResource to select diff comments that match only one file id

Adding filter to FillDiffCommentResource to only fetch comments for requested
file, and not an entire list of comments for every file in a given diffset.

Manual testing: following the steps described in the issue will no longer
produce the issue. Checked that this did not affect alternate method for
fetching diff comments (ie fetching specific comments for a given diffset)

Fixes bug 2327.
Reviewed at http://reviews.reviewboard.org/r/3377/
Add a missing file

I forgot to add this file for b64dd03 (I being David Trowbridge). Whoops.
Fix webapi list of review requests when using ?shipit=0

When accessing the /api/review-requests resource with a shipit=0 parameter, it
would return the wrong results (often returning review requests that did have
reviews with ship-its). This code was initially written to do a join with the
reviews table and query the shipit parameter of that model, which was checking
whether *all* of the associated review requests didn't have ship-its (which
wasn't completely incorrect but not what a user really wants when passing
shipit=0).

To fix this, instead of excluding review requests with reviews that have ship
its, I've changed it to add a test for the shipit_count field on the
ReviewRequest model. I've also updated the unit tests for this API to use the
same query to test against.

While updating the unit tests, I discovered a couple issues. All of the
shipit_count fields in the fixtures were zero, despite there being the presence
of some reviews with a ship-it. I've updated the test_reviewrequests fixture to
fix this issue. I also had to add the test_site fixture to the
test_get_reviewrequests_with_ship_it_* tests in order for the data to be valid.

Fixes issue 2635.
Reviewed at http://reviews.reviewboard.org/r/3371/
Add all new contributors in 1.6.13.
Catch first file patch failed exception.

Catch exception when first file diff fails to apply. This makes the view diff
display the "Diff currently unavailable" error message instead of just throwing
out the traceback onto screen.

Tested on local 1.7 beta 3 (dev) server.
Reviewed at http://reviews.reviewboard.org/r/3324/
Fix a bug with linkify js globbing parentheses

This further fixes the bug with regards to linkifying text. Now, URLs with
anchors (open and close parens, with or without text in between them) work
properly. This fix doesn't affect the behaviour observed with url wrapped in
parens.

Testing done:
Self tested on own environment with different types of urls
- blah blah (http://www.foo.com/)
- http://download.oracle.com/javase/1.4.2/docs/api/java/lang/Object.html#clone()
- http://download.oracle.com/javase/1.4.2/docs/api/java/lang/Object.html#equals(java.lang.Object)
- gwegdss http://www.foo.com/

Fixes bug 2002
Reviewed at http://reviews.reviewboard.org/r/3374/
Fix diff headers in e-mails.

The e-mails weren't updated for the new diff header data, causing the
e-mails to show json blocks instead of header text.

We now use the same logic that we use in the diff renderer template.
Fix alt text for many tags in the Dashboard.

The Dashboard columns were rendering HTML with localized strings being
fed in. These strings used ugettext_lazy, which, when inserted into a
plain string, will basically show a repr() instead of the string
contents.

We now mark these all as unicode strings, which fixes the problem.

Fixes bug #2747
Consolidate and fix the expand diff links.

There's now a couple template tags for rendering the diff links in a
simpler way. This ends up going through a single utility template to
render each link.

As part of this, we now have a smarter place to handle the diff headers.
We no longer show expansion links next to headers showing
functions/classes that are not contained within that collapsed region.
The headers are still shown, but you can't expand.
Add release notes for 1.6.13.