PEP-8'ifying prompt_if_not_set

Merge remote-tracking branch 'remotes/manolinux/645_gmg_adduser_prompt'

Raise a slightly useful exception when we can't find the media type.

Updated translations

Merge remote branch 'remotes/manolinux/671_spaces_in_tag_list_edit'

* remotes/manolinux/671_spaces_in_tag_list_edit:
  * Feature #678: Drop custom delimiters in tags   * Eliminate the definition of the tag delimiter for tests.   * Remove a test that was related to custom tags delimiter. * Bug #671: Tags list on Edit page is not seperated by spaces and hard to read   * Modify a test to include this space.
  * Bug #671: Tags list on Edit page is not seperated by spaces and hard to read : Make 'media_tags_as_string' function put a space after each comma. * Feature #678: Drop custom delimiters in tags : I declare a constant in the begining of text.py file.

Bug #685: only provide CSRF token if it exists

This was suggested by Nathan Yergler in the bug logs.
Just implementing it.

- Let render_csrf_form_token return None, if the CSRF_TOKEN
  is not available in the environ, because the
  process_request part of the meddleware has not yet run.

- In render_template: If the returned value from above is
  None, then do not add the csrf_token to the templates
  context.

Bug #685: Add failing unit test

The simplest way to reproduce Bug #685 is to ask for a non
existent page. This should return a proper 404. It
currently doesn't.
So add a unit test exactly for this.

This unit test fails currently! It will fail until the bug
gets fixed.

Prompt for arguments in adduser if not present (I created a function in util.py to check and prompt for arguments).

Change adduser arguments from positional to --keyword style.

Changed comment error message wording slightly. Btw, should we translate these things?

We should use the variable local_templates instead of user_template_path

Expect 404 in unit tests, if we now use 404.

Our unit tests for auth were expecting a 400.
Well, now we give a 404. So expect that!

I'm not completely sure, if the 404 is the right thing
here, but that's another topic.

Merge branch 'master' of gitorious.org:mediagoblin/mediagoblin

Should be 404 for 404s, not 400 :)

Don't barf on templates that use the autoescaping extension

Merge remote branch 'remotes/nyergler/issue-680-csrf-optout'

* remotes/nyergler/issue-680-csrf-optout:
  Issue 680 Allow decorating views to prevent CSRF protection.
  Issue 680: Dispatch meddleware request processing post-routing

Mark two strings for translation

1. "Go to page:" in pagination
2. "Submit" in the forget password form

Merge remote branch 'remotes/manolinux/683_text_separate_tags_by_commas_and_spaces'

* remotes/manolinux/683_text_separate_tags_by_commas_and_spaces:
  Fix the text "Seperate tags by commas and spaces" since spaces are not used to seperate anymore.

Updating translations

Fix the text "Seperate tags by commas and spaces" since spaces are not used to seperate anymore.

* Feature #678: Drop custom delimiters in tags
  * Eliminate the definition of the tag delimiter for tests.
  * Remove a test that was related to custom tags delimiter.
* Bug #671: Tags list on Edit page is not seperated by spaces and hard to read
  * Modify a test to include this space.

* Bug #671: Tags list on Edit page is not seperated by spaces and hard to read : Make 'media_tags_as_string' function put a space after each comma.
* Feature #678: Drop custom delimiters in tags : I declare a constant in the begining of text.py file.

Issue 680 Allow decorating views to prevent CSRF protection.

Issue 680: Dispatch meddleware request processing post-routing

Merge remote branch 'remotes/inconexo/588_lowercase_extension'

* remotes/inconexo/588_lowercase_extension:
  Filename extensions are lowercased before uploading the image.

The merge failed really. It tried to apply the changes to
the wrong file. So applied them by hand to
mediagoblin/media_types/image/processing.py

Conflicts:
mediagoblin/processing.py

Use setup_global_and_app_config in gmg's migrate.

Instead of using read_mediagoblin_config, forgetting to
check the validation report and then finding the main app
section by hand, just use setup_global_and_app_config.

Add base class for Meddleware

Created a BaseMeddleware which all Meddleware should derive
from. This is not strictly needed, but will greatly help.

The base class has the common __init__ of all the other
Meddlwares and fall backs for all hooks. That way a new
Meddlware only needs to override what it actually wants to
implement.

Uncommenting requires=['gst'] till I figure out why Joar added it there :)

Rename MediaGoblin middleware to meddleware to avoid confusion w/ wsgi middleware

hehehehehe, "meddleware"

Removing these video javascript files, which are currently unused

Remove old enable_video config option

Load multiple media types based on the media_types section of the config file

Merge remote-tracking branch 'remotes/jwandborg/master'

Rename MediaEntry.uploader() to .get_uploader()

The .uploader() method conflicts with the uploader database
field. As we're moving to .FIELD for db field access, this
is a relevant conflict.

So renaming .uploader() to .get_uploader()

652: Don't show empty field labels.

If the label for a field is empty, don't show it at all.
And don't translate it!

Wrap long line.

Nothing else.

Added a message noticing the user the image has been successfully
deleted.

Fixes for video branch

- Removed superfluous code from media_types.image
- Updated lazy_pass_to_ugettext imports

Fixes #597. Add a visible error when user tries to delete an image
without cheking the "I'm sure" checkbox.

Video support is disabled by default, set enable_video to true to enable

Fixes after merging video into master - part 2

- Added handling of InvalidFileType to submit.views
- Updated test_celery_setup and test_submission tests to reflect the changes to the
  media procesing infrastructure

Fixes after merging video branch into master

