Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Implement ActionView::Template::Types | Piotr Sarnacki | 2012-08-28 | 1 | -1/+1 |
| | | | | | | AV::Template::Types is a small abstraction to allow to specify template types that can be used in ActionView. When Action Pack is loaded it's replaced with Mime::Type. | ||||
* | Remove Mime::Type translations from Action View | Piotr Sarnacki | 2012-08-28 | 1 | -1/+1 |
| | | | | | Action View should not be responsible for translating mime types. Any translation that's needed should be handled at controller level. | ||||
* | use mutex_m rather than use a delegate system | Aaron Patterson | 2012-06-21 | 1 | -6/+3 |
| | |||||
* | Merge pull request #6428 from pinetops/resolver_concurrency_fix | Yehuda Katz | 2012-06-21 | 1 | -19/+76 |
|\ | | | | | Make the Resolver template cache threadsafe | ||||
| * | Improve the readability of the Resolver change detection code | Tom Clarke | 2012-05-22 | 1 | -11/+19 |
| | | |||||
| * | More granular locking of the Resolver template cache | Tom Clarke | 2012-05-21 | 1 | -5/+23 |
| | | | | | | | | | | | | | | In order to avoid holding a global lock when doing template resolution, instead add individual locks on a per cache entry basis. The global lock is now only used for manipulation of the main cache data structure. | ||||
| * | Make the Resolver template cache threadsafe - closes #6404 | Tom Clarke | 2012-05-21 | 1 | -19/+50 |
| | | | | | | | | | | | | | | | | | | The Template cache in the Resolver can be accessed by multiple threads similtaneously in multi-threaded environments. The cache is implemented using a Hash, which isn't threadsafe in all VMs (notably JRuby). This commit extracts the cache to a new Cache class and adds mutexes to prevent concurrent access. | ||||
* | | stop subclassing string | Aaron Patterson | 2012-06-21 | 1 | -3/+10 |
|/ | |||||
* | Revert "Merge pull request #6425 from pinetops/resolver_concurrency_fix" | José Valim | 2012-05-21 | 1 | -68/+19 |
| | | | | | This reverts commit 254c04286c5916ae7f91eb6e173b312e7a74e364, reversing changes made to 513a0525c24c2944630acfa465b22cd2f4601adf. | ||||
* | More granular locking of the Resolver template cache | Tom Clarke | 2012-05-21 | 1 | -5/+23 |
| | | | | | | | In order to avoid holding a global lock when doing template resolution, instead add individual locks on a per cache entry basis. The global lock is now only used for manipulation of the main cache data structure. | ||||
* | Make the Resolver template cache threadsafe - closes #6404 | Tom Clarke | 2012-05-21 | 1 | -19/+50 |
| | | | | | | | | | The Template cache in the Resolver can be accessed by multiple threads similtaneously in multi-threaded environments. The cache is implemented using a Hash, which isn't threadsafe in all VMs (notably JRuby). This commit extracts the cache to a new Cache class and adds mutexes to prevent concurrent access. | ||||
* | Created a Raw handler for templates. | Steve Klabnik | 2012-05-17 | 1 | -1/+3 |
| | | | | Fixes #2394 | ||||
* | Move require to where it's needed | Santiago Pastorino | 2012-05-11 | 1 | -0/+1 |
| | |||||
* | Minor clarification in documentation language and spelling | Matthew Daubert | 2012-03-13 | 1 | -4/+4 |
| | |||||
* | Remove some of the ActiveSupport core extensions related to 1.8. | Uģis Ozols | 2011-12-21 | 1 | -1/+0 |
| | |||||
* | Revert accidental change from cattr_accessor to class_attribute. | José Valim | 2011-12-08 | 1 | -1/+1 |
| | |||||
* | Some small optimizations and improvements to benchmark code. | José Valim | 2011-12-08 | 1 | -1/+1 |
| | |||||
* | minor details revised in a gsub | Xavier Noria | 2011-08-18 | 1 | -1/+1 |
| | | | | | | | | | Regexps have a construct to express alternation of characters, which is character classes. In addition to being the most specific idiom to write this pattern, it reads better without the backslashes. Also, it is better not to use a capture if none is needed. As a side-effect of these changes, the gsub is marginally faster, but speed is not the point of this commit. | ||||
* | Properly escape glob characters. | Aaron Patterson | 2011-08-16 | 1 | -3/+12 |
| | |||||
* | Just remove the sort_locals method | Aaron Patterson | 2011-08-09 | 1 | -12/+1 |
| | |||||
* | the freeze trick does nothing on arrays used as hash keys. | Aaron Patterson | 2011-08-09 | 1 | -2/+1 |
| | |||||
* | use functional style to build a list of template objects | Aaron Patterson | 2011-08-09 | 1 | -9/+12 |
| | |||||
* | refactor the optimized build_query a bit | Aaron Patterson | 2011-08-08 | 1 | -8/+3 |
| | |||||
* | reduce file stats by improving our dir glob pattern | Aaron Patterson | 2011-08-08 | 1 | -1/+1 |
| | |||||
* | hash on the template directory in order to improve cache hits | Aaron Patterson | 2011-08-08 | 1 | -2/+3 |
| | |||||
* | use meaningful names with our variables | Aaron Patterson | 2011-08-08 | 1 | -6/+6 |
| | |||||
* | use binread to read the files | Aaron Patterson | 2011-08-08 | 1 | -1/+2 |
| | |||||
* | Use shorter class-level File methods instead of going through File.stat. | thedarkone | 2011-07-25 | 1 | -1/+1 |
| | |||||
* | Optimize the most common resolver case. | José Valim | 2011-05-09 | 1 | -13/+26 |
| | |||||
* | No need for replace. | José Valim | 2011-05-09 | 1 | -12/+11 |
| | |||||
* | removes a remaining reference to .rjs in template resolver's RDoc | Xavier Noria | 2011-04-13 | 1 | -1/+1 |
| | |||||
* | fixes a couple of regexps, the suite showed warnings about them | Xavier Noria | 2011-03-27 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | A couple of things worth mentioning here: - "{" is a metacharacter, should be escaped if it is meant to match a "{". The code worked, though, because the regexp engine is tolerant to this, but issued warnings. - gsub accepts a string as first argument. That's the best idiom to use when your pattern has no metacharacters, since gsub interprets the string as an exact substring to look for, rather than a regexp. The benefit is that your pattern is crystal clear and needs no backslashes. | ||||
* | Improved resolver docs a bit | José Valim | 2011-03-19 | 1 | -2/+10 |
| | |||||
* | [action_view] docs for FileSystemResolver | Chris Kowalik | 2011-03-20 | 1 | -2/+30 |
| | |||||
* | [action_view] added custom patterns to template resolver | Chris Kowalik | 2011-03-20 | 1 | -18/+49 |
| | |||||
* | Add additional text to NotImplementedErrors [#6328 state:resolved] | Mike Gehard | 2011-03-12 | 1 | -1/+1 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | reduce string append funcalls | Aaron Patterson | 2011-02-08 | 1 | -3/+3 |
| | |||||
* | Ensure render is case sensitive even on systems with case-insensitive ↵ | José Valim | 2011-02-08 | 1 | -3/+12 |
| | | | | | | filesystems. This fixes CVE-2011-0449 | ||||
* | A bunch of cleanup on the inherited template patch | wycats | 2010-12-26 | 1 | -1/+1 |
| | |||||
* | Bring back config.action_view.cache_template_loading [#5847 state:resolved] | Piotr Sarnacki | 2010-12-16 | 1 | -3/+8 |
| | |||||
* | Ensure resolvers backward compatibility. | José Valim | 2010-12-09 | 1 | -1/+1 |
| | |||||
* | Deprecate handler_class_for_extension as template handlers can be any Ruby ↵ | José Valim | 2010-12-01 | 1 | -1/+1 |
| | | | | object, not necessarily a class. | ||||
* | Do not allow templates coming from Fallback resolvers to store a virtual path. | José Valim | 2010-10-10 | 1 | -0/+13 |
| | |||||
* | Resolvers now consider timestamps. | José Valim | 2010-10-10 | 1 | -21/+46 |
| | | | | | | | | | | | Before this patch, every request in development caused the template to be compiled, regardless if it was updated in the filesystem or not. This patch now checks the timestamp and only compiles it again if any change was done. While this probably won't show any difference for current setups, but it will be useful for asset template handlers (like SASS), as compiling their templates is slower than ERb, Haml, etc. | ||||
* | Remove locals dependency from template. | José Valim | 2010-10-07 | 1 | -7/+23 |
| | | | | | | This means that templates does not need to store its source anymore, allowing us to reduce the ammount of memory taken by our Rails processes. Naively speaking, if your app/views contains 2MB of files, each of your processes (after being hit by a bunch of requests) will take 2MB less of memory after this commit. This is extremely important for the upcoming features. Since Rails will also render CSS and JS files, their source won't be stored as well allowing us to decrease the ammount of memory taken. | ||||
* | Get rid of ruby warnings in Resolvers. Move a few methods up to the abstract ↵ | José Valim | 2010-10-07 | 1 | -21/+24 |
| | | | | class. | ||||
* | Initialize @path. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | |||||
* | Revert "Setup explicit requires for files with exceptions. Removed them from ↵ | José Valim | 2010-09-02 | 1 | -1/+0 |
| | | | | | | | | autoloading." Booting a new Rails application does not work after this commit [#5359 state:open] This reverts commit 38a421b34d0b414564e919f67d339fac067a56e6. | ||||
* | Setup explicit requires for files with exceptions. Removed them from ↵ | Łukasz Strzałkowski | 2010-09-02 | 1 | -0/+1 |
| | | | | | | autoloading. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Avoid using Pathname on Resolver and AS::Dependencies. | José Valim | 2010-06-24 | 1 | -1/+1 |
| |