aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge pull request #9221 from thenickcox/image_alt_attributeCarlos Antonio da Silva2013-02-113-4/+40
|\ \ \ | | | | | | | | Improve img alt attribute for screen readers
| * | | Improve img alt attribute for screen readersthenickcox2013-02-093-4/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the img_alt method in ActionView keeps underscores in the alt attribute. Because underscores are pronounced in Apple's VoiceOver Utility, this has serious implications for accessibility. This patch makes underscored or hyphenated file names (both common in projects) read more naturally in screen readers by replacing them with spaces. See method documentation for details. Added documentation to image_alt method
* | | | the `:controller` option for routes can contain numbers. closes #9231.Yves Senn2013-02-092-1/+13
| | | |
* | | | Add missing require to APCarlos Antonio da Silva2013-02-081-0/+1
| | | |
* | | | We don't need a CHANGELOG entry for an issue of something that was not ↵Santiago Pastorino2013-02-081-5/+0
| | | | | | | | | | | | | | | | released yet
* | | | Merge pull request #9196 from AndreyChernyh/fix-cookies-with-null-sessionSantiago Pastorino2013-02-084-5/+49
|\ \ \ \ | | | | | | | | | | Fix #9168 Initialize NullCookieJar with all options needed for KeyGenerator
| * | | | Fix #9168 Initialize NullCookieJar with all options needed for KeyGeneratorAndrey Chernih2013-02-084-5/+49
| |/ / /
* / / / Bump rack dependency to 1.5.2Santiago Pastorino2013-02-081-1/+1
|/ / /
* | | Update actionpack/CHANGELOG.mdRafael Mendonça França2013-02-061-4/+4
| | | | | | | | | [ci skip]
* | | ruby constant syntax is not supported as routing `:controller` option.Yves Senn2013-02-063-9/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current implementation only works correctly if you supply the `:controller` with directory notation (eg. `:controller => 'admin/posts'`). The ruby constant notation (eg. `:controller => 'Admin::Posts`) leads to unexpected problems with `url_for`. This patch prints a warning for every non supported `:controller` option. I also added documentation how to work with namespaced controllers. The warning links to that documentation in the rails guide.
* | | partials inside directory work with `assert_template`Yves Senn2013-02-046-3/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 '/'.
* | | descriptive `assert_template` error when partial wasn't renderedYves Senn2013-02-043-0/+16
| | | | | | | | | | | | | | | | | | | | | When `assert_template` is used with the :locals option, and the partial was not rendered, a method_missing error was raised. This changes first checks, if the partial actually was rendered and raises a descriptive error.
* | | Put .rdoc at the end of the files named RUNNING_UNIT_TESTSYuki Nishijima2013-02-031-0/+0
| |/ |/| | | | | So that they are printed nicely on GitHub.
* | Fix markdown syntax in actionpack CHANGELOG.Steve Klabnik2013-02-011-1/+2
| | | | | | | | Starting a line with an octothorpe makes an <h1>
* | Missing require hash/keysAkira Matsuda2013-02-012-0/+3
| |
* | Missing or unneeded require extract_optionsAkira Matsuda2013-02-0110-1/+10
| |
* | Use \A in RegexpsEgor Homakov2013-02-011-1/+1
| | | | | | | | | | So, if there is redirect_to params[:q] i can send ?q=javascript:asdf()%0A/localpath Or something more nasty, so please use \A
* | remove dead codeAaron Patterson2013-01-301-4/+1
| |
* | change parameter name for positional argsAaron Patterson2013-01-301-2/+1
| |
* | nodoc the helper classes, cache stuff for optimized helperAaron Patterson2013-01-301-13/+17
| |
* | cache path parts in the instanceAaron Patterson2013-01-301-2/+3
| |
* | stop evaling a string every timeAaron Patterson2013-01-301-4/+4
| |
* | moving helper classes outside the private blockAaron Patterson2013-01-301-90/+88
| |
* | pushing specialization down to the optimized classAaron Patterson2013-01-301-23/+25
| |
* | use polymorphism to remove conditionalAaron Patterson2013-01-301-35/+27
| |
* | move conditionals to instanceAaron Patterson2013-01-301-6/+16
| |
* | pull stuff out of the caller and hide in the instanceAaron Patterson2013-01-301-4/+3
| |
* | moving more stuff on to the instanceAaron Patterson2013-01-301-18/+18
| |
* | move optimize_helper? to the helper instanceAaron Patterson2013-01-301-9/+12
| |
* | moving more stuff to the instanceAaron Patterson2013-01-301-6/+6
| |
* | moving some stuff to the initializerAaron Patterson2013-01-301-14/+38
| |
* | moved more evald codeAaron Patterson2013-01-301-8/+7
| |
* | factored out some of the dynamic codeAaron Patterson2013-01-301-19/+31
| |
* | Fix `content_tag_for` with array html option.Semyon Perepelitsa2013-01-313-1/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It would embed array as string instead of joining it like `content_tag` does: content_tag(:td, class: ["foo", "bar"]){} #=> '<td class="foo bar"></td>' Before: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item [&quot;foo&quot;, &quot;bar&quot;]" id="item_1"></td>' After: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item foo bar" id="item_1"></td>'
* | Changelog about BestStandardsSupport removalGuillermo Iguaran2013-01-291-5/+6
| |
* | Add 'X-UA-Compatible' => 'chrome=1' to default headersGuillermo Iguaran2013-01-292-2/+5
| |
* | Remove BestStandardsSupport middlewareGuillermo Iguaran2013-01-294-67/+0
| |
* | Merge pull request #9104 from bemurphy/remove_bad_idea_parser_testCarlos Antonio da Silva2013-01-291-13/+0
|\ \ | | | | | | Remove yaml Proc param parser test
| * | Remove yaml Proc param parser testBrendon Murphy2013-01-281-13/+0
| | | | | | | | | | | | | | | | | | | | | | | | I don't believe this test is exercising any explicit params_parser behavior that the other two Proc tests aren't already doing. Given that we now know it's a bad idea to load user input via YAML.load, somebody reading this test might get a dangerous idea about building out a YAML params parser.
* | | s/ERb/ERB/Akira Matsuda2013-01-291-1/+1
|/ /
* | Merge pull request #9096 from tricknotes/fix-code-commentRafael Mendonça França2013-01-281-4/+4
|\ \ | | | | | | Fix code comment
| * | Fix JavaScript syntax in code comment [ci skip]Ryunosuke SATO2013-01-291-4/+4
| | |
* | | Move AS changelog entry to the top, improve AP changelog a bitCarlos Antonio da Silva2013-01-281-4/+4
| | | | | | | | | | | | [ci skip]
* | | Added a Changelog entry for fixing HEAD requests.Michiel Sikkes2013-01-281-0/+5
| | |
* | | Make current_url? work with a HEAD methodMichiel Sikkes2013-01-282-1/+7
| | | | | | | | | | | | | | | | | | ActionDispatch::Head was removed in favor of Rack::Head. But Rack::Head does not convert GET requests to HEAD requests so we need to do checking for HEAD requests ourselves.
* | | Remove caching_allowed? from ActionController::CachingMichiel Sikkes2013-01-281-4/+0
|/ / | | | | | | | | Where is this used? No other code references to this method and it isn't being tested anywhere. No tests fail when commented out.
* | Merge pull request #9032 from firmhouse/head-breaks-csrfSantiago Pastorino2013-01-282-2/+6
|\ \ | | | | | | Make HEAD work / convert to GET once more
| * | Added request.head? to forgery protection codeMichiel Sikkes2013-01-221-2/+2
| | |
| * | Added a test that shows that a HEAD request does not normally pass CSRF ↵Michiel Sikkes2013-01-221-0/+4
| | | | | | | | | | | | protection
* | | Use Encoding::UTF_8 constant :do_not_litter:Akira Matsuda2013-01-287-11/+11
| | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-08-07 09:30:26 +0000