Issue #362 - Updated the MediaComment model

*   `MediaComment.get_comments()` now uses pagination
*   `MediaComment.get_comments()` now sorts by `created` DESC
*   `MediaComment.media_entry` is now **required**
*   `MediaComment.media_entry()` now returns parent `MediaEntry`

Issue #362 - Added a route to MediaComment POST handler

Issue 362 - Add simple comments

*   Added MediaComment database model
    Holds `media_entry` (`ObjectId`), `author` (`ObjectId`), `created`, `content` and `content_html`.

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

Celery wasn't really being properly connected during tests.

Lots of fixes to do this.

 - setup_celery_from_config no longer responsible for checking
   'celery_setup_elsewhere'; that's the app's job.  (This was a problem
   because more than the app was relying on using this function)
 - Allow us to specifically set the config file we're setting up
   celery from with setup_self
 - Set up celery_always_eager.  This is something we strongly want
   while doing tests.
 - Instead of setting up the app in the get_test_app method, let's set
   that up simply by importing from_tests, which should itself up via
   from_celery being the environment variable being set.

Reordering the uploaded on/by and description sections of media display template

Align images in a gallery to the center

Just adjusting indentation a little bit in the base template.

Merge remote branch 'remotes/gullydwarf-cfdv/is330'

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

Make the width of these fields 100% rather than providing a specific pixel width.

Put the thumbnail on the edit details part on the top and center

Setting height for media thumbnail to also 200px

Not sure why height wasn't at 200px but anyway not having at 200px
means some images would hit overlap.

Adjust the width of the fields on the form submission

Did several things, including:

 - Call the individual form field rendering macros instead of the auto
   whole-form renderer so we get better control over the TextArea
 - Use the textarea specific form field rendering macro
 - Adds textarea styling to the css
 - Sets the input, textarea styling to be 460 px wide (the width of
   the form section, not including padding)

Separates out the field rendering part from the whole form rendering macro

Also adds a textarea-specific version that renders rows and cols as
part of the input.

Implement basic footer

Remove excess grids, change CSS to center forms

Add <br /> for correct text layout

Submit form should not have edit_box class

Make several small changes to wording and top bar links

Merge remote branch 'remotes/schendje/960-grid-system'

Slightly clearer message about email marked as verified w/ adduser gmg command

Setting status to active when adding users with ./bin/gmg adduser

Making arguments positional and removing the "usage" functions

All these arguments were required and might as well be positional
thusly.  If you don't provide all required positional arguments,
argparser will correct you and request usage.

No need to describe usage of subcommands, gmg subcommand --help does that

Adding new function to gmg_commands, adduser,makeadmin and changepw issue 373

Switch css over to 960.gs. If I did everything right, things should look exactly the same.

Check on media['uploader']'s id rather than the username

This skips making an extra query to retrieve the uploader object and
also checks on the canonical ID, which is probably good (even if
indexing should ensure uniqueness there).

Merge remote branch 'remotes/gullydwarf-cfdv/is388_only_media_owner_edits'

Two not three ways to contribute now, reflecting that in docs :)

adds conditional for display of media edit link

Add updates background images

Several small UI changes to edit profile page: moved header, changed form wording, changed css class

Merge remote-tracking branch 'gitorious/master'

Merge remote branch 'remotes/elrond/idea/invalidid'

Merge remote-tracking branch 'gitorious/master'

I don't think we need the username to be part of the profile editing URL

Adding an extra space between function definitions

Indenting some sections between if / else blocks in root.html

Merge remote branch 'refs/remotes/gullydwarf-cfdv/is315'

Extra checks so that we don't even RUN tests unless the right celery environ set

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

Make sure we *ONLY* ever kill the test database, EVER.

Merge remote-tracking branch 'gitorious/master'

333. Adds explanation of command lines; tweaks format-patch line

* Adds explanation of command lines and their arguments (gullydwarf's
  suggestion).
* Switched the ``git format-patch ...`` line from using -o patches to
  using --stdout.  This creates a single patch file and is (probably)
  easier to deal with.
* Fixed a typo.
* Adds toc to top of mediagoblin chapter

Also fix clean_html so that it doesn't barf on an empty string.

Merge remote-tracking branch 'gitorious/master'

assigns migration steps to User database objects

adds the migration_handler to the User db class, connecting the
migration steps in ../db/migrations.py to the migration code in
gmg_commands

Only migrate to description_html if description also exists.

Fix cleaned_markdown_conversion so that it doesn't bork on empty strings

Basically, clean_html would throw an error on '', so we just return ''
"if not text"

removes unecessary dependence on existence of username for User migration01, + fix whitespace issues

adds accommmodations for migration of fields in User: bio and url

Merge remote-tracking branch 'origin/master' into is315

Merge remote-tracking branch 'gitorious/master'

Also moving the test_server.ini to test_paste.ini to avoid ambiguity.

Moving server.ini->paste.ini because willkg points out server.ini is ambiguous

Updating the file, updating lazyserver.sh, updating docs.

Updated git documentation to have more useful branch names.

