doctrine · greg0ire · Oct 11, 2023 · Oct 9, 2023 · Oct 10, 2023 · Oct 10, 2023
diff --git a/UPGRADE.md b/UPGRADE.md
@@ -1,5 +1,13 @@
 # Upgrade to 3.0
 
+## BC BREAK: Partial objects are removed
+
+- The `PARTIAL` keyword in DQL no longer exists.
+- `Doctrine\ORM\Query\AST\PartialObjectExpression`is removed.
+- `Doctrine\ORM\Query\SqlWalker::HINT_PARTIAL` and
+  `Doctrine\ORM\Query::HINT_FORCE_PARTIAL_LOAD` are removed.
+- `Doctrine\ORM\EntityManager*::getPartialReference()` is removed.
+
 ## BC BREAK: `Doctrine\ORM\Persister\Entity\EntityPersister::executeInserts()` return type changed to `void`
 
 Implementors should adapt to the new signature, and should call

diff --git a/docs/en/index.rst b/docs/en/index.rst
@@ -73,7 +73,6 @@ Advanced Topics
 * :doc:`TypedFieldMapper <reference/typedfieldmapper>`
 * :doc:`Improving Performance <reference/improving-performance>`
 * :doc:`Caching <reference/caching>`
-* :doc:`Partial Objects <reference/partial-objects>`
 * :doc:`Change Tracking Policies <reference/change-tracking-policies>`
 * :doc:`Best Practices <reference/best-practices>`
 * :doc:`Metadata Drivers <reference/metadata-drivers>`

diff --git a/docs/en/reference/dql-doctrine-query-language.rst b/docs/en/reference/dql-doctrine-query-language.rst
@@ -518,33 +518,6 @@ when the DQL is switched to an arbitrary join.
     - HAVING is applied to the results of a query after
       aggregation (GROUP BY)
 
-
-Partial Object Syntax
-^^^^^^^^^^^^^^^^^^^^^
-
-By default when you run a DQL query in Doctrine and select only a
-subset of the fields for a given entity, you do not receive objects
-back. Instead, you receive only arrays as a flat rectangular result
-set, similar to how you would if you were just using SQL directly
-and joining some data.
-
-If you want to select partial objects you can use the ``partial``
-DQL keyword:
-
-.. code-block:: php
-
-    <?php
-    $query = $em->createQuery('SELECT partial u.{id, username} FROM CmsUser u');
-    $users = $query->getResult(); // array of partially loaded CmsUser objects
-
-You use the partial syntax when joining as well:
-
-.. code-block:: php
-
-    <?php
-    $query = $em->createQuery('SELECT partial u.{id, username}, partial a.{id, name} FROM CmsUser u JOIN u.articles a');
-    $users = $query->getResult(); // array of partially loaded CmsUser objects
-
 "NEW" Operator Syntax
 ^^^^^^^^^^^^^^^^^^^^^
 
@@ -1417,15 +1390,6 @@ exist mostly internal query hints that are not be consumed in
 userland. However the following few hints are to be used in
 userland:
 
-
--  ``Query::HINT_FORCE_PARTIAL_LOAD`` - Allows to hydrate objects
-   although not all their columns are fetched. This query hint can be
-   used to handle memory consumption problems with large result-sets
-   that contain char or binary data. Doctrine has no way of implicitly
-   reloading this data. Partially loaded objects have to be passed to
-   ``EntityManager::refresh()`` if they are to be reloaded fully from
-   the database. This query hint is deprecated and will be removed
-   in the future (\ `Details <https://github.com/doctrine/orm/issues/8471>`_)
 -  ``Query::HINT_REFRESH`` - This query is used internally by
    ``EntityManager::refresh()`` and can be used in userland as well.
    If you specify this hint and a query returns the data for an entity
@@ -1678,10 +1642,8 @@ Select Expressions
 
 .. code-block:: php
 
