PHPLIB-158: delete removes files document first

Will Banfield · jmikola · commit 2ca40c7d7597 · 2016-03-14T10:58:40.000-04:00
diff --git a/src/GridFS/Bucket.php b/src/GridFS/Bucket.php
@@ -87,13 +87,13 @@ public function __construct(Manager $manager, $databaseName, array $options = []
     public function delete(ObjectId $id)
     {
         $file = $this->collectionsWrapper->getFilesCollection()->findOne(['_id' => $id]);
+        $this->collectionsWrapper->getFilesCollection()->deleteOne(['_id' => $id]);
         $this->collectionsWrapper->getChunksCollection()->deleteMany(['files_id' => $id]);
 
         if ($file === null) {
             throw new GridFSFileNotFoundException($id, $this->collectionsWrapper->getFilesCollection()->getNameSpace());
         }
 
-        $this->collectionsWrapper->getFilesCollection()->deleteOne(['_id' => $id]);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -87,13 +87,13 @@ public function __construct(Manager $manager, $databaseName, array $options = []`
`87`	`87`	`public function delete(ObjectId $id)`
`88`	`88`	`{`
`89`	`89`	`$file = $this->collectionsWrapper->getFilesCollection()->findOne(['_id' => $id]);`
	`90`	`+ $this->collectionsWrapper->getFilesCollection()->deleteOne(['_id' => $id]);`
`90`	`91`	`$this->collectionsWrapper->getChunksCollection()->deleteMany(['files_id' => $id]);`
`91`	`92`
`92`	`93`	`if ($file === null) {`
`93`	`94`	`throw new GridFSFileNotFoundException($id, $this->collectionsWrapper->getFilesCollection()->getNameSpace());`
`94`	`95`	`}`
`95`	`96`
`96`		`- $this->collectionsWrapper->getFilesCollection()->deleteOne(['_id' => $id]);`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`/**`