Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Require action_view explicitly in AC::Base | Łukasz Strzałkowski | 2013-12-08 | 1 | -0/+1 |
| | |||||
* | Merge pull request #13189 from strzalek/retain-ap-av-dep | Jeremy Kemper | 2013-12-05 | 1 | -0/+1 |
|\ | | | | | Retain ActionPack dependency on ActionView. Fixes #12979. | ||||
| * | Retain ActionPack dependency on ActionView | Łukasz Strzałkowski | 2013-12-05 | 1 | -0/+1 |
| | | |||||
* | | Improve a couple exception messages related to variants and mime types | Carlos Antonio da Silva | 2013-12-03 | 1 | -8/+10 |
|/ | | | | | Avoid one-liner conditionals when they are too big. Avoid concatenating strings to build error messages. Improve messages a bit. | ||||
* | Action Pack Variants | Łukasz Strzałkowski | 2013-12-04 | 2 | -1/+11 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, variants in the templates will be picked up if a variant is set and there's a match. The format will be: app/views/projects/show.html.erb app/views/projects/show.html+tablet.erb app/views/projects/show.html+phone.erb If request.variant = :tablet is set, we'll automatically be rendering the html+tablet template. In the controller, we can also tailer to the variants with this syntax: class ProjectsController < ActionController::Base def show respond_to do |format| format.html do |html| @stars = @project.stars html.tablet { @notifications = @project.notifications } html.phone { @chat_heads = @project.chat_heads } end format.js format.atom end end end The variant itself is nil by default, but can be set in before filters, like so: class ApplicationController < ActionController::Base before_action do if request.user_agent =~ /iPad/ request.variant = :tablet end end end This is modeled loosely on custom mime types, but it's specifically not intended to be used together. If you're going to make a custom mime type, you don't need a variant. Variants are for variations on a single mime types. | ||||
* | calculate the ivars to remove in advance as a set and cache them in a | Aaron Patterson | 2013-11-06 | 1 | -8/+2 |
| | | | | | | | constant. `view_assigns` can use the precalculated sets and remove instance variables without allocating any extra arrays | ||||
* | use a set and reject to avoid array allocations | Aaron Patterson | 2013-11-06 | 1 | -4/+11 |
| | |||||
* | each_with_object on the view_assigns hash | Aaron Patterson | 2013-11-06 | 1 | -3/+1 |
| | |||||
* | use slice to avoid range allocation | Aaron Patterson | 2013-11-06 | 1 | -1/+3 |
| | |||||
* | these variables are also private | Aaron Patterson | 2013-11-06 | 1 | -0/+1 |
| | |||||
* | instance_variables returns symbols, so we should use symbols in our list | Aaron Patterson | 2013-11-06 | 1 | -1/+1 |
| | |||||
* | Make AC standalone rendering work | Santiago Pastorino | 2013-09-10 | 1 | -2/+5 |
| | |||||
* | Remove BasicRendering tests | José Valim | 2013-09-09 | 1 | -9/+1 |
| | |||||
* | Remove remaining coupling with AV in MimeResponds | José Valim | 2013-09-09 | 1 | -0/+7 |
| | |||||
* | Remove BasicRendering and remove template functionality from AbsC::Rendering | José Valim | 2013-09-09 | 1 | -31/+12 |
| | |||||
* | Move BasicRendering to AbstractController | Łukasz Strzałkowski | 2013-09-03 | 1 | -0/+28 |
| | |||||
* | Make Mime::TEXT default format in AbstractController | Łukasz Strzałkowski | 2013-09-03 | 1 | -0/+1 |
| | |||||
* | Move skeleton methods from AV to AbsC | Łukasz Strzałkowski | 2013-09-03 | 1 | -7/+19 |
| | | | | | | | | | | The methods: * #render_to_body * #render_to_string * #_normalize_render Haven't had anything specyfic to ActionView. This was common code which should belong to AbstractController | ||||
* | Return to using protected_instance_variables in AV | Łukasz Strzałkowski | 2013-09-02 | 1 | -5/+6 |
| | |||||
* | Revert "Port all remaining self.protected_instance_variables to class methods" | Łukasz Strzałkowski | 2013-09-02 | 2 | -10/+11 |
| | | | | This reverts commit 7de994fa215e9f4c2856d85034bc4dd7b65d0c01. | ||||
* | Port all remaining self.protected_instance_variables to class methods | Łukasz Strzałkowski | 2013-08-29 | 2 | -11/+10 |
| | |||||
* | Add #rendered_format method to controllers | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+5 |
| | |||||
* | Improve AV::Rendering docs | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+3 |
| | |||||
* | Code formatting & typo fixes | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 |
| | |||||
* | Remove abstract_controller load hooks | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+0 |
| | |||||
* | Move protected_instance_variables & view_assigns to AbstractController | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+20 |
| | |||||
* | Create AbstractController::Rendering interface | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+50 |
| | | | | This interface should be use when implementing renderers. | ||||
* | Revert "Rename abstract_controller/rendering. to errors.rb" | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+0 |
| | | | | This reverts commit 6fe91ec5008838338e54ab8570f7c95ee0cdd8e8. | ||||
* | Rename abstract_controller/rendering. to errors.rb | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+0 |
| | | | | Since all rendering stuff was extracted to AV, the only thing that left was single class with error so file name wasn't relevant anymore | ||||
* | Hook up AV::Rendering on AV intialization | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+2 |
| | |||||
* | Move rendering from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 2 | -610/+0 |
| | |||||
* | Move view_paths from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -96/+0 |
| | |||||
* | Execute conditional procs on controller filters only for current action. | Nicholas Jakobsen | 2013-08-10 | 1 | -1/+1 |
| | | | | | | :only and :except options for controller filters are now added before :if and :unless. This prevents running :if and :unless procs when not on the specified. Closes #11786. | ||||
* | move `MissingHelperError` out of the `ClassMethods` module. | Yves Senn | 2013-07-12 | 1 | -15/+18 |
| | |||||
* | Show real LoadError on helpers require | Piotr Niełacny | 2013-07-10 | 1 | -1/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | When helper try to require missing file rails will throw exception about missing helper. # app/helpers/my_helper.rb require 'missing' module MyHelper end And when we try do load helper class ApplicationController helper :my end Rails will throw exception. This is wrong because there is a helper file. Missing helper file helpers/my_helper.rb Now when helper try to require non-existed file rails will throw proper exception. No such file to load -- missing | ||||
* | use extract_options! | Neeraj Singh | 2013-07-02 | 1 | -1/+1 |
| | |||||
* | deprecating string based terminators | Aaron Patterson | 2013-05-14 | 1 | -1/+3 |
| | |||||
* | Avoid leak into controller's action_methods | printercu | 2013-04-18 | 1 | -0/+1 |
| | |||||
* | Prefer find_by over dynamic finders in rdoc | Sam Ruby | 2013-04-02 | 1 | -1/+1 |
| | |||||
* | Fix typos in AP: "overriden" => "overridden" | Carlos Antonio da Silva | 2013-03-30 | 1 | -1/+1 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-03-30 | 2 | -2/+2 |
|\ | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb activerecord/test/cases/adapter_test.rb guides/source/testing.md [ci skip] | ||||
| * | Capitalize the first letter of sentence | Tatsuro Baba | 2013-03-18 | 1 | -1/+1 |
| | | |||||
| * | Remove :all from *args options in AbstractController.helper | Bryan Ricker | 2013-03-16 | 1 | -1/+1 |
| | | |||||
* | | Fixed grammar | Anupam Choudhury | 2013-03-28 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #8458 from ↵ | Rafael Mendonça França | 2013-03-27 | 1 | -5/+9 |
|\ \ | |/ |/| | | | | | | | | | | | | | lucisferre/improve-layout-override-fallback-behavior Provides standard layout lookup behavior for method and proc cases Conflicts: actionpack/CHANGELOG.md | ||||
| * | Provides standard layout lookup behavior for method and proc cases | Chris Nicola | 2013-03-27 | 1 | -5/+9 |
| | | | | | | | | | | | | | | When setting the layout either by referencing a method or supplying a Proc there is no way to fall back to the default lookup behavior if desired. This patch allows fallback to the layout lookup behavior when returning nil from the proc or method. | ||||
* | | change useless gsub to tr | robertomiranda | 2013-03-05 | 1 | -1/+1 |
| | | |||||
* | | Improve docs for AbsC::Rendering | José Valim | 2013-02-27 | 1 | -3/+11 |
|/ | |||||
* | Review #translate docs [ci skip] | Carlos Antonio da Silva | 2013-01-20 | 1 | -8/+7 |
| | |||||
* | Add documentation for abstract controller #translate and #localize method. | Jens Bissinger | 2013-01-20 | 1 | -0/+10 |
| |