Django FileField.save（）は重複ファイルを作成します

Question

ファイルに書き込もうとしているコンテンツをユーザーが投稿してからFileFieldに保存しました。

ので、私はこのようになりますモデルあります

class Revision(models.Model): 
    def custom_revision_file_path(instance, filename): 
     return '/'.join(['content/revisions', filename]) 
    path = models.FileField(upload_to=custom_revision_file_path) 
    document = models.ForeignKey(Document) 
    ... 

をしてインスタンスを作成しますビューは次のようになります。

def handle_revisions(request): 
    document = Document.objects.get(id=request.GET['docid']) 
    basename = os.path.basename(str(document.path)) 

    revision = Revision.objects.create(
     document = document, 
    ) 
    revision.path.save(basename, ContentFile(request.GET['revision'])) 

このすべては、比較的細かい動作しますが、用2つの問題：

1）ContentFileは、文字列内の各文字の間にスペースを入れます。'test'は't e s t'に変わります。

2）何らかの理由で私がビューを実行するたびに、2つのRevisionインスタンスがほぼ同じパスで保存されます。すなわち、 2つ目のパスが全く存在してはならない場合、一方のパスは'content/revisions/test.txt'になり、もう一方のパスは'content/revisions/test_1.txt'になります。

何がありますか？

Answer 1

まず第一に、あなたはパスを作成するためにそのようなものを使用しないでください：

'/'.join(['content/revisions', filename]) 

しかし：

os.path.join("my_dir", "my_subdir", ..., "filename.txt") 

あなたは、あなたのアプリケーションが、UNIX上で動作するかどうかを知ることになっていませんまたはWindows（はい、ウェブサーバーとしてWindowsを使用する人もいます）のようになります。

また、FileField属性pathと呼ぶべきではありません。FilePathFieldであいまいです。

このフィールドはNOT NULLですか？あなたのcreate()ステートメントでは、あなたはそれを提供していないからです。これによりエラーが発生するはずです。

私はこれを得ることはありません：

revision.path.save(basename, ContentFile(request.GET['revision'])) 

あなたが達成しようとしていますか？あなたはGETパラメータをファイルに保存してもよろしいですか？

あなたの質問に答えるために、Djangoはファイルシステム上に存在するファイルを上書きする責任を負いません。そのため、接尾辞を追加することで自動的にunique pathで保存します。

この動作が適合しない場合は、custom file storageを作成することを検討してください。