Ticket #11402: exists.3.diff

django/db/models/base.py

@@ -467 +467 @@
             if pk_set:
                 # Determine whether a record with the primary key already exists.
                 if (force_update or (not force_insert and
-                        manager.filter(pk=pk_val).extra(select={'a': 1}).values('a').order_by())):
+                        manager.filter(pk=pk_val).exists())):
                     # It does already exist, so do an UPDATE.
                     if force_update or non_pks:
                         values = [(f, None, (raw and getattr(self, f.attname) or f.pre_save(self, False))) for f in non_pks]

django/db/models/manager.py

@@ -172 +172 @@
 
     def only(self, *args, **kwargs):
         return self.get_query_set().only(*args, **kwargs)
+    
+    def exists(self, *args, **kwargs):
+        return self.get_query_ste().exists(*args, **kwargs)
 
     def _insert(self, values, **kwargs):
         return insert_query(self.model, values, **kwargs)

django/db/models/query.py

@@ -443 +443 @@
         self._result_cache = None
         return query.execute_sql(None)
     _update.alters_data = True
+    
+    def exists(self):
+        if self._result_cache is None:
+            return self.query.has_results()
+        return bool(self._result_cache)
 
     ##################################################
     # PUBLIC METHODS THAT RETURN A QUERYSET SUBCLASS #

django/db/models/sql/query.py

@@ -383 +383 @@
             number = min(number, self.high_mark - self.low_mark)
 
         return number
+    
+    def has_results(self):
+        """
+        Returns whether or not there are any items in the result set this Query
+        represents.  Does this in a super tricking way.  Basically the 
+        equivalent of QuerySet.extra(select={'a': 1}).values('a').order_by()
+        to maximize efficiency.
+        """
+        c = self.clone()
+        c.add_extra({'a': 1}, None, None, None, None, None)
+        c.add_fields(())
+        c.set_extra_mask(('a',))
+        c.set_aggregate_mask(())
+        c.clear_ordering()
+        results = self.execute_sql()
+        return bool(results)
+
 
     def as_sql(self, with_limits=True, with_col_aliases=False):
         """

django/forms/models.py

@@ -319 +319 @@
             if self.instance.pk is not None:
                 qs = qs.exclude(pk=self.instance.pk)
 
-            # This cute trick with extra/values is the most efficient way to
-            # tell if a particular query returns any results.
-            if qs.extra(select={'a': 1}).values('a').order_by():
+            if qs.exists():
                 if len(unique_check) == 1:
                     self._errors[unique_check[0]] = ErrorList([self.unique_error_message(unique_check)])
                 else:
@@ -354 +352 @@
             if self.instance.pk is not None:
                 qs = qs.exclude(pk=self.instance.pk)
 
-            # This cute trick with extra/values is the most efficient way to
-            # tell if a particular query returns any results.
-            if qs.extra(select={'a': 1}).values('a').order_by():
+            if qs.exists():
                 self._errors[field] = ErrorList([
                     self.date_error_message(lookup_type, field, unique_for)
                 ])

docs/ref/models/querysets.txt

@@ -1114 +1114 @@
 
 .. _field-lookups:
 
+``exists()``
+~~~~~~~~~~~~
+
+Returns whether or not there are any items in the ``QuerySet``.  This will
+perform the query in the most efficient way possible.
+
 Field lookups
 -------------