Path: csiph.com!usenet.pasdenom.info!weretis.net!feeder4.news.weretis.net!ecngs!feeder2.ecngs.de!newsfeed.freenet.ag!news2.euro.net!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
Date: Tue, 09 Jul 2013 09:07:58 -0700
From: Ethan Furman <ethan@stoneleaf.us>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121010 Thunderbird/16.0.1
MIME-Version: 1.0
To: python-list@python.org
Subject: Re: Default scope of variables
References: <51d4eb9c$0$29999$c3e8da3$5496439d@news.astraweb.com><mailman.4200.1372910878.3114.python-list@python.org><51d508ed$0$6512$c3e8da3$5496439d@news.astraweb.com><mailman.4208.1372916885.3114.python-list@python.org><51d5a504$0$29999$c3e8da3$5496439d@news.astraweb.com><CAPTjJmpgmABYwn0xB4T-9VbYMP=8JfpRcy8TPSerzyGvzPbMmA@mail.gmail.com><alpine.DEB.2.02.1307070759350.18702@gilgamesh><mailman.4357.1373204606.3114.python-list@python.org><krg4k4$tea$1@dont-email.me><mailman.4423.1373346452.3114.python-list@python.org><krg91s$e6j$1@dont-email.me><CAPTjJmqkmFd4-Jpugr-vubuB6riBV6K_Mwnxc_U3CVaBr_Wgbg@mail.gmail.com><krgefq$gji$1@ger.gmane.org> <CALwzid=FzgjPebifx1stDBkh8iwLtWggwwPTPhZ1ykYg+05wEg@mail.gmail.com> <krgi5f$n2v$1@ger.gmane.org>
In-Reply-To: <krgi5f$n2v$1@ger.gmane.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.4461.1373387509.3114.python-list@python.org>
Lines: 78
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:50260

On 07/09/2013 01:38 AM, Frank Millman wrote:
>
> "Ian Kelly" <ian.g.kelly@gmail.com> wrote in message
> news:CALwzid=FzgjPebifx1stDBkh8iwLtWggwwPTPhZ1ykYg+05wEg@mail.gmail.com...
>> On Tue, Jul 9, 2013 at 1:35 AM, Frank Millman <frank@chagford.com> wrote:
>>> When any of them need any database access, whether for reading or for
>>> updating, they execute the following -
>>>
>>>      with db_session as conn:
>>>          conn.transaction_active = True  # this line must be added if
>>> updating
>>>          conn.cur.execute(__whatever__)
>>
>> I'd probably factor out the transaction_active line into a separate
>> DbSession method.
>>
>>     @contextmanager
>>     def updating(self):
>>         with self as conn:
>>             conn.transaction_active = True
>>             yield conn
>>
>> Then you can do "with db_session" if you're merely reading, or "with
>> db_session.updating()" if you're writing, and you don't need to repeat
>> the transaction_active line all over the place.
>>
>
> I'll bear it in mind, but I will have to expend some mental energy to
> understand it first <g>, so it will have to wait until I can find some time.

You could also do it like this:

     def updating(self):
         self.transaction_active = True
         return self

and a sample object:

     class Tester(object):
         def __init__(self):
             self.transaction_active = False
             print 'initializied'
         def __enter__(self, *args):
             print '__enter__: transaction_active =', self.transaction_active
             return self
         def __exit__(self, *args):
             self.transaction_active = False
             print '__exit__: transaction_active =', self.transaction_active
             return
         def updating(self):
             self.transaction_active = True
             print 'updating: self.transaction_active =', self.transaction_active
             return self

     with Tester() as conn:
         print 'in first with block'

     print '-' * 50

     with Tester().updating() as conn:
         print 'in second with block'

with it's test run:

     ethan@hydra:~$ python test_cm.py
     initialized
     __enter__: transaction_active = False
     in first with block
     __exit__: transaction_active = False
     --------------------------------------------------
     initialized
     updating: self.transaction_active = True
     __enter__: transaction_active = True
     in second with block
     __exit__: transaction_active = False

--
~Ethan~