Update project module for new Transaction

[pheller, 2010-08-09 14:45:22]

http://codereview.appspot.com/1860047/diff/1/3
File work.py (right):

http://codereview.appspot.com/1860047/diff/1/3#newcode117
work.py:117: with Transaction().set_context(active_test=False):
need to import Transaction

[pheller, 2010-08-09 14:55:51]

looks good here

[zodman, 2010-08-29 23:40:54]

i have a bug on project.

http://img837.imageshack.us/img837/9944/pantallazofo.png

http://codereview.appspot.com/1860047/diff/5001/6002
File work.py (right):

http://codereview.appspot.com/1860047/diff/5001/6002#newcode81
work.py:81: if context.get('type') == 'project':
Transaction().context.get('type') ... must be ?

[pheller, 2010-08-30 00:59:35]

Sorry guys, it seems I might have missed a thing or two in Project....

http://codereview.appspot.com/1860047/diff/5001/6002
File work.py (right):

http://codereview.appspot.com/1860047/diff/5001/6002#newcode27
work.py:27: self.parent = copy.copy(self.parent)
ced: Is this needed?  Doesn't this just assign self.parent to a week copy of
itself?

http://codereview.appspot.com/1860047/diff/5001/6002#newcode28
work.py:28: self.parent.context = copy.copy(self.parent.context)
Ibid.

http://codereview.appspot.com/1860047/diff/5001/6002#newcode29
work.py:29: self.parent.context['type'] = Eval('type')
??

http://codereview.appspot.com/1860047/diff/5001/6002#newcode49
work.py:49: 'invisible': Get(Eval('context', {}), 'type', False),
??

http://codereview.appspot.com/1860047/diff/5001/6002#newcode81
work.py:81: if context.get('type') == 'project':
On 2010/08/29 23:40:54, zodman wrote:
> Transaction().context.get('type') ... must be ?

Yes, right.

[ced, 2010-08-30 05:57:20]

http://codereview.appspot.com/1860047/diff/5001/6002
File work.py (right):

http://codereview.appspot.com/1860047/diff/5001/6002#newcode27
work.py:27: self.parent = copy.copy(self.parent)
On 2010/08/30 00:59:35, pheller wrote:
> ced: Is this needed?  Doesn't this just assign self.parent to a week copy of
> itself?
parent is a class attribute so we must copy it to not change it on all
TimesheetWork instance in every pool.

http://codereview.appspot.com/1860047/diff/5001/6002#newcode29
work.py:29: self.parent.context['type'] = Eval('type')
On 2010/08/30 00:59:35, pheller wrote:
> ??

This is an other context

http://codereview.appspot.com/1860047/diff/5001/6002#newcode81
work.py:81: if context.get('type') == 'project':
On 2010/08/30 00:59:35, pheller wrote:
> On 2010/08/29 23:40:54, zodman wrote:
> > Transaction().context.get('type') ... must be ?
> 
> Yes, right.

Fixed but next time fill an issue in the roundup when the codereview is closed