Unit tests

Added unit tests for lowercasify username on login.

Pass validated username to check_login_simple

The login function in mediagoblin/auth/views.py grabs the username prior to
form validation. If validation passes, the pre-validated username is passed to
the check_login_simple function.

Lowercasifying of the username occurs as part of form validation. By sending
the pre-validated username, there's a chance of sending a username with
uppercase letters. This will fail to match any user ids in the database, as
all of the usernames are lowercased during the registration process.

This change sends the post-validated username to check_login_simple, so that
any username that was entered by the user with uppercase letters has a chance
to be properly lowercased before being passed.

Template hook to allow plugin authors to add extra things to the header

Thanks to Andrew Browning for requesting this and suggesting where to
put it!

Fix #1017 - Exception caused by activity being set on incorrect foreign key

The .activity ForeignKey on several models such as MediaEntry expects
a ActivityIntmediatory ID not an Activity ID however in submit code the
activity attribute was being set the Activity, simply removing this
assignment should fix the issue as everything is set correctly at this point.

I have also moved the creation of the activity above the processing of media
to avoid race conditions.

Fix #1007 - get location object not string of location name; caused 500 when editing profile

Fix #1010 - Typos in mediagoblin/media_types/image/migrations.py

Fix #1005 - location.position is a dictionary so 'value' in dict check needed

Only create a slug after a media upload via the API when there is a title

Fix problem where duplicate slug would cause an update exception

Merge branch '984-ai-improvements'

Fix #984 - Improvements to Activity and ActivityIntermediator

- Add unit tests to cover get and set methods on Activity
- Rewrite the set to remove set and use Session.flush instead
- Use sqlalchemy's validator instead of .save hack

airy.css: Override white text in TOS

Fixes bug https://issues.mediagoblin.org/ticket/808

Somehow Shawn and Sergio got in there twice

Make a mediagoblin.ini file for the user if needed.

This is to prevent our docs from confusing people in this
transitionary time period...

Do git submodule init and git submodule update in bootstrap.sh

Idea borrowed from guix bootstrap script.

If this proves to not be great we can move it :)

Bring back the ascii art goblin!  For python 2 anyway.

This does a "friendly" fallback where it skips the middleware on python 3.

Remove the paste.ini rule.

So, we're not really using AC_CONFIG_MACRO_DIR for now.

Update docs: it's now ./bootstrap.sh, also, you need to get a copy of mediagoblin.ini

Clarification on the mediagoblin.ini copying

Give a user a mediagoblin.ini if they don't have one

Conditionally give paste or gunicorn help

Move paste.ini, lazystarter.sh back to the right place

Use python imports to determine whether or not we're using gunicorn

This commit sponsored by Andrés Muñiz Piniella.  Thank you!

Add and use paste server selector.  Yeah!  Smart solutions!

This commit sponsored by Tryggvi Björgvinsson.  Thank you!

That's right, *append* to the list, don't call it :P

Only install gunicorn on py2

Also remove the include directory

Update lazyserver to use paste or gunicorn conditionally

Also get rid of env.in, we're not using that for now.

Various makefile fixes

 - Actually use $(PYTHON) in virtualenv creation
 - Make paste.ini if need be
 - distclean is more forgiving if files aren't there
 - clean up virtualenvs also

Moving experimental-bootstrap.sh back to bootstrap.sh

No rule, yet, to clean house.

Users now get an editable copy of mediagoblin.example.ini->mediagoblin.ini

This way we no longer have people confusedly editing mediagoblin.ini
when they meant to edit mediagoblin.example.ini.  Hopefully it should be
a lot simpler.

Ignore paste.ini because it's just a symlink to the appropriate python
version paste

Symlink the appropriate paste.ini

Set USE_PYTHON3 to yes/no depending on whether --with-python3 enabled

No acceptable *python* could be found

Moved where the python init command was

Most of the way to having a desirable `./configure && make` setup

Tries to simplify a lot of the configure code.  This is mostly because I
found what I expected the previous configure setup to do wasn't quite
what it was doing.

Also supports a --with-python3 option.

Ignore that env file when built

Continuing to iterate toward the right autoconf/make solution

Updated MediaGoblin config stuff to latest pyconfigure release

Iterating towards what we want in the configure setup.

Merge branch 'location'

Add Location model which holds textual, geolocation coordiantes
or postal addresses. This migrates data off Image model metadata
onto the general Location model. It also adds the ability for location
to be set on MediaEntry, User, MediaComment and Collection models.

The geolocation plugin has been updated so that the location can be displayed
in more general places rather than explicitely on the MediaEntry view.
If GPS coordiantes are set for the User the profile page will also have the
OSM provided by the geolocation plugin.

Add location model and migrations

Merge branch '905-activities'

    Add Activity and Generator models which allow for
    activities to be created. This now works with the
    feed API.

Add __repr__ to Activity and Generator class

Fix bug where activity.content was not populated

Fix all the unit tests and clean up code

Fix typo in client registration API for logo_uri

Install ExifRead from GitHub on Python 3.

Add initial migration guide.

This can be moved to docs/source/devel/migrations.rst.

Do not call AlembicMigrationManager.init_tables() for now.

