HTML fallback when format is :js should be also localized

Trying again:

The current implementation of localized templates on rails/actionpack/lib/action_view/paths.rb is incomplete.

At the line 48, when we check for a html file when format is js (this is used in when xhr? is true) should be also localized. So what we have now as:

  # Try to find html version if the format is javascript
  elsif format == :js && template = load_path["#{template_path}.html"]
    return template
  end

Should actually be:

  # Try to find html version if the format is javascript
  elsif format == :js && template = load_path["#{template_path}.#{I18n.locale}.html"]
    return template
  end
  elsif format == :js && template = load_path["#{template_path}.html"]
    return template
  end

I'm not building a patch, since it's a quick fix.

Joshua, patch attached.

I could not see tests for HTML fallback even when not localized. So I added tests for both cases.

Joshua,

Don't forget to add this one. The tests provide a failing scenario.

Thanks!