aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Added thingiview.js symlink, fixes webgl viewJoar Wandborg2013-03-101-0/+1
|
* 655: Fix collection fetching for media_collect()Elrond2013-03-101-3/+3
| | | | | | | | | | | | | | | | | | | | | | The problem is: Collection.query.filter_by(id=X, ...) 1. X = form.collection.data This works nicely for the completely empty form (X = None). It does not work for a selected collection, because X will be the collection, not its id. 2. X = request.form.get('collection') (old code). This one works mostly, except for the completely empty form on postgres, because in this case X = u"__None" and postgres does not like comparing an integer column with a string. Fix: collection = form.collection.data if collection and collection.creator != request.user.id: collection = None
* tyop fix in docs, lazyserer.sh->lazyserver.shChristopher Allan Webber2013-03-091-1/+1
| | | | This commit sponsored by S J Bennett. Thanks!
* Change the _original_ design decisions.Elrond2013-03-091-7/+7
| | | | | - Rename the chapter to "Original *". - Fix links.
* Revive old "Design Decisions".Elrond2013-03-092-0/+330
| | | | | This brings back the exact version that was removed in 65e7ce634cfecc87ed6f390f9ccf91be513d2eea.
* Providing warning to users about instability of OAuth/APIChristopher Allan Webber2013-03-081-0/+4
|
* Making a fix so that video codec name switched from "vp8 video" to "vp8"Christopher Allan Webber2013-03-061-3/+10
| | | | | | | | "vp8 video" is what vp8 is marked as in gstreamer's metadata. However, the browser expects it just as the name "vp8". So fixing that. This commit sponsored by Tyng-Ruey Chuang. Thank you!
* Committing extracted and compiled translationsChristopher Allan Webber2013-03-0457-1956/+1820
|
* Merge branch 'master' into 419_cherrypick_large_uploadsChristopher Allan Webber2013-03-0416-56/+275
|\
| * Removing an unnecessary video writeChristopher Allan Webber2013-03-041-3/+0
| | | | | | | | | | | | | | In the case of if we're skipping transcoding, we don't need to copy this file at all! This commit sponsored by Frank Zambrini III. Thanks!
| * Merge branch 'joar-skip_transcoding'Christopher Allan Webber2013-03-0414-52/+268
| |\
| | * self.media_data->self.media_manager in the docstring. Thanks for catching, ↵Christopher Allan Webber2013-03-031-1/+1
| | | | | | | | | | | | | | | | | | Elrond. This commit sponsored by Sebastian Hugentobler. Thank you!
| | * No reason really to pass in fetch_order anyway...Christopher Allan Webber2013-03-031-7/+5
| | | | | | | | | | | | | | | | | | | | | I think this is legacy code from get_display_media being a utility, or something. Removed! (Thanks for pointing this out, Elrond!) This commit sponsored by Tristan Chambers. Thank you!
| | * Another elrond suggestion: only init orig_metadata if there's anything in ↵Christopher Allan Webber2013-03-031-2/+4
| | | | | | | | | | | | | | | | | | the dict. This commit sponsored by Joshua Rosen. Thank you!
| | * Per Elrond's suggestions moving DEFAULT_WEBM_TYPE to media managerChristopher Allan Webber2013-03-033-4/+4
| | | | | | | | | | | | | | | | | | | | | The reason for this is to avoid defining this twice as we were previously (once in the template, once in video/models.py) This commit sponsored by Roland McIntosh. Thank you!
| | * Extrapolate type= based on the video metadata that we have, if we can.Christopher Allan Webber2013-03-033-2/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | It's kind of awkward because it relies on there being a entry.media_data, but that's not guaranteed... (see http://issues.mediagoblin.org/ticket/650) so we use a dopey fallback in the template in that case (kind of annoying info duplication). This commit sponsored by Piotr Wieczorek. Thank you!
| | * Mild cosmetic cleanups to video.html and adding missing </li> elements.Christopher Allan Webber2013-03-031-8/+16
| | | | | | | | | | | | This commit sponsored by Jukka Hellen. Thanks!
| | * {% set %} the display_type and display_path and use that elsewhereChristopher Allan Webber2013-03-031-2/+3
| | | | | | | | | | | | | | | | | | This will make some stuff a bit cleaner that's coming up... This commit sponsored by J B Nicholson-Owens. Thanks!
| | * pdb.set_trace() in mediagoblin code is Not Allowed(TM), removing from audio codeChristopher Allan Webber2013-03-031-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | Of course, the version that appears here is not really dangerous because it's for the "call the file individually" form of debugging, but it isn't allowed anyway. This commit sponsored by Michael Faryniarz. Thanks!
| | * Now store metadata info from processing into the media type.Christopher Allan Webber2013-03-033-0/+62
| | | | | | | | | | | | | | | | | | | | | | | | | | | This comes in several parts: - Store the metadata from gstreamer during processing - Add a new JSONEncoded field to the VideoData table - And, of course, add a migration for that field! This commit sponsored by Julius Tuomisto. Thank you, Julius!
| | * Huge amount of work to (mostly) allow .ogg (and maybe other) formats to skip ↵Christopher Allan Webber2013-03-027-17/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | transcode - Update get_display_media in several ways: - now uses the media type's own declaration of the order of things - returns both the media_size and the media_path, as per the docstring - implicitly uses self.media_files as opposed to forcing you to pass it in - update videos to use get_display_media - update images to declare media_fetch_order in the media manager (videos also) - update stl to use media.media_files['original'] instead of weird use of get_display_media - update sidebar to only conditionally show webm_640 TODO still: identify video type information *during* processing, show that in the <video><source /></video> element. This commit sponsored by Nathan Yergler. Thanks, nyergler!
| | * If we're not transcoding, copy this file directly over to ['original']Christopher Allan Webber2013-03-011-6/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There's no reason to copy it over to 'webm_640' in such a case, clearly. Added logic so we don't do it twice either. Haven't tested this yet though ;) This commit sponsored by Algot Runeman. Thank you!
| | * Merge branch 'master' into joar-skip_transcodingChristopher Allan Webber2013-03-01140-6676/+10025
| | |\ | | | | | | | | | | | | | | | | Conflicts: mediagoblin/config_spec.ini
| | * | Added option to skip transcodingJoar Wandborg2013-01-224-20/+109
| | | | | | | | | | | | | | | | | | | | - If the video input matches the configurable rules, just copy it to the output without transcoding it.
| * | | Elrond points out that we should use form.collection.dataChristopher Allan Webber2013-03-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | That's true; I'm not sure what it's fixing, but he thinks it's fixing something. Anyway, it's correct :) This commit sponsored by Philippe Gauthier. Thanks!
| * | | Give a more useful error if a table already exists and so we can't create it ↵Christopher Allan Webber2013-03-041-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | during migrations This commit sponsored by Andrzej Prochyra. Thanks!
| * | | Three fixes to collection adding view, one of them a serious security bugChristopher Allan Webber2013-03-041-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Don't let people who aren't the authors of a collection from adding things to it (handled by forcing the user check in the query) - request url in case invalid collection selected fixed - collection_item.author doesn't yet exist; removing the selection (we might want multiple people to be able to edit a collection in the future but that future does not yet exist; as Elrond said, remove this "false hope") Thanks to Elrond to pointing out these issues. And thanks to David Kindler for sponsoring this commit!
* | | | Make copying to/from storage systems memory efficient (#419)Sebastian Spaeth2013-03-032-7/+8
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The copy_locally and copy_local_to_storage (very inconsistent terms BTW) were simply slurping in everything in RAM and writing it out at once. (the copy_locally was actually memory efficient if the remote system was local) Use shutil.copyfileobj which does chunked reads/writes on file objects. The default buffer size is 16kb, and as each chunk means a separate HTTP request for e.g. cloudfiles, we use a chunksize of 4MB here (which has just been arbitrarily set by me without tests). This should help with the failure to upload large files issue #419.
* | | Removed debugging exceptionJoar Wandborg2013-03-031-1/+0
| | | | | | | | | | | | | | | .. that accidentally got included due to not testing the video media type in the unit tests.
* | | Updated raven pluginJoar Wandborg2013-03-038-56/+71
| | | | | | | | | | | | | | | | | | | | | - Added wrap_wsgi, celery_setup, celery_logging_setup hooks - Updated raven plugin docs - Updated production considerations docs - Added raven logging setup
* | | plugin/raven: Fix paster's celery config issueJoar Wandborg2013-03-031-2/+6
| | | | | | | | | | | | | | | | | | Check for CELERY_CONFIG_MODULE before we import raven.contrib.celery. It seems that the import otherwise sets up the celery client before we get to pass it our mediagoblin-specific settings.
* | | Added raven plugin docs to docsJoar Wandborg2013-03-032-0/+2
| | |
* | | Removed stray )Joar Wandborg2013-03-031-1/+1
| | |
* | | Removed PLUGIN_DIR from ravenJoar Wandborg2013-03-031-2/+0
| | |
* | | Don't look for the oauth config in raven pluginJoar Wandborg2013-03-031-1/+1
| | |
* | | Refactored raven pluginJoar Wandborg2013-03-031-3/+4
| | |
* | | Added raven pluginJoar Wandborg2013-03-032-0/+68
| | |
* | | Added .send method to cloudfiles storage objectJoar Wandborg2013-03-031-1/+9
| | | | | | | | | | | | | | | .. wrapper. Also added some logging - A .warn() for the legacy .write() method and a .debug() for the new copy_local_to_storage()
* | | Make Cloudfiles copy memory efficient too (#419)Sebastian Spaeth2013-03-021-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems that (our implementation of) cloudfiles.write() takes all existing data and appends write(data) to it, sending the full monty over the wire everytime. This would of course absolutely kill chunked writes with some O(1^n) performance and bandwidth usage. So, override this method and use the Cloudfile's "send" interface instead. Also make the Cloudfile file wrapper an iterator that allows us to simply do "for data in cloudfile:" which will stream the data in a memory-efficient way. DO NOTE THAT THIS PATCH IS COMPLETELY UNTESTED DUE TO LACK OF SETUP PLEASE REVIEW AND VERIFY.
* | | Fixed hidden fields in oauth client authorizationJoar Wandborg2013-03-022-5/+4
| | | | | | | | | | | | | | | | | | Removed the translation marking and passed in empty strings to avoid WTForms automagically creating the labels from the field names (i.e. client_id => 'Client Id').
* | | Do not encode the next kwarg twiceJoar Wandborg2013-03-021-1/+1
| | |
* | | Turn "License preference" into a normal field.Elrond2013-03-021-6/+3
| |/ |/| | | | | | | | | | | | | | | | | | | | | instead of rendering it by hand, use the normal field rendering tools. Old: [choose box] License preference New: License preference [choose box] This will be your default license on upload forms.
* | Fix some media page redirects in media_collect.Elrond2013-03-011-3/+3
| | | | | | | | Use .slug_or_id instead of only .id.
* | Fixup _("...") % (...) in media_collect.Elrond2013-03-011-6/+6
| | | | | | | | | | | | | | Get some messages translated. Bad: _("..." % (...)) Good: _("...") % (...)
* | MediaGoblin not really anything to do with FooCorp anymore.Christopher Allan Webber2013-02-271-15/+0
| |
* | Fixing translations stuff for command line tools and such.Christopher Allan Webber2013-02-271-2/+7
| | | | | | | | | | | | | | | | | | | | | | We had switched mg_globals to have the default translations set to None so that it would be set up by the mediagoblin app later. However, this would mean that things like scripts would try to call gettext and error out. Thanks to Tumulte for catching this. This commit sponsored by Aurimas Fišeras. Thank you!
* | Simplifying the test for whether or not a user and slug combo exists.Christopher Allan Webber2013-02-271-3/+1
| | | | | | | | | | | | | | Thanks to tchernobog for catching this (it was breaking on postgres) and Elrond for the suggestion on how to fix it. This commit sponsored by Caleb Cooper. Thanks Caleb!
* | Removing html5shiv for not complying with its own licenses and racismChristopher Allan Webber2013-02-264-28/+0
| | | | | | | | | | | | | | | | | | | | Issues of racism seem to have been resolved and removed from upstream, but make having this as a dependency somewhat uncomfortable: https://github.com/aFarkas/html5shiv/issues/91 Regardless, at the time of writing the project doesn't comply with its own license... it states to be dual licensed under MIT and GPLv2 but distributes neither of these licenses with its source.
* | Allow media managers without sniff_handlerJoar Wandborg2013-02-271-1/+2
| |
* | Copying the license from the jquery repository rather than the generic MIT one.Christopher Allan Webber2013-02-261-1/+2
| | | | | | | | | | Damned MIT and BSD licenses encouraging modification by every project makes compliance annoying.
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-02 17:27:58 +0000