Fixed EXIF tests

Modifies EXIF section with Camera Info, display toggle and template styling

Changes to display all EXIF information

Added upload processing to the piwigo/addSimple

Just word-wrapping the recent changes to the deployment docs.

Merge remote-tracking branch 'refs/remotes/merge-requests/58'

Noting why we don't have an email uniqueness constraint in the db.

This commit sponsored by Guido Günther.  Thanks!

Merge remote-tracking branch 'refs/remotes/elrond/misc/pytest_enable_testing'

Reverting "Always activate testing in every test module ever."
Revert "Always activate testing in every test module ever."

This reverts commit 0536306048daa0970d2e43411ba2a9bf073e570e.

Always activate testing using pytest fixture.

This is possibly the cleaner approach to the problem solved
in the previous commit.

The previous commit makes sure, that testing is enabled at
import time. While this one enables it just before the
individual test is run. So this one takes a bit more CPU.

Maybe revert the previous commit?

References: 0536306048daa0970d2e43411ba2a9bf073e570e

Always activate testing in every test module ever.

Kind of a dorky way to implement this, but...

Fixing bug in dbupdate where it would explode on plugin that is missing MODELS or MIGRATIONS

The reason it blew up was because in the latter caught exception, it
wouldn't set models/migrations to an empty set, so it would actually
use the previous run's models/migrations!  That's what we get for
"leaky" variables on python for loops :)

This commit sponsored by Pascal Diogo Antunes.  Thank you!

Merge branch '623_context_hooks'

Testing the template_context_prerender hook

This allows for modifying any context *right before render*, including
access to the variables that are passed in.  This test takes advantage
of that and takes one of the variables, "doubleme", and modifies
it (doubles it!)

In our case it turns "happy" and "joy" into "happyhappy" and "joyjoy".

This commit sponsored by Mark Holmquist.  Thank you!

Simple tyop, view->test... I was writing too many tests at the time :)

Oh right, actually add that hook we just documented, "template_context_prerender" :)

This commit sponsored by William Goudie.  Thanks Bill! :)

Hook->hooks since there's more than one of them :)

Adding global context hooks & fixing method names->symbolic view names in docs

This commit sponsored by Sheila Miguez.  Thanks Sheila!

Added documentation on view specific hooks

This commit sponsored by Matthew Woodward.  Thank you!

Normalizing indentation a bit in media.html

This commit sponsored by Christian Frank.  Thank you!

fix for bug when comments are disabled, link attribute is never closed

Documenting the test_modify_context and context_modified_app methods

Fully working context hooks, both template/view and global level, with tests

Needs documentation though... that's coming next :)

This commit sponsored by Luca Tius.  Thanks Luca!

piwigo: Start first tests.

This first test checks the session logic of the piwigo
plugin. login, logout, getStatus.

piwigo: Fix validator usage.

wtforms.validators.Optional doesn't take an argument.
I don't know, why I gave it one.

Merge branch 'master' into 623_context_hooks

Removing reference to nose (thanks for catching, Elrond!)

This commit sponsored by Kevin McCarthy.  Thank you!

Tests should no longer use the environment-variable-related-setup and set up in app instead

This should remove a bunch of confusing cruft.  I hate using that
environment variable!  Also that old code was fragile.

This commit sponsored by Stephen Milton.  Thanks!

Making the uncomment-able db string match the one that we use in the install docs

This commit sponsored by Denise Merkel.  Thanks, Mom!  Happy Mother's Day!

Moving the get_original_date method over to the MediaManager.

This fixes a bug so that it doesn't explode if the media_data doesn't
exist.

This commit sponsored by Ole-Morten Duesund.  Thank you!

Updated "created on" feature to reflect new "hover for real time" tooling

Merge remote-tracking branch 'refs/remotes/tryggvib/532-exif-creation-date'

Conflicts:
mediagoblin/config_spec.ini
mediagoblin/templates/mediagoblin/user_pages/media.html

Merge remote-tracking branch 'gabithume/146_debug_message'

Use the controller's symbolic/lookup name as part of the key for context hooks

This commit sponsored by David Collins.  Thank you!

Merge branch 'master' into 623_context_hooks

Merge remote-tracking branch 'refs/remotes/dthompson/453_disable_comments'

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

Merge remote-tracking branch 'refs/remotes/merge-requests/47'

piwigo: Better logging for login.

piwigo: Let getStatus return the current user.

If there is a user logged in, show his name.

Create new session system for piwigo plugin.

Using the brand new itsdangerous sessions to power the
sessions for piwigo.
The real point is: Clients want to have the session in a
"pwg_id" cookie and don't accept any other cookie name.

piwigo: Add .images.add including form handling.

To make things a bit easier, switch to WTForms
for validating the received data.

piwigo: Remove possibly_add_cookie.

This one was a fake thing to make clients happy.
Real sessions coming sonn.

Fix the new config "plugin spec" loading code so it doesn't break on no plugins section

Fixing API setup with new plugin "config spec" world

It shouldn't reference the config until in the setup_plugin() method,
else there's a race condition.

Ah right, and here's the config we intend to use for these context modified tests :)

This commit sponsored by Xavier Gulliot.  Thanks Xavier!

The beginning of context hooks.

Not the working solution, but getting there conceptually.  Basically
we'll have a key with the view and the template as a tuple which is
the context hook that anyone can attach to.

However, some changes have still to be made:
 - The unit test doesn't work yet and contains a set_trace ;)
 - We'll probably switch the "view" component from being the callable
   to the "urlgen"'able name per Elrond's suggestion
 - Found a bug in unit tests related to running custom apps for
   different configs... hm.  I need to fix this!

