Re: ZPsycopgDA Isolation Levels Bug

On 09/08/11 17:23, Daniele Varrazzo wrote:
> 2011/8/9 Lex Berezhny <eukreign@gmail.com>:
>> Hello,
>>
>>  I have updated ZPsycopgDA add/edit screens to include all of the
>> isolation levels provided by psycopg2. It seems that even the ones
>> that were there had the wrong mappings between the isolation level
>> name and the internal integer identifier. "Read Commited" in the drop
>> down was actually mapping to "Read Uncommited" and "Serializable" was
>> mapping to "Read Committed".
>
> I'm not entirely sure Autocommit should be included in the list: it
> isn't exactly a transaction isolation level. But I know little about
> Zope so I may be wrong.

IMO, autocommit should not be included. Zope has the concept of "page
transaction" and we don't want to break that.

>>  I have attached a diff of my changes, if there is a way for me to
>> git push my changes where someone with write access can pull from I
>> can do that too.
>
> You may have a clone on github. I have my repos there and Federico
> usually pulls my code from it.

Yep, just send to me or to this list a pull request.

federico

--
Federico Di Gregorio                         federico.digregorio@dndg.it
Studio Associato Di Nunzio e Di Gregorio                  http://dndg.it
  Se consideri l'uso del software libero una concessione tu stesso,
   come potrai proporla agli altri?                         -- Nick Name