activity sidebar for invoices.HEADmaster

6 files changed, 277 insertions, 70 deletions

diff --git a/src/administration.cpp b/src/administration.cpp
index cb158e2..d7e4eed 100644
--- a/src/administration.cpp
+++ b/src/administration.cpp
@@ -129,6 +129,7 @@ void administration_create()
     list_init(&g_administration.contacts);
     list_init(&g_administration.projects);
     list_init(&g_administration.tax_rates);
+    list_init(&g_administration.activities);
     list_init(&g_administration.all_tax_rates);
     list_init(&g_administration.cost_centers);
     strops::copy(g_administration.path, "", sizeof(g_administration.path));
@@ -176,6 +177,18 @@ static void administration_destroy_invoices()
     list_iterator_stop(&g_administration.invoices);
 }
 
+int sort_invoice_time_t(const void *a, const void *b) {
+    invoice* inv1 = (invoice*)a;
+    invoice* inv2 = (invoice*)b;
+    return (int)(inv1->issued_at - inv2->issued_at);
+}
+
+void administration::sort_data()
+{
+    list_attributes_comparator(&g_administration.invoices, sort_invoice_time_t);
+    list_sort(&g_administration.invoices, 1);
+}
+
 void administration::destroy()
 {
     is_initialized = false;
@@ -183,6 +196,7 @@ void administration::destroy()
     administration_destroy_list(&g_administration.invoices);
     administration_destroy_list(&g_administration.contacts);
     administration_destroy_list(&g_administration.projects);
+    administration_destroy_list(&g_administration.activities);
     administration_destroy_list(&g_administration.tax_rates);
     administration_destroy_list(&g_administration.all_tax_rates);
     administration_destroy_list(&g_administration.cost_centers);
@@ -353,8 +367,8 @@ void administration::create_tax_statement(tax_statement* statement)
     time_t youngest = 0;
     for (u32 i = 0; i < invoice_count; i++)
     {
-        if (invoice_buffer[i].delivered_at < oldest) oldest = invoice_buffer[i].delivered_at;
-        if (invoice_buffer[i].delivered_at > youngest) youngest = invoice_buffer[i].delivered_at;
+        if (invoice_buffer[i].delivered_at < oldest) oldest = country::get_invoice_date_to_use_for_tax_report(country_code, &invoice_buffer[i]);
+        if (invoice_buffer[i].delivered_at > youngest) youngest = country::get_invoice_date_to_use_for_tax_report(country_code, &invoice_buffer[i]);
     }
 
     u16 oldest_year;
@@ -527,7 +541,7 @@ void administration::create_income_statement(income_statement* statement)
 
         u16 yy;
         u8 qq;
-        time_t_to_quarter(inv->issued_at, &yy, &qq);
+        time_t_to_quarter(inv->delivered_at, &yy, &qq);
 
         u32 report_index = (qq + (yy*4)) - (oldest_quarter + (oldest_year*4));
 
@@ -1494,6 +1508,7 @@ a_err administration::invoice_update(invoice* inv)
 
             if (invoice_changed_event_callback) invoice_changed_event_callback(c);
 
+            administration::activity_add(ACTIVITY_USER, c->id, "activity.update_invoice", 0);
             return A_ERR_SUCCESS;
         }
     }
@@ -1547,9 +1562,11 @@ a_err administration::invoice_add(invoice* inv)
     
     memops::copy(new_inv, &copy, sizeof(invoice));
 
-    if (!list_append(&g_administration.invoices, new_inv)) {
+    if (!list_prepend(&g_administration.invoices, new_inv)) {
         return A_ERR_GENERIC;
-    }  
+    }
+
+    administration::activity_add(ACTIVITY_USER, inv->id, "activity.add_invoice", 0);
 
     g_administration.next_id++;
     g_administration.next_sequence_number++;
@@ -1900,4 +1917,50 @@ billing_item administration::billing_item_create_empty()
     memops::zero(&item, sizeof(billing_item));
     item.amount = 1;
     return item;
