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

Excitedly send a user a message that their comment was posted.

Use the cleaned_markdown_conversion method instead of doing that manually

Merge remote branch 'remotes/jwandborg/feature_362-simple_comments'

Merge branch 'feature_362-simple_comments'

Making the messages conform to the 960.gx widths

Styling for the messaging list items.  A rainbow of messages!

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

Issue #362 - Added new `wtforms.Form`; `MediaCommentForm()`

*   ADDED `MediaCommentForm` is a form for `MediaComment`
    user entry and posting.

Issue #362 - Updated `mediagoblin.user_pages.views` to handle new "Simple comments" feature

*   ADDED `media_post_comment(request)` which creates `MediaComments`
    from POST requests
*   UPDATED `media_home(request, media, **kwargs)` now passes `comments`,
    `pagination` and `comments_form` to `media.html`

Issue #362 - Updated media.html with things necessary for the simple comment feature

*   `media.html` now imports `wtforms.html` for use in comment form rendering
*   `media.html` now outputs a commenting form if `request.user` evals to `True`
*   `media.html` now outputs `MediaComments` if `comments` evals to `True`
*   `media.html` now includes `pagination.html` to handle comment pagination

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

adds message queue unit testing

removes messages.add_message from the global template context

templates don't add messages. views are responsible for that.

Documenging index utils and adjusting how remove_deprecated_indexes gets arguments

More specifically, we now just take a dictionary of lists, like:
   {'collection': ['index_identifier1', 'index_identifier2']}

Previously we took something with more info like in add_new_indexes,
but that extra info isn't really necessary.

Documentation updates to indexes.py's docstring

Includes information on how to deprecate indexes.

Updating migrate.py to actually run the indexing commands

Adding our current indexes and removing the index that was in models.py

Tools to add / remove indexes from collections

Issue 362 - Add simple comments

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

adds message queue

Feature #368 - messaging framework
* message levels are debug, info, success, warning, and error
* display mechanism - base.html includes messages.html for now
* fetch_messages and add_message are available to all templates
  using jinja2 environment.globals
* messages are displayed and cleared. no persistence

Store the collection information in the ACTIVE_INDEXES keys

A base set of indexes for us to use with our new indexing tool.

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