| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
StorageObjectWrapper, added with-support to CloudFilesStorageWrapper
* Removed custom `closing()` method
* Removed usage of `closing()` in process_media/__init__.py
* Renamed StorageObjectWrapper -> CloudFilesStorageObject wrapper
In my first version of the StorageOjbectWrapper it was located
inside the CloudFilesStorage object, things have changed since
then but there has been no renaming, thank you Elrond for the
good point.
* CloudFilesStorageObjectWrapper now supports context manager
methods such as `__enter__()` and `__exit__()` (and `close()`)
|
| |
|
|
|
|
| |
- `medium.jpg` issue should no longer exist.
|
|
|
|
|
|
| |
Also removing .generator_error_message() which doesn't make sense
really... we need to get the message when we don't have an instance of
the exception, and this method requires an instance.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Handled in several places:
- In the run() of the ProcessMedia itself for
handled (BaseProcessingFail derived) errors (best to do these not
in on_failure because the errors are highlighted in celeryd in a
way that looks inappropriate for when the errors are well handled)
- In ProcessMedia.on_failure() for all other errors
- In the submit view where all exceptions are caught, media is marked
at having failed, then the error is re-raised. (The reason for
this is that users running in "lazy" mode will get errors
propagated by celery and so on_failure won't run for them.)
|
|
|
|
|
|
| |
BaseProcessingFail based exceptions recorded and marked appropriately
in the database. Other exceptions also caught and marked (or rather
not marked) appropriately in the database as well.
|
| |
|
|
|
|
|
|
|
|
|
| |
* Added configuration options to mediagoblin.ini
* process_media supports the python-cloudfiles
almost-file-like objects by wrapping them in a
contextlib.contextmanager-decorated func.
* storage now has the CloudFilesStorage
* New dependency added to setup.py; `python-cloudfiles`
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
mediagoblin/db/migrations.py
mediagoblin/db/models.py
mediagoblin/process_media/__init__.py
mediagoblin/templates/mediagoblin/user_pages/media.html
mediagoblin/util.py
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* `migrations.py`
* Removed empty line
* Added empty line
* `models.py`
* Added `MediaEntry.get_display_media()` helper function
* `process_media.__init__.py`
* Updated `process_media_initial()`
* Renamed `main` => `original`.
* Added condition to `medium`, it's only created if the original
dimensions exceed the MEDIUM_SIZE dimensions.
* `media.html`
* The image tag is now populated by `MediaEntry.get_display_media()`
* `util.py`
* Added `DISPLAY_IMAGE_FETCHING_ORDER`, used by `MediaEntry.get_display_media()`
|
| | |
|
| | |
|
|/
|
|
| |
We don't know if that's permanent but if it isn't it seems "Mostly Harmless"
|
|
|
|
|
|
|
|
|
|
| |
* `mediagoblin.process_media.__init__`
* Added `medium` size image conversion
* Updated `thumbnail` conversion to use `queued_filename` instead
of `queued_file`
* `media.html`
* If there exists a `medium` size for the `MediaEntry`, it will
display instead of the original `main` image.
|
|\ |
|
| |
| |
| |
| |
| | |
And add a lot of warnings, as the checks for "being part of
the main Manager" are all gone.
|
| |
| |
| |
| |
| | |
Some references to Workbench.dir look ugly, I'm happy to
hear suggestions there.
|
|\ \
| |/
|/|
| |
| | |
Conflicts:
mediagoblin/process_media/__init__.py
|
| |
| |
| |
| |
| | |
Just a small refactoring of the filename setup in the
public store. Very simple.
|
| | |
|
| |
| |
| |
| | |
be clear that it's okay to do by doing it *somewhere* :)
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \ |
|
| |/
| |
| |
| | |
set up in the right order.
|
| | |
|
| |
| |
| |
| | |
functions but haven't actually tested them yet. :)
|
|/ |
|
|
|
|
|
|
| |
We used to import those from pymongo and mongokit directly.
We should import them from a single place. So let's try
db.util for this.
|
| |
|
| |
|
|
Haven't completely checked it for workingness, and not the final form
this will take :)
|