Drop deprecated features

Author: IonBazan

src/Configuration.php

@@ -374,17 +374,6 @@ public function getDefaultCommitOptions(): array
     /** @phpstan-param CommitOptions $defaultCommitOptions */
     public function setDefaultCommitOptions(array $defaultCommitOptions): void
     {
-        foreach (UnitOfWork::DEPRECATED_WRITE_OPTIONS as $deprecatedOption) {
-            if (array_key_exists($deprecatedOption, $defaultCommitOptions)) {
-                trigger_deprecation(
-                    'doctrine/mongodb-odm',
-                    '2.6',
-                    'The "%s" commit option used in the configuration is deprecated.',
-                    $deprecatedOption,
-                );
-            }
-        }
-
         $this->attributes['defaultCommitOptions'] = $defaultCommitOptions;
     }
 

src/Mapping/Annotations/EmbeddedDocument.php

@@ -12,12 +12,4 @@
 #[Attribute(Attribute::TARGET_CLASS)]
 final class EmbeddedDocument extends AbstractDocument
 {
-    /** @var Index[] */
-    public $indexes;
-
-    /** @param Index[] $indexes */
-    public function __construct(array $indexes = [])
-    {
-        $this->indexes = $indexes;
-    }
 }

src/Mapping/Annotations/File.php

@@ -21,9 +21,6 @@ final class File extends AbstractDocument
     /** @var string|null */
     public $repositoryClass;
 
-    /** @var Index[] */
-    public $indexes;
-
     /** @var bool bool */
     public $readOnly;
 
@@ -34,14 +31,12 @@ final class File extends AbstractDocument
     public $chunkSizeBytes;
 
     /**
-     * @param Index[]         $indexes
      * @param string|int|null $writeConcern
      */
     public function __construct(
         ?string $db = null,
         ?string $bucketName = null,
         ?string $repositoryClass = null,
-        array $indexes = [],
         bool $readOnly = false,
         ?string $shardKey = null,
         public $writeConcern = null,
@@ -50,7 +45,6 @@ public function __construct(
         $this->db              = $db;
         $this->bucketName      = $bucketName;
         $this->repositoryClass = $repositoryClass;
-        $this->indexes         = $indexes;
         $this->readOnly        = $readOnly;
         $this->shardKey        = $shardKey;
         $this->chunkSizeBytes  = $chunkSizeBytes;

src/Mapping/ClassMetadata.php

@@ -609,15 +609,6 @@ final class ClassMetadata implements BaseClassMetadata
      */
     public private(set) array $subClasses = [];
 
-    /**
-     * The ReflectionProperty instances of the mapped class.
-     *
-     * @deprecated Since 2.13, use $propertyAccessors instead.
-     *
-     * @var LegacyReflectionFields|array<ReflectionProperty>
-     */
-    public private(set) LegacyReflectionFields|array $reflFields = [];
-
     /** @var array<string, PropertyAccessors\PropertyAccessor> */
     public private(set) array $propertyAccessors = [];
 
@@ -819,7 +810,6 @@ public function __construct(string $documentName)
         $this->rootDocumentName  = $documentName;
         $this->reflectionService = new RuntimeReflectionService();
         $this->reflClass         = new ReflectionClass($documentName);
-        $this->reflFields        = new LegacyReflectionFields($this, $this->reflectionService);
         $this->setCollection($this->reflClass->getShortName());
         $this->instantiator = new Instantiator();
     }
@@ -1424,29 +1414,6 @@ public function isChangeTrackingDeferredImplicit(): bool
         return $this->changeTrackingPolicy === self::CHANGETRACKING_DEFERRED_IMPLICIT;
     }
 
