home · contact · privacy
Improve ledger.py.
[misc] / ledger.py
index 6e0a482bffa4b66a57202d84027c1f718dfca933..2b2a80764104057ccf3e03ef62a29ecd291c15b1 100755 (executable)
--- a/ledger.py
+++ b/ledger.py
@@ -1,9 +1,9 @@
 from http.server import BaseHTTPRequestHandler, HTTPServer
-import sys
 import os
 import html
 import jinja2
 import decimal
+import datetime
 from urllib.parse import parse_qs, urlparse
 hostName = "localhost"
 serverPort = 8082
@@ -13,11 +13,6 @@ class HandledException(Exception):
     pass
 
 
-def handled_error_exit(msg):
-    print(f"ERROR: {msg}")
-    sys.exit(1)
-
-
 def apply_booking_to_account_balances(account_sums, account, currency, amount):
     if not account in account_sums:
         account_sums[account] = {currency: amount}
@@ -80,6 +75,7 @@ def parse_lines(lines, validate_bookings=True):
     bookings = []
     comments = []
     lines = lines.copy() + [''] # to ensure a booking-ending last line
+    last_date = ''
     for i, line in enumerate(lines):
         prefix = f"line {i}"
         # we start with the case of an utterly empty line
@@ -116,6 +112,9 @@ def parse_lines(lines, validate_bookings=True):
                 datetime.datetime.strptime(date_string, '%Y-%m-%d')
             except ValueError:
                 raise HandledException(f"{prefix} bad date string: {date_string}")
+            if last_date > date_string:
+                raise HandledException(f"{prefix} out-of-order-date")
+            last_date = date_string
             try:
                 description = toks[1]
             except IndexError:
@@ -181,11 +180,10 @@ def parse_lines(lines, validate_bookings=True):
                             raise HandledException(f"{prefix} amount has multiple dots: {value}")
                         dots_counted += 1
                     amount_string += c
-            if len(amount_string) == 0:
-                raise HandledException(f"{prefix} amount missing: {value}")
             if len(currency) == 0:
                 raise HandledException(f"{prefix} currency missing: {value}")
-            amount = decimal.Decimal(amount_string)
+            if len(amount_string) > 0:
+                amount = decimal.Decimal(amount_string)
         booking_lines += [(account_name, amount, currency)]
     if inside_booking:
         raise HandledException(f"{prefix} last booking unfinished")
@@ -275,31 +273,69 @@ class Database:
     def get_lines(self, start, end):
         return self.real_lines[start:end]
 
-    def replace(self, start, end, lines):
+    def write_db(self, text, mode='w'):
         import shutil
         if os.path.exists(self.lock_file):
             raise HandledException('Sorry, lock file!')
-        if os.path.exists(self.db_file):
-            shutil.copy(self.db_file, self.db_file + ".bak")
         f = open(self.lock_file, 'w+')
         f.close()
-        total_lines = self.real_lines[:start] + lines + self.real_lines[end:]
-        text = '\n'.join(total_lines)
-        with open(self.db_file, 'w') as f:
+
+        # always back up most recent to .bak
+        bakpath = f'{self.db_file}.bak'
+        shutil.copy(self.db_file, bakpath)
+
+        # collect modification times of numbered .bak files
+        bak_prefix = f'{bakpath}.'
+        backup_dates = []
+        i = 0
+        bak_as = f'{bak_prefix}{i}'
+        while os.path.exists(bak_as):
+            mod_time = os.path.getmtime(bak_as)
+            backup_dates += [str(datetime.datetime.fromtimestamp(mod_time))]
+            i += 1
+            bak_as = f'{bak_prefix}{i}'
+
+        # collect what numbered .bak files to save
+        to_save = []
+        datetime_len = 19
+        now = str(datetime.datetime.now())[:datetime_len]
+        while datetime_len > 2:
+            for i, date in reversed(list(enumerate(backup_dates))):
+                if date[:datetime_len] == now:
+                    if i not in to_save:
+                        to_save += [i] 
+                        break
+            datetime_len -= 1 
+            now = now[:datetime_len]
+
+        # remove redundant backup files 
+        j = 0
+        for i in reversed(to_save):
+            if i != j:
+                source = f'{bak_prefix}{i}'
+                target = f'{bak_prefix}{j}'
+                shutil.move(source, target)
+            j += 1
+        for i in range(j, len(backup_dates)):
+            try:
+                os.remove(f'{bak_prefix}{i}')
+            except FileNotFoundError:
+                pass
+
+        # 
+        shutil.copy(self.db_file, f'{bak_prefix}{j}')
+        with open(self.db_file, mode) as f:
             f.write(text);
         os.remove(self.lock_file)
 
