From 8822ee7e3069193162469438bb54ee4629e1ae19 Mon Sep 17 00:00:00 2001
From: Christian Heller  {{date}} {{desc}} {{head_comment|e}}
-"""
-booking_html = """
-
-
-
-{% for l in booking_lines %}
-{% if l.acc %}
-
{{l.acc|e}} {{l.money|e}} {{l.comment|e}} 
{{date}} {{desc}} {{head_comment|e}}
+# 
+# 
| {{l.acc|e}} | {{l.money|e}} | {{l.comment|e}} | 
| {{l.comment|e}} | 
{content}"
-
-    def add_free(self, start=0, end=0, copy=False):
-        tmpl = jinja2.Template(add_form_header + add_free_html + add_form_footer) 
-        lines = self.get_lines(start, end)
-        if copy:
-            start = end = 0
-        return tmpl.render(action=self.prefix + '/add_free', start=start, end=end, lines=lines)
-
-    def add_structured(self, start=0, end=0, copy=False, temp_lines=[], add_empty_line=None):
-        tmpl = jinja2.Template(add_form_header + add_structured_html + add_form_footer) 
-        lines = temp_lines if len(''.join(temp_lines)) > 0 else self.get_lines(start, end)
-        bookings, comments = parse_lines(lines, validate_bookings=False)
-        if len(bookings) > 1:
-            raise PlomException('can only structurally edit single Booking')
-        if add_empty_line is not None:
-            comments = comments[:add_empty_line+1] + [''] + comments[add_empty_line+1:]
-            booking = bookings[0]
-            booking.lines = booking.lines[:add_empty_line+1] + [''] + booking.lines[add_empty_line+1:]
-        action = self.prefix + '/add_structured'
-        datalist_sets = {'descriptions': set(), 'accounts': set(), 'currencies': set()}
-        for b in self.bookings:
-            datalist_sets['descriptions'].add(b.description)
-            for account, moneys in b.account_changes.items():
-                datalist_sets['accounts'].add(account)
-                for currency in moneys.keys():
-                    datalist_sets['currencies'].add(currency)
-        content = ''
-        today = str(datetime.now())[:10]
-        booking_lines = []
-        if copy:
-            start = end = 0
-        desc = head_comment = ''
-        if len(bookings) == 0:
-            date=today
+        bookings = self.bookings[:(until if until is None else int(until)+1)]
+        account_trunk = Account('', None)
+        accounts = {account_trunk.full_name: account_trunk}
+        for booking in bookings:
+            for full_account_name, moneys in booking.account_changes.items():
+                toks = full_account_name.split(':')
+                path = [] 
+                for tok in toks:
+                    parent_name = ':'.join(path) 
+                    path += [tok] 
+                    account_name = ':'.join(path)
+                    if not account_name in accounts.keys():
+                        accounts[account_name] = Account(own_name=tok, parent=accounts[parent_name])
+                accounts[full_account_name].add_wealth(moneys)
+        class Node:
+            def __init__(self, indent, name, moneys):
+                self.indent = indent
+                self.name = name
+                self.moneys = moneys.money_dict
+        nodes = []
+        def walk_tree(nodes, indent, account):
+            nodes += [Node(indent, account.own_name, account.full_moneys)]
+            for child in account.children:
+                walk_tree(nodes, indent+1, child)
+        for acc in account_trunk.children:
+            walk_tree(nodes, 0, acc)
+        return j2env.get_template('balance.html').render(nodes=nodes)
+
+    # def balance_as_html(self, until=None):
+    #     bookings = self.bookings[:until if until is None else int(until)]
+    #     lines = []
+    #     account_tree, account_sums = bookings_to_account_tree(bookings)
+    #     def print_subtree(lines, indent, node, subtree, path):
+    #         line = f"{indent}{node}"
+    #         n_tabs = 5 - (len(line) // 8)
+    #         line += n_tabs * "\t"
+    #         if "â¬" in account_sums[path + node].keys():
+    #             amount = account_sums[path + node]["â¬"]
+    #             line += f"{amount:9.2f} â¬\t"
+    #         else:
+    #             line += f"\t\t"
+    #         for currency, amount in account_sums[path + node].items():
+    #             if currency != 'â¬' and amount != 0:
+    #                 line += f"{amount:5.2f} {currency}\t"
+    #         lines += [line]
+    #         indent += "  "
+    #         for k, v in sorted(subtree.items()):
+    #             print_subtree(lines, indent, k, v, path + node + ":")
+    #     for k, v in sorted(account_tree.items()):
+    #         print_subtree(lines, "", k, v, "")
+    #     content = "\n".join(lines)
+    #     return f"{content}"
+
+    def edit(self, index, sent=None, error_msg=None, edit_mode='table'):
+        accounts = set() 
+        if sent or -1 == index:
+            content = sent if sent else ([] if 'textarea'==edit_mode else None)
         else:
