CachedStaticFilesStorage eats @import statement

[anonymous]

Input:

@import '../../3rdparty/pt-sans-generated/stylesheet.css';

html, body {
    padding: 0;
    margin: 0;
    background: transparent;
}

Output:

url("../../3rdparty/pt-sans-generated/stylesheet.cc11c6fa559e.css");

html, body {
    padding: 0;
    margin: 0;
    background: transparent;
}

Without @import statement!

Problem located in:

class CachedFilesMixin(object):
    patterns = (
        ("*.css", (
            r"""(url\(['"]{0,1}\s*(.*?)["']{0,1}\))""",
            r"""(@import\s*["']\s*(.*?)["'])""",
        )),
    )

last statement replace whole (@import…) construction with just url(). This quick fix solves a problem:

class FixedCachedFilesMixin(CachedFilesMixin):
    patterns = (
        ("*.css", (
            r"""(url\(['"]{0,1}\s*(.*?)["']{0,1}\))""",
            r"""@import\s*(["']\s*(.*?)["'])""",
            )),
        )


class CachedStaticFilesStorage(CachedFilesMixin, StaticFilesStorage):
    """
    A static file system storage backend which also saves
    hashed copies of the files it saves.
    """
    pass

But it is definitely a serious bug because @import is completely broken by default.

[Jannis Leidel <jannis@…>]

In [3047981517ffa0c75c97f05446bd0d41865e323b]:

Fixed #18050 -- Fixed a rather glaring bug in the handling of @import statements when using the cached staticfiles storage.