Changes between Initial Version and Version 1 of Ticket #16180, comment 7


Ignore:
Timestamp:
Jun 9, 2011, 10:30:57 PM (13 years ago)
Author:
Stephen Burrows

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #16180, comment 7

    initial v1  
    1 I attached a patch with corrections to the docs for language and for clarity. However, I wonder if this is really the solution? Really, shouldn't lookup_params that aren't fields on the model always be ignored? They certainly aren't security risks the same way as other items. get_ignored_lookup_params should only be necessary if (for some reason) there's a conflict between the name of the lookup field you want to handle and one of the fields on the model.
     1I attached a patch with corrections to the docs for language and for clarity. However, I wonder if this is really the solution? Really, shouldn't lookup_params that aren't fields on the model always be ignored? They certainly aren't security risks the same way as other items. get_ignored_lookup_params should only be necessary if (for some reason) there's a conflict between the name of the param you want to handle and one of the fields on the model.
Back to Top