Comments
Patch
@@ -934,6 +934,7 @@
def __get__(self, obj, type=None):
# do we need to check if the file changed?
if self.name in obj.__dict__:
+ assert self.name in obj._filecache, self.name
return obj.__dict__[self.name]
entry = obj._filecache.get(self.name)
@@ -955,8 +956,15 @@
return entry.obj
def __set__(self, obj, value):
- if self.name in obj._filecache:
- obj._filecache[self.name].obj = value # update cached copy
+ if self.name not in obj._filecache:
+ # we add an entry for the missing value because X in __dict__
+ # implies X in _filecache
+ ce = filecacheentry(self.join(obj, self.path), False)
+ obj._filecache[self.name] = ce
+ else:
+ ce = obj._filecache[self.name]
+
+ ce.obj = value # update cached copy
obj.__dict__[self.name] = value # update copy returned by obj.x
def __delete__(self, obj):
@@ -101,6 +101,17 @@
# it
repo.commit('.')
+def setbeforeget(repo):
+ os.remove('x')
+ repo.cached = 0
+ repo.invalidate()
+ print repo.cached
+ repo.invalidate()
+ f = open('x', 'w')
+ f.write('a')
+ f.close()
+ print repo.cached
+
print 'basic:'
print
basic(fakerepo())
@@ -109,3 +120,7 @@
print
fakeuncacheable()
test_filecache_synced()
+print
+print 'setbeforeget:'
+print
+setbeforeget(fakerepo())
@@ -17,3 +17,9 @@
working directory now based on revision -1
repository tip rolled back to revision -1 (undo commit)
working directory now based on revision -1
+
+setbeforeget:
+
+0
+creating
+None