Merge branch 'master' of git@github.com:geemus/fog

lib/fog/aws/models/s3/bucket.rb

@@ -22,11 +22,9 @@ module Fog
         end
 
         def location
-          @location ||= begin
           data = s3.get_bucket_location(name)
           data.body['LocationConstraint']
         end
-        end
 
         def objects
           Fog::AWS::S3::Objects.new(
@@ -36,11 +34,9 @@ module Fog
         end
 
         def payer
-          @payer ||= begin
           data = connection.get_request_payment(name)
           data.body['Payer']
         end
-        end
 
         def payer=(new_payer)
           connection.put_request_payment(name, new_payer)

lib/fog/aws/models/s3/object.rb

@@ -6,6 +6,7 @@ module Fog
 
         attr_accessor :body,
           :content_length,
+          :content_type,
           :etag,
           :key,
           :last_modified,
@@ -26,14 +27,6 @@ module Fog
           super
         end
 
-        def body
-          @body ||= get.body
-        end
-
-        def content_length
-          @content_length ||= head.content_length
-        end
-
         def copy(target_bucket_name, target_object_key)
           data = connection.copy_object(bucket.name, key, target_bucket_name, target_object_key).body
           copy = self.dup
@@ -51,14 +44,6 @@ module Fog
           connection.delete_object(bucket, key)
         end
 
-        def etag
-          @etag ||= head.etag
-        end
-
-        def last_modified
-          @last_modified ||= head.last_modified
-        end
-
         def save(options = {})
           data = connection.put_object(bucket.name, key, body, options)
           @etag = data.headers['ETag']
@@ -66,28 +51,6 @@ module Fog
 
         private
 
-        def get
-          data = connection.get_object(bucket.name, key, options)
-          object_data = { :body => data.body}
-          for key, value in data.headers
-            if ['Content-Length', 'ETag', 'Last-Modified'].include?(key)
-              object_data[key] = value
-            end
-          end
-          update_attributes(object_data)
-        end
-
-        def head
-          data = connection.head_object(bucket.name, key, options)
-          object_data = {}
-          for key, value in data.headers
-            if ['Content-Length', 'ETag', 'Last-Modified'].include?(key)
-              object_data[key] = value
-            end
-          end
-          update_attributes(object_data)
-        end
-
         def bucket=(new_bucket)
           @bucket = new_bucket
         end

lib/fog/aws/models/s3/objects.rb

@@ -48,6 +48,34 @@ module Fog
           object
         end
 
+        def get
+          data = connection.get_object(bucket.name, key, options)
+          object_data = { :body => data.body}
+          for key, value in data.headers
+            if ['Content-Length', 'Content-Type', 'ETag', 'Last-Modified'].include?(key)
+              object_data[key] = value
+            end
+          end
+          Fog::AWS::S3::Object.new({
+            :bucket     => bucket,
+            :connection => connection
+          }.merge!(object_data))
+        end
+
+        def head
+          data = connection.head_object(bucket.name, key, options)
+          object_data = {}
+          for key, value in data.headers
+            if ['Content-Length', 'Content-Type', 'ETag', 'Last-Modified'].include?(key)
+              object_data[key] = value
+            end
+          end
+          Fog::AWS::S3::Object.new({
+            :bucket     => bucket,
+            :connection => connection
+          }.merge!(object_data))
+        end
+
         def new(attributes = {})
           Fog::AWS::S3::Object.new({
             :bucket     => bucket,