Feature #28228

FLOW3 should be usable without Doctrine

Added by Christopher Hlubek about 4 years ago. Updated about 4 years ago.

Status:Resolved Start date:2011-07-15
Priority:Must have Due date:
Assigned To:Christopher Hlubek % Done:

100%

Category:-
Target version:-
PHP Version: Complexity:
Has patch:

Description

FLOW3 should work without the Doctrine package. We should try to move the code that relies on Doctrine to a separate package to make FLOW3 more modular.

If I don't active the Doctrine package in the current version I'll get an exception during compilation:

PHP Fatal error:  Class 'Doctrine\ORM\Query\TreeWalkerAdapter' not found in .../Packages/Framework/TYPO3.FLOW3/Classes/Persistence/Doctrine/CountWalker.php on line 24

As a general goal we should try to push FLOW3 more into a component based architecture (though we provide the user with a full-stack framework with the base distribution).

Associated revisions

Revision 7f648f8a
Added by Christopher Hlubek about 4 years ago

[BUGFIX] It should be possible to disable Doctrine integration

This change introduces a setting to control the integration of Doctrine
in the bootstrap of FLOW3. For non-Doctrine persistence backends no
Doctrine code should be invoked, otherwise warnings about missing
driver or driverClass options will occur. And there is no need to
do proxy code compilation for Doctrine if it's not used.

Change-Id: I0ae80d2106c758d12242805ed20becb696e75da8
Resolves: #28228

History

#1 Updated by Christopher Hlubek about 4 years ago

One note: It would be okay to bundle the Doctrine\Common package as it just provides general tools and datastructures. Though I still feel uncomfortable with the naming implications since it looks like we have this strong dependency on Doctrine.

#2 Updated by Mr. Hudson about 4 years ago

  • Status changed from New to Under Review

Patch set 2 of change I0ae80d2106c758d12242805ed20becb696e75da8 has been pushed to the review server.
It is available at http://review.typo3.org/3407

#3 Updated by Christopher Hlubek about 4 years ago

  • Assigned To set to Christopher Hlubek

#4 Updated by Mr. Hudson about 4 years ago

Patch set 3 of change I0ae80d2106c758d12242805ed20becb696e75da8 has been pushed to the review server.
It is available at http://review.typo3.org/3407

#5 Updated by Christopher Hlubek about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Resolved for now

Also available in: Atom PDF