Django-compressor: RemovedInDjango31警告：FILE_CHARSET設定は非推奨です

リリースノートには次のように書かれています。

FILE_CHARSET設定は非推奨です。 Django 3.1以降、ディスクから読み取られるファイルはUTF-8でエンコードする必要があります。

• https://docs.djangoproject.com/en/2.2/releases/2.2/#id2

Djangoでの関連するコミット：

https://github.com/django/django/commit/0cd465b63aa7c03a3d14bd5fd6543628d585f8da

切符：

https://code.djangoproject.com/ticket/29817

この問題に対処するには、django-compressorのすべてのsettings.FILE_CHARSETをutf-8に置き換えるだけで十分ですか？

報告してくれてありがとう。 すべてのオカレンスをutf-8置き換えると、実際にこの設定を使用している人にとっては問題が発生する可能性があります。 適切な修正は、設定が存在する場合はその設定を使用し、存在しない場合は問題のパラメーターを設定しないことです。

@karyonお返事ありがとうございます。 パッチを書いてみました。 私の理解が正しいかどうかを確認していただきたいと思います。

このレポにPRを送るのはこれが初めてで、何か重要なことを見逃したかもしれません。 何かおかしいと思ったら教えてください。 前もって感謝します。

＃934で修正

新しいコードは、私のセットアップで非常によく似た警告を引き起こすようです：

  /usr/local/lib/python3.7/site-packages/compressor/filters/base.py:123: RemovedInDjango31Warning: The FILE_CHARSET setting is deprecated. Starting with Django 3.1, all files read from disk must be UTF-8 encoded.
    settings.FILE_CHARSET if settings.is_overridden('FILE_CHARSET') else

と

django-compressor==2.3
Django==2.2.2

settings.FILE_CHARSETプロパティにアクセスすると、オーバーライドされているかどうかをテストする場合でも、警告がトリガーされるようです。 これは、 utf-8デフォルトを使用する必要があるかどうかを確認するために行うことです。

下位互換性を維持したい場合、最も簡単な方法は、次のようなコードを使用して、コードのこの特定の行に対してこの警告を無効にすることです。

import warnings

with warnings.catch_warnings():
    warnings.filterwarnings("ignore", message="popo popo")
    warnings.warn("popo popo")  # the code that will trigger the warning
# warnings are back to normal filtering from here

そのような解決策があなたにとって大丈夫だと思われる場合、私はPRを提供することができます。

はい、PRがいいでしょう。 必要に応じて、元のPRを元に戻すことから始めることができます:)

PRのパッチを作成しているときに、警告が実際にはis_overridden('FILE_CHARSET')非常に奇妙な偽の検出によってトリガーされていることを発見しました。 私のプロジェクトでは、設定でFILE_CHARSETを設定していませんが、 is_overridden('FILE_CHARSET')はTrueを返しているようです。

しかし、django-compressorをインストールしただけの最小限のdjangoプロジェクトでテストした場合->警告なし

したがって、私のPRは実際には必要ありません。 is_overridden('FILE_CHARSET')はTrueを返さないため、 compress()とCompilerFilter()は警告を表示しません。 通常の状況下で。

この警告pbのリグレッションを検出するためにdjango-compressorコードベースに追加したテストでさえ、それほど煩わしいものではありません：django / django @ 3d716467 commit（間もなく登場するdjango 3.1に含まれています）は、前述の警告を削除しました、 djangoの非推奨タイムラインに従い

明るい面：新しいdjangoコード（ FILE_CHARSETと関連する警告が表示されなくなったコード）に対してdjango-compressorをテストしたところ、検出可能な問題は発生しませんでした。

ご不便おかけしてすみません。 私はあなたが（再び）問題を閉じることができると思います。