Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

Applied changes to stylesheet_link_tag from javascript_include_tag which corrects issues with ordering and duplicates.

This corrects two issues with javascript_include_tag, the order at which they are expanded, and removing duplicates.

When individual js assets are specified, they will override the order of the same asset specified in an expansion.

[#5938 state:resolved]

Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

As per the HTML 4.01 spec:

  Buttons created with the BUTTON element function just like buttons
  created with the INPUT element, but they offer richer rendering
  possibilities: the BUTTON element may have content. For example, a
  BUTTON element that contains an image functions like and may resemble
  an INPUT element whose type is set to "image", but the BUTTON element
  type allows content.

Since rich content is the main purpose of the <button> element, it makes
sense for the button_tag helper to accept a block.

http://www.w3.org/TR/html401/interact/forms.html#edef-BUTTON
http://dev.w3.org/html5/spec/the-button-element.html#the-button-elementSigned-off-by: NSantiago Pastorino and Emilio Tagua <santiago+emilioe@wyeworks.com>

"submit" is the default value of the <button> element's type attribute
according to the HTML 4.01 and the HTML5 draft specs, so if button_tag
is going to have a default, type="submit" is a more sensible choice than
type="button".

http://www.w3.org/TR/html401/interact/forms.html#adef-type-BUTTON
http://dev.w3.org/html5/spec/the-button-element.html#attr-button-typeSigned-off-by: NSantiago Pastorino and Emilio Tagua <santiago+emilioe@wyeworks.com>

Removed Array#safe_join in AS core_ext and moved it to a view helper with the same same. This also changes how safe_join works, if items or the separator are not html_safe they are html_escape'd, a html_safe string is always returned.
Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Applied the wrong version.

This reverts commit 98c0c5db.

Be sure to javascript_escape the email address to prevent apostrophes inadvertently causing javascript errors.

This fixes CVE-2011-0446

Add tests showing the LH issue #6381: fields_for with inline blocks and nested attributes already persisted
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

[#6369 state:committed]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

To make sure it will show block contents if it is placed after 'render
:partial'

[#5557 state:resolved]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

fixes regression by http://github.com/rails/rails/commit/133ada6ab0f0cb7bef2bd40dbc18f2d5bc6b964e

[#5503 state:committed]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

[#6222 state:committed]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

Improve select helpers by allowing a selected value of false.  This is useful when using a select helper with a boolean attribute, and the attribute is false. (e.g. f.select :allow_comments)

This tag is similar in nature to submit_tag, but allows more control.
It also doesn't submit if submit type isn't used, allowing JavaScript to
control the flow where required.

For more information: http://www.whatwg.org/specs/web-apps/current-work/multipage/the-button-element.html#the-button-element

Don't be so picky on MissingTemplate error details, this fails randomly on 1.8.7 because of not ordered hash

[#6044 state:committed]
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>

Using the same hash instance makes using the same expansions for
both javascripts and stylesheets.

This reverts commit 006ec77e.

Don't catch exceptions here. Instead only declare that we want exceptions to be rescued as :html, but also let users configure reactions to exceptions in I18n.
Signed-off-by: NJosé Valim <jose.valim@gmail.com>

Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>