aboutsummaryrefslogtreecommitdiffstats
path: root/activestorage/app/models
Commit message (Collapse)AuthorAgeFilesLines
* Enable links in Active Storage docs [ci skip]Yoshiyuki Hirano2017-08-262-2/+2
|
* Fix link in doc for active storage [ci skip]Yoshiyuki Hirano2017-08-251-1/+1
|
* Add frozen string literal pragmaGeorge Claghorn2017-08-211-0/+2
|
* Encode Content-Disposition filenames according to RFC 2231George Claghorn2017-08-213-1/+39
| | | | Closes #30134.
* DRYGeorge Claghorn2017-08-201-1/+1
|
* Rename ActiveStorage::Filename#extname to extension_with_delimiterGeorge Claghorn2017-08-201-9/+11
|
* restores some double newlines deleted in ae87217Xavier Noria2017-08-152-0/+4
| | | | | | These were intentional, see https://github.com/rails/rails/pull/30061#issuecomment-320068368
* minor tweaks in Active Storage after a walkthroughXavier Noria2017-08-154-29/+27
|
* Use frozen string literal in Active StorageKoichi ITO2017-08-125-0/+10
|
* Fix formatting of Active Storage docs [ci skip]Koichi ITO2017-08-114-30/+30
| | | | Follow up of #30188.
* Fix a typoqsona2017-08-101-1/+1
|
* Use `content_type.start_with?("...")` than `content_type =~ /^.../`Ryuta Kamizono2017-08-051-4/+4
| | | | | `start_with?` is a little faster than regexp for prefix matching by a fixed string.
* Merge pull request #30061 from claudiob/ast-rubocopRafael França2017-08-032-2/+2
|\ | | | | Make Rubocop happy about ActiveStorage
| * Make Rubocop happier about ActiveStorageclaudiob2017-08-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | 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)
* | Do not eager load ActiveRecord::BaseRafael Mendonça França2017-08-032-7/+0
| | | | | | | | Everything inside the app directory of a engine is autoload/eager loaded automatically so we don't need to require them.
* | There is no reason to single line methods hereRafael Mendonça França2017-08-031-4/+12
|/ | | | | I know those methods are unlikely to change but having one line method is hard to read and also hard to modify.
* Remove circular dependencyDavid Heinemeier Hansson2017-08-011-2/+0
|
* Add 'activestorage/' from commit '3f4a7218a4a4923a0e7ce1b2eb0d2888ce30da58'Rafael Mendonça França2017-07-315-0/+406
git-subtree-dir: activestorage git-subtree-mainline: 0d58e7e478e79c2d6b2a39a4444d2a17a903b2a6 git-subtree-split: 3f4a7218a4a4923a0e7ce1b2eb0d2888ce30da58