Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | partials inside directory work with `assert_template` | Yves Senn | 2013-02-04 | 2 | -0/+2 | |
| | | | | | | | | | | previously when a partial was placed inside a directory (eg. '/dir/_partial'), `assert_template` did not replace the '_' prefix when looking through rendered tempaltes, which resulted in an error. I modified it to replace both, the leading '_' and the last '_' after a '/'. | |||||
* | Added AR integration tests for form helpers | virusman | 2013-01-21 | 1 | -0/+1 | |
| | ||||||
* | Do not generate local vars for partials without object or collection | Carlos Antonio da Silva | 2013-01-08 | 1 | -0/+1 | |
| | | | | | | | | | Previously rendering a partial without giving :object or :collection would generate a local variable with the partial name by default. This was noticed due to warnings in Ruby 2.0 of not used variables, which turned out to be the generation of not used variables inside partials that do not contain objects related to them. | |||||
* | Digestor explicit dependency should not contain trailing whitespace | Brian Alexander | 2012-12-21 | 1 | -1/+5 | |
| | | | | test for rails/rails#8586 | |||||
* | Add explicit opt-out for fragment cache digesting | Drew Ulmer | 2012-11-25 | 1 | -0/+3 | |
| | | | | | | | | | This add support for sending an explicit opt-out of the "Russian-doll" cache digest feature on a case-by-case basis. This is useful when cache- expiration needs to be performed manually and it would be otherwise difficult to know the exact name of a digested cache key. More information: https://github.com/rails/cache_digests/pull/16 | |||||
* | Allow for deep directory path for view templates. | Andy Shipman | 2012-10-11 | 2 | -0/+1 | |
| | ||||||
* | recognizes when a partial was rendered twice. Closes #3675 | Yves Senn | 2012-10-11 | 1 | -0/+2 | |
| | ||||||
* | `assert_template` no more passing with what ever string that matches. | Hugo Roque | 2012-09-29 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | Given Im rendering an template `/layout/hello.html.erb`, assert_template was passing with any string that matches. This behavior allowed false passing like: assert_template "layout" assert_template "out/hello" Now the passing possibilities are: assert_template "layout/hello" assert_template "hello" fixing assert_template bug when template matches expected, but not ends with Cherry Pick Merge: Fixes issue #3849 assert_template false positive taking redundant test off prevening incorrect assert_template when rendering with repeated names in path updating CHANGELOG with bugfix: assert_template false passing | |||||
* | Merge pull request #7251 from rails/integrate-strong_parameters | David Heinemeier Hansson | 2012-09-18 | 1 | -1/+0 | |
|\ | | | | | Integrate strong_parameters in Rails 4 | |||||
| * | Remove integration between attr_accessible/protected and ↵ | Guillermo Iguaran | 2012-09-16 | 1 | -1/+0 | |
| | | | | | | | | AC::Metal::ParamsWrapper | |||||
* | | Rename .rb template handler to .ruby to avoid conflicts with mustache views ↵ | Guillermo Iguaran | 2012-09-17 | 1 | -0/+0 | |
|/ | | | | classes | |||||
* | warning removed. | Arun Agrawal | 2012-09-12 | 1 | -1/+0 | |
| | | | | | 1. Unused variable 2. possibly useless use of a variable in void context | |||||
* | Add .rb template handler | Guillermo Iguaran | 2012-09-11 | 1 | -0/+3 | |
| | | | | This handler simply allows arbitrary Ruby code as a template | |||||
* | `Digestor` can now parse old style hash syntax for `render` | Christos Zisopoulos | 2012-08-30 | 1 | -0/+1 | |
| | ||||||
* | `Digestor` ignores most whitespace when parsing `render` invocations | Christos Zisopoulos | 2012-08-30 | 2 | -0/+4 | |
| | ||||||
* | Add automatic template digests to all CacheHelper#cache calls (originally ↵ | David Heinemeier Hansson | 2012-08-29 | 8 | -0/+14 | |
| | | | | spiked in the cache_digests plugin) *DHH* | |||||
* | Indentation should consider line number character count. | Lucas Uyezu | 2012-08-13 | 1 | -0/+13 | |
| | ||||||
* | Remove deprecation warning, since scoped waas deprecated | Rafael Mendonça França | 2012-07-29 | 1 | -1/+1 | |
| | ||||||
* | Remove deprecation warnings | Carlos Galdino + Rafael Mendonça França | 2012-07-21 | 2 | -2/+2 | |
| | ||||||
* | Common behavior with adding formats to lookup_context for TemplateRenderer ↵ | Dmitry Vorotilin | 2012-07-18 | 7 | -0/+7 | |
| | | | | and PartialRenderer | |||||
* | Remove more tests related to draw external routes files | Rafael Mendonça França | 2012-06-29 | 2 | -2/+0 | |
| | | | | Related with 5e7d6bba79393de0279917f93b82f3b7b176f4b5 | |||||
* | Respect absolute paths in compute_source_path. | Steve Klabnik | 2012-06-16 | 1 | -0/+3 | |
| | | | | | | | | | | | | | | | | When using compute_source_path to determine the full path of an asset, if our source begins with '/', we don't want to include the directory. Examples are illustrative: > compute_source_path("foo", "stylesheets", "css") => "/Users/steve/src/my_app/public/stylesheets/foo.css" > compute_source_path("/foo", "stylesheets", "css") => "/Users/steve/src/my_app/public/foo.css" Before this patch, the second example would return the same as the first. Fixes #5680. | |||||
* | Fix sorting of helpers from different paths | Piotr Sarnacki | 2012-05-28 | 2 | -0/+10 | |
| | | | | | | | | | | | | | | | | When more than one directory for helpers is provided to a controller, it should preserver the order of directories. Given 2 paths: MyController.helpers_paths = ["dir1/helpers", "dir2/helpers"] helpers from dir1 should be loaded first. Before this commit, all helpers were mixed and then sorted alphabetically, which essentially would require to rename helpers to get desired order. This is a problem especially for engines, where you would like to be able to predict accurately which engine helpers will load first. (closes #6496) | |||||
* | Merge pull request #6309 from steveklabnik/fix-2394 | José Valim | 2012-05-17 | 2 | -0/+2 | |
|\ | | | | | Created a Raw handler for templates. | |||||
| * | Created a Raw handler for templates. | Steve Klabnik | 2012-05-17 | 2 | -0/+2 | |
| | | | | | | | | Fixes #2394 | |||||
* | | add tests and external file backtrace for Routing::Mapper#draw | Kornelius Kalnbach | 2012-05-15 | 2 | -0/+2 | |
|/ | ||||||
* | Fix assert_template assertion with :layout option | Alexey Vakhov | 2012-05-04 | 1 | -0/+2 | |
| | ||||||
* | wrap translate defaults to use translate helper features, closes #1102 | Sergey Nartimov | 2012-04-30 | 1 | -0/+1 | |
| | ||||||
* | Allow access to current object_counter variable from layout when rendering ↵ | Carlos Antonio da Silva | 2012-04-29 | 1 | -0/+1 | |
| | | | | with partial + collection | |||||
* | Allow layout rendering to access current object being rendered when using ↵ | Carlos Antonio da Silva | 2012-04-29 | 1 | -0/+1 | |
| | | | | partial + collection | |||||
* | Make controller namespace partial prefix optional | Grant Hutchins | 2012-03-28 | 3 | -2/+3 | |
| | | | | | | | config.action_view.prefix_partial_path_with_controller_namespace This allows you to choose to render @post using /posts/_post.erb instead of /admin/posts/_post.erb inside Admin::PostsController. | |||||
* | If partial is rendered in controller, grab format from template | Piotr Sarnacki | 2012-03-27 | 1 | -0/+1 | |
| | | | | | | | | | Previously `rendered_format` was set only based on mime types passed in Accept header, which was wrong if first type from Accept was different than rendered partial. The fix is to simply move setting rendered_format to the place where template is available and grab format from the template. If it fails we can fallback to formats passed by Accept header. | |||||
* | Use the new scope syntax | Rafael Mendonça França | 2012-03-23 | 1 | -1/+1 | |
| | ||||||
* | Merge pull request #5480 from drogus/rendering-issues | José Valim | 2012-03-17 | 6 | -0/+6 | |
| | | | | Fix for #5440 | |||||
* | Remove tests and fixtures for Sprockets on ActionPack | Guillermo Iguaran | 2012-03-13 | 12 | -3/+0 | |
| | ||||||
* | Adds :layout option to render :partial when a collection is given. | Sergio Arbeo | 2012-03-06 | 1 | -0/+1 | |
| | ||||||
* | format lookup for partials is derived from the format in which the template ↵ | Santiago Pastorino | 2012-02-22 | 1 | -0/+1 | |
| | | | | | | is being rendered Closes #5025 part 2 | |||||
* | Use the right format when a partial is missing. | Santiago Pastorino | 2012-02-20 | 1 | -0/+1 | |
| | | | | Closes #5025 | |||||
* | Remove fixture files with Windows incompatible filenames | Andrew White | 2012-02-20 | 12 | -12/+0 | |
| | | | | | | Windows doesn't allow `\ / : * ? " < > |` in filenames so create the fixture files at runtime and ignore the incompatible ones when running on Windows. | |||||
* | Fix ActionDispatch::Static to serve files with unencoded PCHAR | Andrew White | 2012-02-17 | 12 | -1/+12 | |
| | | | | | | | | | | | RFC 3986[1] allows sub-delim characters in path segments unencoded, however Rack::File requires them to be encoded so we use URI's unescape method to leave them alone and then escape them again. Also since the path gets passed to Dir[] we need to escape any glob characters in the path. [1]: http://www.ietf.org/rfc/rfc3986.txt | |||||
* | escape static file path to prevent double unescaping | Sergey Nartimov | 2012-02-16 | 1 | -0/+1 | |
| | ||||||
* | Removing old Controller test | Thiago Pradi | 2012-02-12 | 1 | -1/+0 | |
| | ||||||
* | Add testcase for non english filename. Related to #2982. | kennyj | 2012-02-05 | 1 | -0/+1 | |
| | ||||||
* | Replaced all 'for' loops with Enumerable#each | Nathan Broadbent | 2012-01-30 | 1 | -1/+1 | |
| | ||||||
* | Add font_path helper method | Santiago Pastorino | 2012-01-03 | 2 | -0/+0 | |
| | ||||||
* | test helpers in erb using erb | lest | 2011-11-30 | 1 | -0/+3 | |
| | ||||||
* | Deprecate set_sequence_name in favour of self.sequence_name= | Jon Leighton | 2011-11-29 | 1 | -2/+2 | |
| | ||||||
* | Deprecate set_table_name in favour of self.table_name= or defining your own ↵ | Jon Leighton | 2011-11-29 | 1 | -1/+1 | |
| | | | | method. | |||||
* | fix label with block in erb | lest | 2011-11-27 | 1 | -0/+4 | |
| | ||||||
* | Merge pull request #2219 from kommen/fix_fragment_caching_squashed | Santiago Pastorino | 2011-10-13 | 1 | -0/+1 | |
| | | | | Fix fragment caching (squashed commits) |