| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|/ / / /
| | | |
| | | |
| | | | |
For test with latest thor.
|
|\ \ \ \
| | | | |
| | | | | |
Allow `table_name_prefix` and `table_name_suffix` have `$`
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
MySQL 5.7 and PostgreSQL 9.6 allow table identifiers have the dollar sign.
* MySQL 5.7
https://dev.mysql.com/doc/refman/5.7/en/identifiers.html
> Permitted characters in unquoted identifiers:
> ASCII: [0-9,a-z,A-Z$_] (basic Latin letters, digits 0-9, dollar, underscore)
* PostgreSQL 9.6
https://www.postgresql.org/docs/9.6/static/sql-syntax-lexical.html
> SQL identifiers and key words must begin with a letter (a-z, but also letters with diacritical marks and non-Latin letters) or an underscore (_). Subsequent characters in an identifier or key word can be letters, underscores, digits (0-9), or dollar signs ($). Note that dollar signs are not allowed in identifiers according to the letter of the SQL standard, so their use might render applications less portable. The SQL standard will not define a key word that contains digits or starts or ends with an underscore, so identifiers of this form are safe against possible conflict with future extensions of the standard.
Address #30044
[Yasuo Honda & Ryuta Kamizono]
|
|\ \ \ \ \
| |_|_|_|/
|/| | | | |
Pass missing name attribute to execute_hook
|
| | |_|/
| |/| |
| | | |
| | | | |
Fixes commit 10bf93ef92a70ae511036134290bf0e2de184b5c created to solve issue #30025
|
|\ \ \ \
| | | | |
| | | | | |
Remove unused `action` arg for `generate_routing_code`
|
| | | | | |
|
| |/ / /
|/| | |
| | | |
| | | | |
thor 0.20.0 was released so we can test with the released version.
|
| | | |
| | | |
| | | |
| | | | |
[ci skip]
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Changed sentence to be more clear
[ci skip]
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
`it` and `has` are for singular, not plural
[ci skip]
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Periods should be outside of the <a> tags
[ci skip]
|
|\ \ \ \
| | | | |
| | | | | |
Change namespace of Active Storage rake task
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix RuboCop offenses
|
| |/ / / /
| | | | |
| | | | |
| | | | | |
And enable `context_dependent` of Style/BracesAroundHashParameters cop.
|
|/ / / /
| | | |
| | | | |
Since c51f9b61 changed the "_before_type_cast" expectation for enum.
|
| | | |
| | | |
| | | |
| | | | |
Test case for #30045
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | | |
albertoalmagro/fix-raise-unpermitted-parameters-regression
Load Parameters configurations on :action_controller only once
|
| | | |
| | | |
| | | |
| | | | |
Fixes regression ActionController::UnpermittedParameters not raised. The inner hook was being executed twice, once when ActionController::Base was loaded and again when ActionController::API was loaded. As options.delete operations inside the block are not idempotent, the second time it was run there was no configuration option available
|
| | | |
| | | |
| | | |
| | | | |
Provide run_once: true option to on_load in case you want a hook only to be executed once. This may be useful in cases where executing a hook several times may have undesired side effects
|
|\ \ \ \
| | | | |
| | | | | |
Add `--skip-yarn` option to the plugin generator
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Add SharedGeneratorTests#application_path
This method will help to DRY in files app_generator_test.rb, plugin_generator_test.rb
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
These were intentional, see
https://github.com/rails/rails/pull/30061#issuecomment-320068368
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
kamipo/through_scope_should_not_be_affected_by_scoping
Through scope should not be affected by scoping
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Follow up of #29834.
Fixes #30266.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Update guides/source/plugins.md [ci skip]
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
kamipo/dont_expose_write_attribute_without_type_cast
Don't expose `write_attribute_without_type_cast`
|
| | |/ / / /
| |/| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
`write_attribute_without_type_cast` is defined as a private method in
`AttributeMethods::Write`, but `AttributeMethods::Dirty` overrode it as
a public method. It should be kept the original visibility.
|
| | | | | | |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
Inserted system_test ones into scaffolding logs [ci skip]
|
|/ / / / / |
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Fix typo for api_app_generator_test
|
| | | | | |
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
Add missing backticks
Add missing *
Add missing .
|
|\ \ \ \
| |_|/ /
|/| | | |
Fix format of generator docs [ci skip]
|
|/ / / |
|
|\ \ \
| | | |
| | | |
| | | | |
Ensure sum honors distinct on has_many through
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
When using a has_many through relation and then summing an attribute
the distinct was not being used. This will ensure that when summing
an attribute, the number is only used once when distinct has been used.
|
|\ \ \ \
| | | | |
| | | | | |
Clarify base_class tests on abstract STI vs concrete STI
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Add `binary` helper method to fixtures.
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Refactor `--api` option of the app generator
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
It can't use JavaScript in api mode.
Action cable should be disabled by default.
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Fix guide about command_line [ci skip]
|
| | |_|_|_|/ /
| |/| | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Extract `primary_key` to `AbstractReflection`
|
| | |_|_|/ / /
| |/| | | | | |
|