Suggestion #52846: Add isConnected() to Persistence interface - TYPO3 Flow Base Distribution - TYPO3 Forge

TYPO3 Flow Base Distribution TYPO3.Eel TYPO3.Flow TYPO3.Fluid TYPO3.Kickstart TYPO3.Welcome Packages Applications

Suggestion #52846

Add isConnected() to Persistence interface

Added by Rens Admiraal almost 2 years ago. Updated about 1 year ago.

Status:

Resolved

Start date:

2013-10-15

Priority:

Could have

Due date:

Assigned To:

% Done:

100%

Category:

Target version:

Description

Change https://review.typo3.org/#/c/24609 introduced and isConnected() method to the roleRepository to verify if the repository can actually be used for fetching and storing data. In the comments of the change it was suggested to move this method up in the class hierarchy and make it api. As this is still not fully decided and the bugfix has to be merged we will push this later on based on this ticket.

Basically the following should be decided:

Should it be api
Should it be added to an interface
To which interface

When this method is moved up it should of course be removed from the roleRepository

Associated revisions

Revision 2df9c7cf
Added by Andreas Förthner almost 2 years ago

[TASK] Move isConnected from role repository to persistence manager

To make isConnected api, we have to move it to some central
place. The persistence manager seems to be a lot better place,
than the persistence repository, as it is not reference as much
in user code.

Resolves: #52846
Releases: master
Change-Id: Ia28daa34e69beb41eb7a1e07f03e251b27dd13c9