Refactored url_for in AV to have its own instances of the helpers instead of 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) 

Require persisted? in ActiveModel::Lint and remove new_record? and 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.

For performance reasons, you can no longer call html_safe! on Strings. Instead, all Strings are always not html_safe?. Instead, you can get a SafeBuffer from a String by calling #html_safe, which will SafeBuffer.new(self).

  * Additionally, instead of doing concat("</form>".html_safe), you can do
    safe_concat("</form>"), which will skip both the flag set, and the flag
    check.
  * For the first pass, I converted virtually all #html_safe!s to #html_safe,
    and the tests pass. A further optimization would be to try to use
    #safe_concat as much as possible, reducing the performance impact if
    we know up front that a String is safe.

This reverts commit 3aa1ea1a, reversing
changes made to 2c12a713.

Conflicts:

	actionpack/lib/action_view/helpers/javascript_helper.rb
	actionpack/lib/action_view/helpers/url_helper.rb
	actionpack/test/template/url_helper_test.rb

Removed all helpers from PrototypeHelper that are implemented in AjaxHelper. Modified tests that extended with PrototypeHelper to now extend using AjaxHelper. AjaxHelper is now included by default in view helper

Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NJosé Valim <jose.valim@gmail.com>

[#3259 state:resolved]
Signed-off-by: NEloy Duran <eloy.de.enige@gmail.com>

  This consists of:

  * String#html_safe! a method to mark a string as 'safe'
  * ActionView::SafeBuffer a string subclass which escapes anything unsafe which is concatenated to it
  * Calls to String#html_safe! throughout the rails helpers
  * a 'raw' helper which lets you concatenate trusted HTML from non-safety-aware sources (e.g. presantized strings in the DB)
  * New ERB implementation based on erubis which uses a SafeBuffer instead of a String

Hat tip to Django for the inspiration.

Allow fields_for on a nested_attributes association to accept an explicit collection to be used. [#2648 state:resolved]
Signed-off-by: NEloy Duran <eloy.de.enige@gmail.com>

Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NPratik Naik <pratiknaik@gmail.com>

Signed-off-by: NYehuda Katz + Carl Lerche <ykatz+clerche@engineyard.com>

Made label target radio button tags with values. Radio button now respects inherited :index options when generating id.
Signed-off-by: NMichael Koziarski <michael@koziarski.com>

Signed-off-by: NDavid Heinemeier Hansson <david@loudthinking.com>

Changed API of NestedAttributes to take an array, or hash with index keys, of hashes that have the id on the inside of the attributes hash and updated the FormBuilder to produce such hashes. Also fixed NestedAttributes with composite ids.
Signed-off-by: NMichael Koziarski <michael@koziarski.com>
Signed-off-by: NEloy Duran <eloy.de.enige@gmail.com>
[#1892 state:committed]

Move checkbox hidden field before the actual checkbox so the actual value doesn't get clobbered [#1863 state:resolved]

Allowing an object to be passed explicitly to a fields_for with nested_attributes on one-to-one associations
Signed-off-by: NMichael Koziarski <michael@koziarski.com>
[#1849 state:committed]

Signed-Off-By: NMichael Koziarski <michael@koziarski.com>

[#1202 state:committed]

Signed-off-by: NJoshua Peek <josh@joshpeek.com>

Use output_buffer reader and writer methods exclusively instead of hitting the instance variable so others can override the methods.