Incorrect docstring in Options.get_fields()

[Eric Zarowny]

According to the docstring for ​Options.get_fields():

Returns a list of fields associated to the model. By default will only return forward fields.

However, the call to _get_fields does not set reverse=False when its default is True. I can't really tell if this was intentional or not. I would be happy to make a patch / pull request to fix this either way.

[Tim Graham]

Proposed update of the docstring:

django/db/models/options.py

@@ -755 +755 @@
 
     def get_fields(self, include_parents=True, include_hidden=False):
         """
-        Returns a list of fields associated to the model. By default will only
-        return forward fields. This can be changed by enabling or disabling
-        field types using the parameters:
+        Returns a list of fields associated to the model. By default will
+        include forward and reverse fields, fields derived from inheritance,
+        but not hidden fields. The returned fields can be changed by using the
+        parameters:
 
         - include_parents: include fields derived from inheritance
         - include_hidden:  include fields that have a related_name that

[Tim Graham <timograham@…>]

In b99114f6:

Fixed #24758 -- Corrected Options.get_fields() docstring.

[Tim Graham <timograham@…>]

In 6a7b27c:

[1.8.x] Fixed #24758 -- Corrected Options.get_fields() docstring.

Backport of b99114f67f0e5eba3b1d31641906b75a14fd2fa0 from master