carrierwave-mongoid 0.1.1 → 0.1.2

data/README.md

@@ -1,33 +1,20 @@
 # CarrierWave for Mongoid
 
-This gem adds support for Mongoid and MongoDB's GridFS to CarrierWave, see the
-CarrierWave documentation for more detailed usage instructions.
+This gem adds support for Mongoid and MongoDB's GridFS to [CarrierWave](https://github.com/jnicklas/carrierwave/)
 
-### This version supports ONLY version of mongoid ~> 2.1
-Keep in mind that if you came up from previous versions you should make a few steps to go with it:
+This functionality used to be part of CarrierWave but has since been extracted into this gem.
 
-* change(rename in db) field name from `avatar_name` to `avatar`(appropriate to your uploader name)
-* fix you code where you need to operate with uploaded file's filename from `avatar` to `avatar_identifier`
-
-Install it like this:
+## Installation
 
     gem install carrierwave-mongoid
 
-Use it like this:
-
-```ruby
-require 'carrierwave/mongoid'
-```
-
-Make sure to disable auto_validation on the mounted column.
+## Requiring the gem
 
-Using bundler:
+    require 'carrierwave/mongoid'
 
-```ruby
-gem 'carrierwave-mongoid', :require => 'carrierwave/mongoid'
-```
+## Using Bundler
 
-This used to be part of CarrierWave but has been extracted.
+    gem 'carrierwave-mongoid', :require => 'carrierwave/mongoid'
 
 ## Using MongoDB's GridFS store
 
@@ -61,8 +48,26 @@ CarrierWave.configure do |config|
 end
 ```
 
-## Known issues/ limitations
+## Changes from earlier versions of CarrierWave with Mongoid support built in
+
+This version supports ONLY version of mongoid ~> 2.1
+
+You can use `upload_identifier` to retrieve the original name of the uploaded file.
+
+The default mount column used to be the name of the upload column plus  `_filename`. Now it is simply the name of the column. Most of the time, the column was called `upload`, so it would have been mounted to `upload_filename`.
+If you'd like to avoid a database migration, simply use the `:mount_on` option to specify
+the field name explicitly. Therefore, you only have to add a `_filename` to your column name. For example, if your column is called `:upload`:
+
+```ruby
+class Dokument
+  mount_uploader :upload, DokumentUploader, mount_on: :upload_filename
+end
+```
+
+## Known issues and limitations
+
+It is recommended that you disable Mongoid's auto_validation on the mounted column.
 
-If using Mongoid, note that embedded documents files aren't saved when parent documents are saved.
+Note that embedded documents files aren't saved when parent documents are saved.
 You must explicitly call save on embedded documents in order to save their attached files.
 You can read more about this [here](https://github.com/jnicklas/carrierwave/issues#issue/81)

data/lib/carrierwave/mongoid.rb

@@ -41,7 +41,8 @@ module CarrierWave
         def find_previous_model_for_#{column}
           if self.embedded?
             ancestors       = [[ self.metadata.key, self._parent ]].tap { |x| x.unshift([ x.first.last.metadata.key, x.first.last._parent ]) while x.first.last.embedded? }
-            reloaded_parent = ancestors.first.last.reload
+            first_parent = ancestors.first.last
+            reloaded_parent = first_parent.class.find(first_parent.to_key.first)
             ancestors.inject(reloaded_parent) { |parent,(key,ancestor)| (parent.is_a?(Array) ? parent.find(ancestor.to_key.first) : parent).send(key) }.find(to_key.first)
           else
             self.class.find(to_key.first)

data/lib/carrierwave/mongoid/version.rb

@@ -1,5 +1,5 @@
 module Carrierwave
   module Mongoid
-    VERSION = "0.1.1"
+    VERSION = "0.1.2"
   end
 end

data/spec/mongoid_spec.rb

@@ -450,6 +450,14 @@ describe CarrierWave::Mongoid do
         File.exists?(public_path('uploads/old.jpeg')).should be_true
       end
 
+      it "should not touch parent's dirty attributes" do
+        @class.field :title
+        @doc.title = "Title"
+        @embedded_doc.image = stub_file('new.jpeg')
+        @embedded_doc.save.should be_true
+        @doc.title.should == "Title"
+      end
+
       describe 'with double embedded documents' do
 
         before do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: carrierwave-mongoid
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 31
   prerelease: 
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - Jonas Nicklas
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-08-12 00:00:00 -05:00
+date: 2011-08-16 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency