Dependencies: don't override get_available_name (Django Storages)

[humitos]

The fuction get_available_overwrite_name was removed. There is a setting file_overwrite which is set to True by default and our override shouldn't be required anymore.

https://django-storages.readthedocs.io/en/latest/backends/amazon-S3.html#settings

Related: #11502

[humitos]

@stsewd does this change look good for you?

[stsewd]

Makes sense. Tests are broken.

[humitos]

It seems that it's not overriding the files but adding a suffix:

>       self.assertEqual(
            {result.path for result in results},
            {"index.html", "404.html", "test.html", "api/index.html"},
        )
E       AssertionError: Items in the first set but not the second:
E       'index_ISlyKDd.html'
E       'test_EzpP2AY.html'
E       '404_nNCfKDs.html'
E       'index_Iha4OsZ.html'
E       '404_QBsoCyP.html'
E       '404_2qHEqDO.html'
E       'test_NAXp5j8.html'
E       Items in the second set but not the first:
E       'api/index.html'

In theory, it should behave like that by default in the newer version, but 🤷🏼

[humitos]

OK, it's seems the problem here is a combination of things between django-storages and Django itself. There is a long conversation at jschneier/django-storages#1430. I will block this PR for now and keep an eye on that conversation for now.