Consider not using built-in functions as parameters
|Reported by:||Richard Eames||Owned by:||nobody|
|Cc:||Richard Eames||Triage Stage:||Accepted|
|Has patch:||no||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
Currently the staticfiles finders'
find function has a paramter
all which is also a built-in function. Personally, I consider using built-ins as parameters/variables to be bad form, and would usually just rename the parameter in my subclasses. Unfortunately,
find is explicitly called with
django.contrib.staticfiles.finders.find(). So, to use the built in
all() one needs to use
from __builtin__ import all as all_.
There are probably more examples throughout the codebase, but this is the one I've run into.