Issue 10495

Title
trytond-console should have an option to run the transaction readonly
Priority
feature
Status
resolved
Nosy list
ced, nicoe, reviewbot, roundup-bot
Assigned to
nicoe
Keywords
review

Created on 2021-06-08.17:52:14 by nicoe, last changed 1 month ago by roundup-bot.

Messages

New changeset c6d0c9bc547b by Nicolas Évrard in branch 'default':
Add option to trytond-console to start a readonly transaction
https://hg.tryton.org/tryton-env/rev/c6d0c9bc547b
New changeset c8f1dc9a357c by Nicolas Évrard in branch 'default':
Add option to trytond-console to start a readonly transaction
https://hg.tryton.org/trytond/rev/c8f1dc9a357c
Author: [hidden] (nicoe) Tryton committer
Date: 2021-06-09.09:06:03
* Cédric Krier  [2021-06-09 08:59 +0200]: 

>I found the name of the flag "writable" very astonishing and not related to SQL naming.

I searched for an antonym of readonly. It's the best I came up for.

>At the end I think we should keep by default a normal transaction and have an
>option to start a readonly transaction as SQL does. Because it is only if the
>user is calling commit that it will alter the data.
>
>Also it is better to keep a backward compatible behavior.

OK I'll use readonly if we're going for the reverse.
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-06-09.08:59:39

I found the name of the flag "writable" very astonishing and not related to SQL naming.
At the end I think we should keep by default a normal transaction and have an option to start a readonly transaction as SQL does. Because it is only if the user is calling commit that it will alter the data.
Also it is better to keep a backward compatible behavior.

Author: [hidden] (nicoe) Tryton committer
Date: 2021-06-08.17:52:13

Currently the transaction is by default a writable transaction.
It should default to a readonly transaction and the command line should have a switch to make it writable.

History
Date User Action Args
2021-06-16 14:59:53roundup-botsetmessages: + msg68294
2021-06-16 14:59:50roundup-botsetmessages: + msg68293
nosy: + roundup-bot
status: in-progress -> resolved
2021-06-15 19:31:13reviewbotsetmessages: + msg68277
2021-06-09 09:29:07reviewbotsetmessages: + msg68159
2021-06-09 09:06:03nicoesetmessages: + msg68158
2021-06-09 08:59:39cedsetmessages: + msg68157
nosy: + ced
2021-06-08 18:05:11reviewbotsetmessages: + msg68152
nosy: + reviewbot
2021-06-08 17:52:14nicoecreate

Showing 10 items. Show all history (warning: this could be VERY long)