Task #6249
closed
Support for TCA fields of type "group" - allowes DAM support (has patch)
Added by Franz Koch almost 15 years ago. Updated over 5 years ago.
70%
Description
Hi, extbase currently doesn't support TCA fields of type "group" natively. A easy workaround currently is to additionally define a "foreign_table" in your columns configuration together with a MM relation of course (no comma separated lists).
But unfortunately this little trick doesn't work together with DAM relations and probably some other multicolumn relations, because DAM itself is not configuring all MM_match_fields (tablenames missing by default) and is letting TYPO3 do the job of filling in the correct 'tablenames' match fields in MM relations. This patch is a first attempt to do in extbase the same as TYPO3 does in the backend. So with this patch you get:
- native support of group fields (type: group, interal_type: db, allowed: onlyOneTableNameSoFar)
- handling of multi-table relations like the CORE of TYPO3 does (adding tablenames if needed)
With this patch applied, DAM relations will work by default.
What will not work:
Retrieving/persisting objects to multiple tables for a single db field (so more then one allowed relation table, like the RECORDS CType of TYPO3 is using). To support this, some major rework is needed I think, that should/could be done together with singleTableInheritance.
Files
| extbase-SupportForTcaFieldType-Group.diff (9.33 KB) extbase-SupportForTcaFieldType-Group.diff | Franz Koch, 2010-01-27 15:18 | ||
| extbase-SupportForTcaFieldType-Group_v2.diff (10.9 KB) extbase-SupportForTcaFieldType-Group_v2.diff | Franz Koch, 2010-01-27 23:32 | ||
| Issue_6249_TcaGroupSupport.diff (18.1 KB) Issue_6249_TcaGroupSupport.diff | Franz Koch, 2011-01-21 17:09 |
Updated by Martin Kutschker almost 15 years ago
The issue is related to #6232 and #6233 as methods changed in these issues are also changed with this.
Updated by Franz Koch almost 15 years ago
- File extbase-SupportForTcaFieldType-Group_v2.diff extbase-SupportForTcaFieldType-Group_v2.diff added
version 2 of the patch, adding enhanced compatibility for group fields and incorporating the changes (drop of the check for 'foreign_label') from #6232 - so with this patch #6232 is obsolete.
Patch #6233 should still apply correctly.
Updated by Franz Koch over 14 years ago
ok - the patch won't apply anymore due to recent changes in extbase. I already merged the changes in my local svn version, but still need to extract a clean patch for this, whithout all my other adoptions ;) So a new patch is upcomming.
Updated by Jochen Rau over 14 years ago
- Assignee set to Jochen Rau
- Priority changed from Must have to Should have
- Target version changed from Extbase 1.1.0 to Extbase 1.2.0beta2
Updated by Jochen Rau over 14 years ago
- Target version changed from Extbase 1.2.0beta2 to Extbase 1.3.0beta1
Updated by Sebastian Kurfuerst almost 14 years ago
- Tracker changed from Feature to Story
- Target version changed from Extbase 1.3.0beta1 to Extbase 1.3.0
any updates here? That's definitely an important one.
Updated by Jochen Rau almost 14 years ago
- Assignee deleted (
Jochen Rau) - Priority changed from Should have to Must have
Sorry, no updates here. Due to the fact that this is a feature, it can't go into 1.3 anymore.
Updated by Jochen Rau almost 14 years ago
- Target version deleted (
Extbase 1.3.0) - Translation missing: en.field_position deleted (
5882) - Translation missing: en.field_position set to 5727
Updated by Jochen Rau almost 14 years ago
- Tracker changed from Story to Bug
- Parent task set to #10797
Updated by Sebastian Kurfuerst almost 14 years ago
- Target version changed from Extbase 1.3.0RC1 to Extbase 1.3.0
not possible to be included in 1.3.0RC1 anymore, that's why I move them to 1.3.0 for now.
Updated by Sebastian Kurfuerst almost 14 years ago
moving issues to 1.3.0, as they will not be fixed in RC1 anymore.
Updated by Franz Koch almost 14 years ago
- File Issue_6249_TcaGroupSupport.diff Issue_6249_TcaGroupSupport.diff added
- Assignee set to Franz Koch
- % Done changed from 50 to 80
- test for correct behavior when adding/removing MM relations
- tests with MM_opposite_field configurations in read and write processes.
This patch still doesn't add support for multiTable relations, but is already providing the basics. The "only" thing missing is to find a way on how to map the objects correctly - writing should already work correctly (MM-relation wise).
Updated by Franz Koch almost 14 years ago
- Status changed from Accepted to Under Review
Updated by Franz Koch almost 14 years ago
- Target version changed from Extbase 1.3.0 to Extbase 1.4
- % Done changed from 80 to 70
I did some further tests and noticed some remaining issues that have to be addressed in order to get full group field support. The patch itself seems to work (besides of one bug I found), but extbase is missing other features that have to be implemented first in order to have it work correctly (like relation type BELONGS_TO_MANY as far as I have seen). So it won't make it into 1.3.0 - sorry.
Updated by Thomas Deinhamer over 12 years ago
Updates on this? Is it still relevant?
Updated by Lorenz Ulrich over 12 years ago
Does this have a relation with http://bugs.typo3.org/view.php?id=16956 ?
Updated by Markus Günther over 12 years ago
Is this still relevant or could we close the issue?
Updated by Markus Günther over 12 years ago
- Status changed from Under Review to Needs Feedback
Updated by Thomas Deinhamer over 12 years ago
As far as I have experienced in a recent project
this does not work yet, so in my oppinion this is
still a relevant feature request.
So, not sure how up to date the patch is. I guess
it won't work with the current git master.
Furthermore there are some other
tickets related to the group field:
http://forge.typo3.org/issues/32549
http://forge.typo3.org/issues/23980
Updated by Franz Koch over 12 years ago
yes, having a look at current codebase this is still missing.
Updated by Markus Günther about 12 years ago
- Status changed from Needs Feedback to Accepted
Ok i create a list for me with issues like that and will have a look at this.
Updated by Helmut Hummel almost 12 years ago
- Status changed from Accepted to Needs Feedback
Franz Koch wrote:
yes, having a look at current codebase this is still missing.
Can you please add a full TCA configuration of what is not yet supported in extbase?
Updated by Franz Koch almost 12 years ago
AFAIK currently users still have to define a "foreign_table" in TCA in order to support fields of type "group". Also some special cases of the group type are not treated correctly. See attached patch v2 which should still be valid. It's unfortunately almost 3 years ago that I had this issue and debugged this issue for several hours.
Updated by Alexander Schnitzler over 11 years ago
- Target version changed from Extbase 1.4 to Extbase 6.2
Updated by Anja Leichsenring over 11 years ago
- Target version changed from Extbase 6.2 to Extbase 6.3
Updated by Alexander Opitz about 10 years ago
- Project changed from 534 to TYPO3 Core
- Category deleted (
Extbase: MVC) - Target version changed from Extbase 6.3 to 7.0
- TYPO3 Version set to 4.5
Hi Franz,
can you push your patch to Gerritt?
Updated by Franz Koch about 10 years ago
I highly doubt it'll apply easily on current version and I don't have the time to dig into Extbase internals for days again. I wrote this patch back when I was involved with Extbase development and knew my ways around in the persistence layer and everything, but now I'm out and I haven't been following the internal changes in the last year(s). So somebody else is probably better qualified.
Updated by Alexander Opitz about 10 years ago
- Status changed from Needs Feedback to New
- Assignee deleted (
Franz Koch)
Updated by Franz Koch about 10 years ago
please note that my patch was working fine back then and I refactored the implementation several times. So it should still be a good starting ground if somebody want's to pick it up.
Updated by Mathias Schreiber almost 10 years ago
- Target version changed from 7.0 to 7.1 (Cleanup)
Updated by Benni Mack over 9 years ago
- Target version changed from 7.1 (Cleanup) to 7.4 (Backend)
Updated by Susanne Moog over 9 years ago
- Target version changed from 7.4 (Backend) to 7.5
Updated by Susanne Moog over 5 years ago
- Is duplicate of Bug #62277: TCA > Config > type group - foreign_table necessary for extbase to work added
Updated by Susanne Moog over 5 years ago
- Status changed from New to Closed
As the patch does not apply anymore and the same topic is handled in #62277 I'm closing this task for now.