alltyme / appengine-ndb-experiment

Automatically exported from code.google.com/p/appengine-ndb-experiment
Other
0 stars 0 forks source link

Deleting a ComputedProperty silently fails #217

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Creating a Model with a ComputedProperty
2. Creating an instance of the model
3. Deleting the property of said model

What is the expected output? What do you see instead?
An exception or similar. Instead it silently fails.

Please use labels and text to provide additional information.
https://groups.google.com/forum/?fromgroups=#!topic/appengine-ndb-discuss/eiLdsw
U26jU

Original issue reported on code.google.com by beechhorn on 16 Oct 2012 at 6:35

GoogleCodeExporter commented 9 years ago
Created a fix https://codereview.appspot.com/6725046/

but am not entirely happy with the finer details such as using an equality test 
on __class__ instead of an isinstance test, etc.

Original comment by beechhorn on 17 Oct 2012 at 9:29

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 72bb0691fb78.

Original comment by beechhorn on 17 Oct 2012 at 8:21

GoogleCodeExporter commented 9 years ago
Thanks!

Original comment by guido@google.com on 17 Oct 2012 at 8:24