-            booking = bookings[0]
-            desc = booking.description
-            date = today if copy else booking.date_string
-            head_comment=comments[0]
-            for i in range(1, len(comments)):
-                account = amount = currency = ''
-                if i < len(booking.lines) and booking.lines[i] != '':
-                    account = booking.lines[i][0]
-                    amount = booking.lines[i][1]
-                    currency = booking.lines[i][2]
-                booking_lines += [{
-                        'i': i,
-                        'acc': account,
-                        'amt': amount,
-                        'curr': currency if currency else 'â¬',
-                        'comment': comments[i],
-                        'comm_cols': len(comments[i])}]
-        for i in range(len(comments), len(comments) + 8):
-            booking_lines += [{'i': i, 'acc': '', 'amt': '', 'curr': 'â¬', 'comment': ''}]
-        content += tmpl.render(
-                action=action,
-                date=date,
-                desc=desc,
-                head_comment=head_comment,
-                booking_lines=booking_lines,
-                datalist_sets=datalist_sets,
-                start=start,
-                end=end)
-        return content
-
-    def move_up(self, start, end):
-        prev_booking = None
-        for redir_nth, b in enumerate(self.bookings):
-            if b.start_line >= start:
-                break
-            prev_booking = b
-        start_at = prev_booking.start_line 
-        self.make_move(start, end, start_at)
-        return redir_nth - 1
-
-    def move_down(self, start, end):
-        next_booking = None
-        for redir_nth, b in enumerate(self.bookings):
-            if b.start_line > start:
-                next_booking = b
-                break
-        # start_at = next_booking.start_line + len(next_booking.lines) - (end - start) + 1 
-        # self.make_move(start, end, start_at-1)
-        start_at = next_booking.start_line + len(next_booking.lines) - (end - start)
-        print("DEBUG", start, end, start_at)
-        self.make_move(start, end, start_at)
-        return redir_nth
-
-    def make_move(self, start, end, start_at):
-        # FIXME currently broken due to changed self.write_lines_in_total_lines_at, easy fix would be lines += [""] maybe?
-        lines = self.get_lines(start, end)
-        if start == 0:
-            total_lines = self.real_lines[end+1:]
-            lines = [''] + lines
-            start_at += 1
-        else: 
-            total_lines = self.real_lines[:start-1] + self.real_lines[end:]  # -1 because we reduce the original position's two empty limit lines to one in-between line
-            lines += ['']
-        self.write_lines_in_total_lines_at(total_lines, start_at, lines)
-
-    def booking_lines_from_postvars(self, postvars):
-        add_empty_line = None
-        date = postvars['date'][0]
-        description = postvars['description'][0]
-        start_comment = postvars['line_0_comment'][0]
-        start_line = f'{date} {description}'
-        if start_comment.rstrip() != '':
-            start_line += f' ; {start_comment}' 
-        lines = [start_line]
-        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}']
-                    lines += [f'Assets  {amount} {currency}']
-            except PlomException:
-                pass
-        elif 'add_taxes' in postvars.keys():
-            lines += self.add_taxes(lines, finish=False)
-        elif 'add_taxes2' in postvars.keys():
-            lines += self.add_taxes(lines, finish=True)
-        elif 'replace' in postvars.keys():
-            for i, line in enumerate(lines):
-                lines[i] = line.replace(postvars['replace_from'][0], postvars['replace_to'][0])
-        elif 'add_mirror' in postvars.keys():
-            lines += self.add_mirror(lines)
-        return lines, add_empty_line
+            content = self.bookings[index]
+        if 'textarea' == edit_mode and content:
+            content = content.for_writing()
+        else:
+            for booking in self.bookings:
+                for transfer_line in booking.transfer_lines:
+                    accounts.add(transfer_line.account)
+        return j2env.get_template('edit.html').render(content=content, index=index, error_msg=error_msg, edit_mode=edit_mode, accounts=accounts)
+
+    # def add_free(self, start=0, end=0, copy=False):
+    #     tmpl = jinja2.Template(add_form_header + add_free_html + add_form_footer) 
+    #     lines = self.get_lines(start, end)
+    #     if copy:
+    #         start = end = 0
+    #     return tmpl.render(action=self.prefix + '/add_free', start=start, end=end, lines=lines)
+
+    # def add_structured(self, start=0, end=0, copy=False, temp_lines=[], add_empty_line=None):
+    #     tmpl = jinja2.Template(add_form_header + add_structured_html + add_form_footer) 
+    #     lines = temp_lines if len(''.join(temp_lines)) > 0 else self.get_lines(start, end)
+    #     bookings, comments = parse_lines(lines, validate_bookings=False)
+    #     if len(bookings) > 1:
+    #         raise PlomException('can only structurally edit single Booking')
+    #     if add_empty_line is not None:
+    #         comments = comments[:add_empty_line+1] + [''] + comments[add_empty_line+1:]
+    #         booking = bookings[0]
+    #         booking.lines = booking.lines[:add_empty_line+1] + [''] + booking.lines[add_empty_line+1:]
+    #     action = self.prefix + '/add_structured'
+    #     datalist_sets = {'descriptions': set(), 'accounts': set(), 'currencies': set()}
+    #     for b in self.bookings:
+    #         datalist_sets['descriptions'].add(b.description)
+    #         for account, moneys in b.account_changes.items():
+    #             datalist_sets['accounts'].add(account)
+    #             for currency in moneys.keys():
+    #                 datalist_sets['currencies'].add(currency)
+    #     content = ''
+    #     today = str(datetime.now())[:10]
+    #     booking_lines = []
+    #     if copy:
+    #         start = end = 0
+    #     desc = head_comment = ''
+    #     if len(bookings) == 0:
+    #         date=today
+    #     else:
+    #         booking = bookings[0]
+    #         desc = booking.description
+    #         date = today if copy else booking.date_string
+    #         head_comment=comments[0]
+    #         for i in range(1, len(comments)):
+    #             account = amount = currency = ''
+    #             if i < len(booking.lines) and booking.lines[i] != '':
+    #                 account = booking.lines[i][0]
+    #                 amount = booking.lines[i][1]
+    #                 currency = booking.lines[i][2]
+    #             booking_lines += [{
+    #                     'i': i,
+    #                     'acc': account,
+    #                     'amt': amount,
+    #                     'curr': currency if currency else 'â¬',
+    #                     'comment': comments[i],
+    #                     'comm_cols': len(comments[i])}]
+    #     for i in range(len(comments), len(comments) + 8):
+    #         booking_lines += [{'i': i, 'acc': '', 'amt': '', 'curr': 'â¬', 'comment': ''}]
+    #     content += tmpl.render(
+    #             action=action,
+    #             date=date,
+    #             desc=desc,
+    #             head_comment=head_comment,
+    #             booking_lines=booking_lines,
+    #             datalist_sets=datalist_sets,
+    #             start=start,
+    #             end=end)
+    #     return content
+
+    def move_up(self, nth):
+        return self.move(nth, -1) 
+
+    def move_down(self, nth):
+        return self.move(nth, +1) 
+
+    def move(self, nth, direction):
+        to_move = self.bookings[nth]
+        swap_nth = nth+1*(direction)
+        to_swap = self.bookings[swap_nth]
+        self.bookings[nth] = to_swap 
+        self.bookings[nth+1*(direction)] = to_move 
+        return swap_nth
+
+    def write_db(self):
+        lines = []
+        for i, booking in enumerate(self.bookings):
+            if i > 0:
+                lines += ['']
+            lines += booking.for_writing()
+        self.write_text_to_db('\n'.join(lines) + '\n')
+
+    # def move_up(self, start, end):
+    #     prev_booking = None
+    #     for redir_nth, b in enumerate(self.bookings):
+    #         if b.start_line >= start:
+    #             break
+    #         prev_booking = b
+    #     start_at = prev_booking.start_line 
+    #     self.make_move(start, end, start_at)
+    #     return redir_nth - 1
+
+    # def move_down(self, start, end):
+    #     next_booking = None
+    #     for redir_nth, b in enumerate(self.bookings):
+    #         if b.start_line > start:
+    #             next_booking = b
+    #             break
+    #     # start_at = next_booking.start_line + len(next_booking.lines) - (end - start) + 1 
+    #     # self.make_move(start, end, start_at-1)
+    #     start_at = next_booking.start_line + len(next_booking.lines) - (end - start)
+    #     self.make_move(start, end, start_at)
+    #     return redir_nth
+
+    # def make_move(self, start, end, start_at):
+    #     # FIXME currently broken due to changed self.write_lines_in_total_lines_at, easy fix would be lines += [""] maybe?
+    #     lines = self.get_lines(start, end)
+    #     if start == 0:
+    #         total_lines = self.real_lines[end+1:]
+    #         lines = [''] + lines
+    #         start_at += 1
+    #     else: 
+    #         total_lines = self.real_lines[:start-1] + self.real_lines[end:]  # -1 because we reduce the original position's two empty limit lines to one in-between line
+    #         lines += ['']
+    #     self.write_lines_in_total_lines_at(total_lines, start_at, lines)
+
+    # def booking_lines_from_postvars(self, postvars):
+    #     add_empty_line = None
+    #     date = postvars['date'][0]
+    #     description = postvars['description'][0]
+    #     start_comment = postvars['line_0_comment'][0]
+    #     start_line = f'{date} {description}'
+    #     if start_comment.rstrip() != '':
+    #         start_line += f' ; {start_comment}' 
+    #     lines = [start_line]
+    #     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}']
+    #                 lines += [f'Assets  {amount} {currency}']
+    #         except PlomException:
+    #             pass
+    #     elif 'add_taxes' in postvars.keys():
+    #         lines += self.add_taxes(lines, finish=False)
+    #     elif 'add_taxes2' in postvars.keys():
+    #         lines += self.add_taxes(lines, finish=True)
+    #     elif 'replace' in postvars.keys():
+    #         for i, line in enumerate(lines):
+    #             lines[i] = line.replace(postvars['replace_from'][0], postvars['replace_to'][0])
+    #     elif 'add_mirror' in postvars.keys():
+    #         lines += self.add_mirror(lines)
+    #     return lines, add_empty_line
 
 
 
