Very serious bug in table component: when you have several tables in one page, Drupal/Flexiblock will be show only one (I think, last). Reason (AFAIK):

$form['table'][... (line 22 in field_table.inc),

after

foreach($items as $key => $item) {
$form[$key] = $item;
}
(line about 580 in flexinode.module)

we have only one array with name 'table'.

I was changed this line in field_table.inc like this:

$form[$fieldname][$fieldname.'_rows'] = array( //заменим table на $fieldname...

and many other items from 'table' to $fieldname, it seems a slolution.

P. S. Sorry for bad English, it is my non-native language ;-)

Comments

Bèr Kessels’s picture

Category: bug » support

I do not understand your problem?. So I am making this a support request, untill we know what the prolem is.

* Are you using any non-released field .inc files?
* Did you follow the install instructions? Are you seeing problesm with non-existing database tables? Is that the problem?
* What page can we see the bug?
* What do we need to do to reproduce it?

Losik’s picture

>I do not understand your problem?
Hmmm... The problem: if you have 3 flexinode tables in one Web page you will see only one.

>* Are you using any non-released field .inc files?
No, the problem in contribution file, included in distributive (http://ftp.osuosl.org/pub/drupal/files/projects/flexinode-4.7.0.tar.gz)

>* Did you follow the install instructions? Are you seeing problesm with non-existing database tables? Is that the problem?
Yes, I follow all install instructions. Database tables is exist ;-)

>* What page can we see the bug?
Any page with Flexinode Table widget.

>What do we need to do to reproduce it?
1. Create Flexinode Content Type with name Test Table.
2. Add 3 tables (Table 1, Table 2, Table 3)
3. Create Content, Test Table.
4. See the page 8)

Losik’s picture

Addition: I test it on clean Drupal install, with last released version Flexinode.

Bèr Kessels’s picture

Title: Very serious bug in table component » Make table field ready for 4.7
Priority: Critical » Normal

Table field was not ready for 4.7. Whe are repairing the broken fields one at the time.

I am removing this for now.

Losik’s picture

Thanks.

P. S. Solution, which described in my first post in this thread seems is fully working.

Bèr Kessels’s picture

Category: support » bug
Status: Active » Needs work

Not a suppoort req. but a feature request: update table field inc to 4.7

However, we need a real patch: http://drupal.org/diffandpatch