Closed (fixed)
Project:
TableField
Version:
7.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Sep 2011 at 07:35 UTC
Updated:
12 Nov 2015 at 11:15 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
ddrozdik commentedI will try to fix this problem, a bit later.
Comment #2
stevesmename commentedSee applied patch, solves problem of rebuilding the table. Still having issues with importing of csv file.
Comment #3
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 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 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 commentedComment #7
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 commentedAnyone looking into this? I tried but without success :(
Comment #9
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 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 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 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 commentedThese patches fail with security update 7.23
Comment #16
jaydee1818 commentedThis should work with 7.23
Comment #17
jaydee1818 commentedSorry, ignore previous miss-fire ... this patch should work with 7.23
Comment #18
inakilz commentedI've tested the patch from #17 but it only works with the first table of a field collection.
Comment #19
inakilz commentedComment #20
tchopshop commentedThe new patch on #17 is not working for me.
Comment #21
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 commented#21 works for me too, tnx Sam152 ;)
This patch fix also the same issue with "multifield" module
Comment #23
johnny5th commented#21 seems to be working for me too!
Comment #24
stevesmename commentedComment #25
likewhoa commentedworks for me.
Comment #26
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 commentedComment #28
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
#nameon 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 commentedOpened #2387001: Import CSV option does not work with Field Collection module to track the import CSV issue.
Comment #30
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 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 !