@@ -756,7 +1195,7 @@ class LedgerHandler(PlomHandler):
     def app_init(self, handler):
         default_path = '/ledger'
         handler.add_route('GET', default_path, self.forward_gets) 
-        handler.add_route('POST', default_path, self.forward_posts) 
+        handler.add_route('POST', default_path, self.forward_posts)
         return 'ledger', default_path 
 
     def do_POST(self):
@@ -764,88 +1203,133 @@ class LedgerHandler(PlomHandler):
 
     def forward_posts(self):
         prefix = self.apps['ledger'] if hasattr(self, 'apps') else '' 
-        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])
-        print("DEBUG start, end", start, end)
-        db = LedgerDB(prefix)
-        add_empty_line = None
-        lines = []
-        # get inputs
-        if prefix + '/add_structured' == parsed_url.path and not 'revert' in postvars.keys():
-            lines, add_empty_line = db.booking_lines_from_postvars(postvars) 
-        elif prefix + '/add_free' == parsed_url.path and not 'revert' in postvars.keys():
-            lines = postvars['booking'][0].splitlines()
-        # validate where appropriate
-        if ('save' in postvars.keys()) or ('check' in postvars.keys()):
-            _, _ = parse_lines(lines)
-        # if saving, process where to and where to redirect after
-        if 'save' in postvars.keys():
-            last_date = str(datetime.now())[:10]
-            if len(db.bookings) > 0:
-                last_date = db.bookings[-1].date_string
-            target_date = last_date[:] 
-            first_line_tokens = lines[0].split() if len(lines) > 0 else ''
-            first_token = first_line_tokens[0] if len(first_line_tokens) > 0 else ''
-            try:
-                datetime.strptime(first_token, '%Y-%m-%d')
-                target_date = first_token
-            except ValueError:
-                 pass
-            if start == end == 0:
-                start = db.insert_at_date(lines, target_date)
-                nth = db.get_nth_for_booking_of_start_line(start) 
+        db = LedgerDB(prefix, ignore_editable_exceptions=True)
+        index = 0
+        parsed_url = urlparse(self.path)
+        for string in {'save', 'copy', 'check', 'mirror', 'fill sink', 'as textarea', 'as table', 'move up', 'move down', 'add taxes'}:
+            if string in postvars.keys():
+                submit_button = string
+                break
+        if prefix + '/ledger' == parsed_url.path:
+            if submit_button == 'move up':
+                index = db.move_up(int(postvars['move up'][0]))
+            elif submit_button == 'move down':
+                index = db.move_down(int(postvars['move down'][0]))
+        elif prefix + '/edit' == parsed_url.path:
+            index = int(postvars['index'][0])
+            starts_at = '?' if index == -1 else db.bookings[index].starts_at
+            edit_mode = postvars['edit_mode'][0]
+            validate = submit_button in {'save', 'copy', 'check'}
+            if 'textarea' == edit_mode:
+                lines = postvars['booking'][0].rstrip().split('\n')
+                booking = Booking(lines, starts_at, validate=validate)
             else:
