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

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.

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'

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.

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!

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

Moving the rest of the relevant tests to have test_app set up in "def setup()" as well

at pytest author's advice, changing the way _setup back to -> setup

They showed me how to use @pytest.fixture, which allowed us to pass the
test into setup!

piwigo: Send NotImplemented for unknown methods.

That's somewhat, what piwigo does.

Use check_file_field in pwg_images_addSimple.

Start to use six for basestring.

six allows us to smoothly get more forward compatible with
py3.  The idea is to change things over to use six, when/if
we feel a need for it.

stl:blender_render() - Remove useless arg_string.

Fixing typo in calling render_http_exception

ppoppler -> poppler tyop fix

Turning image's media manager into a new style class.

images are the first media type to use the new style class
based media manager.

Add sniff_handler method to mediamanagers.

The MediaManagerBase gets a fallback "return False" one.
The Compat-Manager checks in the dict and calls that one or
returns False.

Add MediaManagerBase class.

This is the base class for all new style class based media
managers.

MediaManager: Use .foo instead of ['foo'].

To make .media_fetch_order work, create a property.

First step towards a MediaManager class: Compat one.

To get us moving towards a MediaManager class, the first
idea is to create a class that wraps our current dict based
manager and makes all users happy.

update documentation for s/nose/py.test/

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

PIL: Support systems with Pillow and without

Fixes for systems with Pillow, but leaves a "try: except ImportError" to
support anything that doesn't have a PIL top level import.

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

pdf/processing.py: don't use check_output, only in python>=2.7

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

Refactor test resources into new resources.py

PDF: Use pytest.mark.skipif for skipping tests

Instead of leaving test early if they can not run, use the
pytest.mark.skipif marked to tell the test system not to
even run the test.

This also adds to the stats, because skipped tests are
counted differently. Thus making it obvious, that some
tests did not run, because of any reason.

Switch to an apparently newer version of pdf.js per irc.mozilla.org #pdfjs recommendations

Updating branch...
Merge branch 'master' of git://gitorious.org/mediagoblin/mediagoblin into ticket-660

Changed the max character length for tags on media to 255. Only had to change one line in mediagoblin/config_spec.ini
in response to isses.mediagoblin.org/ticket/601

Transfered information from wiki.mediagoblin.org/Storage into docs/source/devel/storage
For info check out ticket: issues.mediagoblin.org/ticket/660

test_timesince() doesn't need a complete test_app.

Removing unused imports

Removing unused imports and an unused media_files dict reference

pdf: use pdf.js in an iframe

add pdf media type

The new media type supports pdf and a subset of media recognized by libreoffice via
unoconv.

Every document added goes through:
* conversion to pdf with unoconv if not already a pdf
* creation of thumbnail and medium sized image, and pdfinfo generates
 some information (even for unoconv produces docs - should fix this)

Poppler (pdftocairo, pdfinfo) is used.  http://poppler.freedesktop.org/

A working but uglified pdf.js integration exists, which is enabled by
setting pdf.pdf_js=true mediagoblin_local.ini (disabled in mediagoblin.ini)

Adds one test to the test_submission test suite, and another separate test_pdf suite.
The tests are only run if media_types.pdf.processing.check_prerequisites passes, so
the test suite will not require any extra package.

TODO: make test suite say 'skipped' in that case instead of just 'ok'

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

document submodule usage

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