Project

General

Profile

Actions

Feature #105188

closed

Extbase DataMapper column UID for external database

Added by Maxime DAL FABBRO 4 months ago. Updated 12 days ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2024-10-03
Due date:
% Done:

0%

Estimated time:
PHP Version:
8.1
Tags:
Complexity:
easy
Sprint Focus:

Description

If you want to connect to an external database with Extbase Model you have to configure a table mapping between properties and fieldname (Extbase/Persistence/Classes.php).
But, to be able to use Extbase Model your table must have a UID fieldname because in typo3/sysext/extbase/Classes/Persistence/Generic/Mapper/DataMapper.php there is a direct reference to UID string name.
A possible solution would be to use the table mapping to map UID property to another fieldname (see attachment patch sample).


Files

Actions #1

Updated by Garvin Hicking 4 months ago

  • Tracker changed from Bug to Feature
  • TYPO3 Version deleted (11)
  • Is Regression deleted (No)

Thanks for sharing! I believe this could have some more side effects, as the getUid() method is very commonly used and "uid" is really an extbase convention.

If you want, you could try to contribute this patch (see https://docs.typo3.org/m/typo3/guide-contributionworkflow/main/en-us/ - or use the github PR feature) - that would run your patch through our CI Pipeline, and if tests are created to verify the behavior, this idea could be evaluated some more!

Actions #2

Updated by Georg Ringer 12 days ago

  • Status changed from New to Rejected

as pointed out by Garvin, this might lead to a lot of other issues.

if you want to use extbase you need to follow some requirements, having a uid is one of those. otherwise I suggest to skip the mapping and just use direct queries.

I reject the issue because nothing is planned to be changed here.

Actions

Also available in: Atom PDF