Removing of Bucket.new_blob.

Opting to just use Blob() constructor where relevant.

Author: dhermes

docs/_components/storage-getting-started.rst

@@ -91,11 +91,10 @@ Python built-in ``object``.
 If you want to set some data, you just create a ``Blob`` inside your bucket
 and store your data inside the blob::
 
-  >>> blob = bucket.new_blob('greeting.txt')
+  >>> blob = storage.Blob('greeting.txt', bucket=bucket)
   >>> blob.upload_from_string('Hello world!')
 
-:func:`new_blob <gcloud.storage.bucket.Bucket.new_blob>` creates a
-:class:`Blob <gcloud.storage.blob.Blob>` object locally and
+This creates a :class:`Blob <gcloud.storage.blob.Blob>` object locally and
 :func:`upload_from_string <gcloud.storage.blob.Blob.upload_from_string>`
 allows you to put a string into the blob.
 
@@ -117,8 +116,7 @@ Then you can look at the file in a terminal::
 And what about when you're not dealing with text?
 That's pretty simple too::
 
-  >>> blob = bucket.new_blob('kitten.jpg')
-  >>> blob.upload_from_filename('kitten.jpg')
+  >>> blob = bucket.upload_file('kitten.jpg')
 
 And to test whether it worked?
 
@@ -136,9 +134,9 @@ remotely.  If the blob doesn't exist, it will return ``None``.
 .. note:: ``get_blob`` is **not** retrieving the entire object's data.
 
 If you want to "get-or-create" the blob (that is, overwrite it if it
-already exists), you can use :func:`new_blob
-<gcloud.storage.bucket.Bucket.new_blob>`.  However, keep in mind, the blob
-is not created until you store some data inside of it.
+already exists), you can just construct a :class:`gcloud.storage.blob.Blob`
+locally and update it.  However, keep in mind, the blob is not created
+remotely until you store some data inside of it.
 
 If you want to check whether a blob exists, you can use the ``in`` operator
 in Python::

docs/_components/storage-quickstart.rst

@@ -61,7 +61,7 @@ you can create buckets and blobs::
   >>> bucket = storage.create_bucket('my-new-bucket', connection=connection)
   >>> print bucket
   <Bucket: my-new-bucket>
-  >>> blob = bucket.new_blob('my-test-file.txt')
+  >>> blob = storage.Blob('my-test-file.txt', bucket=bucket)
   >>> print blob
   <Blob: my-new-bucket, my-test-file.txt>
   >>> blob = blob.upload_from_string('this is test content!')

docs/index.rst

@@ -49,5 +49,5 @@ Cloud Storage
   from gcloud import storage
   storage.set_defaults()
   bucket = storage.get_bucket('<your-bucket-name>')
-  blob = bucket.new_blob('my-test-file.txt')
+  blob = storage.Blob('my-test-file.txt', bucket=bucket)
   blob = blob.upload_contents_from_string('this is test content!')

gcloud/storage/bucket.py

