Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#28184 closed New feature (fixed)

FileField storage param should allow a callable

Reported by: miigotu Owned by: miigotu
Component: File uploads/storage Version: dev
Severity: Normal Keywords: callable storage filefield
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

For example, when using S3 I may have a field that I want to use private acl, while other fields are not using private storage. Locally or in different deployments I may not be using S3, and the storage is different.

Change History (12)

comment:1 by Tim Graham, 7 years ago

Easy pickings: unset
Patch needs improvement: set
Summary: FileField storages param should allow a callableFileField storage param should allow a callable
Triage Stage: UnreviewedAccepted

Tentatively accepting. This idea also came up in #24648. The PR has test failures and documentation changes are also required. See the PatchReviewChecklist.

comment:2 by miigotu, 7 years ago

Patch needs improvement: unset

Improved patch, passes all tests now.

comment:3 by Carlton Gibson, 6 years ago

Needs documentation: set
Patch needs improvement: set

comment:4 by Carlton Gibson, 6 years ago

Left some comments on PR.

in reply to:  4 comment:5 by miigotu, 6 years ago

Replying to Carlton Gibson:

Left some comments on PR.

PR Updated, please review

comment:6 by miigotu, 6 years ago

Needs documentation: unset
Patch needs improvement: unset

comment:7 by miigotu, 6 years ago

Version: 1.11master

comment:8 by Carlton Gibson, 6 years ago

Patch needs improvement: set

comment:9 by Carlton Gibson, 4 years ago

Patch needs improvement: unset

comment:10 by Carlton Gibson, 4 years ago

Triage Stage: AcceptedReady for checkin

comment:11 by Carlton Gibson <carlton@…>, 4 years ago

Resolution: fixed
Status: assignedclosed

In 210657b7:

Fixed #28184 -- Allowed using a callable for FileField and ImageField storage.

comment:12 by Carlton Gibson <carlton@…>, 4 years ago

In 94b32a2:

Refs #28184 -- Added test for FileField storage default.

Co-authored-by: miigotu <miigotu@…>

Note: See TracTickets for help on using tickets.
Back to Top