-                new_start = db.update(start, end, lines, target_date)
-                print("DEBUG save", new_start, start, end, lines)
-                nth = db.get_nth_for_booking_of_start_line(new_start)
-                if new_start > start: 
-                    nth -= 1 
-            self.redirect(prefix + f'/#{nth}')
-        # otherwise just re-build editing form
-        else:
-            if prefix + '/add_structured' == parsed_url.path: 
-                edit_content = db.add_structured(start, end, temp_lines=lines, add_empty_line=add_empty_line)
+                booking = Booking.from_postvars(postvars, starts_at, validate)
+            if submit_button in {'save', 'copy'}:
+                if index != -1 and submit_button != 'copy':
+                     if booking.date == db.bookings[index].date:
+                        db.bookings[index] = booking 
+                        booking_is_placed = True
+                     else:
+                        db.bookings = db.bookings[:index] + db.bookings[index+1:]
+                        db.insert_booking_at_date(booking)
+                else: 
+                    db.insert_booking_at_date(booking)
             else:
-                edit_content = db.add_free(start, end)
-            header = jinja2.Template(html_head).render(prefix=prefix)
-            self.send_HTML(header + edit_content)
+                error_msg = None
+                if 'check' == submit_button:
+                    error_msg = 'All looks fine!'
+                elif 'mirror' == submit_button:
+                    booking.add_mirror()
+                elif 'fill sink' == submit_button:
+                    booking.fill_sink()
+                elif 'add taxes' == submit_button:
+                    booking.add_taxes()
+                elif submit_button in {'as textarea', 'as table'}:
+                    edit_mode = submit_button[len('as '):]
+                page = db.edit(index, booking, error_msg=error_msg, edit_mode=edit_mode)
+                self.send_HTML(page)
+                return
+        db.write_db() 
+        index = index if index >= 0 else len(db.bookings) - 1
+        self.redirect(prefix + f'/ledger#{index}')
+
+    # def forward_posts(self):
+    #     prefix = self.apps['ledger'] if hasattr(self, 'apps') else '' 
+    #     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])
+    #     print("DEBUG start, end", start, end)
+    #     db = LedgerDB(prefix)
+    #     add_empty_line = None
+    #     lines = []
+    #     # get inputs
+    #     if prefix + '/add_structured' == parsed_url.path and not 'revert' in postvars.keys():
+    #         lines, add_empty_line = db.booking_lines_from_postvars(postvars) 
+    #     elif prefix + '/add_free' == parsed_url.path and not 'revert' in postvars.keys():
+    #         lines = postvars['booking'][0].splitlines()
+    #     # validate where appropriate
+    #     if ('save' in postvars.keys()) or ('check' in postvars.keys()):
+    #         _, _ = parse_lines(lines)
+    #     # if saving, process where to and where to redirect after
+    #     if 'save' in postvars.keys():
+    #         last_date = str(datetime.now())[:10]
+    #         if len(db.bookings) > 0:
+    #             last_date = db.bookings[-1].date_string
+    #         target_date = last_date[:] 
+    #         first_line_tokens = lines[0].split() if len(lines) > 0 else ''
+    #         first_token = first_line_tokens[0] if len(first_line_tokens) > 0 else ''
+    #         try:
+    #             datetime.strptime(first_token, '%Y-%m-%d')
+    #             target_date = first_token
+    #         except ValueError:
+    #              pass
+    #         if start == end == 0:
+    #             start = db.insert_at_date(lines, target_date)
+    #             nth = db.get_nth_for_booking_of_start_line(start) 
+    #         else:
+    #             new_start = db.update(start, end, lines, target_date)
+    #             print("DEBUG save", new_start, start, end, lines)
+    #             nth = db.get_nth_for_booking_of_start_line(new_start)
+    #             if new_start > start: 
+    #                 nth -= 1 
+    #         self.redirect(prefix + f'/#{nth}')
+    #     # otherwise just re-build editing form
+    #     else:
+    #         if prefix + '/add_structured' == parsed_url.path: 
+    #             edit_content = db.add_structured(start, end, temp_lines=lines, add_empty_line=add_empty_line)
+    #         else:
+    #             edit_content = db.add_free(start, end)
+    #         header = jinja2.Template(html_head).render(prefix=prefix)
+    #         self.send_HTML(header + edit_content)
 
     def do_GET(self):
         self.try_do(self.forward_gets)
 
     def forward_gets(self):
         prefix = self.apps['ledger'] if hasattr(self, 'apps') else '' 
