Opened 8 years ago

Last modified 8 years ago

#27309 closed Bug

Add CallableBool.__hash__() — at Version 2

Reported by: Reto Aebersold Owned by: nobody
Component: Utilities Version: 1.10
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Tim Graham)

As CallableBool has no hash method, membership test operations for sets do not work:

from django.utils.deprecation import CallableTrue
assert CallableTrue in {True}

TypeError: unhashable type: 'CallableBool'

Adding a hash method solves this problem:

def __hash__(self):
    return hash(self.value)

Change History (2)

comment:1 by Reto Aebersold, 8 years ago

Component: UncategorizedUtilities
Description: modified (diff)
Type: UncategorizedBug

comment:2 by Tim Graham, 8 years ago

Description: modified (diff)
Has patch: set
Summary: Missing Hash function in CallableBoolAdd CallableBool.__hash__()
Triage Stage: UnreviewedReady for checkin

It seems a bit odd to have such code, but I suppose it doesn't hurt.

Note: See TracTickets for help on using tickets.
Back to Top