aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | do not break params filtering on nil valuesVasiliy Ermolovich2013-09-072-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | closes #12149
* | | | | | | | Removed semicolon and added spaceAnupam Choudhury2013-09-131-2/+2
| | | | | | | |
* | | | | | | | Revert "Merge pull request #12208 from mjhoy/patch-1"Rafael Mendonça França2013-09-121-0/+4
| |_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ab5cd54b7e791f8419f689d1bef5394890268a6f, reversing changes made to cdc10c898d4865302740340eedec4f5f4ca76565. Reason: This way of defining root path is still supported. See https://github.com/rails/rails/blob/d262773ab7f0aae5de2d354ac2eca168e95b653d/actionpack/test/controller/routing_test.rb#L450-457
* | | | | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2013-09-131-10/+10
|\ \ \ \ \ \ \
| * | | | | | | Change documentation to consistently refer to the same objectEarl J St Sauver2013-09-081-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The documentation in this section is referring to a profile, so the resource that's created should probably also be a profile of some sort.
| * | | | | | | Blacklist refferenced in docs is actually whitelistEarl J St Sauver2013-09-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The docs refference a blacklist, but really what's being described is a whitelist. Anything that matches the constraint gets through to the path.
* | | | | | | | remove outdated docsMichael Hoy2013-09-121-4/+0
| | | | | | | |
* | | | | | | | Reset ActionView::Base.logger instead of AC::Base.loggerAkira Matsuda2013-09-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | see: 9b0ac0bc74569db460f87ea6888b3847be0ff5be
* | | | | | | | Refactor handling of action normalizationMax Shytikov2013-09-101-11/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reference: Bloody mess internals http://gusiev.com/slides/rails_contribution/static/#40
* | | | | | | | Make AC standalone rendering workSantiago Pastorino2013-09-102-2/+20
| | | | | | | |
* | | | | | | | Remove helper fixtures not used in any testclaudiob2013-09-091-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fixture for module AbcHelper defines three functions bare_a, bare_b and bare_c, but only bare_a is used in the code that tests helper functions.
* | | | | | | | Remove BasicRendering testsJosé Valim2013-09-092-10/+1
| | | | | | | |
* | | | | | | | Remove remaining coupling with AV in MimeRespondsJosé Valim2013-09-093-5/+13
| | | | | | | |
* | | | | | | | Remove BasicRendering and remove template functionality from AbsC::RenderingJosé Valim2013-09-094-40/+19
| |/ / / / / / |/| | | | | |
* | | | | | | Do not use instance variables if they are not reused elsewhereCarlos Antonio da Silva2013-09-061-2/+1
| | | | | | |
* | | | | | | render_to_string shouldn't play with response_bodySantiago Pastorino2013-09-041-4/+5
| | | | | | |
* | | | | | | Merge pull request #12077 from estsauver/doc_fixSteve Klabnik2013-09-041-2/+3
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Update mapper documenation for match helper [ci skip]
| * | | | | | Update mapper documenation for match helper [ci skip]Earl St Sauver2013-08-301-2/+3
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This piece of documentation is out of date. The use of match without any via option is prevented, now the HTTP verbs have to be explicitly set. If they're not set then the error message in normalize_conditions! (around line 186) is shown.
* | | | | | Fix a few typos. [ci skip]namusyaka2013-09-041-1/+1
| | | | | |
* | | | | | Move BasicRendering to AbstractControllerŁukasz Strzałkowski2013-09-033-29/+29
| | | | | |
* | | | | | Make Mime::TEXT default format in AbstractControllerŁukasz Strzałkowski2013-09-032-4/+1
| | | | | |
* | | | | | Move skeleton methods from AV to AbsCŁukasz Strzałkowski2013-09-031-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łkowski2013-09-021-5/+6
| | | | | |
* | | | | | Revert "Port all remaining self.protected_instance_variables to class methods"Łukasz Strzałkowski2013-09-024-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 7de994fa215e9f4c2856d85034bc4dd7b65d0c01.
* | | | | | Reuse variable to avoid symbol usageCarlos Antonio da Silva2013-08-311-2/+2
| | | | | |
* | | | | | Merge pull request #11912 from jonkessler/respond-to-signature-fixesRafael Mendonça França2013-08-312-1/+6
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Update Rails::Railtie::Configuration and ActionDispatch::Response#respond_to? to accept include_private argument
| * | | | | update Rails::Railtie::Configuration and ↵Jon Kessler2013-08-162-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | ActionDispatch::Response#respond_to? to accept include_private argument
* | | | | | Stop messing up with instance variables, use protected as it was meant forCarlos Antonio da Silva2013-08-301-2/+7
| | | | | |
* | | | | | Merge pull request #12062 from strzalek/default-protected-varsJosé Valim2013-08-294-17/+17
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Port all remaining self.protected_instance_variables to class methods
| * | | | | Port all remaining self.protected_instance_variables to class methodsŁukasz Strzałkowski2013-08-294-17/+17
| | | | | |
* | | | | | Improve AP changelog regarding AV extraction [ci skip]Carlos Antonio da Silva2013-08-261-2/+2
|/ / / / /
* | | | | Fix formatting of error messageŁukasz Strzałkowski2013-08-251-2/+1
| | | | | | | | | | | | | | | | | | | | Make it oneliner
* | | | | Change description of BasicRendering#renderŁukasz Strzałkowski2013-08-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We're not dealing with templates here [ci skip]
* | | | | Simplify if statementsŁukasz Strzałkowski2013-08-251-6/+4
| | | | |
* | | | | Move abstract's controller tests to AVŁukasz Strzałkowski2013-08-2515-891/+0
| | | | | | | | | | | | | | | | | | | | The ones that were actually testing AV functionality and should belong in there
* | | | | Describe BasicRendering shortlyŁukasz Strzałkowski2013-08-251-0/+4
| | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | Extend basic rendering, test it in railtiesŁukasz Strzałkowski2013-08-252-22/+16
| | | | |
* | | | | Load HTML in ActionView not ActionPackŁukasz Strzałkowski2013-08-251-1/+0
| | | | | | | | | | | | | | | | | | | | HTML Scanner is part of ActionView and it should be loaded along with it
* | | | | Update AP changelogŁukasz Strzałkowski2013-08-251-0/+9
| | | | |
* | | | | Basic rendering testŁukasz Strzałkowski2013-08-251-0/+19
| | | | |
* | | | | Move AP's capture tests to AVŁukasz Strzałkowski2013-08-251-79/+0
| | | | |
* | | | | Move remaining layouts tests to AVŁukasz Strzałkowski2013-08-2510-270/+0
| | | | |
* | | | | Remove unused fixtures and models from AP testsŁukasz Strzałkowski2013-08-257-12/+0
| | | | |
* | | | | Move render_test to AVŁukasz Strzałkowski2013-08-251-1298/+106
| | | | |
* | | | | Fist stab on basic renderingŁukasz Strzałkowski2013-08-253-0/+20
| | | | |
* | | | | Do not include action_view/layoutsŁukasz Strzałkowski2013-08-251-1/+0
| | | | | | | | | | | | | | | | | | | | We don't need them. They'll be loaded automaticly if AV will be included
* | | | | Add #rendered_format method to controllersŁukasz Strzałkowski2013-08-252-4/+9
| | | | |
* | | | | Do not load AV inside APŁukasz Strzałkowski2013-08-252-9/+0
| | | | | | | | | | | | | | | | | | | | Move that part to AV railtie
* | | | | Remove dependency on AVŁukasz Strzałkowski2013-08-251-3/+3
| | | | |
* | | | | Revert "Move setting content_type to AV"Łukasz Strzałkowski2013-08-251-0/+1
| | | | | | | | | | | | | | | | | | | | This reverts commit f4d602aff6cec80304b131ecfcc2676d0304f0cc.