-    SelectExpression        ::= (IdentificationVariable | ScalarExpression | AggregateExpression | FunctionDeclaration | PartialObjectExpression | "(" Subselect ")" | CaseExpression | NewObjectExpression) [["AS"] ["HIDDEN"] AliasResultVariable]
+    SelectExpression        ::= (IdentificationVariable | ScalarExpression | AggregateExpression | FunctionDeclaration | "(" Subselect ")" | CaseExpression | NewObjectExpression) [["AS"] ["HIDDEN"] AliasResultVariable]
     SimpleSelectExpression  ::= (StateFieldPathExpression | IdentificationVariable | FunctionDeclaration | AggregateExpression | "(" Subselect ")" | ScalarExpression) [["AS"] AliasResultVariable]
-    PartialObjectExpression ::= "PARTIAL" IdentificationVariable "." PartialFieldSet
-    PartialFieldSet         ::= "{" SimpleStateField {"," SimpleStateField}* "}"
     NewObjectExpression     ::= "NEW" AbstractSchemaName "(" NewObjectArg {"," NewObjectArg}* ")"
     NewObjectArg            ::= ScalarExpression | "(" Subselect ")"
 

diff --git a/docs/en/reference/partial-objects.rst b/docs/en/reference/partial-objects.rst
diff --git a/docs/en/sidebar.rst b/docs/en/sidebar.rst
@@ -40,7 +40,6 @@
       reference/query-builder
       reference/native-sql
       reference/change-tracking-policies
-      reference/partial-objects
       reference/attributes-reference
       reference/xml-mapping
       reference/php-mapping

diff --git a/lib/Doctrine/ORM/Cache/DefaultQueryCache.php b/lib/Doctrine/ORM/Cache/DefaultQueryCache.php
@@ -16,7 +16,6 @@
 use Doctrine\ORM\PersistentCollection;
 use Doctrine\ORM\Query;
 use Doctrine\ORM\Query\ResultSetMapping;
-use Doctrine\ORM\Query\SqlWalker;
 use Doctrine\ORM\UnitOfWork;
 
 use function array_map;
@@ -211,10 +210,6 @@ public function put(QueryCacheKey $key, ResultSetMapping $rsm, mixed $result, ar
             throw FeatureNotImplemented::nonSelectStatements();
         }
 
-        if (($hints[SqlWalker::HINT_PARTIAL] ?? false) === true || ($hints[Query::HINT_FORCE_PARTIAL_LOAD] ?? false) === true) {
-            throw FeatureNotImplemented::partialEntities();
-        }
-
         if (! ($key->cacheMode & Cache::MODE_PUT)) {
             return false;
         }

diff --git a/lib/Doctrine/ORM/Cache/Exception/FeatureNotImplemented.php b/lib/Doctrine/ORM/Cache/Exception/FeatureNotImplemented.php
@@ -20,9 +20,4 @@ public static function nonSelectStatements(): self
     {
         return new self('Second-level cache query supports only select statements.');
     }
-
-    public static function partialEntities(): self
-    {
-        return new self('Second level cache does not support partial entities.');
-    }
 }
diff --git a/lib/Doctrine/ORM/Decorator/EntityManagerDecorator.php b/lib/Doctrine/ORM/Decorator/EntityManagerDecorator.php
@@ -8,7 +8,6 @@
 use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\LockMode;
-use Doctrine\Deprecations\Deprecation;
 use Doctrine\ORM\Cache;
 use Doctrine\ORM\Configuration;
 use Doctrine\ORM\EntityManagerInterface;
@@ -103,18 +102,6 @@ public function getReference(string $entityName, mixed $id): object|null
         return $this->wrapped->getReference($entityName, $id);
     }
 
-    public function getPartialReference(string $entityName, mixed $identifier): object|null
-    {
-        Deprecation::trigger(
-            'doctrine/orm',
-            'https://github.com/doctrine/orm/pull/10987',
-            'Method %s is deprecated and will be removed in 3.0.',
-            __METHOD__,
-        );
-
-        return $this->wrapped->getPartialReference($entityName, $identifier);
-    }
-
     public function close(): void
     {
         $this->wrapped->close();

diff --git a/lib/Doctrine/ORM/EntityManager.php b/lib/Doctrine/ORM/EntityManager.php
@@ -10,7 +10,6 @@
 use Doctrine\Common\Util\ClassUtils;
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\LockMode;
-use Doctrine\Deprecations\Deprecation;
 use Doctrine\ORM\Exception\EntityManagerClosed;
 use Doctrine\ORM\Exception\InvalidHydrationMode;
 use Doctrine\ORM\Exception\MissingIdentifierField;
@@ -401,37 +400,6 @@ public function getReference(string $entityName, mixed $id): object|null
         return $entity;
     }
 
