Skip to content
This repository has been archived by the owner on May 6, 2021. It is now read-only.

CarrierWave::SanitizedFile cleanup #22

Open
richardkmichael opened this issue Feb 2, 2013 · 0 comments
Open

CarrierWave::SanitizedFile cleanup #22

richardkmichael opened this issue Feb 2, 2013 · 0 comments

Comments

@richardkmichael
Copy link
Owner

Re-factor move_to and copy_to, the cut and paste is bad.
https://github.com/jnicklas/carrierwave/blob/master/lib/carrierwave/sanitized_file.rb#L173

Perhaps test inside the file operation, instead of before?
https://github.com/jnicklas/carrierwave/blob/master/lib/carrierwave/uploader/cache.rb#L128

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant