yup we still need this to support xml mapping

Author: jrushlow

src/Doctrine/EntityRegenerator.php

@@ -202,9 +202,9 @@ private function createClassManipulator(string $classPath): ClassSourceManipulat
             sourceCode: $this->fileManager->getFileContents($classPath),
             overwrite: $this->overwrite,
             // if properties need to be generated then, by definition,
-            // some non-annotation config is being used, and so, the
+            // some non-annotation config is being used (e.g. XML), and so, the
             // properties should not have annotations added to them
-            useAttributesForDoctrineMapping: true // We always want to use attributes?
+            useAttributesForDoctrineMapping: false
         );
     }