For text fields (filtered text) along with "field_[name]_value" column there is also "field_[name]_format" column in database. I have to manually rename the "field_[name]_format" column.

Besides, everything works, so thanks for very needed module.

Comments

drupalnesia’s picture

Status: Active » Fixed

6.x-1.0-beta3 : Bug fix: formatted text-field contains format column name (reported by mki)

Status: Fixed » Closed (fixed)

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

eneko1907’s picture

Version: 6.x-1.0-beta2 » 6.x-1.0-beta13
Status: Closed (fixed) » Needs work

Hi, thanks for the module.

This bug does not seem to be 100% fixed. It may not matter, but it would seem that the fix proposed on version beta 3, created the "format" field OK.. and that field was made of type "longtext". I think it should be "int(10) unsigned", as the Input formats are encoded as integers 0,1,2 (in lieu of "Filtered HTML", PHP Code, Full HTML").

I retested this problem to be sure, [[BTW, a practice I recommend before changing the status of the issue to closed (i think there are intermediate statuses such as "checked by the community, needs review, etc"]]

I updated your nice cck_field_rename module to Beta13. Created a cck field of type "text" and widget "textarea". When configuring the field (last step) I check the "text processing" "Filtered text (user selects input format" box. This creates the "value" and "format" associated fields in the table...

| field_test_value | longtext | YES | | NULL | |
| field_test_format | int(10) unsigned | YES | | NULL |

The I renamed the field using the nice form you created. I change the field_test to field_testtest

this resulted in a "longtext" for the type "format", not int(10) unsigned.
| field_testtest_value | longtext | YES | | NULL |
| field_testtest_format | longtext | YES | | NULL |

Other than the wysiwyg editor and text area not showing properly, the rest seems unaffected.
I have to do the bookeeping manually (alter the field again).

cheers, inigo

eneko1907’s picture

i'll take a second look at the code for a fix, meanwhile, you can issue this statement if you care.

ALTER TABLE $yourcontenttypeorfieldtable CHANGE field_$target_format field_$target_format int(10) unsigned;

inigo

drupalnesia’s picture

1. I can not find how to disable the automatically issue closing, I will check later.
2. Thanks for bug finding, patched available now:

6.x-1.0-beta14 :
-------------------
New: support new field: email
Bug fix: format of texfield, thanks to eneko1907 (http://drupal.org/user/422339)

drupalnesia’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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