-    /**
-     * Whether the change tracking policy of this class is "notify".
-     *
-     * @deprecated This method was deprecated in doctrine/mongodb-odm 2.4. Please use DEFERRED_EXPLICIT tracking
-     * policy and isChangeTrackingDeferredImplicit method to detect it.
-     */
-    public function isChangeTrackingNotify(): bool
-    {
-        return $this->changeTrackingPolicy === self::CHANGETRACKING_NOTIFY;
-    }
-
-    /**
-     * Gets the ReflectionProperties of the mapped class.
-     *
-     * @deprecated Since 2.13, use getPropertyAccessors() instead.
-     *
-     * @return array<ReflectionProperty>|LegacyReflectionFields
-     */
-    public function getReflectionProperties(): array|LegacyReflectionFields
-    {
-        return $this->reflFields;
-    }
-
     /**
      * Gets the ReflectionProperties of the mapped class.
      *
@@ -1457,16 +1424,6 @@ public function getPropertyAccessors(): array
         return $this->propertyAccessors;
     }
 
-    /**
-     * Gets a ReflectionProperty for a specific field of the mapped class.
-     *
-     * @deprecated Since 2.13, use getPropertyAccessor() instead.
-     */
-    public function getReflectionProperty(string $name): ReflectionProperty
-    {
-        return $this->reflFields[$name];
-    }
-
     public function getPropertyAccessor(string $name): PropertyAccessor|null
     {
         return $this->propertyAccessors[$name] ?? null;
@@ -2407,13 +2364,7 @@ public function mapField(array $mapping): array
         }
 
         if (isset($mapping['targetDocument']) && isset($mapping['discriminatorMap'])) {
-            trigger_deprecation(
-                'doctrine/mongodb-odm',
-                '2.2',
-                'Mapping both "targetDocument" and "discriminatorMap" on field "%s" in class "%s" is deprecated. Only one of them can be used at a time',
-                $mapping['fieldName'],
-                $this->name,
-            );
+            throw MappingException::targetDocumentCanNotBeCombinedWithDiscriminatorMap($this->name, $mapping['fieldName']);
         }
 
         if (isset($mapping['reference']) && $mapping['type'] === self::ONE) {
@@ -2535,7 +2486,6 @@ public function mapField(array $mapping): array
      *
      * Parts that are also NOT serialized because they cannot be properly unserialized:
      *      - reflClass (ReflectionClass)
-     *      - reflFields (ReflectionProperty array)
      *      - propertyAccessors (ReflectionProperty array)
      *
      * @return array The names of all the fields that should be serialized.
@@ -2646,7 +2596,6 @@ public function __wakeup(): void
         $this->reflClass         = new ReflectionClass($this->name);
         $this->instantiator      = new Instantiator();
         $this->reflectionService = new RuntimeReflectionService();
-        $this->reflFields        = new LegacyReflectionFields($this, $this->reflectionService);
 
         foreach ($this->fieldMappings as $field => $mapping) {
             $accessor = PropertyAccessorFactory::createPropertyAccessor($mapping['declared'] ?? $this->name, $field);

src/Mapping/ClassMetadataFactory.php

@@ -197,16 +197,6 @@ protected function doLoadMetadata(ClassMetadataInterface $class, ?ClassMetadataI
             Events::loadClassMetadata,
             new LoadClassMetadataEventArgs($class, $this->dm),
         );
-
-        // phpcs:ignore SlevomatCodingStandard.ControlStructures.EarlyExit.EarlyExitNotUsed
-        if ($class->isChangeTrackingNotify()) {
-            trigger_deprecation(
-                'doctrine/mongodb-odm',
-                '2.4',
-                'NOTIFY tracking policy used in class "%s" is deprecated. Please use DEFERRED_EXPLICIT instead.',
-                $class->name,
-            );
-        }
     }
 
     /**

src/Mapping/Driver/AttributeDriver.php

@@ -189,20 +189,6 @@ public function loadMetadataForClass($className, PersistenceClassMetadata $metad
             $metadata->setWriteConcern($documentAttribute->writeConcern);
         }
 
-        if (isset($documentAttribute->indexes) && count($documentAttribute->indexes)) {
-            trigger_deprecation(
-                'doctrine/mongodb-odm',
-                '2.2',
-                'The "indexes" parameter in the "%s" attribute for class "%s" is deprecated. Specify all "@Index" and "@UniqueIndex" attributes on the class.',
-                $documentAttribute::class,
-                $className,
-            );
-
-            foreach ($documentAttribute->indexes as $index) {
-                $this->addIndex($metadata, $index);
-            }
-        }
-
         if (! empty($documentAttribute->readOnly)) {
             $metadata->markReadOnly();
         }

src/Mapping/LegacyReflectionFields.php

@@ -232,6 +232,11 @@ public static function repositoryMethodCanNotBeCombinedWithSkipLimitAndSort(stri
         return new self(sprintf("'repositoryMethod' used on '%s' in class '%s' can not be combined with skip, limit or sort.", $fieldName, $className));
     }
 
+    public static function targetDocumentCanNotBeCombinedWithDiscriminatorMap(string $className, string $fieldName): self
+    {
+        return new self(sprintf("'targetDocument' used on '%s' in class '%s' can not be combined with 'discriminatorMap'.", $fieldName, $className));
+    }
+
     public static function xmlMappingFileInvalid(string $filename, string $errorDetails): self
     {
         return new self(sprintf("The mapping file %s is invalid: \n%s", $filename, $errorDetails));

src/Mapping/MappingException.php

@@ -174,12 +174,6 @@ private function changed(): void
         }
 
         $this->isDirty = true;
-
-        if (! $this->needsSchedulingForSynchronization() || $this->owner === null) {
-            return;
-        }
-
-        $this->uow->scheduleForSynchronization($this->owner);
     }
 
     public function isDirty(): bool
@@ -739,15 +733,6 @@ private function isOrphanRemovalEnabled(): bool
         return isset($this->mapping['reference']) && $this->mapping['isOwningSide'] && $this->mapping['orphanRemoval'];
     }
 
-    /**
-     * Checks whether collection owner needs to be scheduled for dirty change in case the collection is modified.
-     */
-    private function needsSchedulingForSynchronization(): bool
-    {
-        return $this->owner && isset($this->dm) && ! empty($this->mapping['isOwningSide'])
-            && $this->dm->getClassMetadata(get_class($this->owner))->isChangeTrackingNotify();
-    }
-
     /**
      * @phpstan-param Closure(TKey, T):bool $p
      *