+}
+
+// Activity functions.
+// ===================
+a_err administration::activity_add(char* user, char* ref_id, char* message, ...)
+{
+    activity* new_activity = (activity*)memops::alloc(sizeof(activity));
+    strops::copy(new_activity->user_name, user, MAX_LEN_SHORT_DESC);
+    strops::copy(new_activity->ref_id, ref_id, MAX_LEN_ID);
+    strops::copy(new_activity->message, message, MAX_LEN_LONG_DESC);
+    new_activity->timestamp = time(NULL);
+
+    va_list args;
+    va_start(args, message);
+
+    char* param;
+    u32 param_count = 0;
+    do
+    {
+        param = va_arg(args, char*);
+        if (param != 0) {
+            strops::copy(new_activity->params[param_count++], param, MAX_LEN_LONG_DESC);
+        }
+    } while (param && param_count < ACTIVITY_MAX_PARAMS);
+    
+    va_end(args);
+
+    if (!list_prepend(&g_administration.activities, new_activity)) {
+        return A_ERR_GENERIC;
+    }
+
+    return A_ERR_SUCCESS;
+}
+
+u32 administration::activity_get_all_for_object(activity* buffer, char* ref_id)
+{
+    u32 write_cursor = 0;
+    
+    list_iterator_start(&g_administration.activities);
+    while (list_iterator_hasnext(&g_administration.activities)) {
+        activity c = *(activity *)list_iterator_next(&g_administration.activities);
+        if (strops::equals(c.ref_id, ref_id)) buffer[write_cursor++] = c;
+    }
+    list_iterator_stop(&g_administration.activities);
+
+    return write_cursor;
 }
\ No newline at end of file
diff --git a/src/administration_reader.cpp b/src/administration_reader.cpp
index 05133f8..34cfe8f 100644
--- a/src/administration_reader.cpp
+++ b/src/administration_reader.cpp
@@ -101,6 +101,8 @@ bool administration_reader::open_existing(char* file_path)
 
     zip_close(zip);
 
+    administration::sort_data();
+    
     logger::info("Imported '%s' in %.3fms.", file_path, STOPWATCH_TIME);
 
     return true;
diff --git a/src/administration_writer.cpp b/src/administration_writer.cpp
index c699a0a..f12cf1f 100644
--- a/src/administration_writer.cpp
+++ b/src/administration_writer.cpp
@@ -357,7 +357,8 @@ static char* get_eas_id_for_contact(contact contact)
     if (strops::equals(country_code, "IT")) return contact.businessid; // Italy
     if (strops::equals(country_code, "FI")) return contact.businessid; // Finland
     if (strops::equals(country_code, "DK")) return contact.businessid; // Denmark
-    return NULL; // Unknown country code
+
+    return contact.businessid; // Unknown country code
 }
 
 static char* get_eas_scheme_for_contact(contact contact)
@@ -397,7 +398,8 @@ static char* get_eas_scheme_for_contact(contact contact)
     if (strops::equals(country_code, "SI")) return "9949"; // Slovenia
     if (strops::equals(country_code, "SK")) return "9950"; // Slovakia
     if (strops::equals(country_code, "ES")) return "9920"; // Spain
