summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJon Dufresne <jon.dufresne@gmail.com>2016-07-12 17:50:17 -0700
committerTim Graham <timograham@gmail.com>2016-07-13 21:55:25 -0400
commit1a2ee697e0631475d95197a795aa2af762537b10 (patch)
tree62b6215883e142a3079134cd41eef7a4ec0b0183
parentded502024191053475bac811d365ac29dca1db61 (diff)
[1.10.x] Fixed #26889 -- Fixed missing PostgreSQL index in SchemaEditor.add_field().
Backport of 2e4cfcd2b9a0984ad6c4087a5deebbf33413835c from master
-rw-r--r--django/db/backends/postgresql/schema.py6
-rw-r--r--docs/releases/1.8.14.txt14
-rw-r--r--docs/releases/1.9.8.txt4
-rw-r--r--docs/releases/index.txt1
-rw-r--r--tests/schema/tests.py26
5 files changed, 50 insertions, 1 deletions
diff --git a/django/db/backends/postgresql/schema.py b/django/db/backends/postgresql/schema.py
index cd9e6cfd5e..c10b18b00f 100644
--- a/django/db/backends/postgresql/schema.py
+++ b/django/db/backends/postgresql/schema.py
@@ -17,6 +17,12 @@ class DatabaseSchemaEditor(BaseDatabaseSchemaEditor):
def quote_value(self, value):
return psycopg2.extensions.adapt(value)
+ def add_field(self, model, field):
+ super(DatabaseSchemaEditor, self).add_field(model, field)
+ like_index_statement = self._create_like_index_sql(model, field)
+ if like_index_statement is not None:
+ self.deferred_sql.append(like_index_statement)
+
def _model_indexes_sql(self, model):
output = super(DatabaseSchemaEditor, self)._model_indexes_sql(model)
if not model._meta.managed or model._meta.proxy or model._meta.swapped:
diff --git a/docs/releases/1.8.14.txt b/docs/releases/1.8.14.txt
new file mode 100644
index 0000000000..6311172abc
--- /dev/null
+++ b/docs/releases/1.8.14.txt
@@ -0,0 +1,14 @@
+===========================
+Django 1.8.14 release notes
+===========================
+
+*Under development*
+
+Django 1.8.14 fixes several bugs in 1.8.13.
+
+Bugfixes
+========
+
+* Fixed missing ``varchar/text_pattern_ops`` index on ``CharField`` and
+ ``TextField`` respectively when using ``AddField`` on PostgreSQL
+ (:ticket:`26889`).
diff --git a/docs/releases/1.9.8.txt b/docs/releases/1.9.8.txt
index 7a3b55bae5..7265e166b8 100644
--- a/docs/releases/1.9.8.txt
+++ b/docs/releases/1.9.8.txt
@@ -9,4 +9,6 @@ Django 1.9.8 fixes several bugs in 1.9.7.
Bugfixes
========
-* ...
+* Fixed missing ``varchar/text_pattern_ops`` index on ``CharField`` and
+ ``TextField`` respectively when using ``AddField`` on PostgreSQL
+ (:ticket:`26889`).
diff --git a/docs/releases/index.txt b/docs/releases/index.txt
index 3dbbe4ae33..6bedc78e6f 100644
--- a/docs/releases/index.txt
+++ b/docs/releases/index.txt
@@ -47,6 +47,7 @@ versions of the documentation contain the release notes for any later releases.
.. toctree::
:maxdepth: 1
+ 1.8.14
1.8.13
1.8.12
1.8.11
diff --git a/tests/schema/tests.py b/tests/schema/tests.py
index 2aaaa1e64a..f74db77fb5 100644
--- a/tests/schema/tests.py
+++ b/tests/schema/tests.py
@@ -1739,6 +1739,32 @@ class SchemaTests(TransactionTestCase):
editor.add_field(Author, new_field)
@unittest.skipUnless(connection.vendor == 'postgresql', "PostgreSQL specific")
+ def test_add_indexed_charfield(self):
+ field = CharField(max_length=255, db_index=True)
+ field.set_attributes_from_name('nom_de_plume')
+ with connection.schema_editor() as editor:
+ editor.create_model(Author)
+ editor.add_field(Author, field)
+ # Should create two indexes; one for like operator.
+ self.assertEqual(
+ self.get_constraints_for_column(Author, 'nom_de_plume'),
+ ['schema_author_95aa9e9b', 'schema_author_nom_de_plume_7570a851_like'],
+ )
+
+ @unittest.skipUnless(connection.vendor == 'postgresql', "PostgreSQL specific")
+ def test_add_unique_charfield(self):
+ field = CharField(max_length=255, unique=True)
+ field.set_attributes_from_name('nom_de_plume')
+ with connection.schema_editor() as editor:
+ editor.create_model(Author)
+ editor.add_field(Author, field)
+ # Should create two indexes; one for like operator.
+ self.assertEqual(
+ self.get_constraints_for_column(Author, 'nom_de_plume'),
+ ['schema_author_nom_de_plume_7570a851_like', 'schema_author_nom_de_plume_key']
+ )
+
+ @unittest.skipUnless(connection.vendor == 'postgresql', "PostgreSQL specific")
def test_alter_field_add_index_to_charfield(self):
# Create the table and verify no initial indexes.
with connection.schema_editor() as editor: