Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Notice: Undefined index: field_fixture in tablefield_field_widget_form() (line 237 of /var/www/test/sites/all/modules/contrib/tablefield/tablefield.module).
Notice: Undefined index: field_fixture in tablefield_field_widget_form() (line 312 of /var/www/test/sites/all/modules/contrib/tablefield/tablefield.module).
Notice: Undefined index: field_fixture in tablefield_field_widget_form() (line 313 of /var/www/test/sites/all/modules/contrib/tablefield/tablefield.module).
Comment | File | Size | Author |
---|---|---|---|
#21 | tablefield-rebuild_in_field_collection-1288510-21.patch | 6.51 KB | Sam152 |
Comments
Comment #1
ddrozdik CreditAttribution: ddrozdik commentedI will try to fix this problem, a bit later.
Comment #2
stevesmename CreditAttribution: stevesmename commentedSee applied patch, solves problem of rebuilding the table. Still having issues with importing of csv file.
Comment #3
Nexsoft CreditAttribution: Nexsoft commentedThanks a lot! There is still a problem while having multiple instances of a field_collection, can't find a way right now to solve it.
Comment #4
Taxoman CreditAttribution: Taxoman commentedCan some more people who have already tested this patch chime in here in case we can get it committed to the -dev branch?
Comment #5
Gr3fweN CreditAttribution: Gr3fweN commentedTried the patch, it kind of works but data that already exist got deleted :(
Also, it's only possible to rebuild tables once, if I do it twice, page reloads and I get ugly errors:
Notice: Undefined index: rebuild in tablefield_rationalize_table() (line 606 of /home/.../modules/tablefield/tablefield.module).
Notice: Undefined index: rebuild in tablefield_rationalize_table() (line 607 of /home/.../modules/tablefield/tablefield.module).
Notice: Undefined index: rebuild in tablefield_field_widget_form() (line 322 of /home/.../modules/tablefield/tablefield.module).
Notice: Undefined index: rebuild in tablefield_field_widget_form() (line 323 of /home/.../modules/tablefield/tablefield.module).
Havn't tried csv...
Comment #6
Taxoman CreditAttribution: Taxoman commentedComment #7
MostlyGhostly CreditAttribution: MostlyGhostly commentedI've applied the above patch, and don't really notice any difference, despite clearing cache, etc.
It returns:
Notice: Undefined index: colour in tablefield_field_widget_form() (line 238 of /home/.../sites/all/modules/tablefield/tablefield.module).
Notice: Undefined index: colour in tablefield_field_widget_form() (line 322 of /home/.../sites/all/modules/tablefield/tablefield.module).
Notice: Undefined index: colour in tablefield_field_widget_form() (line 323 of /home/.../sites/all/modules/tablefield/tablefield.module).
Followed by:
Status message Table structure rebuilt.
--
The result of this is the entire table disappearing. Clicking on "Show/Change number of rows and columns" shows the boxes to be empty. Entering anything in to this is fruitless, as they return empty anyway.
Interesting to see the difference in errors between myself and Gr3fweN though.
Comment #8
Gr3fweN CreditAttribution: Gr3fweN commentedAnyone looking into this? I tried but without success :(
Comment #9
jamsilver CreditAttribution: jamsilver commentedI have been trying to use a tablefield inside a referenced entity using inline_entity_form. Basically the tablefield module ajax code assumes that it will be on the top-level of the form when this is not always necessarily the case.
The attached patch works for my use case - and I've given it a quick test on tablefields in a field_collection too and it seems to work.
Comment #10
markhalliwellMarking #2027271: TableField module content does not display in Bootstrap as a dup of this issue.
Comment #11
PinoloThis patch is based on #9, applies to latest 7.x-2.x branch and hopefully 7.x-2.2
Comment #12
dahousecat CreditAttribution: dahousecat commentedJust used the patch from #11 and solved the issue for me - the rebuild table function now works fine in a field collection. Thanks.
The only remaining issue is after having re-built the table I am no longer able to edit any of the default values in the table. Where there were inputs there are now just labels.
Also any text I entered before re-building the table is lost after the table is re-built.
This patch still makes the module usable (with field_collection) but is by no means perfect...
Comment #13
dragonfire353 CreditAttribution: dragonfire353 commentedThe patch did not work for me, well completely. It made it possible at least but I have to jump through hoops; I have to try rebuilding the one that isn't last, then rebuild the last one, which will actually rebuild the one I just tried rebuilding, very odd. Same thing for csv. Have yet to be able to fix this problem as I'm still new to drupal development..
Also, if anyone else is having trouble with it not importing right and deal with macs, take a look at this: http://us3.php.net/manual/en/function.fgetcsv.php#87196
My work uses all macs and that solved it not importing correctly at least... wish I could figure out the other problem though...
Comment #14
Alexey.y CreditAttribution: Alexey.y commented#13 I have the same problem. When i rebuild one table in field_collection others table changed to. I supplemented patch from #11
Comment #15
jaydee1818 CreditAttribution: jaydee1818 commentedThese patches fail with security update 7.23
Comment #16
jaydee1818 CreditAttribution: jaydee1818 commentedThis should work with 7.23
Comment #17
jaydee1818 CreditAttribution: jaydee1818 commentedSorry, ignore previous miss-fire ... this patch should work with 7.23
Comment #18
inakilz CreditAttribution: inakilz commentedI've tested the patch from #17 but it only works with the first table of a field collection.
Comment #19
inakilz CreditAttribution: inakilz commentedComment #20
tchopshop CreditAttribution: tchopshop commentedThe new patch on #17 is not working for me.
Comment #21
Sam152 CreditAttribution: Sam152 commented#17 was a bit messed up for me. I have rerolled #14 against 7.x-2.3 which is now working correctly for me.
Comment #22
sistro CreditAttribution: sistro commented#21 works for me too, tnx Sam152 ;)
This patch fix also the same issue with "multifield" module
Comment #23
johnny5th CreditAttribution: johnny5th commented#21 seems to be working for me too!
Comment #24
stevesmename CreditAttribution: stevesmename commentedComment #25
likewhoa CreditAttribution: likewhoa commentedworks for me.
Comment #26
dragonfire353 CreditAttribution: dragonfire353 commented#21 appeared to work for me at first, at least for rebuilding. But when saving it comes up blank. Importing csv doesn't work either, the first field collection item gets deleted and importing in the second one applies to the first one; although when you save it shows up for both.
Comment #27
dragonfire353 CreditAttribution: dragonfire353 commentedComment #28
mikeker CreditAttribution: mikeker commented+1 for #21 with regards to rebuild table working. I tested with single- and multi-valued Tablefields as well as Tablefields in Field Collections. The rebuild table works as expected.
However the import CSV option does not work for Tablefields in a collection as the
#name
on the import buttons is the same.In the interest of moving this issue along, I'll open another issue around the import problems.
Comment #29
mikeker CreditAttribution: mikeker commentedOpened #2387001: Import CSV option does not work with Field Collection module to track the import CSV issue.
Comment #30
dragonfire353 CreditAttribution: dragonfire353 commentedI can confirm the rebuilding works, something is messed up on my site causing it to not work correctly. Tested in a fresh environment, as I should have before, and it worked as expected.
Comment #31
vitalie CreditAttribution: vitalie commentedFixed as part of this issue: #2387001: Import CSV option does not work with Field Collection module
Comment #32
Tart0#21's patch work for me. Thanks !