aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view/helpers/url_helper.rb
Commit message (Collapse)AuthorAgeFilesLines
* Allow customization of form class for button_toAndrei Bocan2011-02-011-3/+13
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Call html_escape in ERB::Util module and don't mix it in in the helpersSantiago Pastorino2010-10-181-5/+6
|
* Refactor a bit this code to add data-confirm and data-disable-withSantiago Pastorino2010-10-111-12/+4
|
* data-disable-with in button_to helperPaco Guzman2010-10-111-0/+6
| | | | | | [#4993 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* refactoring to use fewer intermediate variablesAaron Patterson2010-10-041-11/+12
|
* cleaning up more crazy!Aaron Patterson2010-10-041-3/+3
|
* deleting more crazyAaron Patterson2010-10-041-6/+4
|
* dry up some crazy codesAaron Patterson2010-10-041-8/+6
|
* Performance: refactor convert_options_to_data_attributes.thedarkone2010-09-271-10/+14
|
* Remove dead code.thedarkone2010-09-271-2/+2
|
* Clean up url_for.thedarkone2010-09-271-3/+1
|
* Start cleanup of deprecations in ActionViewCarlos Antonio da Silva2010-09-061-6/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Refactor RoutesProxy to avoid using _with_routes in helpersPiotr Sarnacki2010-09-031-3/+5
|
* Fix generating urls with mounted helpers in view contextPiotr Sarnacki2010-09-031-1/+3
| | | | | | | | | | There were actually 2 problems with this one: * script_name was added to options as a string and then it was used in RouteSet#url_for with usage of <<, which was changing the original script_name * the second issue was with _with_routes method. It was called in RoutesProxy to modify _routes in view_context, but url_helpers in views is just delegating it to controller, so another _with_routes call is needed there
* Merge remote branch 'docrails/master'Xavier Noria2010-09-031-1/+1
|\
| * fix typo(?)Akira Matsuda2010-09-011-1/+1
| |
* | Remove unnecessary code from UrlHelper#link_to.yury2010-09-011-7/+2
|/ | | | | | convert_options_to_data_attributes always returns not nil stringified html_options [#5445 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Fix API docs for button_to optionsJaime Iniesta2010-08-311-2/+3
|
* Restore consistency with the rest of the doc.Rémy Coutable2010-08-251-2/+2
|
* Fixed a missing simple quote.Rémy Coutable2010-08-251-1/+1
|
* options could be of any kind of Hash (Hash, HashWithIndifferentAccess or ↵Santiago Pastorino2010-07-211-1/+1
| | | | OrderedHash) this way we keep the properties of the options passed as an argument
* Use capture instead of yield in link_to_unless.José Valim2010-07-211-1/+1
| | | | [#5162 state:resolved]
* Unify routes naming by renaming router to routesPiotr Sarnacki2010-07-021-1/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* s/escape_once/html_escape/, since html safety is the contract that now says ↵Xavier Noria2010-06-301-14/+17
| | | | whether something has to be escaped
* url_for no longer escapes HTML, the :escape option is also goneXavier Noria2010-06-301-16/+5
| | | | Rationale: url_for is just a path/URL generator, it is the responsability of the caller to escape conveniently HTML needs it, JavaScript needs different escaping, a text mail needs no escaping at all, etc.
* Restores the escaping of urls generated from hashes. [#4765 state:resolved]Andrew White2010-06-281-1/+1
| | | | | | | | | | | | | | HTML specifications recommend the escaping of urls in web pages, which url_for does by default for string urls and consquently urls generated by path helpers as these return strings. Hashes passed to url_for are not escaped by default and this commit reverses this default so that they are escaped. Undoes the changes of this commit: http://github.com/rails/rails/commit/1b3195b63ca44f0a70b61b75fcf4991cb2fbb944 Signed-off-by: José Valim <jose.valim@gmail.com>
* Action View is now titled.Rizwan Reza2010-06-211-1/+1
|
* Added titles and description.Rizwan Reza2010-06-161-0/+1
|
* Merge branch 'master' of github.com:rails/railswycats2010-04-261-1/+1
|\
| * HWIA symbolize_keys now returns a hash so no need to do this anymoreSantiago Pastorino2010-04-221-1/+1
| | | | | | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* | Allow :remote => false to be passed to link_toNicolas Sanguinetti2010-04-221-3/+1
|/ | | | | | | And add tests for `button_to` and `form_tag` which currently behave as expected, so we avoid a regression. Signed-off-by: wycats <wycats@gmail.com>
* url_for now works with HashWithIndifferentAccess ht jay [#4391 state:committed]Santiago Pastorino2010-04-211-1/+1
| | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* mail_to with :encode => :javascript now outputs safe htmlSam Elliott2010-04-161-1/+1
| | | | Signed-off-by: Carl Lerche <carllerche@mac.com>
* Consistently use lowercase instead of camelCase for all JS class names in RailsDavid Heinemeier Hansson2010-04-081-5/+5
|
* * Change the object used in routing constraints to be an instance ofwycats2010-04-031-1/+6
| | | | | | | | | | | ActionDispatch::Request rather than Rack::Request. * Changed ActionDispatch::Request#method to return a String, to be compatible with the Rack::Request superclass. * Changed ActionDispatch::Request#method to return the original method in the case of methodoverride and #request_method not to, to be compatible with Rack::Request
* Refactored url_for in AV to have its own instances of the helpers instead of ↵wycats2010-04-031-15/+37
| | | | proxying back to the controller. This potentially allows for more standalone usage of AV. It also kicked up a lot of dust in the tests, which were mocking out controllers to get this behavior. By moving it to the view, it made a lot of the tests more standalone (a win)
* Merge branch 'master' into docrailswycats2010-03-261-3/+9
|\
| * Remove data-url support from url_helper [#4236 state:resolved]Denis Odorcic2010-03-211-2/+0
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
| * Fix missing requirewycats2010-03-161-0/+1
| |
| * Do not always include the named URL helpers into AC::Base and AV::Base.Carl Lerche2010-03-161-0/+7
| |
| * Fix link_to with blockJeremy Kemper2010-03-151-1/+1
| |
* | Merge remote branch 'mainstream/master'Pratik Naik2010-03-151-1/+1
|\|
| * Making escaped things more readableSantiago Pastorino2010-03-141-1/+1
| |
* | Merge remote branch 'mainstream/master'Pratik Naik2010-03-121-13/+10
|\| | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/base.rb railties/lib/rails/configuration.rb railties/lib/rails/log_subscriber.rb
| * Tweak the semantic of various URL related methods of ActionDispatch::RequestCarlhuda2010-03-031-2/+3
| |
| * Use ActionDispatch::Routing everywhereMartin Schürrer2010-02-211-1/+1
| |
| * Require persisted? in ActiveModel::Lint and remove new_record? and ↵José Valim2010-02-211-1/+1
| | | | | | | | destroyed? methods. ActionPack does not care if the resource is new or if it was destroyed, it cares only if it's persisted somewhere or not.
| * Remove empty line and trailing hash, breaks documentation generationDirkjan Bussink2010-02-191-2/+0
| | | | | | | | Signed-off-by: Yehuda Katz <yehudakatz@YK.local>
| * Explicit html_escape removed when not neededSantiago Pastorino and José Ignacio Costa2010-02-141-1/+1
| | | | | | | | Signed-off-by: Yehuda Katz <yehudakatz@YK.local>
| * html_escape mail_to when encode javascript and not hexSantiago Pastorino and José Ignacio Costa2010-02-141-5/+3
| | | | | | | | Signed-off-by: Yehuda Katz <yehudakatz@YK.local>