| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
kamipo/eval_active_storage_attached_name_only_once
Evaluate `@active_storage_attached_#{name}` only once
|
| |
| |
| |
| |
| | |
Currently `"@active_storage_attached_#{name}"` in `define_method` is
evaluated every call. It is better to evaluate it only once.
|
|\ \
| | |
| | | |
Add `null: false` to Active Storage tables
|
| |/
| |
| |
| | |
These columns aren't intended nullable.
|
|/ |
|
| |
|
|
|
|
| |
Follow up of #30188.
|
|
|
|
|
| |
* Use `+` instead of backquote.
* Remove escape from class to be link
|
| |
|
|
|
|
|
|
|
| |
* Isolate ActiveStorage namespaces
* Rename migrations task
[Rafael Mendonça França + Dino Maric]
|
| |
|
|\
| |
| | |
ActiveStorage:Filter out Minitest backtrace.
|
| |
| |
| |
| |
| | |
Filter out Minitest backtrace while allowing backtrace from other
libraries to be shown.
|
|\ \
| | |
| | | |
Set ActiveStorage::Blob.service when ActiveStorage::Blob is loaded
|
| | |
| | |
| | |
| | |
| | | |
Fixes that ActiveStorage::Blob.service is unset when ActiveStorage::Blob
is reloaded.
|
|\ \ \
| |/ /
|/| |
| | | |
Support content_type in AzureStorageService#url
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Add in the content_type option, which is defined as part of the generic
ActiveStorage::Service class.
Without this option, attempts to generate a URL for an attached file
fail with "ArgumentError (unknown keyword: content_type)"
|
|/ /
| |
| |
| |
| |
| | |
* Add migrations per rails engine conventions
* Fix failing tests
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Extend image_tag to accept ActiveStorage's Attachments and Variants
* Flip resolve_image_source around
* Add tests for the new use-cases of image_tag
* Remove the higher-level test
* Update image_tag documentation
* Add error states into the test suite
* Re-raise polymorhic_url's NoMethodError as ArgumentError
* delegate_missing_to will raise DelegationError instead of NoMethodError
|
|
|
|
| |
Follow #30083
|
| |
|
|\ |
|
| |\
| | |
| | | |
Use `content_type.start_with?("...")` than `content_type =~ /^.../`
|
| | |
| | |
| | |
| | |
| | | |
`start_with?` is a little faster than regexp for prefix matching by a
fixed string.
|
| |\ \
| | | |
| | | | |
Fix ruby warnings
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This fixes following warnings:
```
test/models/variant_test.rb:11: warning: ambiguous first argument; put parentheses or a space even after `/' operator
lib/active_storage/attached/macros.rb:63: warning: instance variable @active_storage_attached_highlights not initialized
lib/active_storage/attached/macros.rb:25: warning: instance variable @active_storage_attached_avatar not initialized
```
|
| |\ \ \
| | | | |
| | | | | |
Active Storage: check for `app.secrets.secret_key_base`, not `app.config.secret_key_base`
|
| | |/ /
| | | |
| | | |
| | | | |
By default, apps only have the former set.
|
| |\ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
koic/change_gem_version_of_active_storage_to_5_2_0
Change gem version of Active Storage to 5.2.0.alpha
|
| | |/ / |
|
| |/ / |
|
| |/
| |
| |
| | |
changed `rails/activestorage` to `rails/rails`.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The reasons for this commit are:
- uniformity with the other Rails libraries
- (possibly) behave better with respect to autoloading
- fix the index in the generated documentation
Before this commit, run `rake rdoc` generates this left sidebar
(ActiveStorage entries are indexed twice, both inside and outside the module):
<img width="308" alt="before" src="https://user-images.githubusercontent.com/10076/28939523-7c087dec-7846-11e7-9289-38ed4a2930cd.png">
After this commit, run `rake rdoc` generates this left sidebar:
(ActiveStorage entries are only indexed inside the module):
<img width="303" alt="after" src="https://user-images.githubusercontent.com/10076/28939524-7c090be0-7846-11e7-8ee5-29dfecae548e.png">
|
| |
|
|
|
|
|
|
|
|
|
| |
If an AWS bucket name includes a `.` (e.g. `bucket.name`), then the canonical
URL for an object will start with "https://s3.amazonaws.com/bucket.name/"
and not with "https://bucket.name.s3.amazonaws.com/".
The URL tests have now been separated into two separate asserts, to ensure
that both the "s3.amazonaws.com" and the "bucket.name" components are included,
but not specifically in that order.
|
|
|
|
|
|
| |
We are already removing the braces around hash parameters in the last
argument in other places so we should not change the entire codebase
because of two places.
|
| |
|
|\
| |
| | |
Make Rubocop happy about ActiveStorage
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Running `rubocop activestorage` before this commit resulted in 20 offenses.
This commit only fixes:
- Trailing whitespace detected
- Space inside } missing
- Put one space between the method name and the first argument.
The other offenses are left since they are intentional according to
@georgeclaghorn (https://github.com/rails/rails/pull/30061#issuecomment-319999190)
|
| | |
|
| |
| |
| |
| | |
Everything inside the app directory of a engine is autoload/eager loaded automatically so we don't need to require them.
|
|/
|
|
|
| |
I know those methods are unlikely to change but having one line method
is hard to read and also hard to modify.
|
| |
|
|
|
| |
Omit azure-core (it's required by azure-storage), remove require:false to match other points.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
```
% be rubocop -a --only Style/StringLiterals activestorage
Inspecting 74 files
........................................CCCCCCCCCC.C........CC.......C.C..
(snip)
74 files inspected, 31 offenses detected, 31 offenses corrected
```
|
| |
|
|
|
| |
previously https://github.com/rails/activestorage/pull/86
|