Title as field and devel generate problem

peximo - October 13, 2009 - 13:16
Project:Devel
Version:7.x-1.x-dev
Component:devel_generate
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

#1

peximo - October 13, 2009 - 13:17
Status:active» needs review

This patch fix the problem.

AttachmentSize
devel-603266-1.patch 3.39 KB

#2

moshe weitzman - October 13, 2009 - 16:56
Status:needs review» fixed

committed. thx.

#3

salvis - October 17, 2009 - 16:58
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?

#4

peximo - October 17, 2009 - 17:38

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.

#5

salvis - October 17, 2009 - 19:04
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.

#6

System Message - October 31, 2009 - 19:10
Status:fixed» closed

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

#7

salvis - November 5, 2009 - 22:05
Status:closed» 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.

#8

salvis - November 7, 2009 - 21:38

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...

#9

System Message - November 21, 2009 - 21:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.