aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Changelog improvements, use 1.9 style hash in examples [ci skip]Carlos Antonio da Silva2013-02-221-1/+1
| | | | |
* | | | | Address edge case for number_to_human with units option.hoffm2013-02-221-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActiveSupport::NumberHelper#number_to_human now returns the number unaltered when the units hash does not contain the needed key, e.g. when the number provided is less than the largest key provided.
* | | | | Merge pull request #9334 from NARKOZ/time_tagAndrew White2013-02-212-19/+21
|\ \ \ \ \ | | | | | | | | | | | | use ISO 8601 format in time_tag helper
| * | | | | document setting custom datetime attribute for time_tag helperNihad Abbasov2013-02-221-0/+2
| | | | | |
| * | | | | fix time_tag helper and update tests to not rely on dynamically generated valuesNihad Abbasov2013-02-222-19/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before `time_tag Date.today` will return `<time datetime="2013-02-14T00:00:00+00:00">February 14, 2013</time>`. This commit fixes time_tag helper to use RFC-3339 full-date format (Y-m-d) in datetime attribute.
* | | | | | completes pass over the AP changelog [ci skip]Xavier Noria2013-02-212-15/+6
| | | | | |
* | | | | | tweaks to the AP changelog [ci skip]Xavier Noria2013-02-211-9/+3
| | | | | |
* | | | | | Merge pull request #9361 from senny/improved_match_shorthand_syntaxAndrew White2013-02-213-14/+53
|\ \ \ \ \ \ | | | | | | | | | | | | | | determine the match shorthand target early.
| * | | | | | determine the match shorthand target early.Yves Senn2013-02-213-14/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #7554. This patch determines the `controller#action` directly in the `match` method when the shorthand syntax is used. this prevents problems with namespaces and scopes.
* | | | | | | Revert "Revert "Merge pull request #4803 from lucascaton/master""Santiago Pastorino2013-02-213-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 6871bd9818a9a7d9d8c7e21e253d64c0410fde1d.
* | | | | | | This cache is not neededSantiago Pastorino2013-02-211-2/+1
| | | | | | |
* | | | | | | Use composition to figure out the forgery protection strategySantiago Pastorino2013-02-211-9/+27
| |_|_|_|_|/ |/| | | | |
* | | | | | Use #grep to filter priority zones when a regexp is givenCarlos Antonio da Silva2013-02-212-3/+5
| | | | | |
* | | | | | Use Array Difference to calculate the zones array, instead of iteratingCarlos Antonio da Silva2013-02-211-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | It's faster, and doesn't force us to dup the timezones array.
* | | | | | Fix test name and refactor fake timezones creation in form options testCarlos Antonio da Silva2013-02-211-3/+3
| | | | | |
* | | | | | Remove not necessary changelog about time zone array mutationCarlos Antonio da Silva2013-02-211-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seemed to be introduced in master by 9dd64f2e0c1b3ce85b503b0f81c50c93f89ffbcd, so there's no need to add a changelog entry for fixing it. [ci skip]
* | | | | | tweaks to the AP changelog [ci skip]Xavier Noria2013-02-211-42/+27
|/ / / / /
* | | | | Merge pull request #9354 from NARKOZ/remove-warningsSteve Klabnik2013-02-211-2/+2
|\ \ \ \ \ | | | | | | | | | | | | remove assigned but unused variables
| * | | | | remove assigned but unused variable warningNihad Abbasov2013-02-211-2/+2
| | | | | |
* | | | | | Rack::Test::UploadedFile is a permitted scalarFabio Kreusch2013-02-212-4/+6
|/ / / / /
* | / / / Revert "Merge pull request #4803 from lucascaton/master"Santiago Pastorino2013-02-203-5/+3
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit bb842e8d2111e50b21a14b8bd6d89371a4b9cd68, reversing changes made to 40c287c7983c20c498f6a8a2ea49e9a6455347f7. This was causing issues in one of our apps we just upgraded. ActionController::RoutingError: No route matches [GET] "/images/favicon.ico" favicon_link_tag now returns '/images/favicon.ico' and in 3.2 returned '/favicon.ico' Browsers by default look for favicon.ico in the root directory Conflicts: actionpack/CHANGELOG.md actionpack/lib/action_view/helpers/asset_tag_helper.rb
* | | | Remove XML Parser from ActionDispatchPrem Sichanugrist2013-02-204-339/+26
|/ / / | | | | | | | | | | | | If you want an ability to parse XML parameters, please install `actionpack-xml_parser` gem.
* | | Fix time_zone_options_for_select to not mutate TimeZones arrayBrian McManus2013-02-193-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previous implementation of time_zone_options_for_select did not dup the ActiveSupport::TimeZone.all array. When :priority_zones were provided the method would reject! the zones from the memoized TimeZones array thus affecting future requests to the server. Essentially whatever zones were specified as :priority_zones would show up for the first request but then disappear from the time zone options on future requests.
* | | moves a CHANGELOG entry from AR to APXavier Noria2013-02-191-0/+4
| | |
* | | InvalidMessage is in ActiveSupport::MessageEncryptor namespaceSantiago Pastorino2013-02-192-4/+19
| | | | | | | | | | | | Closes #9302
* | | Fix AP tests related to routes inspector output and the new column titlesCarlos Antonio da Silva2013-02-191-20/+65
| | |
* | | Add missing require to routes inspectorCarlos Antonio da Silva2013-02-191-0/+1
| | |
* | | Require AS deps in ActionDispatch::Http::URLMiha Filej2013-02-191-0/+3
| | | | | | | | | | | | Allows us to use url_for and friends in isolation.
* | | Duck typing is hard. Add header to HtmlTableFormatter.Steve Klabnik2013-02-181-0/+4
| | | | | | | | | | | | | | | | | | | | | Somehow I missed this method, which was then throwing an error when viewing routes as HTML. Make @rubys happy: ✔
* | | editorial tweaksXavier Noria2013-02-191-3/+6
| | |
* | | use strip_heredoc to keep indentation consistent.Steve Klabnik2013-02-181-12/+12
| | | | | | | | | | | | Thanks @sikachu. :heart:
* | | Add message when you have no routes defined.Steve Klabnik2013-02-182-0/+31
| | | | | | | | | | | | | | | | | | Print a message in both `rake routes` and at GET "/rails/info/routes" that lets you know you have no routes defined, as well as linking to the Rails Guide on the topic.
* | | Add headings to rake routes tableSteve Klabnik2013-02-181-3/+19
| | |
* | | Gist URLs are now namespacedAkira Matsuda2013-02-181-1/+1
| | | | | | | | | | | | see: https://github.com/blog/1406-namespaced-gists
* | | Return false on toggle on error pagesBartlomiej Kozal2013-02-172-6/+7
| | |
* | | Merge pull request #9307 from wangjohn/remove_unneeded_md5_digestXavier Noria2013-02-161-1/+0
|\ \ \ | |/ / |/| | Removed an old require of an MD5 digest which is no longer used.
| * | Removed an old require of an MD5 digest which is no longer used.wangjohn2013-02-151-1/+0
| | |
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-02-153-4/+4
|\ \ \ | |/ / |/| | | | | | | | Conflicts: guides/source/upgrading_ruby_on_rails.md
| * | Correct bullet content line wrappingTim "Roger" Harvey2013-02-121-1/+1
| | | | | | | | | | | | | | | | | | The `:prefix` bullet content was not properly wrapping, dropping to the following line because the additional content was not indented properly. With this change, it will now display properly the way `:discard_type` does.
| * | Fix typo.Carson McDonald2013-02-111-1/+1
| | | | | | | | | | | | [ci skip]
| * | Fixing the examples for button_toMarc Baumbach2013-02-061-2/+2
| | | | | | | | | The examples do not generate the output they intend to due to not explicitly declaring the hash separations. This causes it to be treated as one parameter instead of the intended two parameters.
* | | Merge pull request #8704 from senny/remove_regexp_global_from_url_forAaron Patterson2013-02-142-2/+9
|\ \ \ | | | | | | | | replace regexp global in #url_for
| * | | replace regexp global in #url_forYves Senn2013-01-162-2/+9
| | | |
* | | | Bump rack-test dependency to 0.6.2Rahul P. Chaudhari2013-02-141-1/+1
| | | |
* | | | Avoid to_sym callsSantiago Pastorino2013-02-131-1/+1
| | | |
* | | | Improve changelog entry from #9221 [ci skip]Carlos Antonio da Silva2013-02-112-18/+16
| | | |
* | | | 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
| | | |