| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | | |
|
| | | | | |
|
|\ \ \ \ \
| |_|_|/ /
|/| | | | |
Possibly clearer way of getting rid of ` and "
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Incidentally it's also faster...
>> a = 'hello "id` world'; Benchmark.realtime { 500_000.times { a.tr('`"', "") } }
=> 0.7388770580291748
>> a = 'hello "id` world'; Benchmark.realtime { 500_000.times { a.gsub(/[`"]/, "") } }
=> 1.7843739986419678
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Remove code duplication in InclusionValidator and ExclusionValidator.
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Orginal code by @jamescook
Closes #1352
[Rafael Mendonça França + James Cook]
|
|/ / / /
| | | |
| | | |
| | | | |
Closes #5632
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
If embedding auth_token in remote forms is off and we
pass a value for auth_token it should respect it.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Changed the config.assets.manifest to nil
|
| | | | |
| | | | |
| | | | |
| | | | | |
config template.
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Rails.initialized? can be called at any time without raising an exception [Closes #2507]
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
[Closes #2507]
Changes:
* `Rails.initialized=` has been removed
* `Rails.initialized?` and `Rails.application.initialized?` are now
* delegating to `MyApp::Application.initialized?`
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
Remove redundant variable from core_ext/hash/deep_dup.rb
|
| |/ / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix indenting in migration generator
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
$ rails generate migration remove_foo_from_bars foo:string
This currently generates:
def up
remove_column :bars, :foo
end
Fix it:
def up
remove_column :bars, :foo
end
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
Fix label_tag to merge the options hash with the object hash
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
hash.
Closes #2492 and #5615
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
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
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Don't break Haml with textarea newline fix. (master)
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
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.
|
| | | | | | | |
|
|/ / / / / /
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
if you want to read the file you may need to ask if there is something
to read from
|
| | | | | | |
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This reverts commit 7feec40b89ada892ca6f107ada27ceedde505d83.
Reason: The guide had the contents of railties/bin/rails and I don't see
any reason to change it.
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
It is in response for confusion in issue #5301
|
| | | | | |
| | | | | |
| | | | | | |
(and changed the use of true for a more database agnostic example).
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
this adds documentation for -e flag which sets the environment of the rails console like the `rails server` command.
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
removed unused variables
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
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.
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
prevent returning a negative zero from number_with_precision
|