aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
* remove calls to deprecated find(:first) in actionpack test suiteSergey Nartimov2012-04-261-2/+2
|
* Fix the buildYehuda Katz2012-04-251-1/+2
|
* Merge pull request #5980 from gazay/valid_ipsJeremy Kemper2012-04-251-20/+114
|\ | | | | Remote ip logic and validation. IPv6 support.
| * Valid ips v4 and v6. Right logic for working with X-FORWARDED-FOR header and ↵Alexey Gaziev2012-04-251-20/+114
| | | | | | | | tests.
* | Remove default match without specified methodJose and Yehuda2012-04-2439-341/+341
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964
* | Use leap years trick in distance_of_time_in_words only for distances between ↵Alexey Vakhov2012-04-241-0/+2
| | | | | | | | real date points
* | fixed non matching documentation behaviour with method semantics on ↵Marcelo Casiraghi2012-04-241-2/+19
| | | | | | | | distance_of_time_in_words
* | Distance of time in words should work correct if from time > to_timeAlexey Vakhov2012-04-241-0/+1
|/
* Merge pull request #5865 from tiegz/minor_fixesAaron Patterson2012-04-201-34/+21
|\ | | | | Catch nil.to_sym errors in partial_renderer, and raise ArgumentError instead
| * converting some tests to assert_raises, and DRY'ing retrieve_variable changesTieg Zaharia2012-04-161-39/+17
| |
| * catch nil.to_sym errors in partial_renderer, and raise ArgumentError insteadTieg Zaharia2012-04-161-0/+9
| |
* | Remove unused test code.Rafael Mendonça França2012-04-161-5/+0
|/ | | | | ActionController::RoutingError are raised in router execution time and can not be rescued in a controller.
* Merge pull request #5746 from jmbejar/index_method_form_builderSantiago Pastorino2012-04-111-0/+50
|\ | | | | Add method FormBuilder#index
| * Add index method to FormBuilder. Useful when you use field_for and need to ↵Jorge Bejar2012-04-111-0/+50
| | | | | | | | know the index number into the iteration.
* | Adding itemscope to list of boolean attributes.Frankie Roberto2012-04-091-2/+2
| | | | | | | | | | | | 'itemscope' is defined within HTML5 for use in microdata markup. See http://www.whatwg.org/specs/web-apps/current-work/multipage/microdata.ht ml#attr-itemscope
* | Merge pull request #5783 from rafaelfranca/default_url_optionsSantiago Pastorino2012-04-082-2/+2
|\ \ | | | | | | Document that default_url_options must return a hash with symbolized keys
| * | default_url_options does not receive one argument anymoreRafael Mendonça França2012-04-082-2/+2
| | |
* | | Revert "Merge pull request #5750 from ahoward/master"Santiago Pastorino2012-04-081-18/+0
|/ / | | | | | | | | | | | | This reverts commit 174cf8b22064c3999dfa685014d5b31ac993bb54, reversing changes made to 7ecd6a731bd60665bc6de94095137f0b2c4ada2a. The reverted commit improved the performance in the wrong place, now we have added this 6ddbd1844a6fd6aca2992f5f75c9f605cf89808f improvement.
* | Merge pull request #5748 from ai/no_type_in_html5Jeremy Kemper2012-04-085-104/+104
|\ \ | | | | | | Remove unnecessary in HTML 5 type attribute with default value
| * | Remove unnecessary in HTML 5 type attribute with default valueAndrey A.I. Sitnik2012-04-055-104/+104
| | |
* | | Make url_for work when option is nil and default_url_options hasRafael Mendonça França2012-04-051-0/+9
| | | | | | | | | | | | stringified keys
* | | Merge pull request #5750 from ahoward/masterAaron Patterson2012-04-051-0/+9
|\ \ \ | |_|/ |/| | mo fasta and mo betta the url_for
| * | carefully and quickly merge url_for optionsara.t.howard2012-04-041-0/+9
| |/
* | Merge pull request #4446 from ayamomiji/patch-1José Valim2012-04-051-0/+7
|\ \ | |/ |/| prevent the cache sweeper ignores NoMethodError
| * prevent the cache sweeper ignores NoMethodErrorayaya2012-03-291-0/+7
| |
* | add type option to atom feed entry builderSergey Nartimov2012-04-011-0/+31
| | | | | | | | closes #5672
* | Merge pull request #5691 from avakhov/form-label-blockJosé Valim2012-04-011-0/+14
|\ \ | | | | | | Block version of label should wrapped in field_with_errors in case of error
| * | Block version of label should wrapped in field_with_errors in case of errorAlexey Vakhov2012-04-011-0/+14
| | |
* | | Tests :if option of force_ssl methodAlexey Vakhov2012-04-011-0/+23
|/ /
* | Remove the leading \n added by textarea on assert_selectSantiago Pastorino2012-03-301-0/+7
| |
* | Merge pull request #5625 from ↵José Valim2012-03-284-10/+63
|\ \ | | | | | | | | | | | | nertzy/prefix_partial_path_with_controller_namespace Add config option to turn off prefixing partial path with controller namespace
| * | Make controller namespace partial prefix optionalGrant Hutchins2012-03-284-6/+59
| | | | | | | | | | | | | | | | | | | | | config.action_view.prefix_partial_path_with_controller_namespace This allows you to choose to render @post using /posts/_post.erb instead of /admin/posts/_post.erb inside Admin::PostsController.
| * | Test that render gets correct exact template nameGrant Hutchins2012-03-281-4/+4
| |/
* | Merge pull request #5359 from avakhov/missed-commits-from-3-2-stableSantiago Pastorino2012-03-282-1/+27
|\ \ | |/ |/| Missed commits from 3-2-stable
| * datetime_select should work with -/+ infinity datesJoe Van Dyk2012-03-281-0/+12
| |
| * Add a test case for layout nil.José Valim2012-03-281-1/+15
| |
* | Set proper rendered_format when doing render :inlineSantiago Pastorino2012-03-281-0/+1
|/ | | | Closes #5632
* Cover one more case in auth_token and remote formsPiotr Sarnacki2012-03-281-0/+7
| | | | | If embedding auth_token in remote forms is off and we pass a value for auth_token it should respect it.
* config.action_view.embed_authenticity_token_in_remote_forms is true by defaultPiotr Sarnacki2012-03-281-19/+14
| | | | | | | Changed default value for `config.action_view.embed_authenticity_token_in_remote_forms` to `false`. This change breaks remote forms that need to work also without javascript, so if you need such behavior, you can either set it to `true` or explicitly pass `:authenticity_token => true` in form options
* Added config.action_view.embed_authenticity_token_in_remote_formsPiotr Sarnacki2012-03-281-2/+48
| | | | | | | | | | | | There is a regression introduced in 16ee611fa, which breaks remote forms that should also work without javascript. This commit introduces config option that allows to configure this behavior defaulting to the old behavior (ie. include authenticity token in remote forms by default) Conflicts: actionpack/CHANGELOG.md
* Add test for default_url_options[:port]Aaron Suggs2012-03-281-0/+12
|
* Merge pull request #5621 from rafaelfranca/fix-2492-masterJeremy Kemper2012-03-271-0/+34
|\ | | | | Fix label_tag to merge the options hash with the object hash
| * Check if the options hash already exists and merge it with the anotherRafael Mendonça França2012-03-271-6/+18
| | | | | | | | | | | | hash. Closes #2492 and #5615
| * Fixing issue #2492 for master branch. ActionView::Base.field_error_proc ↵Andrey Samsonov2012-03-271-0/+22
| | | | | | | | | | | | | | | | doesn't call for label. objectify_options method should be applied to the proper options arg. See explanation and example of the bug - https://github.com/kryzhovnik/rails_field_error_proc_bug_example
* | Merge pull request #5620 from jcoleman/textarea-newline-fix-breaks-haml-masterSantiago Pastorino2012-03-271-6/+6
|\ \ | | | | | | Don't break Haml with textarea newline fix. (master)
| * | Don't break Haml with textarea newline fix.James Coleman2012-03-271-6/+6
| |/ | | | | | | | | | | See issue #393, issue #4000, issue #5190, and issue #5191. Adds a newline after the textarea opening tag based on @codykrieger's original patch so that we don't cause regressions in Haml-using apps. The regression caused textarea tags to add newlines to the field unintentionally (each update/save added an extra newline.) Also fix 6 more tests that didn't yet have the newline expectation.
* / adds delegetion for eof? to AD::Http::UploadedFileJens Fahnenbruck2012-03-271-0/+6
|/ | | | | if you want to read the file you may need to ask if there is something to read from
* If partial is rendered in controller, grab format from templatePiotr Sarnacki2012-03-272-0/+14
| | | | | | | | | Previously `rendered_format` was set only based on mime types passed in Accept header, which was wrong if first type from Accept was different than rendered partial. The fix is to simply move setting rendered_format to the place where template is available and grab format from the template. If it fails we can fallback to formats passed by Accept header.
* change number_with_precision to not return negative zerosTim McEwan2012-03-271-0/+1
|
* Don't ignore non Enumerable values passed to sanitize (closes #5585)Piotr Sarnacki2012-03-271-0/+18
| | | | | | | | | When someone accidentally passes a string to sanitize like: sanitize("<span>foo</span>", :tags => "b") there is no indication that it's the wrong way and span will not be removed.