aboutsummaryrefslogtreecommitdiffstats
path: root/mediagoblin
Commit message (Collapse)AuthorAgeFilesLines
...
| * Testing the template_context_prerender hookChristopher Allan Webber2013-05-155-5/+17
| | | | | | | | | | | | | | | | | | | | | | 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!
| * Oh right, actually add that hook we just documented, ↵Christopher Allan Webber2013-05-151-0/+7
| | | | | | | | | | | | "template_context_prerender" :) This commit sponsored by William Goudie. Thanks Bill! :)
| * Documenting the test_modify_context and context_modified_app methodsChristopher Allan Webber2013-05-141-0/+7
| |
| * Fully working context hooks, both template/view and global level, with testsChristopher Allan Webber2013-05-147-7/+114
| | | | | | | | | | | | Needs documentation though... that's coming next :) This commit sponsored by Luca Tius. Thanks Luca!
| * Merge branch 'master' into 623_context_hooksChristopher Allan Webber2013-05-1210-76/+68
| |\
| * | Use the controller's symbolic/lookup name as part of the key for context hooksChristopher Allan Webber2013-05-102-3/+3
| | | | | | | | | | | | This commit sponsored by David Collins. Thank you!
| * | Merge branch 'master' into 623_context_hooksChristopher Allan Webber2013-05-109-34/+123
| |\ \
| * | | Ah right, and here's the config we intend to use for these context modified ↵Christopher Allan Webber2013-05-081-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | tests :) This commit sponsored by Xavier Gulliot. Thanks Xavier!
| * | | The beginning of context hooks.Christopher Allan Webber2013-05-083-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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?!
* | | | Normalizing indentation a bit in media.htmlChristopher Allan Webber2013-05-141-9/+9
| | | | | | | | | | | | | | | | This commit sponsored by Christian Frank. Thank you!
* | | | fix for bug when comments are disabled, link attribute is never closedRodney Ewing2013-05-141-2/+2
| | | |
* | | | piwigo: Start first tests.Elrond2013-05-122-1/+70
| | | | | | | | | | | | | | | | | | | | This first test checks the session logic of the piwigo plugin. login, logout, getStatus.
* | | | piwigo: Fix validator usage.Elrond2013-05-121-1/+1
| |_|/ |/| | | | | | | | | | | 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!)Christopher Allan Webber2013-05-121-3/+1
| | | | | | | | | | | | This commit sponsored by Kevin McCarthy. Thank you!
* | | Tests should no longer use the environment-variable-related-setup and set up ↵Christopher Allan Webber2013-05-124-68/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | 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!
* | | Moving the get_original_date method over to the MediaManager.Christopher Allan Webber2013-05-113-18/+24
| | | | | | | | | | | | | | | | | | | | | 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" toolingChristopher Allan Webber2013-05-111-6/+10
| | |
* | | Merge remote-tracking branch 'refs/remotes/tryggvib/532-exif-creation-date'Christopher Allan Webber2013-05-113-0/+29
|\ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: mediagoblin/config_spec.ini mediagoblin/templates/mediagoblin/user_pages/media.html
| * | | Documented the get_original_date methodTryggvi Björgvinsson2012-11-251-0/+5
| | | |
| * | | Moved exif date transformation logic from template to image data model (as a ↵Tryggvi Björgvinsson2012-11-252-7/+20
| | | | | | | | | | | | | | | | function)
| * | | A template fix to show the EXIF DateTimeOriginal (as creation date). Added ↵Tryggvi Björgvinsson2012-11-082-0/+9
| | | | | | | | | | | | | | | | as a configurable option 'original_date_visible' (default false). Since the EXIF tag is a string with the date formatted as 'YYYY:MM:DD HH:MM:SS' the fix slices the string to give only the date and then replaces : by - to conform with the 'Added on' date.
* | | | Merge remote-tracking branch 'gabithume/146_debug_message'Christopher Allan Webber2013-05-112-4/+26
|\ \ \ \ | |_|_|/ |/| | |
| * | | Fixing ticket #146 - Make messaging system testableGabi Thume2013-04-292-4/+26
| | | |
* | | | Merge remote-tracking branch 'refs/remotes/dthompson/453_disable_comments'Christopher Allan Webber2013-05-103-7/+18
|\ \ \ \
| * | | | Do not allow comments to be posted when they are disabled.David Thompson2013-04-131-1/+7
| | | | |
| * | | | Do not render 'Add a comment' button if comments are disabled.David Thompson2013-04-131-6/+8
| | | | |
| * | | | Add allow_comments configuration setting.David Thompson2013-04-131-0/+3
| | | | |
* | | | | piwigo: Better logging for login.Elrond2013-05-091-1/+4
| | | | |
* | | | | piwigo: Let getStatus return the current user.Elrond2013-05-091-1/+5
| | | | | | | | | | | | | | | | | | | | If there is a user logged in, show his name.
* | | | | Create new session system for piwigo plugin.Elrond2013-05-093-7/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.Elrond2013-05-093-3/+41
| | | | | | | | | | | | | | | | | | | | | | | | | To make things a bit easier, switch to WTForms for validating the received data.
* | | | | piwigo: Remove possibly_add_cookie.Elrond2013-05-091-15/+0
| | | | | | | | | | | | | | | | | | | | | | | | | 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 ↵Christopher Allan Webber2013-05-081-1/+1
| | | | | | | | | | | | | | | | | | | | plugins section
* | | | | Fixing API setup with new plugin "config spec" worldChristopher Allan Webber2013-05-081-2/+2
| |_|_|/ |/| | | | | | | | | | | | | | | It shouldn't reference the config until in the setup_plugin() method, else there's a race condition.
* | | | Unit tests for plugins defining their own config_spec.ini!Christopher Allan Webber2013-05-084-0/+76
| | | | | | | | | | | | | | | | This commit sponsored by David Ahmed. Thank you!
* | | | Allow plugins to install their own config_spec.ini if they like.Christopher Allan Webber2013-05-081-0/+42
| | | | | | | | | | | | | | | | This commit sponsored by Timothy Vollmer. Thanks Tim!
* | | | Removed unused imports in httpapiauthJoar Wandborg2013-05-051-2/+1
| | | |
* | | | Use collection.url_for_self more; Add missing /a.Elrond2013-05-054-15/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.Elrond2013-05-021-1/+1
| | | |
* | | | Fix media manager reference in video.Elrond2013-04-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | 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.Praveen Kumar2013-04-305-44/+59
| |/ / |/| |
* | | Create redir_obj and use it around.Elrond2013-04-283-19/+18
| | | | | | | | | | | | | | | | | | | | | | | | 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.Elrond2013-04-271-10/+7
| | | | | | | | | | | | This hopefully makes the code nicer and easier to change.
* | | image resizing: Refactor some decisions into resize_tool.Elrond2013-04-272-30/+39
| | | | | | | | | | | | | | | | | | | | | 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.Elrond2013-04-271-1/+1
| | | | | | | | | | | | | | | | | | 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.Elrond2013-04-271-24/+24
| | | | | | | | | | | | | | | | | | | | | | | | 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.Elrond2013-04-272-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | Switch "sqlite_refcheck" keyword arg to "migrations" which Elrond thinks is ↵Christopher Allan Webber2013-04-262-4/+16
| | | | | | | | | | | | | | | | | | cleaner Also, if migrations is true, *explicitly* say that foreign key checking is off
* | | Don't turn on sqlite refcheck stuff during migrationsChristopher Allan Webber2013-04-262-3/+3
| | |
* | | Merge remote-tracking branch 'refs/remotes/rodney757/issue643' into mergetestChristopher Allan Webber2013-04-262-2/+42
|\ \ \