Tryton - Issues

 

Issue5452

Title Don't create translation entry for empty string
Priority feature Status resolved
Superseder Nosy List ced, pokoli, reviewbot, roundup-bot
Type performance Components trytond
Assigned To ced Keywords review
Reviews 267251002
View: 267251002

Created on 2016-04-01.09:49:56 by ced, last changed by roundup-bot.

Messages
New changeset 6bc2ca141a05 by Cédric Krier in branch 'default':
Do not create empty translations
https://hg.tryton.org/tryton-env/rev/6bc2ca141a05
New changeset 9bf65e1188fb by Cédric Krier in branch 'default':
Do not create empty translations
https://hg.tryton.org/trytond/rev/9bf65e1188fb
review267251002 updated at https://codereview.tryton.org/267251002/#ps265301002
review267251002 updated at https://codereview.tryton.org/267251002/#ps251281002
msg49019 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2019-04-14.18:25:09
Here is review267251002.
I choose to not put a constraint on src for migration reason and also flexibility. Users may want to really create manually a translation for an empty string.
msg25369 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2016-04-01.09:49:55
We should not create translation entries for empty strings because they will always be translated as empty string. Also Pootle mark them as to translate and it is kind of annoying.
History
Date User Action Args
2019-04-15 16:08:13roundup-botsetmessages: + msg49047
2019-04-15 16:08:10roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg49046
2019-04-14 19:09:29reviewbotsetmessages: + msg49023
2019-04-14 18:41:44reviewbotsetnosy: + reviewbot
messages: + msg49021
2019-04-14 18:25:10cedsetstatus: unread -> testing
reviews: 267251002
messages: + msg49019
keyword: + review
assignedto: ced
2016-08-11 11:09:08cedunlinkissue5443 superseder
2016-08-11 09:48:22cedlinkissue5443 superseder
2016-04-05 11:09:14pokolisetnosy: + pokoli
2016-04-01 09:49:56cedcreate

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