As in the subject, this patch allows to see the node language on autocomplete results.

CommentFileSizeAuthor
#9 nodequeue-autocompletelang-1649172-9.patch2.09 KBjenlampton
#4 nodequeue-autocompletelang-1649172-1.patch1.99 KBpaolomainardi
#3 nodequeue-autocompletelang-1649172-1.patch1.99 KBpaolomainardi
#1 nodequeue-1649172-1.patch2.27 KBpaolomainardi

Comments

paolomainardi’s picture

paolomainardi commented
StatusFileSize
new nodequeue-1649172-1.patch2.27 KB

Status: Needs review » Needs work

The last submitted patch, nodequeue-1649172-1.patch, failed testing.

paolomainardi’s picture

paolomainardi commented
StatusFileSize
new nodequeue-autocompletelang-1649172-1.patch1.99 KB

Another try.

paolomainardi’s picture

paolomainardi commented
Status: Needs work » Needs review
StatusFileSize
new nodequeue-autocompletelang-1649172-1.patch1.99 KB

Needs review

paolomainardi’s picture

paolomainardi commented

Up

paolomainardi’s picture

paolomainardi commented

up

paolomainardi’s picture

paolomainardi commented

Up

nils.destoop’s picture

nils.destoop commented

Add a check plain around $node->language in the autocomplete result, and all is ok.

jenlampton’s picture

jenlampton
she / her
commented
Issue summary: View changes
StatusFileSize
new nodequeue-autocompletelang-1649172-9.patch2.09 KB

Isn't $node->language a system-defined value? Is there a way that could be user-provided and unsafe? Attached patch check-plains it anyway. :)

paolomainardi’s picture

paolomainardi commented

Thanks @jenlampton