+    def replace(self, start, end, lines):
+        total_lines = self.real_lines[:start] + lines + self.real_lines[end:]
+        text = '\n'.join(total_lines)
+        self.write_db(text)
+
     def append(self, lines):
-        import shutil
-        if os.path.exists(self.lock_file):
-            raise HandledException('Sorry, lock file!')
-        if os.path.exists(self.db_file):
-            shutil.copy(self.db_file, self.db_file + ".bak")
-        f = open(self.lock_file, 'w+')
-        f.close()
-        with open(self.db_file, 'a') as f:
-            f.write('\n\n' + '\n'.join(lines) + '\n\n');
-        os.remove(self.lock_file)
+        text = '\n\n' + '\n'.join(lines) + '\n\n'
+        self.write_db(text, 'a')
 
     def add_taxes(self, lines, finish=False):
         ret = []
@@ -318,6 +354,7 @@ class Database:
         buffer_expenses = 0
         kk_expenses = 0
         est_expenses = 0
+        months_passed = -int(finish) 
         for b in self.bookings:
             if date == b.date_string:
                 break
@@ -330,11 +367,13 @@ class Database:
                 kk_expenses += b.account_changes[acc_kk]['€']
             if acc_est in acc_keys:
                 est_expenses += b.account_changes[acc_est]['€']
-            if finish and acc_kk_add in acc_keys and acc_kk_minimum in acc_keys:
-                last_monthbreak_kk_add = b.account_changes[acc_kk_add]['€']
-                last_monthbreak_est = b.account_changes[acc_est]['€']
-                last_monthbreak_kk_minimum = b.account_changes[acc_kk_minimum]['€']
-                last_monthbreak_assets = b.account_changes[acc_buffer]['€']
+            if acc_kk_add in acc_keys and acc_kk_minimum in acc_keys:
+                months_passed += 1
+                if finish:
+                    last_monthbreak_kk_add = b.account_changes[acc_kk_add]['€']
+                    last_monthbreak_est = b.account_changes[acc_est]['€']
+                    last_monthbreak_kk_minimum = b.account_changes[acc_kk_minimum]['€']
+                    last_monthbreak_assets = b.account_changes[acc_buffer]['€']
         old_needed_income = last_monthbreak_assets + last_monthbreak_kk_add + last_monthbreak_kk_minimum + last_monthbreak_est
         if finish:
             ret += [f'  {acc_est}  {-last_monthbreak_est}€ ; for old assumption of needed income: {old_needed_income}€']
@@ -349,7 +388,6 @@ class Database:
         E1 = decimal.Decimal(15999)
         E2 = decimal.Decimal(62809)
         E3 = decimal.Decimal(277825)
-        months_passed = int(date[5:7]) - 1
         while True:
             zvE = buffer_expenses - kk_expenses + (12 - months_passed) * needed_income_before_kk
             if finish:
@@ -401,7 +439,8 @@ class Database:
         final_minus = expenses_so_far + old_needed_income + diff
         ret += [f'  {acc_assets}  {-diff} €']
         ret += [f'  {acc_assets}  {final_minus} €']
-        ret += [f'  {acc_buffer}  {-final_minus} €']
+        year_needed = buffer_expenses + final_minus + (12 - months_passed - 1) * final_minus 
+        ret += [f'  {acc_buffer}  {-final_minus} € ; assume as to earn in year: {acc_buffer} + {12 - months_passed - 1} * this = {-year_needed}']
         return ret
 
 
@@ -419,20 +458,19 @@ input[type=number] { text-align: right; font-family: monospace; }
 .full_line_comment { display: block; white-space: nowrap; width: 0; }
 </style>
 <body>
-<a href="/ledger1">ledger1</a>
-<a href="/ledger2">ledger2</a>
+<a href="/">ledger</a>
 <a href="/balance">balance</a>
 <a href="/add_free">add free</a>
 <a href="/add_structured">add structured</a>
 <hr />
 """
     booking_tmpl = jinja2.Template("""
