Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | remove dead code | Aaron Patterson | 2016-02-16 | 1 | -141/+34 |
| | | | | | | this commit removes unused code and changes the monitor to a mutex. Since the digest doesn't recurse on itself anymore, we can just use a mutex | ||||
* | change internal implementation to use a tree | Aaron Patterson | 2016-02-16 | 1 | -2/+22 |
| | |||||
* | convert structs to a regular class | Aaron Patterson | 2016-02-16 | 1 | -8/+12 |
| | |||||
* | clean up classes a little | Aaron Patterson | 2016-02-16 | 1 | -12/+23 |
| | |||||
* | fix recursive templates | Aaron Patterson | 2016-02-16 | 1 | -20/+40 |
| | |||||
* | pull template check up to match existing behavior | Aaron Patterson | 2016-02-16 | 1 | -13/+10 |
| | | | | also remove the EMPTY node since we won't need it | ||||
* | introduce a `tree` factory method for creating the dep tree | Aaron Patterson | 2016-02-16 | 1 | -0/+55 |
| | | | | also add an EMPTY sentinel node | ||||
* | push kwargs up to the user facing API | Aaron Patterson | 2016-02-12 | 1 | -9/+9 |
| | | | | | | | this lets us leverage Ruby's kwarg handling (exceptions for missing params, etc) ASAP which allows us to skip active support method calls and make sure the exception stack is closer to where the user called the methods. | ||||
* | use kwargs to avoid hash slicing | Aaron Patterson | 2016-02-12 | 1 | -3/+3 |
| | | | | | we can use kwargs in this case to avoid values_at and except calls on the options hash | ||||
* | Don't capture the MissingTemplate exception. | Kasper Timm Hansen | 2016-02-11 | 1 | -1/+1 |
| | | | | Wasn't removed in 57ac777. | ||||
* | Oops. :cut: | Aaron Patterson | 2016-02-11 | 1 | -6/+0 |
| | |||||
* | follow indentation rules | Aaron Patterson | 2016-02-11 | 1 | -5/+11 |
| | |||||
* | make the logger method always return something | Aaron Patterson | 2016-02-11 | 1 | -4/+9 |
| | | | | then we can stop scattering nil checks (e.g. `try`) through the class. | ||||
* | Require only necessary concurrent-ruby classes. | Jerry D'Antonio | 2015-11-04 | 1 | -1/+1 |
| | |||||
* | Replaced `ThreadSafe::Map` with successor `Concurrent::Map`. | Jerry D'Antonio | 2015-09-19 | 1 | -3/+3 |
| | | | | | | | The thread_safe gem is being deprecated and all its code has been merged into the concurrent-ruby gem. The new class, Concurrent::Map, is exactly the same as its predecessor except for fixes to two bugs discovered during the merge. | ||||
* | Add wildcard template dependencies. | Kasper Timm Hansen | 2015-07-26 | 1 | -1/+1 |
| | |||||
* | Make digest cache work in development. | Kasper Timm Hansen | 2015-07-18 | 1 | -4/+8 |
| | | | | | | Avoid computing the same fragment digest many times when looping over templates. The cache is cleared on every request so template changes are still picked up. | ||||
* | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-06-21 | 1 | -1/+1 |
|\ | | | | | | | | | Conflicts: guides/source/configuring.md | ||||
| * | [ci skip] Fix to Fixed-width Font | yui-knk | 2015-06-14 | 1 | -1/+1 |
| | | | | | | | | LookupContext is class name | ||||
* | | add error log that notify 'file not found' when using ↵ | Faruk AYDIN | 2015-05-07 | 1 | -1/+2 |
|/ | | | | cache_digests:dependency and cache_digests:nested_dependency tasks | ||||
* | Log digest as :debug instead of :info | Alan Skorkin | 2014-07-15 | 1 | -1/+1 |
| | |||||
* | Digestor should just rely on the finder to know about the format and the ↵ | David Heinemeier Hansson | 2014-03-21 | 1 | -60/+29 |
| | | | | variant -- trying to pass it back in makes a mess of things (oh, and doesnt work) | ||||
* | Log the full path, including variant, that the digestor is trying to find | David Heinemeier Hansson | 2014-03-21 | 1 | -3/+4 |
| | |||||
* | Fix for digestor to consider variants for partials -- this still needs more ↵ | David Heinemeier Hansson | 2014-03-21 | 1 | -2/+2 |
| | | | | testing!! | ||||
* | Warning removed (...) interpreted as grouped expression | Arun Agrawal | 2014-03-20 | 1 | -1/+1 |
| | |||||
* | Clarify AV::Digestor.digest method signature docs and deprecation warning | Jeremy Kemper | 2014-03-15 | 1 | -22/+21 |
| | |||||
* | Fix the resolver cache and stop mutating the lookup_context | Rafael Mendonça França | 2014-03-14 | 1 | -2/+2 |
| | | | | | Before we had a bug in the resolver cache so the disable_cache were not working when passing options to find | ||||
* | Introduce #with_formats_and_variants to prevent problems with mutating ↵ | Łukasz Strzałkowski | 2014-03-14 | 1 | -5/+2 |
| | | | | finder object | ||||
* | Rename _setup_options to _options_for_digest | Łukasz Strzałkowski | 2014-03-14 | 1 | -3/+3 |
| | |||||
* | Disable LookupContext's cache when looking for template | Łukasz Strzałkowski | 2014-03-14 | 1 | -2/+6 |
| | |||||
* | Don't pass hash as keys to #find method | Łukasz Strzałkowski | 2014-03-14 | 1 | -1/+1 |
| | |||||
* | Don't create addition vars, use options[] directly | Łukasz Strzałkowski | 2014-03-13 | 1 | -7/+2 |
| | |||||
* | Ensure LookupContext in Digestor selects correct variant | Piotr Chmolowski | 2014-03-09 | 1 | -1/+4 |
| | | | | | | | | | | Related to: #14242 #14243 14293 Variants passed to LookupContext#find() seem to be ignored, so I've used the setter instead: `finder.variants = [ variant ]`. I've also added some more test cases for variants. Hopefully this time passing tests will mean it actually works. | ||||
* | Variants in ActionView::Digestor | Piotr Chmolowski | 2014-03-04 | 1 | -13/+51 |
| | | | | | | | | | | | Take variants into account when calculating template digests in ActionView::Digest. Digestor#digest now takes a hash as an argument to support variants and allow more flexibility in the future. Old-style arguments have been deprecated. Fixes #14242 | ||||
* | add a new local variable to track if digests are being stored, to ensure the ↵ | Brad Murray | 2013-10-17 | 1 | -2/+2 |
| | | | | cleanup works correctly | ||||
* | update digestor code based on review | Brad Murray | 2013-10-15 | 1 | -2/+2 |
| | |||||
* | Ensure ActionView::Digest.cache is correctly cleaned up when | Brad Murray | 2013-10-15 | 1 | -6/+6 |
| | | | | ActionView::Resolver.caching = false. | ||||
* | Bust the template digest cache key when details are changed | Daniel Schierbeck | 2013-09-19 | 1 | -1/+4 |
| | | | | | | Since the lookup details will influence which template is resolved, they need to be included in the cache key -- otherwise two different templates may erroneously share the same digest value. | ||||
* | Only cache template digests if config.cache_template_loading | Josh Lauer | 2013-08-06 | 1 | -3/+7 |
| | | | | | | since ActionView::Resolver.caching is set to the same value as config.cache_template_loading only cache template digests if config.cache_template_loading is not falsy fixes issues #10752 and #10791 | ||||
* | AV::Digestor thread safety fixes. | thedarkone | 2013-07-30 | 1 | -9/+29 |
| | | | | | This fixes potential thread safety issues introduced in 09f6fe1. The problem with the original code was that the "recursion-stopping" `nil` values could be seen by other threads. | ||||
* | Fix "Stack Level Too Deep" error when rendering recursive partials | Rafael Mendonça França | 2013-07-07 | 1 | -3/+7 |
| | | | | | | | When rendering recursive partial Action View is trying to generate the view digest infinitly causing a stack level error. Fixes #11340 | ||||
* | Move actionpack/lib/action_view* into actionview/lib | Piotr Sarnacki | 2013-06-20 | 1 | -0/+85 |