Update documentation

Author: tanlisu

docs/includes/apiargs-MongoDBCollection-method-rename-param.yaml

@@ -2,10 +2,7 @@ arg_name: param
 name: $toCollectionName
 type: string
 description: |
-  The new name of the collection. If a full namespace is not
-  specified, the database of the original collection will be used. If the new
-  namespace specifies a different database, the command copies the collection
-  to the new database and drops the source collection.
+  The new name of the collection.
 interface: phpmethod
 operation: ~
 optional: false

docs/reference/method/MongoDBCollection-rename.txt

@@ -2,7 +2,7 @@
 MongoDB\\Collection::rename()
 =============================
 
-.. versionadded:: 1.9
+.. versionadded:: 1.10
 
 .. default-domain:: mongodb
 

docs/reference/method/MongoDBDatabase-renameCollection.txt

@@ -2,7 +2,7 @@
 MongoDB\\Database::renameCollection()
 =====================================
 
-.. versionadded:: 1.9
+.. versionadded:: 1.10
 
 .. default-domain:: mongodb
 

src/Collection.php

@@ -1009,9 +1009,9 @@ public function mapReduce(JavascriptInterface $map, JavascriptInterface $reduce,
      * Renames the collection.
      *
      * @see RenameCollection::__construct() for supported options
-     * @param string $toCollectionName New name of the collection
-     * @param string $toDatabaseName   New database name of the collection
-     * @param array  $options          Additional options
+     * @param string  $toCollectionName New name of the collection
+     * @param ?string $toDatabaseName   New database name of the collection. Defaults to the original database.
+     * @param array   $options          Additional options
      * @return array|object Command result document
      * @throws UnsupportedException if options are not supported by the selected server
      * @throws InvalidArgumentException for parameter/option parsing errors

src/Database.php

@@ -475,10 +475,10 @@ public function modifyCollection($collectionName, array $collectionOptions, arra
      * Rename a collection within this database.
      *
      * @see RenameCollection::__construct() for supported options
-     * @param string $fromCollectionName Collection name
-     * @param string $toCollectionName   New name of the collection
-     * @param string $toDatabaseName     New database name of the collection
-     * @param array  $options            Additional options
+     * @param string  $fromCollectionName Collection name
+     * @param string  $toCollectionName   New name of the collection
+     * @param ?string $toDatabaseName     New database name of the collection. Defaults to the original database.
+     * @param array   $options            Additional options
      * @return array|object Command result document
      * @throws UnsupportedException if options are unsupported on the selected server
      * @throws InvalidArgumentException for parameter/option parsing errors

src/Operation/RenameCollection.php

@@ -78,7 +78,7 @@ class RenameCollection implements Executable
      * @param array  $options            Command options
      * @throws InvalidArgumentException for parameter/option parsing errors
      */
-    public function __construct($fromDatabaseName, $fromCollectionName, $toDatabaseName, $toCollectionName, array $options = [])
+    public function __construct(string $fromDatabaseName, string $fromCollectionName, string $toDatabaseName, string $toCollectionName, array $options = [])
     {
         if (isset($options['session']) && ! $options['session'] instanceof Session) {
             throw InvalidArgumentException::invalidType('"session" option', $options['session'], Session::class);
@@ -100,8 +100,8 @@ public function __construct($fromDatabaseName, $fromCollectionName, $toDatabaseN
             throw InvalidArgumentException::invalidType('"dropTarget" option', $options['dropTarget'], 'boolean');
         }
 
-        $this->fromNamespace = (string) $fromDatabaseName . '.' . (string) $fromCollectionName;
-        $this->toNamespace = (string) $toDatabaseName . '.' . (string) $toCollectionName;
+        $this->fromNamespace = $fromDatabaseName . '.' . $fromCollectionName;
+        $this->toNamespace = $toDatabaseName . '.' . $toCollectionName;
         $this->options = $options;
     }
 

tests/FunctionalTestCase.php

@@ -186,8 +186,6 @@ protected function assertCollectionDoesNotExist(string $collectionName, ?string
      * the given name is found, it will be passed to the callback, which may
      * perform additional assertions.
      *
-     * @param string   $collectionName
-     * @param string   $databaseName   Defaults to TestCase::getDatabaseName()
      * @param callable $callback
      */
     protected function assertCollectionExists(string $collectionName, ?string $databaseName = null, ?callable $callback = null): void