ModelAdmin.search_fields crashes for a search term with unbalanced quotes.

[Dlis]

django.contrib.admin.ModelAdmin.search_fields now allows searching against quoted phrases with spaces but unfortunately search crashes for a search term like Foo "Bar "Baz" (with three quotes, some company names match such the pattern)

[Dlis]

The bug can be reproduced by the following way:

from django.utils.text import smart_split
from django.utils.text import unescape_string_literal

search_term = 'Foo "Bar "Baz"'
parts = list(smart_split(search_term))
unescape_string_literal(parts[1])

Result is ValueError: Not a string literal: '"Bar "Baz'

[Dlis]

If somebody has such the problem, here is our temporary fix which was added to our internal basic ModelAdmin:

from django.utils import text


class ModelAdmin(...):

    ...

    def get_search_results(self, request, queryset, search_term):
        parts = search_term.split()
        try:
            for part in text.smart_split(search_term):
                text.unescape_string_literal(part)
        except ValueError:
            parts = map(lambda t: t.strip('"').strip("'"), parts)
        return super().get_search_results(request, queryset, ' '.join(parts))

[Mariusz Felisiak]

Thanks for the report.

Regression in 26a413507abb38f7eee4cf62f2ee9727fdc7bf8d,
Reproduced at 9760e262f85ae57df39abe2799eff48a82b14474.

[Mariusz Felisiak]

​PR

[GitHub <noreply@…>]

In 23fa29f6:

Fixed #32649 -- Fixed ModelAdmin.search_fields crash when searching against phrases with unbalanced quotes.

Thanks Dlis for the report.

Regression in 26a413507abb38f7eee4cf62f2ee9727fdc7bf8d.

[Mariusz Felisiak <felisiak.mariusz@…>]

In 59cce823:

[3.2.x] Fixed #32649 -- Fixed ModelAdmin.search_fields crash when searching against phrases with unbalanced quotes.

Thanks Dlis for the report.

Regression in 26a413507abb38f7eee4cf62f2ee9727fdc7bf8d.
Backport of 23fa29f6a6659e0f600d216de6bcb79e7f6818c9 from main