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


Ignore:
Timestamp:
Sep 21, 2022, 2:00:01 PM (2 years ago)
Author:
Simon Charette

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #33579, comment 7

    initial v1  
    1 Another alternative we would be to simply raise a `base.DoesNotExists` error instead.
     1Another alternative we would be to simply raise a `cls.DoesNotExists` error instead.
    22
    33It would be backward incompatible in the sense that a different exception type would be raised but it seems it would fit elegantly with the message it's meant to convey; the object that was attempted to be updated was not found. It would also make code that handles generic `DoesNotExist` and turn them into HTTP 404 automatically pick up errors of this form without any form of special casing.
Back to Top