Skip to content

M
mongo-php-library
Commit 6aa739c4 authored by Will Banfield's avatar Will Banfield Committed by Jeremy Mikola
Browse files
Options

PHPLIB-159: rename uses UpdateOne() per the spec

parent 2ca40c7d
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 6 deletions
src/GridFS/Bucket.php
View file @ 6aa739c4
...@@ -279,14 +279,10 @@ class Bucket ...@@ -279,14 +279,10 @@ class Bucket
public function rename(ObjectId $id, $newFilename) public function rename(ObjectId $id, $newFilename)
{ {
$filesCollection = $this->collectionsWrapper->getFilesCollection(); $filesCollection = $this->collectionsWrapper->getFilesCollection();
$file = $filesCollection->findOne(['_id' => $id]); $result = $filesCollection->updateOne(['_id' => $id], ['$set' => ['filename' => $newFilename]]);
if($result->getModifiedCount() == 0) {
if ($file === null) {
throw new GridFSFileNotFoundException($id, $this->collectionsWrapper->getFilesCollection()->getNameSpace()); throw new GridFSFileNotFoundException($id, $this->collectionsWrapper->getFilesCollection()->getNameSpace());
} }
$file->filename = $newFilename;
$filesCollection->replaceOne(['_id' => $id], $file);
} }
/** /**
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment