Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Do not load AV inside AP | Łukasz Strzałkowski | 2013-08-25 | 2 | -0/+2 |
| | | | | Move that part to AV railtie | ||||
* | Add missing require | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+2 |
| | |||||
* | Revert "Move setting content_type to AV" | Łukasz Strzałkowski | 2013-08-25 | 2 | -3/+1 |
| | | | | This reverts commit f4d602aff6cec80304b131ecfcc2676d0304f0cc. | ||||
* | Move setting content_type to AV | Łukasz Strzałkowski | 2013-08-25 | 2 | -1/+3 |
| | |||||
* | Use concat to avoid allocating additional array | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 |
| | |||||
* | Pass args to render's super method | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+2 |
| | |||||
* | Improve AV::Rendering docs | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+8 |
| | |||||
* | Include AV::Layouts before setting view_paths | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+2 |
| | |||||
* | Load AV::Layout to AM::Base in railties | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+6 |
| | |||||
* | Don not require AC::Caching | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 |
| | |||||
* | Remove ApplicationController & RoutingTestHelpers | Łukasz Strzałkowski | 2013-08-25 | 1 | -9/+0 |
| | |||||
* | Remove unused ActionDispatch::RoutingVerbs | Łukasz Strzałkowski | 2013-08-25 | 1 | -15/+1 |
| | |||||
* | Remove unused in AV fixture-controllers | Łukasz Strzałkowski | 2013-08-25 | 1 | -29/+0 |
| | |||||
* | Remove view_assigns from AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -7/+0 |
| | | | | It wasn't adding anything, just invoking super | ||||
* | Load AV::Layouts dynamicly via railties | Łukasz Strzałkowski | 2013-08-25 | 2 | -1/+3 |
| | |||||
* | Code formatting & typo fixes | Łukasz Strzałkowski | 2013-08-25 | 1 | -35/+35 |
| | |||||
* | Remove abstract_controller load hooks | Łukasz Strzałkowski | 2013-08-25 | 1 | -4/+0 |
| | |||||
* | Move protected_instance_variables & view_assigns to AbstractController | Łukasz Strzałkowski | 2013-08-25 | 1 | -15/+4 |
| | |||||
* | Remove 'api plugin' from docs | Łukasz Strzałkowski | 2013-08-25 | 1 | -3/+0 |
| | | | | AbstractController::Rendering is module which defines thie api | ||||
* | Create AbstractController::Rendering interface | Łukasz Strzałkowski | 2013-08-25 | 1 | -17/+4 |
| | | | | This interface should be use when implementing renderers. | ||||
* | Fix eager_autoload for layouts | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 |
| | | | | Name was singular, should be plural | ||||
* | Fix AV tests, I18nProxy was moved to AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 |
| | |||||
* | Revert "Require only path_set && lookup_context instead of whole base" | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+1 |
| | | | | This reverts commit 77bb03df2ca3687b290e9fe77d21007642f0329c. | ||||
* | Hook up AV::Rendering on AV intialization | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+5 |
| | |||||
* | Move layouts to AV | Łukasz Strzałkowski | 2013-08-25 | 3 | -0/+431 |
| | |||||
* | Move rendering from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 2 | -0/+187 |
| | |||||
* | Require only path_set && lookup_context instead of whole base | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+2 |
| | |||||
* | Move view_paths from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 5 | -0/+273 |
| | |||||
* | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2013-08-17 | 1 | -1/+1 |
|\ | | | | | | | | | | | | | | | Conflicts: actionview/README.rdoc activerecord/lib/active_record/migration.rb guides/source/development_dependencies_install.md guides/source/getting_started.md | ||||
| * | Revert "Merge branch 'master' of github.com:rails/docrails" | Vijay Dev | 2013-08-17 | 2 | -1/+3 |
| | | | | | | | | | | | | | | This reverts commit 70d6e16fbad75b89dd1798ed697e7732b8606fa3, reversing changes made to ea4db3bc078fb3093ecdddffdf4f2f4ff3e1e8f9. Seems to be a code merge done by mistake. | ||||
* | | RDoc tweaks | Xavier Noria | 2013-08-07 | 1 | -5/+10 |
| | | |||||
* | | Remove privatizing of Fixnum#/ from assert_distance_of_time_in_words | Alex Tambellini | 2013-08-05 | 1 | -5/+7 |
| | | | | | | | | | | | | | | | | MRI reimplemented Date in C so it doesn't hit this division anymore while JRuby still uses the old stdlib implementation of Date so it will always hit this. With this change the actionview date_helper_test.rb tests should pass on JRuby. | ||||
* | | Merge pull request #11743 from gaurish/tmp-magic | Guillermo Iguaran | 2013-08-05 | 1 | -0/+0 |
|\ \ | | | | | | | [Jruby] Make all tests in ActionView::TemplateDigestorTest green | ||||
| * | | Add tmp dir in actionview/test/ | Gaurish Sharma | 2013-08-05 | 1 | -0/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | Missing tmp directory will cause the test to fail on JRuby. will commit adds tmp directory & 22 TemplateDigestorTests which were failing before are now green. Further Discussion on this can be seen at #11743 | ||||
* | | | Removed an unnecessary loop - it kills performance on large texts. | Jan Szumiec | 2013-08-03 | 1 | -4/+6 |
| | | | |||||
* | | | Add "extname" option to javascript_include_tag | Nathan Stitt | 2013-08-03 | 3 | -3/+27 |
|/ / | | | | | | | | | | | | | | | | | | | | | | | ActionView::Helpers.asset_path is where the logic for javascript_include_tag resides. It takes an extname option for specifying the extension or false to not append it. This exposes that option to javascript_include_tag. Without the option files that didn't end with ".js" would get the extension appended to them. This broke JST templates and other file types that should be interpreted as JavaScript but who's file extension isn't ".js" | ||||
* | | Using URI.parser.unescape | Arun Agrawal | 2013-08-03 | 1 | -2/+2 |
| | | | | | | | | | | | | Fixes warning warning: URI.unescape is obsolete | ||||
* | | fix the changelog typo[ci skip] | Rajarshi Das | 2013-08-02 | 1 | -1/+1 |
| | | |||||
* | | Make current_page? compare binary strings | Rafael Mendonça França | 2013-08-01 | 2 | -3/+12 |
| | | |||||
* | | Fix `current_page?` when the URL contains escaped characters | Rafael Mendonça França | 2013-08-01 | 3 | -2/+20 |
| | | | | | | | | | | | | | | In some cases webservers like nginx send the escaped characters lowercased to the Rails application. The current_page? helper was comparing the escaped strings that are different since Ruby escapes the URL using uppercased characters. | ||||
* | | Moved README into action_view [ci skip] | Arun Agrawal | 2013-07-31 | 1 | -0/+5 |
| | | |||||
* | | Change from `map` => `map!` and `collect!` to save creation of extra array. | Vipul A M | 2013-07-31 | 1 | -2/+2 |
| | | |||||
* | | 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. | ||||
* | | grab executable from rubygems | Arun Agrawal | 2013-07-26 | 1 | -2/+1 |
|/ | | | As done here d7fc97d3f90c0e30865d32ce202658f03248cacc | ||||
* | Merge pull request #11341 from arunagw/added-missing-actionview-to-install | Rafael Mendonça França | 2013-07-25 | 2 | -3/+1 |
|\ | | | | | Added description in actionview gemspec. | ||||
| * | Added some description to generate actionview gem [ci skip] | Arun Agrawal | 2013-07-17 | 2 | -3/+1 |
| | | |||||
* | | text_area should handle nil value option like text_field | Joel Cogen | 2013-07-24 | 4 | -3/+24 |
| | | |||||
* | | Leave the knowledge of boolean tag values to content tag | Carlos Antonio da Silva | 2013-07-22 | 1 | -3/+3 |
| | | | | | | | | | | | | content tag already knows which tags are boolean and the values that should generate them when a truthy value is passed, so leave this logic for it instead of duplicating when generating options tags. | ||||
* | | Use merge! to avoid a new hash and change some spots to 1.9 hash style | Carlos Antonio da Silva | 2013-07-22 | 1 | -7/+7 |
| | | |||||
* | | Fix changelog syntax and use Ruby 1.9 hash style [ci skip] | Carlos Antonio da Silva | 2013-07-22 | 1 | -2/+1 |
| | |