Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
the feature title as field (#557292: TF #3: Convert node title to fields) committed to HEAD breaks devel generate.
Comment | File | Size | Author |
---|---|---|---|
#1 | devel-603266-1.patch | 3.39 KB | peximo |
Comments
Comment #1
peximo CreditAttribution: peximo commentedThis patch fix the problem.
Comment #2
moshe weitzman CreditAttribution: moshe weitzman commentedcommitted. thx.
Comment #3
salvisThis 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?Comment #4
peximo CreditAttribution: peximo commentedHi 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.
Comment #5
salvisActually, 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.
Comment #7
salvisagentrickard 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.Comment #8
salvisDuh, 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...