Hi,
the feature title as field (#557292: TF #3: Convert node title to fields) committed to HEAD breaks devel generate.

CommentFileSizeAuthor
#1 devel-603266-1.patch3.39 KBpeximo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

peximo’s picture

Status: Active » Needs review
FileSize
3.39 KB

This patch fix the problem.

moshe weitzman’s picture

Status: Needs review » Fixed

committed. thx.

salvis’s picture

Status: Fixed » Active

This patch replaced $node->title with $node->title[FIELD_LANGUAGE_NONE][0]['value'], which worked for a few days, but now I find that $node->title reverted back to being a simple string. What gives?

peximo’s picture

Hi salvis,
I've the latest checkout and title is a field and I don't have any problem with node title and devel generate.
But I've find another error with devel_generate.module related to (cck) fields.
I'have opened another issue #607310: Devel generate field_info_instances() error for this problem.

salvis’s picture

Status: Active » Fixed

Actually, I found the problem in devel_node_access (which you patched, too). I don't know about devel_generate.

I updated from CVS 12h ago as well as just now, and the $node->title in devel_node_access is a string.

This is weird — let's set it back to fixed and see if we get other people saying the same thing. In the meantime, I can work fine after having removed the indexes.

Status: Fixed » Closed (fixed)

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

salvis’s picture

Status: Closed (fixed) » Fixed

agentrickard mailed me that $node->title[FIELD_LANGUAGE_NONE][0]['value'] wasn't working for him either.

I've reverted it to $node->title for devel_node_access.module.

salvis’s picture

Duh, I finally got to the bottom of this: depending on whether DNA's debug mode was enabled or not, the $node object was retrieved via node_load() or directly from the database. Sorry about all the confusion...

Status: Fixed » Closed (fixed)

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