sqlalchemy-migrate will do this for us.

Port "gmg batchaddmedia" command to Python 3.

Restore exif-py to tox config, for now...

Fix #549 - Deauthorize OAuth applications

Fix migrations on Python 2.

This commit sponsored by Loïc Grobol.  Thank you!

Fix #919 - Use class names in super call not self.__class__

Fix #952 - Force the footer to the bottom of the page

Add logging to console to gunicorn

Merge branch '942-default-location'

* 942-default-location:
  Add documentation on changing data dir and Fix stylistic changes
  Updated config_spec.ini to add %(data_basedir)s and make use of it!
  defaults now pulled from a mashup of mediagoblin.ini and config_spec.ini

Add documentation on changing data dir and Fix stylistic changes

pdf tests: use our own pdf doc as GOOD_PDF

remove non distributable good.pdf

.gitignore celerybeat-schedule.

This commit sponsored by Thomas Løcke.  Thank you!

Updated config_spec.ini to add %(data_basedir)s and make use of it!

This commit sponsored by Loïc Grobol.  Thanks!

defaults now pulled from a mashup of mediagoblin.ini and config_spec.ini

This commit sponsored by Thomas Løcke.  Thank you!

Avoid doing python setup.py sdist, use setup.py develop instead for tox

Berker had this previously, but for some reason it wasn't
working... here's why!  usedevelop needs to be in the testenv!  Aha!

This commit sponsored by Gian-Maria Daffré!  Thank you!

Switch print statement to print function

removing exif-py line

Fix Content-Disposition error for retrieving OAuth request tokens

Making alembic happen with python 3 only... for now.

We want to make sure it works nicely first.

Merge branch 'merge-python3-port'

Conflicts:
setup.py

MediaGoblin 0.7.2 development cycle

Fixing bullet points, sigh

MediaGoblin 0.7.1!

Adding Low Kian Seong to AUTHORS

Updated the release notes

Committing extracted and compiled translations

Committing present MediaGoblin translations before pushing extracted messages

download url removed

Merge remote-tracking branch 'refs/remotes/moggers87/garbage-branch'

Temporarily disable garbage collection

We need to make sure we safely set up transactions here... it looks
pretty clearly like we aren't yet, so we need to review this.  This
should be re-enabled by 0.8.0.

Comment out the "check if the database is up to date" stuff

... it's presently causing session issues :\

Last two issues related to the python 3 merge tests: fixed!

 - Fix the "pulling the error out of excinfo" stuff for py3
 - The u"" only gets embedded in the string on py2.

This commit sponsored by Jeff Gibson.  Thanks, Jeff! :)

Modify deleteuser to fail gracefully

bin/gmg deleteuser will produce a traceback error if you attempt to delete a
user who does not exist. This commit provides a fix to allow deletuser to fail
gracefully.

This commit addresses https://issues.mediagoblin.org/ticket/969

Annnnd another json decode fix for py3!  On a roll with these!

This commit sponsored by Ramana Kumar.  Thanks!

decode to unicode before loading in json again, for py3

This commit sponsored by Chris Cormack.  Thanks!

Much more nicely formed form error check

This doesn't rely on checking HTML output... thus, cleaner.

This commit sponsored by Alexandre Guédon.  Thank you!

Fix error check in test_edit.py for python 3

This isn't the nicest of checks... we should probably be checking the
actual form passed into the context.  But for now, it's a fix.

Fix exception catching on python 3

This commit sponsored by Paul Smith.  Thank you!

Fix iteritems usage on python 3

This commit sponsored by Ben (Free Software Melbourne) Finney.  Thanks!

The scheduled garbage collection task was not being called due to it being misnamed

Fix #968

json.loads(request.body) => json.loads(response.body.decode()))

This fixes python 3 stuff.

This commit sponsored by James Reilly.  Thanks, James!

Import mock correctly on py3

This commit sponsored by Andrew McNicol.  Thank you!

Simplify things here and use io.open

Fix test_legacy_api.py

Or rather, reimplement one of Berker's fixes and add one of mine:
 - add back the http_auth_headers fix Berker wrote
 - decode to json when testing the response.body, since we have no
   idea what the order will be here

Import mock from unittest if on py3

Merge branch 'master' into merge-python3-port

Has some issues, will iteratively fix!

Conflicts:
mediagoblin/gmg_commands/__init__.py
mediagoblin/gmg_commands/deletemedia.py
mediagoblin/gmg_commands/users.py
mediagoblin/oauth/views.py
mediagoblin/plugins/api/views.py
mediagoblin/tests/test_api.py
mediagoblin/tests/test_edit.py
mediagoblin/tests/test_oauth1.py
mediagoblin/tests/test_util.py
mediagoblin/tools/mail.py
mediagoblin/webfinger/views.py
setup.py

Temporarily disabling the mediagoblin errors middleware

We don't have paste (core) in py3, so....

No reason to wrap the rest of the options in a quote

This commit sponsored by Thane Williams.  Thank you!

Upgrading gunicorn.  Referenced issue is resolved.

Also, upgrading seems to fix weird
"RuntimeError: dictionary changed size during iteration" issues
I was seeing.  (Looked like a gunicorn issue, not ours.)