Steps to recreate bug:

Fresh Drupal 7.14 install with tablefield 7.x-2.0

Create content type with Image and Table fields - default settings:
- importantly, the table field must be BEFORE the image field in the node edit form (bug does not occur if table is after image...)

Create test content:
-Rebuild table with 10 rows and insert test data into first column
-Browse and upload test image
-Save node

Only 5 rows are saved in table.

Comments

upchuk’s picture

Issue summary: View changes
Status: Active » Needs review

I confirm this happens in 2.3 but seems to have gotten fixed in the latest dev release.

- importantly, the table field must be BEFORE the image field in the node edit form (bug does not occur if table is after image...)

It's not the order of the field that counts but the order of the operation. I put the table field at the very end and then simply uploaded the file after rebuilding the table. This still caused the rebuild to not be saved.

@Kevin, mind creating a new release?

Thanks!

vitalie’s picture

Version: 7.x-2.0 » 7.x-2.x-dev
Assigned: Unassigned » vitalie
StatusFileSize
new826 bytes

Issue still present in dev when creating new nodes. When editing existing nodes the issue is indeed fixed in dev. Please test the patch here for the case of creating new nodes.

Idea of the patch was suggested here: https://www.drupal.org/node/2052215#comment-9574799

upchuk’s picture

Hey @vitalie,

The patch you mention makes tablefield save the data you put in the table but changes to the table structure are not kept.

D

vitalie’s picture

Thx @Upchuk. I need help with reproducing what you describe. When I execute the steps described by @emilekott, I see that the patch fixes the problem.

upchuk’s picture

Sorry @vitalie,

The patch works. Thanks!

  • vitalie committed 698a4c5 on 7.x-2.x
    Issue #1673416 by vitalie, Upchuk, emilekott: Image upload interferes...
vitalie’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.