+        try:
+            db = LedgerDB(prefix=prefix)
+        except EditableException as e:
+            db = LedgerDB(prefix=prefix, ignore_editable_exceptions=True)
+            page = db.edit(index=e.booking_index, error_msg=f'ERROR: {e}')
+            self.send_HTML(page)
+            return
         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 = LedgerDB(prefix=prefix)
         if parsed_url.path == prefix + '/balance':
             stop = params.get('stop', [None])[0]
             page = db.balance_as_html(stop)
-        elif parsed_url.path == prefix + '/add_free':
-            page = db.add_free(start, end)
-        elif parsed_url.path == prefix + '/add_structured':
-            page = db.add_structured(start, end)
-        elif parsed_url.path == prefix + '/copy_free':
-            page = db.add_free(start, end, copy=True)
-        elif parsed_url.path == prefix + '/copy_structured':
-            page = db.add_structured(start, end, copy=True)
-        elif parsed_url.path == prefix + '/move_up':
-            nth = db.move_up(start, end)
-            self.redirect(prefix + f'/#{nth}')
-            return
-        elif parsed_url.path == prefix + '/move_down':
-            nth = db.move_down(start, end)
-            self.redirect(prefix + f'/#{nth}')
-            return
+        elif parsed_url.path == prefix + '/edit':
+            index = params.get('i', [-1])[0]
+            page = db.edit(int(index))
         else:
             page = db.ledger_as_html()
