Remove docs suggestion to use lambda for models.FileField(storage=...)

[Ran Benita]

The file storage docs ​https://docs.djangoproject.com/en/4.2/topics/files/#using-a-callable currently suggest the following:

upload = models.FileField(storage=lambda: storages["custom_storage"])

The migrations serializer however doesn't handle lambdas. So I think this paragraph should just be removed (the previous paragraph is good enough, using a named function).

Side note: it might have been nice to allow FileField(storage="foo") which will lazily take storages["foo"] and thus avoid the callable annoyance? But that's unrelated to this ticket.

[David Sanders]

Hi bluetech!

Thanks for identifying this 👍 Interested in submitting a PR?

[David Sanders]

So I think this paragraph should just be removed (the previous paragraph is good enough, using a named function).

Personally I think an example involving storages is worth keeping though 🤷‍♂️

Side note: it might have been nice to allow FileField(storage="foo") which will lazily take storagesfoo and thus avoid the callable annoyance? But that's unrelated to this ticket.

Seems like something worth raising on the forum.

[Ran Benita]

Thanks David, agreed about keeping an example using storages. I sent a PR.

Seems like something worth raising on the forum.

If I have some time I'll try checking if it makes sense first :)

[Natalia Bidart]

​PR

[GitHub <noreply@…>]

In 506f93f0:

Fixed #34530 -- Improved docs when customizing storage for FileField.

[Natalia <124304+nessita@…>]

In 4c68482c:

[4.2.x] Fixed #34530 -- Improved docs when customizing storage for FileField.

Backport of 506f93f0c37ed9c44069edeab8baa66da5f7e03a from main