I haven't discussed this with Will yet... if he gets unhappy we can
roll back this documentation change :)

Provide a migration to add description_html to MediaEntries that don't have it

Adds util.cleaned_markdown_conversion() and uses it in the submission process

This simplifies the markdown processing & html cleaning of descritions
and etc by providing a wrapper function that we can use in multiple
locations.

Merge remote branch 'remotes/jwandborg/issue_363'

Conflicts:
setup.py

Merge remote branch 'remotes/cmoylan/test_auth_views_364'

Merge branch 'is330', remote-tracking branch 'origin/master' into is330

Merge remote-tracking branch 'origin/master' into is315

Move InvalidId to db.util

Instead of import InvalidId from the low level bson module,
first import it in our db.util wrapper and second import it
from pymongo.errors.

Making ./runtests.sh the official way to run the tests.

./runtests.sh: run unit tests without having to remember the long command.

Need to run nosetests with CELERY_CONFIG_MODULE set to from_tests again.

Basically, if we don't do this celery sets itself up before it should
and improperly. :\

Documenting the setup_globals function.

Document our new global objects added during the configobj branch

Merge remote branch 'remotes/elrond/idea/globals_check'

Conflicts:
mediagoblin/tests/test_globals.py

Reset the globals parameters while testing parameters

(This way we can be sure that the database is torn down if necessary
but this was the only test that passed last.)

Re-commenting-out lxml from setup.py

Clarifying that this is something to install from the package manager
and not via python setuptools itself.

Updated hacking howto to reflect our new, easier to run lazyserver.sh command.

./lazyserver.sh: launch a server in the most basic way possible

Looks for paster either in buildout or in virtualenv / on the path,
wherever it seems to be.

Actually we need pkg_resources again in test_celery_setup :)

This got removed in master at the same time that it got used in the
configobj branch...

Merge branch 'configobj'

added test coverage for redirecting after login with the next param

Adjusting hackinghowto docs to reflect change in server/tests commands

Removing a couple of unused imports from util.py

Removing "paste config" phrasing where we're no longer using paste's config

renaming storage_system_from_paste_config()->storage_system_from_config()

As Elrond points out, this name doesn't make sense anymore since this
isn't based on the paste config.  Thanks Elrond!

util.read_config_file() no longer needed; removing.

Added tests for all sorts of login form abuse.
Added tests for log out

enables primitive display of profile data on user page

enables entering user details including website url and bio

fixes usage of objects. still need to display the data on user page

Preparing celery unit tests for new setup.

Instead of the previous passing in of dictionaries, we're actually
checking some example config files.

./bin/gmg commands upgraded to work with the new setup.

Nosetests should now be able to run using the new configobj / app init setup

Lots of changes:
 - CELERY_CONFIG_FILE does not need to be set to the from_tests module
   to run tests anymore, in fact it *should not be set at all* and is
   specifically forbidden.
 - moved around the configuration to the new 2-file format
 - and generally adjusting the code appropriately.

Only kill the database if it's really set up.

It's a good idea for us to pass the application itself into mg_globals :)

Updating celery_setup.from_celery to use new config loading / app init

 - The code for this is significantly simpler now.  The app sets up
   everything but celery, and from_celery finishes the job.
 - There's no more specifying the mediagoblin section in the file,
   which doesn't make sense anymore and was already confusing.

Merge remote-tracking branch 'gitorious/master'

Removing option to set celery config section.

There's no real reason to support setting the celery config section;
so we'll just make it always ['celery'].

Removing type conversions from setup_celery_from_config.

These simply aren't needed any more, not now that the config validator
stuff does type conversion for us.

Also fixed the docstring to explain force_celery_always_eager.

Erk!  Forgot to include the new mediagoblin.ini.

Move entire app structure over to using the new config system.

This is a huge change!  This means several things.

 - From the python point of view, launching the application is a heck
   of a lot cleaner.  You just need to pass in the config file path to
   MediaGoblinApp's __init__() and whether or not this funtion should
   setup celery and you're good.
 - There are now two separate config files, separating the server
   setup from the application setup.
   - server.ini: the paste deploy config file, which configures the
     applications and server setup but *NOT* the mediagoblin application
     itself.
   - mediagoblin.ini: where you configure mediagoblin (and possibly celery)
 - Launching the application is now different.  Instead of:
     ./bin/paster serve mediagoblin.ini --reload
   We launch like:
     ./bin/paster serve server.ini --reload

Tests for mediagoblin.config functions

Tests for:
  - read_mediagoblin_config()
  - generate_validation_report()

CONFIG_SPEC_PATH should be config_spec here, fixing.

baby step towards enabling profile edits

adds
* url and bio fields to database
* form for editing the user profile
* route to the edit profile controller
* view for the profile editing page
* template for the profile editing page
* link to edit profile in the welcome page

still needs
* thorough inspection to see if it makes sense
* tests
* ?

Removing unused pkg_resources import from test_celery_setup

Make REPORT_HEADER a unicode string also.  Unicode everywhere, ideally!