- Removed debug output from init/celery
- Moved process_media/__init__ to processing.py
- Centralized the processing.ProcessMedia task class
  - Updated media managers to reference the processing function
    instead of the ProcessMedia instance
  - Updated new-style image processing to previous, newer old-style
    image processing
- Updated video transcoding
  - Changed method in progress output, sometimes
    message.structure['percent'] raises KeyError

Filename extensions are lowercased before uploading the image.

Merge branch 'video_gstreamer-only'

Conflicts:
mediagoblin/db/migrations.py
mediagoblin/submit/views.py

Refractored GStreamer element linking

Tests for StorageInterface*.copy_local_to_storage()

Made the image processing use intermediary conversion file.

This should fix the problem with PIL and the cloudfiles storage system
fighting.

Slightly clearer docs on copy_local_to_storage

copy_local_to_storage, both general and specialized-for-BasicFileStorage versions

This utility should allow for easy copying from a local filesystem to
the storage instance.

Stashing changes

fixed identation

eyecandy for programmers

don't use 'and' anymore, if there is only one tag

Fix redirect to logical path

redirects should in nearly all cases go to a logical path
like 'mediagoblin.auth.login' and not to an absolute path
like "/auth/login".

Merge remote branch 'remotes/aaronw/bug614_verification_crash'

* remotes/aaronw/bug614_verification_crash:
  Update english translation file.
  Reverse order of sanity checks: check email_verified after making sure there's a user in the request.
  Make sure user isn't already verified before resending verification.
  Check request.user to determine if user is logged in.
  Regenerated English .po file to include new string.
  Display and error and redirect to login page if unauthenticated user tries to access resend_verification.

Conflicts:
mediagoblin/i18n/en/LC_MESSAGES/mediagoblin.po

Navigation buttons edits. Removed images as they are no longer needed. Related: bug #504

Remove Edit/Delete icons, since they are not required yet.

Change tag list from a list to a paragraph. Wrap text for translation.

Fix for bug #467, "Add explanatory copy to add/edit picture pages saying that tags are comma-separated"

Slight style changes to button_action_highlight

Small correction, this button should be button_action, not button_form

Slight change to error wording

Another change to button style. Renamed header_submit, header_submit_highlight and button classes, correct all references to these

Change to background of "empty_space", it now uses an image

added unittests, now using form errors and fixed bug when no GET parameter is given for /edit/profile/

added support for changing the password, issue #643

Change button style a bit

Change form structure and add relevant CSS rules

Added back the title block

Merge remote branch 'remotes/inconexo/613_custom_title'

Added Pablo Santos to the AUTHORS file

Allow user to set whether comments are ascending or descending

Merge remote branch 'remotes/inconexo/441_comment_order'

Added Corey Farwell to the list of contributors

incorrect path in nginx config

media_home: order comments by ascending date.

Added parameter ascending to MediaEntry::get_comments, if true, comments
will be ordered ascending, otherwise descending

Allow instance owners to customize html titles of page:
Added html_title config option.
Made base.html template use html_title option as page title.

Add app_config and global_config to the template environment

Adding app_config and global_config to the template environment

Merge branch 'master' of gitorious.org:mediagoblin/mediagoblin

Conflicts:
mediagoblin/templates/mediagoblin/user_pages/user.html

Accidentally had user['profile'] where it shoulda been user['bio']

Fixed video transcoding

- Added audiorate with tolerance 80 million
- Removed deprecated thumbnailer

Committing some futile attempts to make GStreamer transcode the audio properly.

- Added CPU count detection
- Added videorate
- Added audiorate

Dot-Notation: x._id = ObjectId() doesn't seem to work properly

For whatever reason, this does not work as expected:
entry._id = ObjectId()
Need to go this way:
entry['_id'] = ObjectId()

Dot-Notation for "_id"

Note: Migrations can't use "Dot Notation"!

Migrations run on pymongo, not mongokit.
So they can't use the "Dot Notation".
This isn't really a big issue, as migrations are anyway
quite mongo specific.

Enable mongokit's "Dot notation"

mongokit documents can allow to use x.FIELD instead of
x["FIELD"].
First it looks a lot more pythonic.
Second it might allow us an easier migration path towards
an sqlalchemy database backend.

Docs: http://namlook.github.com/mongokit/tutorial.html#dot-notation

export: Handle Unicode titles better in logging

log("ascii %s" % unicode_string) tries to convert unicode
to ascii, which might fail.

Better use log(u"unicode format %s" % unicode_string) and
let the logging framework handle the conversion. This works
much better and the exceptions still happening aren't
stopping the main app.

Also remove one useless import.

moved from videoscale => ffvideoscale *and* put queus before video and audio pipes

Only show "post a comment" link if comments already exist

The purpose of the link is to help you jump past comments to the
comment box, and so...

Even with this new conditional, I'm not entirely sure I like that link ;)

Merge remote-tracking branch 'remotes/nyergler/pep8-ification'

Conflicts:
mediagoblin/db/migrations.py
mediagoblin/db/models.py
mediagoblin/user_pages/views.py
mediagoblin/util.py

Improving <title> on tag page *and* adjusting translation for RTL reasons

Basically moving the variable inside the translation to give
translators more flexibility

Enhanced title on the user's main media gallery

If the gallery view makes sure we have a user anyway, do we need this check?

Seems like the classic annoying "SHOULD NEVER HAPPEN" else: statement :)

Enhanced title for user profile page

Improved title block on media page

Merge remote-tracking branch 'remotes/nyergler/issue-653-csrf-exception'

Issue 653: This time for sure!

Issue 653: Handle the case where request.vary is None

Issue 653: Don't throw exception if response has no vary header.