-<p id="{{start}}"><a href="#{{start}}">{{date}}</a> {{desc}} <span class="comment">{{head_comment|e}}</span><br />
+<p id="{{nth}}"><a href="#{{nth}}">{{date}}</a> {{desc}} <span class="comment">{{head_comment|e}}</span><br />
 <span class="meta">[edit: <a href="/add_structured?start={{start}}&end={{end}}">structured</a>
 / <a href="/add_free?start={{start}}&end={{end}}">free</a>
 | copy:<a href="/copy_structured?start={{start}}&end={{end}}">structured</a>
 / <a href="/copy_free?start={{start}}&end={{end}}">free</a>
-| balance <a href="/balance?stop={{date}}&plus={{steps_after_date}}">before</a> or <a href="/balance?stop={{date}}&plus={{steps_after_date + 1}}">after</a>
+| <a href="/balance?stop={{nth+1}}">balance after</a>
 ]</span>
 <table>
 {% for l in booking_lines %}
@@ -444,122 +482,137 @@ input[type=number] { text-align: right; font-family: monospace; }
 {% endfor %}
 </table></p>
 """)
+    add_form_footer = """
+<input type="hidden" name="start" value={{start}} />
+<input type="hidden" name="end" value={{end}} />
+<input type="submit" name="save" value="save!">
+</form>
+"""
     footer = "</body>\n<html>"
 
     def do_POST(self):
-        db = Database()
-        length = int(self.headers['content-length'])
-        postvars = parse_qs(self.rfile.read(length).decode(), keep_blank_values=1)
-        parsed_url = urlparse(self.path)
-        lines = []
-        add_empty_line = None
-        start = int(postvars['start'][0])
-        end = int(postvars['end'][0])
-        if '/add_structured' == parsed_url.path and not 'revert' in postvars.keys():
-            date = postvars['date'][0]
-            description = postvars['description'][0]
-            start_comment = postvars['line_0_comment'][0]
-            lines = [f'{date} {description} ; {start_comment}']
-            if 'line_0_add' in postvars.keys():
-                add_empty_line = 0
-            i = j = 1
-            while f'line_{i}_comment' in postvars.keys():
-                if f'line_{i}_delete' in postvars.keys():
-                    i += 1
-                    continue
-                elif f'line_{i}_delete_after' in postvars.keys():
-                    break 
-                elif f'line_{i}_add' in postvars.keys():
-                    add_empty_line = j
-                account = postvars[f'line_{i}_account'][0]
-                amount = postvars[f'line_{i}_amount'][0]
-                currency = postvars[f'line_{i}_currency'][0]
-                comment = postvars[f'line_{i}_comment'][0]
-                i += 1
-                new_main = f'{account} {amount} {currency}'
-                if '' == new_main.rstrip() == comment.rstrip():  # don't write empty lines
-                    continue
-                j += 1
-                new_line = new_main
-                if comment.rstrip() != '':
-                    new_line += f' ; {comment}'
-                lines += [new_line]
-            if 'add_taxes' in postvars.keys():
-                lines += db.add_taxes(lines, finish=False)
-            elif 'add_taxes2' in postvars.keys():
-                lines += db.add_taxes(lines, finish=True)
-        elif '/add_free' == parsed_url.path:
-            lines = postvars['booking'][0].splitlines()
         try:
+            parsed_url = urlparse(self.path)
+            length = int(self.headers['content-length'])
+            postvars = parse_qs(self.rfile.read(length).decode(), keep_blank_values=1)
+            start = int(postvars['start'][0])
+            end = int(postvars['end'][0])
+
+            db = Database()
+            add_empty_line = None
+            lines = []
+            if '/add_structured' == parsed_url.path and not 'revert' in postvars.keys():
+                date = postvars['date'][0]
+                description = postvars['description'][0]
+                start_comment = postvars['line_0_comment'][0]
+                lines = [f'{date} {description} ; {start_comment}']
+                if 'line_0_add' in postvars.keys():
+                    add_empty_line = 0
+                i = j = 1
+                while f'line_{i}_comment' in postvars.keys():
+                    if f'line_{i}_delete' in postvars.keys():
+                        i += 1
+                        continue
+                    elif f'line_{i}_delete_after' in postvars.keys():
+                        break 
+                    elif f'line_{i}_add' in postvars.keys():
+                        add_empty_line = j
+                    account = postvars[f'line_{i}_account'][0]
+                    amount = postvars[f'line_{i}_amount'][0]
+                    currency = postvars[f'line_{i}_currency'][0]
+                    comment = postvars[f'line_{i}_comment'][0]
+                    i += 1
+                    new_main = f'  {account}  {amount}'
+                    if '' == new_main.rstrip() == comment.rstrip():  # don't write empty lines, ignore currency if nothing else set
+                        continue
+                    if len(amount.rstrip()) > 0:
+                        new_main += f' {currency}'
+                    j += 1
+                    new_line = new_main
+                    if comment.rstrip() != '':
+                        new_line += f'  ; {comment}'
+                    lines += [new_line]
+                if 'add_sink' in postvars.keys():
+                    temp_lines = lines.copy() + ['_']
+                    try:
+                        temp_bookings, _ = parse_lines(temp_lines)
+                        for currency in temp_bookings[0].sink:
+                            amount = temp_bookings[0].sink[currency]
+                            lines += [f'Assets  {amount:.2f} {currency}']
+                    except HandledException:
+                        pass
+                if 'add_taxes' in postvars.keys():
+                    lines += db.add_taxes(lines, finish=False)
+                elif 'add_taxes2' in postvars.keys():
+                    lines += db.add_taxes(lines, finish=True)
+            elif '/add_free' == parsed_url.path:
+                lines = postvars['booking'][0].splitlines()
+
             if ('save' in postvars.keys()) or ('check' in postvars.keys()):
                 _, _ = parse_lines(lines)
+
             if 'save' in postvars.keys():
                 if start == end == 0:
                     db.append(lines)
                     redir_url = f'/#last'
                 else:
                     db.replace(start, end, lines)
-                    redir_url = f'/#{start}'
-                self.send_response(301)
-                self.send_header('Location', redir_url)
-                self.end_headers()
-            else:
+                    nth = 0
+                    for i, b in enumerate(db.bookings):
+                        if b.start_line == start:
+                            nth = i
+                            break
+                    redir_url = f'/#{nth}'
+                self.send_code_and_headers(301, [('Location', redir_url)])
+            else:  # implicit assumption: this only happens on /add_structured flows
                 page = self.header + self.add_structured(db, start, end, temp_lines=lines, add_empty_line=add_empty_line) + self.footer
-                self.send_response(200)
-                self.send_header("Content-type", "text/html")
-                self.end_headers()
-                self.wfile.write(bytes(page, "utf-8"))
+                self.send_HTML(page)
         except HandledException as e:
-            self.send_response(400)
-            self.send_header("Content-type", "text/html")
-            self.end_headers()
-            page = f'{self.header}ERROR: {e}{self.footer}'
-            self.wfile.write(bytes(page, "utf-8"))
+            self.fail_400(e)
 
     def do_GET(self):
-        self.send_response(200)
-        self.send_header("Content-type", "text/html")
+        try:
+            parsed_url = urlparse(self.path)
+            params = parse_qs(parsed_url.query)
+            start = int(params.get('start', ['0'])[0])
+            end = int(params.get('end', ['0'])[0])
+            db = Database()
+            page = self.header
+            if parsed_url.path == '/balance':
+                stop = params.get('stop', [None])[0]
+                page += self.balance_as_html(db, stop)
+            elif parsed_url.path == '/add_free':
+                page += self.add_free(db, start, end)
+            elif parsed_url.path == '/add_structured':
+                page += self.add_structured(db, start, end)
+            elif parsed_url.path == '/copy_free':
+                page += self.add_free(db, start, end, copy=True)
+            elif parsed_url.path == '/copy_structured':
+                page += self.add_structured(db, start, end, copy=True)
+            else:
+                page += self.ledger_as_html(db)
+            page += self.footer
+            self.send_HTML(page)
+        except HandledException as e:
+            self.fail_400(e)
+
+    def fail_400(self, e):
+        page = f'{self.header}ERROR: {e}{self.footer}'
+        self.send_HTML(page, 400)
+
+    def send_HTML(self, html, code=200):
+        self.send_code_and_headers(code, [('Content-type', 'text/html')])
+        self.wfile.write(bytes(html, "utf-8"))
+
+    def send_code_and_headers(self, code, headers=[]):
+        self.send_response(code)
+        for fieldname, content in headers:
+            self.send_header(fieldname, content)
         self.end_headers()
-        db = Database()
-        parsed_url = urlparse(self.path)
-        page = self.header + ''
-        params = parse_qs(parsed_url.query)
-        start = int(params.get('start', ['0'])[0])
-        end = int(params.get('end', ['0'])[0])
-        if parsed_url.path == '/balance':
-            stop_date = params.get('stop', [None])[0]
-            plus = int(params.get('plus', [0])[0])
-            page += self.balance_as_html(db, stop_date, plus)
-        elif parsed_url.path == '/add_free':
-            page += self.add_free(db, start, end)
-        elif parsed_url.path == '/add_structured':
-            page += self.add_structured(db, start, end)
-        elif parsed_url.path == '/copy_free':
-            page += self.add_free(db, start, end, copy=True)
-        elif parsed_url.path == '/copy_structured':
-            page += self.add_structured(db, start, end, copy=True)
-        elif parsed_url.path == '/ledger2':
-            page += self.ledger2_as_html(db)
-        else:
-            page += self.ledger_as_html(db)
-        page += self.footer
-        self.wfile.write(bytes(page, "utf-8"))
 
-    def balance_as_html(self, db, stop_date=None, plus=0):
+    def balance_as_html(self, db, until=None):
+        bookings = db.bookings[:until if until is None else int(until)]
         lines = []
-        bookings = db.bookings
-        if stop_date:
-            bookings = []
-            i = -1
-            for b in db.bookings:
-                if b.date_string == stop_date:
-                    if -1 == i:
-                        i = 0
-                if i >= plus:
-                    break
-                if i > -1:
-                    i += 1
-                bookings += [b]
         account_tree, account_sums = bookings_to_account_tree(bookings)
         def print_subtree(lines, indent, node, subtree, path):
             line = f"{indent}{node}"
@@ -583,83 +636,19 @@ input[type=number] { text-align: right; font-family: monospace; }
         return f"<pre>{content}</pre>"
 
     def ledger_as_html(self, db):
-        elements_to_write = []
-        account_sums = {}  ##
-        nth_of_same_date = 0
-        same_date = ''
-        for booking in db.bookings:
-            if booking.date_string == same_date:
-                nth_of_same_date += 1
-            else:
-                same_date = booking.date_string
-                nth_of_same_date = 0
-            booking_end = booking.start_line + len(booking.lines)
-            booking_lines = []
-            for booking_line in booking.lines[1:]:
-                if booking_line == '':
-                    continue
-                account = booking_line[0]  ##
-                account_toks = account.split(':')  ##
-                path = ''  ##
-                for tok in account_toks: ##
-                    path += tok  ##
-                    if not path in account_sums.keys():  ##
-                        account_sums[path] = {}  ##
-                    path += ':'  ##
-                moneys = []  ##
-                money = ''
-                if booking_line[1] is not None:
-                    moneys += [(booking_line[1], booking_line[2])]  ##
-                    money = f'{moneys[0][0]} {moneys[0][1]}'
-                else:  ##
-                    for currency, amount in booking.sink.items():  ##
-                        moneys += {(amount, currency)}  ##
-                    money = '['  ##
-                    for m in moneys:  ##
-                        money += f'{m[0]} {m[1]} ' ##
-                    money += ']'  ##
-                balance = ''  ##
-                for amount, currency in moneys:  ##
-                    path = ''  ##
-                    for tok in account_toks:  ##
-                        path += tok  ##
-                        if not currency in account_sums[path].keys():  ##
-                            account_sums[path][currency] = 0  ##
-                        account_sums[path][currency] += amount  ##
-                        path += ':'   ##
-                    balance += f'{account_sums[account][currency]} {currency}'  ##
-                booking_lines += [{'acc': booking_line[0], 'money':money, 'comment':balance}]  ##
-            elements_to_write += [self.booking_tmpl.render(
-                start=booking.start_line,
-                end=booking_end,
-                date=booking.date_string,
-                desc=booking.description,
-                head_comment=db.comments[booking.start_line],
-                steps_after_date=nth_of_same_date,
-                booking_lines = booking_lines)]
-        return '\n'.join(elements_to_write)
-
-    def ledger2_as_html(self, db):
-        single_c_tmpl = jinja2.Template('<span class="comment">{{c|e}}</span><br />')
+        single_c_tmpl = jinja2.Template('<span class="comment">{{c|e}}</span><br />')  ##
         elements_to_write = []
         last_i = i = 0  ##
-        nth_of_same_date = 0
-        same_date = ''
-        for booking in db.bookings:
-            if booking.date_string == same_date:
-                nth_of_same_date += 1
-            else:
-                same_date = booking.date_string
-                nth_of_same_date = 0
-            i = booking.start_line  ##
-            elements_to_write += [single_c_tmpl.render(c=c) for c in db.comments[last_i:i] if c != '']  ##
+        for nth, booking in enumerate(db.bookings):
             booking_end = last_i = booking.start_line + len(booking.lines)
             booking_lines = []
+            i = booking.start_line  ##
+            elements_to_write += [single_c_tmpl.render(c=c) for c in db.comments[last_i:i] if c != '']  ##
             for booking_line in booking.lines[1:]:
-                 i += 1
-                 comment = db.comments[i]
+                 i += 1  ##
+                 comment = db.comments[i]  ##
                  if booking_line == '':
-                     booking_lines += [{'acc': None, 'money': None, 'comment': comment}]
+                     booking_lines += [{'acc': None, 'money': None, 'comment': comment}]  ##
                      continue
                  account = booking_line[0]
                  money = ''
@@ -667,14 +656,14 @@ input[type=number] { text-align: right; font-family: monospace; }
                      money = f'{booking_line[1]} {booking_line[2]}'
                  booking_lines += [{'acc': booking_line[0], 'money':money, 'comment':comment}]  ##
             elements_to_write += [self.booking_tmpl.render(
+                nth=nth,
                 start=booking.start_line,
                 end=booking_end,
                 date=booking.date_string,
                 desc=booking.description,
                 head_comment=db.comments[booking.start_line],
-                steps_after_date=nth_of_same_date,
                 booking_lines = booking_lines)]
-        elements_to_write += [single_c_tmpl.render(c=c) for c in db.comments[last_i:] if c != '']
+        elements_to_write += [single_c_tmpl.render(c=c) for c in db.comments[last_i:] if c != '']  #
         return '\n'.join(elements_to_write)
 
     def add_free(self, db, start=0, end=0, copy=False):
@@ -684,11 +673,7 @@ input[type=number] { text-align: right; font-family: monospace; }
 {% for line in lines %}{{ line }}
 {% endfor %}
 </textarea>
-<input type="hidden" name="start" value={{start}} />
-<input type="hidden" name="end" value={{end}} />
-<input type="submit" name="save" value="save!">
-</form>
-""")
+""" + self.add_form_footer)
         lines = db.get_lines(start, end)
         if copy:
             start = end = 0
@@ -701,6 +686,7 @@ input[type=number] { text-align: right; font-family: monospace; }
 <input type="submit" name="revert" value="revert" />
 <input type="submit" name="add_taxes" value="add taxes" />
 <input type="submit" name="add_taxes2" value="add taxes2" />
+<input type="submit" name="add_sink" value="add sink" />
 <br />
 <input name="date" value="{{date|e}}" size=9 />
 <input name="description" value="{{desc|e}}" list="descriptions" />
@@ -724,12 +710,7 @@ input[type=number] { text-align: right; font-family: monospace; }
 {% endfor %}
 </datalist>
 {% endfor %}
-<input type="hidden" name="start" value={{start}} />
-<input type="hidden" name="end" value={{end}} />
-<input type="submit" name="save" value="save!">
-</form>
-""")
-        import datetime
+""" + self.add_form_footer)
         lines = temp_lines if len(''.join(temp_lines)) > 0 else db.get_lines(start, end)
         bookings, comments = parse_lines(lines, validate_bookings=False)
         if len(bookings) > 1:
@@ -754,7 +735,7 @@ input[type=number] { text-align: right; font-family: monospace; }
         desc = head_comment = ''
         if len(bookings) == 0:
             for i in range(1, 3):
-                booking_lines += [{'i': i, 'acc': '', 'amt': '', 'curr': '', 'comment': ''}]
+                booking_lines += [{'i': i, 'acc': '', 'amt': '', 'curr': '', 'comment': ''}]
             date=today
         else:
             booking = bookings[0]
@@ -772,7 +753,7 @@ input[type=number] { text-align: right; font-family: monospace; }
                         'i': i,
                         'acc': account,
                         'amt': amount,
-                        'curr': currency if currency else '',
+                        'curr': currency if currency else '',
                         'comment': comments[i],
                         'comm_cols': len(comments[i])}]
         content += tmpl.render(