-        header = jinja2.Template(html_head).render(prefix=prefix)
-        self.send_HTML(header + page)
+        self.send_HTML(page)
 
 
 
diff --git a/todo.py b/todo.py
index 8c911fb..29c1226 100644
--- a/todo.py
+++ b/todo.py
@@ -309,29 +309,45 @@ class TodoDB(PlomDB):
             task_rows.sort(key=lambda r: False if not r['todo'] else True, reverse=True)
         elif task_sort == 'comment':
             task_rows.sort(key=lambda r: '' if not r['todo'] else r['todo'].comment, reverse=True)
-        return j2env.get_template('day.html').render(db=self, action=self.prefix+'/day', prev_date=prev_date_str, next_date=next_date_str, task_rows=task_rows, sort=task_sort)
+        done_tasks = []
+        for uuid, task in self.tasks.items():
+            if uuid in self.selected_day.todos.keys():
+                todo = self.selected_day.todos[uuid]
+                if todo.done:
+                    done_tasks += [todo]
+        done_tasks.sort(key=lambda t: t.effort, reverse=True)
+        return j2env.get_template('day.html').render(db=self, action=self.prefix+'/day', prev_date=prev_date_str, next_date=next_date_str, task_rows=task_rows, sort=task_sort, done_tasks=done_tasks)
 
     def show_calendar(self, start_date_str, end_date_str):
         self.t_filter_and = ['calendar']
         self.t_filter_not = ['deleted']
         self.set_visibilities()
         days_to_show = {}
