This is pretty similar to #960596: Node types not quoted in query (preview only).

While I was creating some tests for view I realized that the info provided by simpletest is not really useful.

This patch escapes placeholders in the same way that is done in the view preview.

CommentFileSizeAuthor
#3 1613506.patch3.52 KBdawehner
#1 views.1613506.patch1.17 KBdagmar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
CreditAttribution: dagmar commented
Status: Active » Needs review
FileSize
views.1613506.patch1.17 KB

Patch attached.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

What about adding a method on the query plugin?

This would make the code a bit easier to read.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
FileSize
1613506.patch3.52 KB

Just moved this long code into a new method.

Status: Needs review » Needs work

The last submitted patch, 1613506.patch, failed testing.