|
From: Gregor H. <gh...@gr...> - 2015-01-05 11:37:34
Attachments:
signature.asc
|
Hello,
Problem:
$ dropdb scratch; createdb scratch; python /tmp/scratch.py
Traceback (most recent call last):
File "/tmp/scratch.py", line 25, in <module>
er = Eingangsrechnung(name="test")
File
"/home/gregor/work/gnucash/testve/local/lib/python2.7/site-packages/sqlobject/main.py",
line 1226, in __init__
self._create(id, **kw)
File
"/home/gregor/work/gnucash/testve/local/lib/python2.7/site-packages/sqlobject/main.py",
line 1274, in _create
self._SO_finishCreate(id)
File
"/home/gregor/work/gnucash/testve/local/lib/python2.7/site-packages/sqlobject/main.py",
line 1301, in _SO_finishCreate
self._init(id)
File
"/home/gregor/work/gnucash/testve/local/lib/python2.7/site-packages/sqlobject/main.py",
line 934, in _init
raise SQLObjectNotFound, "The object %s by the ID %s does not
exist" % (self.__class__.__name__, self.id)
sqlobject.main.SQLObjectNotFound: The object Eingangsrechnung by the ID
1 does not exist
--------------------
$ cat /tmp/scratch.py
from sqlobject import SQLObject, sqlhub, connectionForURI, ForeignKey,
StringCol
sqlhub.processConnection =
connectionForURI('postgres://gregor@localhost/scratch?debug=0&autoCommit=1')
class Person(SQLObject):
name = StringCol()
class Eingangsrechnung(SQLObject):
name = StringCol()
class Buchung(SQLObject):
eingangsrechnung = ForeignKey('Eingangsrechnung')
person = ForeignKey('Person')
Person.createTable()
Eingangsrechnung.createTable()
Buchung.createTable()
sqlhub.processConnection.autoCommit = False
t = Person._connection.transaction()
er = Eingangsrechnung(name="test")
pr = Person(name="test")
Buchung(person=pr, eingangsrechnung=er)
t.commit()
----
It works if I remove the transaction lines, or set autoCommit =True.
I have tested this with SQLObject 0.12.4 and 2.0.0, same behaviour.
Is this a bug, or do I do something wrong?
--
Greg
|
|
From: Oleg B. <ph...@ph...> - 2015-01-05 12:19:50
|
Hi!
On Mon, Jan 05, 2015 at 12:19:20PM +0100, Gregor Horvath <gh...@gr...> wrote:
> Problem:
>
> raise SQLObjectNotFound, "The object %s by the ID %s does not
> exist" % (self.__class__.__name__, self.id)
> sqlobject.main.SQLObjectNotFound: The object Eingangsrechnung by the ID
> 1 does not exist
>
> --------------------
>
> sqlhub.processConnection.autoCommit = False
> t = Person._connection.transaction()
> er = Eingangsrechnung(name="test")
> pr = Person(name="test")
> Buchung(person=pr, eingangsrechnung=er)
> t.commit()
>
> ----
> Greg
With SQLObject you can open a dozen different connections to the same
or different backends. Because of that you have to explicitly set your
desired connection for any select/create operation (after select/create
the object remembers its connection and uses it for update/delete); if
you don't set connection explicitly sqlhub is used.
Creating a transaction doesn't automatically use that transaction --
you have to use the new transaction object as a connection. So your
options are:
1. Assign the transaction to sqlhub:
the new t = Person._connection.transaction()
sqlhub.processConnection = t # <= !!!
er = Eingangsrechnung(name="test")
pr = Person(name="test")
Buchung(person=pr, eingangsrechnung=er)
t.commit()
Don't forget to set the original connection back at the end:
sqlhub.processConnection = initialConnection
2. Use the transaction explicitly:
t = Person._connection.transaction()
er = Eingangsrechnung(name="test", connection=t)
pr = Person(name="test", connection=t)
Buchung(person=pr, eingangsrechnung=er, connection=t)
t.commit()
3. Rewrite you code as a function and use sqlhub.doInTransaction(f);
doInTransaction creates and commits a transaction for you, no need to
create t.
Oleg.
--
Oleg Broytman http://phdru.name/ ph...@ph...
Programmers don't die, they just GOSUB without RETURN.
|
|
From: Gregor H. <gh...@gr...> - 2015-01-05 13:27:18
Attachments:
signature.asc
|
Hi Oleg, Am Mon, 5 Jan 2015 13:19:39 +0100 schrieb Oleg Broytman <ph...@ph...>: > On Mon, Jan 05, 2015 at 12:19:20PM +0100, Gregor Horvath > <gh...@gr...> wrote: > > Problem: > > > > raise SQLObjectNotFound, "The object %s by the ID %s does not > > exist" % (self.__class__.__name__, self.id) > > sqlobject.main.SQLObjectNotFound: The object Eingangsrechnung by > > the ID 1 does not exist > Creating a transaction doesn't automatically use that transaction > -- you have to use the new transaction object as a connection. So your Thank you very much for your fast and comprehensive answer! No I understand and it works. To clarify this I would like to update the docs: http://sqlobject.org/SQLObject.html#transactions with for example adding your sentence: "Creating a transaction doesn't automatically use that transaction you have to use the new transaction object as a connection." I did not found a repository for the docs. How can I contribute this to it? -- Greg |
|
From: Oleg B. <ph...@ph...> - 2015-01-05 13:43:23
|
On Mon, Jan 05, 2015 at 02:26:57PM +0100, Gregor Horvath <gh...@gr...> wrote: > Hi Oleg, > > Am Mon, 5 Jan 2015 13:19:39 +0100 > schrieb Oleg Broytman <ph...@ph...>: > > On Mon, Jan 05, 2015 at 12:19:20PM +0100, Gregor Horvath > > <gh...@gr...> wrote: > > > Creating a transaction doesn't automatically use that transaction > > -- you have to use the new transaction object as a connection. So your > > Thank you very much for your fast and comprehensive answer! > No I understand and it works. > > To clarify this I would like to update the docs: > > http://sqlobject.org/SQLObject.html#transactions > > with for example adding your sentence: > > "Creating a transaction doesn't automatically use that transaction > you have to use the new transaction object as a connection." > > I did not found a repository for the docs. > How can I contribute this to it? There is no a separate repository for the docs -- docs are in the main repository: https://github.com/sqlobject/sqlobject/ . Mostly it's because that the way the sources were organized in Subversion and partially because docs are version-dependent (the news, the list of contributors and major features certainly are). Please, create a PR against branch 1.6 (it's the oldest supported branch), I'll merge it upward. Thank you in advance! > -- > Greg Oleg. -- Oleg Broytman http://phdru.name/ ph...@ph... Programmers don't die, they just GOSUB without RETURN. |
