It seems that the node_title_list() function got left behind when node titles were converted to fields in #557292: TF #3: Convert node title to fields.

We now have the odd situation where if you pass a fully-loaded node into this function (i.e., one that came from the Drupal API, via node_load_multiple) then it doesn't work correctly, because it expects $node->title to be a string whereas it's actually an array. It basically only works correctly when it is passed the result of a database query that pulled the node title from {node} directly.

I don't particularly like the fix in the attached patch, but at least it's a start.

CommentFileSizeAuthor
#4 node-title-list-621416-4.patch1009 bytespwolanin
#2 node-title-list-621416-2.patch1015 bytesDavid_Rothstein
node-title-list.patch1.02 KBDavid_Rothstein

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

David_Rothstein’s picture

David_Rothstein commented
Status: Needs work » Needs review
StatusFileSize
new node-title-list-621416-2.patch1015 bytes

Let's try this one instead... Silly PHP :)

pwolanin’s picture

pwolanin commented
Status: Needs review » Needs work

I think FIELD_LANGUAGE_NONE became LANGUAGE_NONE?

pwolanin’s picture

pwolanin commented
Status: Needs work » Needs review
StatusFileSize
new node-title-list-621416-4.patch1009 bytes
David_Rothstein’s picture

David_Rothstein commented
Status: Needs review » Fixed

Node title is no longer a field, so this is "fixed" :)

Status: Fixed » Closed (fixed)

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