@@ -241,27 +241,6 @@ def iterator(self, prefix=None, delimiter=None, max_results=None,
 
         return self._iterator_class(self, extra_params=extra_params)
 
-    def new_blob(self, blob):
-        """Given path name (or Blob), return a :class:`Blob` object.
-
-        This is really useful when you're not sure if you have a ``Blob``
-        instance or a string path name.  Given either of those types, this
-        returns the corresponding ``Blob``.
-
-        :type blob: string or :class:`gcloud.storage.blob.Blob`
-        :param blob: A path name or actual blob object.
-
-        :rtype: :class:`gcloud.storage.blob.Blob`
-        :returns: A Blob object with the path provided.
-        """
-        if isinstance(blob, Blob):
-            return blob
-
-        if isinstance(blob, six.string_types):
-            return Blob(bucket=self, name=blob)
-
-        raise TypeError('Invalid blob: %s' % blob)
-
     def delete(self, force=False):
         """Delete this bucket.
 
@@ -379,12 +358,12 @@ def copy_blob(self, blob, destination_bucket, new_name=None):
         """
         if new_name is None:
             new_name = blob.name
-        new_blob = destination_bucket.new_blob(new_name)
+        new_blob = Blob(bucket=destination_bucket, name=new_name)
         api_path = blob.path + '/copyTo' + new_blob.path
         self.connection.api_request(method='POST', path=api_path)
         return new_blob
 
-    def upload_file(self, filename, blob=None):
+    def upload_file(self, filename, blob_name=None):
         """Shortcut method to upload a file into this bucket.
 
         Use this method to quickly put a local file in Cloud Storage.
@@ -398,9 +377,8 @@ def upload_file(self, filename, blob=None):
           >>> print bucket.get_all_blobs()
           [<Blob: my-bucket, remote-text-file.txt>]
 
-        If you don't provide a blob value, we will try to upload the file
-        using the local filename as the blob (**not** the complete
-        path)::
+        If you don't provide a blob name, we will try to upload the file
+        using the local filename (**not** the complete path)::
 
           >>> from gcloud import storage
           >>> connection = storage.get_connection(project)
@@ -412,22 +390,22 @@ def upload_file(self, filename, blob=None):
         :type filename: string
         :param filename: Local path to the file you want to upload.
 
-        :type blob: string or :class:`gcloud.storage.blob.Blob`
-        :param blob: The blob (either an object or a remote path) of where
-                     to put the file.  If this is blank, we will try to
-                     upload the file to the root of the bucket with the
-                     same name as on your local file system.
+        :type blob_name: string
+        :param blob_name: The name of the blob to upload the file to. If this
+                          is blank, we will try to upload the file to the root
+                          of the bucket with the same name as on your local
+                          file system.
 
         :rtype: :class:`Blob`
         :returns: The updated Blob object.
         """
-        if blob is None:
-            blob = os.path.basename(filename)
-        blob = self.new_blob(blob)
+        if blob_name is None:
+            blob_name = os.path.basename(filename)
+        blob = Blob(bucket=self, name=blob_name)
         blob.upload_from_filename(filename)
         return blob
 
-    def upload_file_object(self, file_obj, blob=None):
+    def upload_file_object(self, file_obj, blob_name=None):
         """Shortcut method to upload a file object into this bucket.
 
         Use this method to quickly put a local file in Cloud Storage.
@@ -441,9 +419,8 @@ def upload_file_object(self, file_obj, blob=None):
           >>> print bucket.get_all_blobs()
           [<Blob: my-bucket, remote-text-file.txt>]
 
-        If you don't provide a blob value, we will try to upload the file
-        using the local filename as the blob (**not** the complete
-        path)::
+        If you don't provide a blob name, we will try to upload the file
+        using the local filename (**not** the complete path)::
 
           >>> from gcloud import storage
           >>> connection = storage.get_connection(project)
@@ -455,19 +432,18 @@ def upload_file_object(self, file_obj, blob=None):
         :type file_obj: file
         :param file_obj: A file handle open for reading.
 
-        :type blob: string or :class:`gcloud.storage.blob.Blob`
-        :param blob: The blob (either an object or a remote path) of where
-                     to put the file.  If this is blank, we will try to
-                     upload the file to the root of the bucket with the
-                     same name as on your local file system.
+        :type blob_name: string
+        :param blob_name: The name of the blob to upload the file to. If this
+                          is blank, we will try to upload the file to the root
+                          of the bucket with the same name as on your local
+                          file system.
 
         :rtype: :class:`Blob`
         :returns: The updated Blob object.
         """
-        if blob:
-            blob = self.new_blob(blob)
-        else:
-            blob = self.new_blob(os.path.basename(file_obj.name))
+        if blob_name is None:
+            blob_name = os.path.basename(file_obj.name)
+        blob = Blob(bucket=self, name=blob_name)
         blob.upload_from_file(file_obj)
         return blob
 

gcloud/storage/demo/demo.py

@@ -24,7 +24,7 @@
 print(list(storage.get_all_buckets(connection)))
 
 # How about we create a new blob inside this bucket.
-blob = bucket.new_blob("my-new-file.txt")
+blob = storage.Blob("my-new-file.txt", bucket=bucket)
 
 # Now let's put some data in there.
 blob.upload_from_string("this is some data!")

gcloud/storage/test_bucket.py

@@ -306,19 +306,6 @@ def test_iterator_explicit(self):
         self.assertEqual(kw['path'], '/b/%s/o' % NAME)
         self.assertEqual(kw['query_params'], EXPECTED)
 
-    def test_new_blob_str(self):
-        from gcloud.storage.blob import Blob
-        BLOB_NAME = 'blob-name'
-        bucket = self._makeOne()
-        blob = bucket.new_blob(BLOB_NAME)
-        self.assertTrue(isinstance(blob, Blob))
-        self.assertTrue(blob.bucket is bucket)
-        self.assertEqual(blob.name, BLOB_NAME)
-
-    def test_new_blob_invalid(self):
-        bucket = self._makeOne()
-        self.assertRaises(TypeError, bucket.new_blob, object())
-
     def test_delete_default_miss(self):
         from gcloud.exceptions import NotFound
         NAME = 'name'