AJAX error when changing number of rows/columns or uploading csv file [#1277760]

Hello

First, thanks for this very helpful module.

I met an error since I updated Drupal Core to 7.8. Always work fine before.
It seems there is a conflict with the Locale module. If Locale is enabled, I can't modify the table structure or upload a file. I get a popup error (capture joined). I have this line in logs : "EntityMalformedException : Missing bundle property on entity of type node. dans entity_extract_ids() (ligne 7389 dans /var/aegir/platforms/drupal-7.8_build_20110901/includes/common.inc)."

If Locale is disabled, Tablefield works again...

Comment	File	Size	Author
#4	tablefield-d7.8_entitymalformed_exception-1277760-4.patch	1.53 KB	acrollet
#4
#2	D7.8_EntityMalformedException-1277760-2.patch	1.7 KB	Schnitzel
#2
#1	D7.8_EntityMalformedException-1277760-1.patch	1.51 KB	Thomas Bosviel
#1
	window-error.png	65.82 KB	Mitsuko

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Thomas Bosviel CreditAttribution: Thomas Bosviel commented 2 November 2011 at 15:38

Version:

7.x-2.0-beta4

» 7.x-2.x-dev

File	Size
D7.8_EntityMalformedException-1277760-1.patch	1.51 KB

Comment #2

Schnitzel CreditAttribution: Schnitzel commented 23 November 2011 at 13:13

File	Size
D7.8_EntityMalformedException-1277760-2.patch	1.7 KB

this patch works for the newest DEV Version

Comment #3

warddr CreditAttribution: warddr commented 7 December 2011 at 01:35

Version:

7.x-2.x-dev

» 7.x-2.0-beta5

I can confirm this patch works, and fixes the problem.

Comment #4

acrollet CreditAttribution: acrollet commented 13 January 2012 at 20:24

Status:

Active

» Reviewed & tested by the community

File	Size
tablefield-d7.8_entitymalformed_exception-1277760-4.patch	1.53 KB

Patch in #2 works for me as well, attaching it re-rolled with --relative.

Comment #5

acrollet CreditAttribution: acrollet commented 13 January 2012 at 20:26

Title:	Error window when changes number of rows/columns or upload csv file with Drupal 7.8 and location module enableb	» AJAX error when changing number of rows/columns or uploading csv file
Version:	7.x-2.0-beta5	» 7.x-2.x-dev

Changing issue title for accuracy, I experience this issue without the Locale (or location) module enabled, and the patch fixes the problem.

Comment #6

Kevin Hankens CreditAttribution: Kevin Hankens commented 13 January 2012 at 22:14

Status:

Reviewed & tested by the community

» Needs work

Thanks for the hard work on this. The error message no longer shows up and CSV import works well, but only for single-value fields. Multi-value fields don't cause the ajax error, but the CSV import fails silently.

I haven't had time to look further into why this breaks for multi-value fields, but that will need to be addressed before we go RC.

I committed this and released beta6. Any more help with the multi-value fields would be really appreciated!!