From 9a519b179f0de487a091490a344cf07f63b683f8 Mon Sep 17 00:00:00 2001
From: webchick <webchick@24967.no-reply.drupal.org>
Date: Sat, 24 Dec 2011 17:23:52 -0800
Subject: [PATCH] Issue #1371256 by ayelet_Cr, oriol_e9g: Document that
 QueryConditionInterface::condition() should not be used to add NULL
 conditions.

---
 includes/database/query.inc | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/includes/database/query.inc b/includes/database/query.inc
index c779687679a..6020b0ea5cf 100644
--- a/includes/database/query.inc
+++ b/includes/database/query.inc
@@ -22,6 +22,9 @@ interface QueryConditionInterface {
    * parameters, they are taken as $field and $value with $operator having a
    * value of IN if $value is an array and = otherwise.
    *
+   * Do not use this method to test for NULL values. Instead, use
+   * QueryConditionInterface::isNull() or QueryConditionInterface::isNotNull().
+   *
    * @param $field
    *   The name of the field to check. If you would like to add a more complex
    *   condition involving operators or functions, use where().
@@ -36,6 +39,9 @@ interface QueryConditionInterface {
    *
    * @return QueryConditionInterface
    *   The called object.
+   *
+   * @see QueryConditionInterface::isNull()
+   * @see QueryConditionInterface::isNotNull()
    */
   public function condition($field, $value = NULL, $operator = NULL);