Nonetheless, making progress.

This commit sponsored by... wait a minute... Christopher Webber?!

Documenting plugin configuration

This commit sponsored by David Krupicz.  Thanks, David!

Unit tests for plugins defining their own config_spec.ini!

This commit sponsored by David Ahmed.  Thank you!

Allow plugins to install their own config_spec.ini if they like.

This commit sponsored by Timothy Vollmer.  Thanks Tim!

Added info about editing mediagoblin.ini
Added link to Apache deployment instructions

Removed unused imports in httpapiauth

Use collection.url_for_self more; Add missing /a.

Using collection.url_for_self(request.urlgen) instead
of request.urlgen(lengthy) is so much nicer, so using
it around the place.

Also added a few missing </a> in one template.

Fix typo in new stl MEDIA_MANAGER.

Fix media manager reference in video.

An uninstantiated bare media manager doesn't any more
support ["something"]. You need to do .something.
Fixed for .source_type().

Make all media types use the new MediaManager classes.

Fixing ticket #146 - Make messaging system testable

Add warning about crypt/itsdangeroussecret.bin.

You should not leak that file, really.

Create redir_obj and use it around.

This is a shortcut function to redirect to the main page
for an object. Objects currently supported: media entries
and collections.

And go around and replace various places to use this.

Rewrite resize_image to use proc_state.store_public.

This hopefully makes the code nicer and easier to change.

image resizing: Refactor some decisions into resize_tool.

Loading the thumb/medium sizes from the config, saving
things to the db, and loading the image is now all done by
resize_tool. It still calls resize_image for the actual
work.

Make uuid look like a uuid.

If we really have to create a visible uuid (for a slug in
this case), don't try to hide the fact that it is a uuid.
So format it like a uuid.

Remove extra indentation left over from previous commit.

This only removes an unneeded extra indentation, left over
from the previous removal of code around.

Extra commit so it is easy to check that it only changes
indentation.

Fix translations for collections and drop useless try.

Don't do:  _("With some value: %s" % value)
Please do: _("WIth some value: %s") % value

Fixed for collection messages.

Also removed a
  try:
    some_code.
  except Exception as e:
    raise

No point in doing that.

Fixing the indentation of some_code comes in an extra
commit, because changing indentation is annoying enough
alone, so don't mix it with other changes.

word-wrapping the public/queue storage explainations in storage.rst

Merge remote-tracking branch 'refs/remotes/npigeon/ticket-660'

Switch "sqlite_refcheck" keyword arg to "migrations" which Elrond thinks is cleaner

Also, if migrations is true, *explicitly* say that foreign key checking is off

Don't turn on sqlite refcheck stuff during migrations

Merge remote-tracking branch 'refs/remotes/rodney757/issue643' into mergetest

Removed an unused import

Unit test for ReallyLazyProxy.

LazyProxy caches its value. So create a unit test for
lazy_pass_to_ugettext to make sure its returned proxy
actually gives a different translation when the locale
is switched.

No lazy_* needed here; Add ReallyLazyProxy.__repr__.

One should use lazy_* only if you have a good reason.

This one found by our unit tests!

For example add_message adds the message to the session,
the session needs to be serialized (with json) and well,
LazyProxy is not serializable.

To aid in debugging, gave our ReallyLazyProxy a __repr__.

Adding ReallyLazyProxy, a proxy that does what we expect :)

Switching non-forms back to using normal pass_to_ugettext

Fix-bug-667-Use-lazy_pass_to_ugettext-for-forms.

Check for duplicate collection slugs and make them unique. Add unique constraint to collection.slug model

Create set_thread_locale and use it.

Setting up a new template env should not directly touch the
translation tools, but go through a function in
translations.py.

resize_image: Change calling style a bit.

- entry: not used, removed.
- proc_state: going to be used, added.
- size_limits: not used, removed.

mediagoblin/submit/lib.py: fix typo

Signed-off-by: Alon Levy <alon@pobox.com>

Create add_media_to_collection()

The ideas is by Alon Levy.

Not only media_collect, but also other places might want to
add media to a collection. So refactor this into a function
for easier usage.

Doc string for ProcessingState.

resize_image: Refactor config loading a small bit.

Well, get the config into a local variable, for easier access.

Cleanup storage after test, and test .delete_dir().

The storage tests work in the system's tmpdir. The python
docs say, we should clean up after using things.  Yes the
directory should be cleaned up on reboot, but if running
tests a lot, the tmpdir could fill up, so we should really
cleanup.

So use the new .delete_dir() on the storage interface to
cleanup test dirs and get them finally removed with
os.rmdir. All nicely packed into cleanup_storage().

Merge remote-tracking branch 'remotes/npigeon/ticket-601'

bumping image quality to 90, after conversation with Elrond

Bumping the default image quality up to 85.

image quality can be set in config file

(cherry-picked from
 https://gitorious.org/~dunkyp/mediagoblin/dunkyps-mediagoblin/commits/image_quality)

Refactor test configs.

All the hook_* tests use the same config, so refactor it.

removing old callable utilities and porting stuff over.

Let's not use reserved keywords :)

changing the things to document in api.rst

Added new tests to test hook_transform()

we should be returning the result, not None!

Adjusting tests to use hook_handle and hook_runall

fixing hook_handle for when result is never assigned, avoid reference without assignent

As Elrond points out, we might as well .pop() default_handler

Hooks: the Next Generation!

I have not tested any of this but it looks right ;)

A warning about the plugin API being unstable.

And... pyblosxom code removed again :)

We agreed on a new direction on IRC last night...

pyblosxom run_callback hoisted