-    return NULL; // Unknown country code
+    
+    return "0203"; // Hack
 }
 
 bool isEmptyTag(const char *start, const char *end) {
diff --git a/src/locales/en.cpp b/src/locales/en.cpp
index d949609..752e328 100644
--- a/src/locales/en.cpp
+++ b/src/locales/en.cpp
@@ -54,6 +54,7 @@ locale_entry en_locales[] = {
     {"form.create", "+ Create"},
     {"form.back", "Back"},
     {"form.save", "Save"},
+    {"form.cancel", "Cancel"},
     {"form.yes", "Yes"},
     {"form.no", "No"},
     {"form.change", "Change"},
@@ -721,7 +722,7 @@ locale_entry en_locales[] = {
     {"taxrate.code.NL_IN/IEUV/21","Intra-EU reverse charge 21% VAT"},
     {"taxrate.code.NL_IN/IEUV/9","Intra-EU reverse charge 9% VAT"},
 
-    // Tax statement strings.
+    // NL Tax statement strings.
     {"taxes.total", "Total"},
 
     {"taxes.nl.1", "Domestic supplies and services"},
@@ -746,6 +747,10 @@ locale_entry en_locales[] = {
     {"taxes.nl.5", "Input tax and small businesses scheme (KOR)"},
     {"taxes.nl.5a", "VAT due (sections 1 to 4)"},
     {"taxes.nl.5b", "Input tax (VAT deductible)"},
+
+    // Activity strings.
+    {"activity.update_invoice", "updated invoice"},
+    {"activity.add_invoice", "created invoice"},
 };
 
 int en_locale_count = sizeof(en_locales) / sizeof(en_locales[0]);
\ No newline at end of file
diff --git a/src/ui/ui_expenses.cpp b/src/ui/ui_expenses.cpp
index 5556cd6..3ea31e1 100644
--- a/src/ui/ui_expenses.cpp
+++ b/src/ui/ui_expenses.cpp
@@ -26,19 +26,34 @@
 #include "administration.hpp"
 #include "administration_writer.hpp"
 
+static activity* activity_buffer = 0;
+static u32 activity_count;
+
 static importer::invoice_request* active_import_request = 0;
 
 static ui::view_state current_view_state = ui::view_state::LIST_ALL;
 static invoice active_invoice = {0};
 static invoice selected_for_removal = {0};
-
+static const float sidepanel_width = 200.0f;
 static billing_item* invoice_items_buffer = 0;
 
 void draw_invoice_items_form(invoice* invoice, bool outgoing = true);
 
+static void _reload_activities()
+{
+    activity_count = administration::activity_get_all_for_object(activity_buffer, active_invoice.id);
+}
+
+static void _set_active_invoice(invoice inv)
+{
+    active_invoice = inv;
+    _reload_activities();
+}
+
 void ui::destroy_expenses()
 {
     memops::unalloc(invoice_items_buffer);
+    memops::unalloc(activity_buffer);
 }
 
 void ui::setup_expenses()
@@ -50,7 +65,8 @@ void ui::setup_expenses()
         current_view_state = ui::view_state::LIST_ALL;
     }
 
-    active_invoice = administration::invoice_create_empty();
+    activity_buffer = (activity*)memops::alloc(sizeof(activity) * 250);
+    _set_active_invoice(administration::invoice_create_empty());
 
     u32 invoice_items_count = MAX_BILLING_ITEMS;
     invoice_items_buffer = (billing_item*)memops::alloc(sizeof(billing_item) * invoice_items_count);
@@ -215,22 +231,35 @@ static void draw_expenses_list()
 
     ImGui::Spacing();
 
-    if (ImGui::BeginTable("TableInvoices", 7, ImGuiTableFlags_Borders | ImGuiTableFlags_RowBg)) {
+    if (ImGui::BeginTable("TableInvoices", 6, ImGuiTableFlags_Borders | ImGuiTableFlags_RowBg)) {
 
         ImGui::TableSetupColumn(locale::get("invoice.table.invoicenumber"), ImGuiTableColumnFlags_WidthFixed, 120);
         ImGui::TableSetupColumn(locale::get("invoice.table.sender"));
         ImGui::TableSetupColumn(locale::get("invoice.table.customer"));
-        ImGui::TableSetupColumn(locale::get("invoice.table.issuedat"));
-        ImGui::TableSetupColumn(locale::get("invoice.table.total"));
-        ImGui::TableSetupColumn(locale::get("invoice.table.status"));
-        ImGui::TableSetupColumn("");
+        ImGui::TableSetupColumn(locale::get("invoice.table.issuedat"), ImGuiTableColumnFlags_WidthFixed, 90);
+        ImGui::TableSetupColumn(locale::get("invoice.table.total"), ImGuiTableColumnFlags_WidthFixed, 90);
+        ImGui::TableSetupColumn(locale::get("invoice.table.status"), ImGuiTableColumnFlags_WidthFixed, 90);
         ImGui::TableHeadersRow();
 
         for (u32 i = 0; i < invoice_count; i++) {
             invoice c = invoice_list[i];
             
             ImGui::TableNextRow();
-            ImGui::TableSetColumnIndex(0); ImGui::Text(c.sequential_number);
+            ImGui::TableSetColumnIndex(0); 
+            
+            ImGui::PushID(i);
+            ImGuiSelectableFlags selectable_flags = ImGuiSelectableFlags_SpanAllColumns | ImGuiSelectableFlags_AllowOverlap;
+            bool selected = false;
+            if (ImGui::Selectable("##invisible_selectable", selected, selectable_flags, ImVec2(0, ImGui::GetFrameHeight()-4.0f)))
+            {
+                _set_active_invoice(c);
+                current_view_state = ui::view_state::VIEW_EXISTING;
+            }
+            ImGui::SetItemAllowOverlap();
+            ImGui::SameLine();
+            ImGui::PopID();
+            
+            ImGui::Text(c.sequential_number);
 
             if (administration::invoice_is_valid(&c) != A_ERR_SUCCESS)
             {
@@ -249,8 +278,8 @@ static void draw_expenses_list()
             ImGui::TableSetColumnIndex(3); ImGui::Text(buf);
             ImGui::TableSetColumnIndex(4); ImGui::Text("%.2f %s", c.total, c.currency);
             ImGui::TableSetColumnIndex(5); ImGui::Text("%s", locale::get(administration::invoice_get_status_string(&c)));
-            ImGui::TableSetColumnIndex(6);
             
+            /*
             char btn_name[20];
             strops::format(btn_name, sizeof(btn_name), "%s##%d", locale::get("form.view"), i);
             if (ImGui::Button(btn_name)) {
@@ -271,23 +300,7 @@ static void draw_expenses_list()
             if (ImGui::Button(btn_name)) {
                 selected_for_removal = c;
                 ImGui::OpenPopup("ConfirmDeletePopup");
-            }
-        }
-
-        // Confirmation popup before contact is deleted definitively.
-        if (ImGui::BeginPopupModal("ConfirmDeletePopup", nullptr, ImGuiWindowFlags_AlwaysAutoResize|ImGuiWindowFlags_NoMove|ImGuiWindowFlags_NoTitleBar)) {
-            ImGui::Text(locale::get("form.confirmDelete"));
-            ImGui::Separator();
-
-            if (ImGui::Button(locale::get("form.yes"), ImVec2(120, 0))) { 
-                administration::invoice_remove(&selected_for_removal);
-                ImGui::CloseCurrentPopup();
-            }
-            ImGui::SameLine();
-            if (ImGui::Button(locale::get("form.no"), ImVec2(120, 0))) {
-                ImGui::CloseCurrentPopup();
-            }
-            ImGui::EndPopup();
+            }*/
         }
 
         ImGui::EndTable();
@@ -299,26 +312,75 @@ static void _reset_to_default_view()
     current_view_state = ui::view_state::LIST_ALL;
     ui::destroy_expenses();
     ui::setup_expenses();
+    activity_count = 0;
+}
+
+static void _reset_to_invoice_view()
+{
+    current_view_state = ui::view_state::VIEW_EXISTING;
+}
+
+static void _draw_activity_sidepanel()
+{
+    ImGui::SameLine();
+    ImGui::SeparatorEx(1 << 1, 3.0f);
+    ImGui::SameLine();
+
+    ImGui::BeginChild("##historyPanel", ImVec2(sidepanel_width, 0), ImGuiChildFlags_None);
+    {
+        for (u32 i = 0; i < activity_count; i++)
+        {
+            activity ac = activity_buffer[i];
+
+            char time_buffer[26];
+            struct tm* tm_info;
+            tm_info = localtime(&ac.timestamp);
+            strftime(time_buffer, 26, "%d/%m/%Y %H:%M", tm_info);
+
+            ImGui::Text(time_buffer);
+
+            ImGui::PushFont(ui::fontBold);
+            ImGui::Text(ac.user_name);
+            ImGui::PopFont();
+            ImGui::SameLine();
+            
+            ImGui::TextWrapped("%s", locale::get(ac.message));
+            ImGui::Spacing();
+        }
+    }
+
+    ImGui::EndChild();
 }
 
 static void draw_expense_update()
 {
-    if (ImGui::Button(locale::get("form.back"), true, false)) {
-        _reset_to_default_view();
+    if (ImGui::Button(locale::get("form.cancel"), true, false)) {
+        current_view_state = ui::view_state::VIEW_EXISTING;
     }
-    
-    draw_expense_form(&active_invoice);
 
-    bool can_save = administration::invoice_is_valid(&active_invoice) == A_ERR_SUCCESS;
-    if (!can_save) ImGui::BeginDisabled();
-    
+    { // Save button
+        bool can_save = administration::invoice_is_valid(&active_invoice) == A_ERR_SUCCESS;
+        if (!can_save) ImGui::BeginDisabled();  
+        ImGui::SameLine();
+        if (ImGui::Button(locale::get("form.save"), true)) {
+            administration_writer::set_write_completed_event_callback(_reset_to_invoice_view);
+            administration::invoice_update(&active_invoice);
+
+            _reload_activities();
+        } 
+        if (!can_save) ImGui::EndDisabled();
+    }
+
+    ImGui::Spacing();
+    ImGui::Separator(3.0f);
     ImGui::Spacing();
-    if (ImGui::Button(locale::get("form.save"), true)) {
-        administration_writer::set_write_completed_event_callback(_reset_to_default_view);
-        administration::invoice_update(&active_invoice);
-    } 
+    
+    float availableWidth = ImGui::GetContentRegionAvail().x;
+    ImGui::BeginChild("##invoicePanel", ImVec2(availableWidth - sidepanel_width, 0), ImGuiChildFlags_None);
+    draw_expense_form(&active_invoice);
+    ImGui::EndChild();
 
-    if (!can_save) ImGui::EndDisabled();
+    _draw_activity_sidepanel();
 }
 
 static void draw_expense_create()
@@ -326,19 +388,28 @@ static void draw_expense_create()
     if (ImGui::Button(locale::get("form.back"), true, false)) {
         _reset_to_default_view();
     }
-    
-    draw_expense_form(&active_invoice);
 
-    bool can_save = administration::invoice_is_valid(&active_invoice) == A_ERR_SUCCESS;
-    if (!can_save) ImGui::BeginDisabled();
-    
-    ImGui::Spacing();
-    if (ImGui::Button(locale::get("form.save"), true)) {
-        administration_writer::set_write_completed_event_callback(_reset_to_default_view);
-        administration::invoice_add(&active_invoice);
-    } 
+    { // Save button
+        bool can_save = administration::invoice_is_valid(&active_invoice) == A_ERR_SUCCESS;
+        if (!can_save) ImGui::BeginDisabled(); 
+        ImGui::SameLine();
+        if (ImGui::Button(locale::get("form.save"), true)) {
+            administration_writer::set_write_completed_event_callback(_reset_to_invoice_view);
+            administration::invoice_add(&active_invoice);
+
+            _reload_activities();
+        } 
+        if (!can_save) ImGui::EndDisabled();
+    }
 
-    if (!can_save) ImGui::EndDisabled();
+    ImGui::Spacing();
+    ImGui::Separator(3.0f);
+    ImGui::Spacing();
+    
+    float availableWidth = ImGui::GetContentRegionAvail().x;
+    ImGui::BeginChild("##invoicePanel", ImVec2(availableWidth - sidepanel_width, 0), ImGuiChildFlags_None);
+    draw_expense_form(&active_invoice);
+    ImGui::EndChild();
 }
 
 static void draw_expense_view()
@@ -347,7 +418,23 @@ static void draw_expense_view()
         _reset_to_default_view();
     }
 
+    ImGui::SameLine();
+
+    if (ImGui::Button(locale::get("form.change"))) {
+        _set_active_invoice(administration::invoice_create_copy(&active_invoice)); // We create a copy because of billing item list pointers.
+        current_view_state = ui::view_state::EDIT_EXISTING;
+    }
+
+    ImGui::Spacing();
+    ImGui::Separator(3.0f);
+    ImGui::Spacing();
+
+    float availableWidth = ImGui::GetContentRegionAvail().x;
+    ImGui::BeginChild("##invoicePanel", ImVec2(availableWidth - sidepanel_width, 0), ImGuiChildFlags_None);
     draw_expense_form(&active_invoice, true);
+    ImGui::EndChild();
+
+    _draw_activity_sidepanel();
 }
 
 static void draw_import_request()
diff --git a/src/ui/ui_invoices.cpp b/src/ui/ui_invoices.cpp
index 0e81259..ff3c73b 100644
--- a/src/ui/ui_invoices.cpp
+++ b/src/ui/ui_invoices.cpp
@@ -25,23 +25,38 @@
 #include "administration.hpp"
 #include "administration_writer.hpp"
 
+static activity* activity_buffer = 0;
+static u32 activity_count;
 
 static ui::view_state current_view_state = ui::view_state::LIST_ALL;
 static invoice active_invoice = {0};
 static invoice selected_for_removal = {0};
 static const float sidepanel_width = 200.0f;
-
 static billing_item* invoice_items_buffer = 0;
 
+static void _reload_activities()
+{
+    activity_count = administration::activity_get_all_for_object(activity_buffer, active_invoice.id);
+}
+
+static void _set_active_invoice(invoice inv)
+{
+    active_invoice = inv;
+    _reload_activities();
+}
+
 void ui::destroy_invoices()
 {
     memops::unalloc(invoice_items_buffer);
+    memops::unalloc(activity_buffer);
 }
 
 void ui::setup_invoices()
 {
+    activity_buffer = (activity*)memops::alloc(sizeof(activity) * 250);
+
     current_view_state = ui::view_state::LIST_ALL;
-    active_invoice = administration::invoice_create_empty();
+    _set_active_invoice(administration::invoice_create_empty());
 
     u32 invoice_items_count = MAX_BILLING_ITEMS;
     invoice_items_buffer = (billing_item*)memops::alloc(sizeof(billing_item) * invoice_items_count);
@@ -279,7 +294,7 @@ static void draw_invoices_list()
     if (ImGui::Button(locale::get("form.create")))
     {
         current_view_state = ui::view_state::CREATE;
-        active_invoice = administration::invoice_create_empty(); // @leak
+        _set_active_invoice(administration::invoice_create_empty());
         active_invoice.supplier = administration::company_info_get();
         active_invoice.is_outgoing = 1;
         active_invoice.status = invoice_status::INVOICE_CONCEPT;
@@ -328,7 +343,7 @@ static void draw_invoices_list()
             bool selected = false;
             if (ImGui::Selectable("##invisible_selectable", selected, selectable_flags, ImVec2(0, ImGui::GetFrameHeight()-4.0f)))
             {
-                active_invoice = c;
+                _set_active_invoice(c);
                 current_view_state = ui::view_state::VIEW_EXISTING;
             }
             ImGui::SetItemAllowOverlap();
@@ -372,11 +387,44 @@ static void _reset_to_default_view()
     current_view_state = ui::view_state::LIST_ALL;
     ui::destroy_invoices();
     ui::setup_invoices();
+    activity_count = 0;
+}
+
+static void _draw_activity_sidepanel()
+{
+    ImGui::SameLine();
+    ImGui::SeparatorEx(1 << 1, 3.0f);
+    ImGui::SameLine();
+
+    ImGui::BeginChild("##historyPanel", ImVec2(sidepanel_width, 0), ImGuiChildFlags_None);
+    {
+        for (u32 i = 0; i < activity_count; i++)
+        {
+            activity ac = activity_buffer[i];
+
+            char time_buffer[26];
+            struct tm* tm_info;
+            tm_info = localtime(&ac.timestamp);
+            strftime(time_buffer, 26, "%d/%m/%Y %H:%M", tm_info);
+
+            ImGui::Text(time_buffer);
+
+            ImGui::PushFont(ui::fontBold);
+            ImGui::Text(ac.user_name);
+            ImGui::PopFont();
+            ImGui::SameLine();
+            
+            ImGui::TextWrapped("%s", locale::get(ac.message));
+            ImGui::Spacing();
+        }
+    }
+
+    ImGui::EndChild();
 }
 
 static void draw_invoice_update()
 {
-    if (ImGui::Button(locale::get("form.back"), true, false)) {
+    if (ImGui::Button(locale::get("form.cancel"), true, false)) {
         current_view_state = ui::view_state::VIEW_EXISTING;
     }
 
@@ -387,6 +435,8 @@ static void draw_invoice_update()
         if (ImGui::Button(locale::get("form.save"), true)) {
             administration_writer::set_write_completed_event_callback(_reset_to_invoice_view);
             administration::invoice_update(&active_invoice);
+
+            _reload_activities();
         } 
         if (!can_save) ImGui::EndDisabled();
     }
@@ -399,6 +449,8 @@ static void draw_invoice_update()
     ImGui::BeginChild("##invoicePanel", ImVec2(availableWidth - sidepanel_width, 0), ImGuiChildFlags_None);
     draw_invoice_form(&active_invoice);
     ImGui::EndChild();
+
+    _draw_activity_sidepanel();
 }
 
 static void draw_invoice_create()
@@ -414,6 +466,8 @@ static void draw_invoice_create()
         if (ImGui::Button(locale::get("form.save"), true)) {
             administration_writer::set_write_completed_event_callback(_reset_to_invoice_view);
             administration::invoice_add(&active_invoice);
+
+            _reload_activities();
         } 
         if (!can_save) ImGui::EndDisabled();
     }
@@ -437,7 +491,7 @@ static void draw_invoice_view()
     ImGui::SameLine();
 
     if (ImGui::Button(locale::get("form.change"))) {
-        active_invoice = administration::invoice_create_copy(&active_invoice); // We create a copy because of billing item list pointers.
+        _set_active_invoice(administration::invoice_create_copy(&active_invoice)); // We create a copy because of billing item list pointers.
         current_view_state = ui::view_state::EDIT_EXISTING;
     }
 
@@ -480,13 +534,7 @@ static void draw_invoice_view()
     draw_invoice_form(&active_invoice, true);
     ImGui::EndChild();
 
-    ImGui::SameLine();
-    ImGui::SeparatorEx(1 << 1, 3.0f);
-    ImGui::SameLine();
-
-    ImGui::BeginChild("##historyPanel", ImVec2(sidepanel_width, 0), ImGuiChildFlags_None);
-    // Sidepanel
-    ImGui::EndChild();
+    _draw_activity_sidepanel();
 }
 
 void ui::draw_invoices()