Better views for bank statements

app/controllers/admin/bank_statements_controller.rb

@@ -1,13 +1,15 @@
 class Admin::BankStatementsController < AdminController
   load_and_authorize_resource
 
+  before_action :set_bank_statement, only: [:show, :download_import_file]
+
   def index
     @q = BankStatement.search(params[:q])
+    @q.sorts  = 'id desc' if @q.sorts.empty?
     @bank_statements = @q.result.page(params[:page])
   end
 
   def show
-    @bank_statement = BankStatement.find(params[:id])
     @q = @bank_statement.bank_transactions.search(params[:q])
     @bank_transactions = @q.result.page(params[:page])
   end
@@ -28,8 +30,17 @@ class Admin::BankStatementsController < AdminController
     end
   end
 
+  def download_import_file
+    filename = @bank_statement.import_file_path.split('/').last
+    send_data File.open(@bank_statement.import_file_path, 'r').read, filename: filename
+  end
+
   private
 
+  def set_bank_statement
+    @bank_statement = BankStatement.find(params[:id])
+  end
+
   def bank_statement_params
     params.require(:bank_statement).permit(:th6_file)
   end

app/models/bank_statement.rb

@@ -15,6 +15,9 @@ class BankStatement < ActiveRecord::Base
       next unless bt_params
       bank_transactions.build(bt_params)
     end
+
+    self.import_file_path = "#{ENV['bank_statement_import_dir']}/#{Time.zone.now.to_formatted_s(:number)}.txt"
+    File.open(import_file_path, 'w') { |f| f.write(th6_file.open.read) }
   end
 
   def parse_th6_row(row)

app/views/admin/bank_statements/show.haml

@@ -5,24 +5,52 @@
     %h2.text-right.text-center-xs
       = link_to(t('back_to_bank_statements'), admin_bank_statements_path, class: 'btn btn-primary')
 %hr
+%row
+  .col-md-12
+.panel.panel-default
+  .panel-heading
+    %h3.panel-title= t(:details)
+  .panel-body
+    %dl.dl-horizontal
+      %dt= t('bank_code')
+      %dd= @bank_statement.bank_code
+
+      %dt= t('iban').upcase
+      %dd= @bank_statement.iban
+
+      %dt= t('queried_at')
+      %dd= l(@bank_statement.queried_at)
+
+      %dt= t('imported_at')
+      %dd= l(@bank_statement.created_at)
+
+      - if @bank_statement.import_file_path
+        %dt= t('import_file')
+        %dd= link_to(t('download'), download_import_file_admin_bank_statement_path(@bank_statement))
+
+%h2.text-center-xs= t('bank_transactions')
+%hr
 .row
   .col-md-12
     .table-responsive
       %table.table.table-hover.table-bordered.table-condensed
         %thead
           %tr
-            %th{class: 'col-xs-2'}
-              = sort_link(@q, 'buyer_name')
-            %th{class: 'col-xs-2'}
+            %th{class: 'col-xs-3'}
               = sort_link(@q, 'paid_at')
-            %th{class: 'col-xs-2'}
-              = sort_link(@q, 'sum', t('sum'))
+            %th{class: 'col-xs-3'}
+              = sort_link(@q, 'buyer_name')
+            %th{class: 'col-xs-3'}
+              = sort_link(@q, 'sum')
+            %th{class: 'col-xs-3'}
+              = sort_link(@q, 'currency')
         %tbody
           - @bank_transactions.each do |x|
             %tr
-              %td= link_to(x.buyer_name, [:admin, x])
-              %td= l(x.paid_at)
+              %td= link_to(l(x.paid_at, format: :date_long), [:admin, x])
+              %td= x.buyer_name
               %td= x.sum
+              %td= x.currency
 .row
   .col-md-12
     = paginate @bank_transactions

app/views/admin/bank_transactions/show.haml

@@ -13,15 +13,15 @@
     %h3.panel-title= t(:details)
   .panel-body
     %dl.dl-horizontal
+      %dt= t('document_no')
+      %dd= @bank_transaction.document_no
+
       %dt= t('bank_reference')
       %dd= @bank_transaction.bank_reference
 
       %dt= t('iban').upcase
       %dd= @bank_transaction.iban
 
-      %dt= t('currency')
-      %dd= @bank_transaction.currency
-
       %dt= t('buyer_bank_code')
       %dd= @bank_transaction.buyer_bank_code
 
@@ -31,14 +31,17 @@
       %dt= t('buyer_name')
       %dd= @bank_transaction.buyer_name
 
-      %dt= t('document_no')
-      %dd= @bank_transaction.document_no
-
       %dt= t('description')
       %dd= @bank_transaction.description
 
       %dt= t('sum')
       %dd= @bank_transaction.sum
 
+      %dt= t('currency')
+      %dd= @bank_transaction.currency
+
       %dt= t('reference_no')
       %dd= @bank_transaction.reference_no
+
+      %dt= t('paid_at')
+      %dd= l(@bank_transaction.paid_at, format: :date_long)

app/views/layouts/application.haml

@@ -42,6 +42,9 @@
                   = t('settings')
                   %span.caret
                 %ul.dropdown-menu{role: "menu"}
+                  %li.dropdown-header= t('accounting')
+                  %li= link_to t('bank_statements'), admin_bank_statements_path
+                  %li.divider
                   %li.dropdown-header= t('system')
                   %li= link_to t('settings'), admin_settings_path
                   %li= link_to t('zonefile'), admin_zonefile_settings_path