-    public function getPartialReference(string $entityName, mixed $identifier): object|null
-    {
-        Deprecation::trigger(
-            'doctrine/orm',
-            'https://github.com/doctrine/orm/pull/10987',
-            'Method %s is deprecated and will be removed in 3.0.',
-            __METHOD__,
-        );
-        $class = $this->metadataFactory->getMetadataFor(ltrim($entityName, '\\'));
-
-        $entity = $this->unitOfWork->tryGetById($identifier, $class->rootEntityName);
-
-        // Check identity map first, if its already in there just return it.
-        if ($entity !== false) {
-            return $entity instanceof $class->name ? $entity : null;
-        }
-
-        if (! is_array($identifier)) {
-            $identifier = [$class->identifier[0] => $identifier];
-        }
-
-        $entity = $class->newInstance();
-
-        $class->setIdentifierValues($entity, $identifier);
-
-        $this->unitOfWork->registerManaged($entity, $identifier, []);
-        $this->unitOfWork->markReadOnly($entity);
-
-        return $entity;
-    }
-
     /**
      * Clears the EntityManager. All entities that are currently managed
      * by this EntityManager become detached.

diff --git a/lib/Doctrine/ORM/EntityManagerInterface.php b/lib/Doctrine/ORM/EntityManagerInterface.php
@@ -163,33 +163,6 @@ public function refresh(object $object, LockMode|int|null $lockMode = null): voi
      */
     public function getReference(string $entityName, mixed $id): object|null;
 
-    /**
-     * Gets a partial reference to the entity identified by the given type and identifier
-     * without actually loading it, if the entity is not yet loaded.
-     *
-     * The returned reference may be a partial object if the entity is not yet loaded/managed.
-     * If it is a partial object it will not initialize the rest of the entity state on access.
-     * Thus you can only ever safely access the identifier of an entity obtained through
-     * this method.
-     *
-     * The use-cases for partial references involve maintaining bidirectional associations
-     * without loading one side of the association or to update an entity without loading it.
-     * Note, however, that in the latter case the original (persistent) entity data will
-     * never be visible to the application (especially not event listeners) as it will
-     * never be loaded in the first place.
-     *
-     * @deprecated 2.7 This method is being removed from the ORM and won't have any replacement
-     *
-     * @param string $entityName The name of the entity type.
-     * @param mixed  $identifier The entity identifier.
-     * @psalm-param class-string<T> $entityName
-     *
-     * @psalm-return T|null
-     *
-     * @template T of object
-     */
-    public function getPartialReference(string $entityName, mixed $identifier): object|null;
-
     /**
      * Closes the EntityManager. All entities that are currently managed
      * by this EntityManager become detached. The EntityManager may no longer

diff --git a/lib/Doctrine/ORM/Query.php b/lib/Doctrine/ORM/Query.php
@@ -70,14 +70,6 @@ class Query extends AbstractQuery
      */
     public const HINT_REFRESH_ENTITY = 'doctrine.refresh.entity';
 
-    /**
-     * The forcePartialLoad query hint forces a particular query to return
-     * partial objects.
-     *
-     * @todo Rename: HINT_OPTIMIZE
-     */
-    public const HINT_FORCE_PARTIAL_LOAD = 'doctrine.forcePartialLoad';
-
     /**
      * The includeMetaColumns query hint causes meta columns like foreign keys and
      * discriminator columns to be selected and returned as part of the query result.

diff --git a/lib/Doctrine/ORM/Query/AST/PartialObjectExpression.php b/lib/Doctrine/ORM/Query/AST/PartialObjectExpression.php