[tryton-commits] changeset in modules/account:default Use getattr instead of test...

[Cédric Krier]

changeset 64b7fe636940 in modules/account:default
details: https://hg.tryton.org/modules/account?cmd=changeset;node=64b7fe636940
description:
        Use getattr instead of testing origin id

        Since changeset b8d76b07910f, the reference value of a negative id is a 
string
        so testing the id does not work. It is safer to call getattr with a 
default
        empty string value.

        issue9500
        review304611002
diffstat:

 general_ledger.fodt |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r 8e089622f33c -r 64b7fe636940 general_ledger.fodt
--- a/general_ledger.fodt       Sat Oct 24 13:45:09 2020 +0200
+++ b/general_ledger.fodt       Sun Oct 25 11:02:23 2020 +0100
@@ -964,7 +964,7 @@
       <text:p text:style-name="P10"><text:placeholder 
text:placeholder-type="text">&lt;line.description or 
&apos;&apos;&gt;</text:placeholder><text:line-break/><text:placeholder 
text:placeholder-type="text">&lt;line.move_description or 
&apos;&apos;&gt;</text:placeholder></text:p>
      </table:table-cell>
      <table:table-cell table:style-name="Table3.G11" 
office:value-type="string">
-      <text:p text:style-name="P10"><text:placeholder 
text:placeholder-type="text">&lt;line.origin.rec_name if line.origin and 
line.origin.id &gt;= 0 else &apos;&apos;&gt;</text:placeholder></text:p>
+      <text:p text:style-name="P10"><text:placeholder 
text:placeholder-type="text">&lt;getattr(line.origin, &apos;rec_name&apos;, 
&apos;&apos;)&gt;</text:placeholder></text:p>
      </table:table-cell>
      <table:table-cell table:style-name="Table3.H11" 
office:value-type="string">
       <text:p text:style-name="P10"><text:placeholder 
text:placeholder-type="text">&lt;line.state_string&gt;</text:placeholder></text:p>