-        todays_date = str(datetime.now())[:10]
-        target_start_str = start_date_str if start_date_str else sorted(self.days.keys())[0]
-        target_start = todays_date if target_start_str == 'today' else target_start_str
-        target_end_str = end_date_str if end_date_str else sorted(self.days.keys())[-1]
-        target_end = todays_date if target_end_str == 'today' else target_end_str
-        start_date = datetime.strptime(target_start, DATE_FORMAT)
-        end_date = datetime.strptime(target_end, DATE_FORMAT)
-        for n in range(int((end_date - start_date).days) + 1):
-            current_date_obj = start_date + timedelta(n)
-            current_date = current_date_obj.strftime(DATE_FORMAT)
-            if current_date not in self.days.keys():
-                days_to_show[current_date] = self.add_day()
+        todays_date_str = str(datetime.now())[:10]
+        todays_date_obj = datetime.strptime(todays_date_str, DATE_FORMAT) 
+        yesterdays_date_obj = todays_date_obj - timedelta(1)
+        yesterdays_date_str = yesterdays_date_obj.strftime(DATE_FORMAT) 
+        start_date_obj = datetime.strptime(sorted(self.days.keys())[0], DATE_FORMAT)
+        if start_date_str and len(start_date_str) > 0:
+            if start_date_str in {'today', 'yesterday'}:
+                start_date_obj = todays_date_obj if start_date_str == 'today' else yesterdays_date_obj
+            else:
+                start_date_obj = datetime.strptime(start_date_str, DATE_FORMAT)
+        end_date_obj = datetime.strptime(sorted(self.days.keys())[-1], DATE_FORMAT)
+        if end_date_str and len(end_date_str) > 0:
+            if end_date_str in {'today', 'yesterday'}:
+                end_date_obj = todays_date_obj if end_date_str == 'today' else yesterdays_date_obj
+            else:
+                end_date_obj = datetime.strptime(start_date_str, DATE_FORMAT)
+        for n in range(int((end_date_obj - start_date_obj).days) + 1):
+            current_date_obj = start_date_obj + timedelta(n)
+            current_date_str = current_date_obj.strftime(DATE_FORMAT)
+            if current_date_str not in self.days.keys():
+                days_to_show[current_date_str] = self.add_day()
             else:
-                days_to_show[current_date] = self.days[current_date]
-            days_to_show[current_date].weekday = datetime.strptime(current_date, DATE_FORMAT).strftime('%A')[:2]
-        return j2env.get_template('calendar.html').render(db=self, days=days_to_show, action=self.prefix+'/calendar', today=str(datetime.now())[:10], start_date=start_date_str, end_date=end_date_str)
+                days_to_show[current_date_str] = self.days[current_date_str]
+            days_to_show[current_date_str].weekday = datetime.strptime(current_date_str, DATE_FORMAT).strftime('%A')[:2]
+        return j2env.get_template('calendar.html').render(db=self, days=days_to_show, action=self.prefix+'/calendar', start_date=start_date_str, end_date=end_date_str)
 
     def show_todo(self, task_uuid, selected_date, referer):
         if selected_date not in self.days.keys():
@@ -552,7 +568,6 @@ class TodoHandler(PlomHandler):
                 page = 'cookie unset!'
         else:
             start_date = get_param('start')
-            start_date = start_date if start_date else 'today'
             end_date = get_param('end')
             page = db.show_calendar(start_date, end_date)
         if parsed_url.path != app_config['prefix'] + '/unset_cookie':
diff --git a/todo_templates/calendar.html b/todo_templates/calendar.html
index 8f0aa8e..5d49b5a 100644
--- a/todo_templates/calendar.html
+++ b/todo_templates/calendar.html
@@ -7,7 +7,7 @@ td.checkbox { width: 0.1em; height: 0.1em; padding: 0em; text-align: center; }
 {% block content %}
 
 
-