rock_books 0.6.1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81be281ee187aa03b43808391235de30c85e05b7983e243a5df887ec6eb12b2b
-  data.tar.gz: 9069620c1ad95d22e35358caa4ed473899c7a897ad741f679047def6b6fee1e7
+  metadata.gz: c30673728b99315fc841b99b96081751f91459663c736758b7340a317407cc56
+  data.tar.gz: 751adad9c6038a85a696deb4076bbc93bbfb8a8650d53159aca691b281388d9f
 SHA512:
-  metadata.gz: e67e7be616f654e5ed67e7f32836b03f56cdfc18821a7d0ee50570aadbb5cb65d0f288d0c4def2999ce2f32b150490e93aa0a6107a1995297b69d178e4ab9de9
-  data.tar.gz: 38819a0cfa909e3171705f11d3151bc2d128d0b946d6a31ff3cb1e6a7278e60670cb820c2fceb8e66be9891891a91dcd317079050675c1b2678b099d26b76676
+  metadata.gz: 84ff76b2001eed18b4f4ec69f9949720a4dbca892f41a4fa6b96ddda9e19dde5660b34c514568f5410a117df8afddf84c33bf23ba31256efc2bceccbdb06bd01
+  data.tar.gz: 2214c3b52b24bf4b9b822059c4f41c67d7d3fc3a6fc666b8a073be79a794c769d1403d8f164085e382e1714320d71cbacd0673f591b26dd958e43f7b4cb1ab50

data/README.md

@@ -33,10 +33,8 @@ To simplify its implementation, RockBooks assumes some conventions:
   
 #### Supported Operating Systems
 
-At this time, RockBooks is supported only on Mac OS and Linux. (It's really the external tools that is the issue, not the Ruby code.) It should probably work with Windows Subsystem for Linux (WSL) but it hasn't been tested. If you're unsuccessful trying to use RockBooks with WSL, give me as much information as possible and I'll try to resolve the issue.
+At this time, RockBooks is tested only on Mac OS and Linux. However, it will probably work fine on Windows.
 
-If you get an error message saying that an external command is missing, install that command using your system's package manager (e.g. `sudo apt install txt2html`). For `wkhtmltopdf` on Linux, see [https://wkhtmltopdf.org/](https://wkhtmltopdf.org/).
-  
 #### Text Files as Input
 
 Instead of a web interface, data input is done in plain text files. This isn't as fancy but has the following advantages:

data/RELEASE_NOTES.md

@@ -1,3 +1,15 @@
+### v0.7.0
+
+* Dependencies on external commands in Linux and Mac OS for generating PDF and HTML files has been eliminated,
+using the prawn gem for PDF and simple ERB templating for HTML.
+* Massive refactoring of reports to separate data generation from presentation.
+* Reports are now computed and then written one at a time. Previously they were all computed, then all written.
+* ERB is now used for generating text reports.
+* Fix receipt hyperlinks in HTML output.
+* Improve some error output.
+* Various minor improvements and bug fixes.
+
+
 ### v0.6.1
 
 * Linux PDF generation fixed by using wkhtmltopdf instead of cupsfilter.

data/lib/rock_books/cmd_line/command_line_interface.rb

@@ -4,7 +4,7 @@ require 'ostruct'
 
 require_relative '../../rock_books'
 require_relative '../version'
-require_relative '../reports/reporter'
+require_relative '../reports/helpers/reporter'
 require_relative '../helpers/book_set_loader'
 
 module RockBooks
@@ -105,7 +105,7 @@ When in interactive shell mode:
       end
     end
 
-    validate_receipts_dir = -> do
+    validate_receipt_dir = -> do
       File.directory?(options.receipt_dir) ? nil : \
           "Receipts directory '#{options.receipt_dir}' does not exist. "
     end
@@ -114,7 +114,7 @@ When in interactive shell mode:
     output << validate_input_dir.()
     output << validate_output_dir.()
     if run_options.do_receipts
-      output << validate_receipts_dir.()
+      output << validate_receipt_dir.()
     end
 
     output.compact!
@@ -298,9 +298,9 @@ When in interactive shell mode:
 
 
   def cmd_d
-    book_set.all_reports($filter).each do |short_name, report_text|
+    book_set.all_reports($filter).each do |short_name, text_report|
       puts "#{short_name}:\n\n"
-      puts report_text
+      puts text_report
       puts "\n\n\n"
     end
     nil
@@ -358,7 +358,7 @@ When in interactive shell mode:
   end
 
   def cmd_w
-    BookSetReporter.new(book_set, run_options.output_dir, $filter).call
+    BookSetReporter.new(book_set, run_options.output_dir, $filter).generate
     nil
   end
 

data/lib/rock_books/cmd_line/main.rb

@@ -1,4 +1,4 @@
-require 'awesome_print'
+require 'amazing_print'
 require 'optparse'
 require 'pry'
 require 'shellwords'
@@ -67,10 +67,6 @@ class Main
         options.verbose_mode = v
       end
 
-      parser.on('-y', '--[no-]say', 'Say error messages.') do |v|
-        options.say = v
-      end
-
       parser.on('', '--[no-]receipts', 'Include report on existing and missing receipts.') do |v|
         options.do_receipts = v
       end
@@ -101,10 +97,6 @@ class Main
 
       HEREDOC
 
-      if run_options.say
-        `say #{error}`
-      end
-
       exit(-1)
       binding.pry
       raise error

data/lib/rock_books/documents/book_set.rb

@@ -1,4 +1,4 @@
-require 'awesome_print'
+require 'amazing_print'
 require 'os'
 
 require_relative 'chart_of_accounts'

data/lib/rock_books/documents/chart_of_accounts.rb

@@ -1,3 +1,4 @@
+require 'stringio'
 require_relative '../types/account'
 require_relative '../types/account_type'
 require_relative '../errors/error'
@@ -10,8 +11,8 @@ class ChartOfAccounts
   REQUIRED_FIELDS.each { |field| attr_reader(field) }
 
 
-  def self.from_file(file)
-    self.new(File.readlines(file).map(&:chomp))
+  def self.from_file(filespec)
+    self.new(File.readlines(filespec).map(&:chomp), filespec)
   end
 
 
@@ -20,11 +21,16 @@ class ChartOfAccounts
   end
 
 
-  def initialize(input_lines)
+  def initialize(input_lines, filespec = nil)
+    @filespec = filespec
     @accounts = []
-    input_lines.each { |line| parse_line(line) }
+    parse_lines(input_lines)
     # TODO: Add validation for required fields.
+    check_for_missing_fields
+  end
+
 
+  def check_for_missing_fields
     missing_fields = REQUIRED_FIELDS.select do |field|
       instance_variable_get("@#{field}").nil?
     end
@@ -35,6 +41,19 @@ class ChartOfAccounts
   end
 
 
+  def parse_lines(input_lines)
+    input_lines.each_with_index do |line, line_num|
+      begin
+        parse_line(line)
+      rescue => e
+        file_message_fragment = (@filespec ? " in file '#{@filespec}'" : '')
+        puts "Error parsing chart of accounts#{file_message_fragment}. Bad line is line ##{line_num}, text is:\n#{line}\n\n"
+        raise
+      end
+    end
+  end
+
+
   def parse_date(date_string)
     # TODO: Add better handling for this error.
     # begin
@@ -69,16 +88,14 @@ class ChartOfAccounts
         rest = matcher[2]
 
         matcher = rest.match(/^(\S+)\s+(.*)$/)
+
         account_type_token = matcher[1]
-        account_type = AccountType.to_type(account_type_token).symbol
+        account_type = AccountType.letter_to_type(account_type_token)
 
         name = matcher[2]
 
-        accounts << Account.new(code, account_type, name)
+        accounts << Account.new(code, account_type.symbol, name)
       end
-    rescue => e
-      puts "Error parsing chart of accounts. Line text is:\n#{line}\n\n"
-      raise
     end
 
   end
@@ -105,7 +122,7 @@ class ChartOfAccounts
 
 
   def report_string
-    result = ''
+    result = StringIO.new
 
     if title
       result << title << "\n\n"
@@ -113,9 +130,9 @@ class ChartOfAccounts
 
     code_width = @accounts.inject(0) { |width, a| width = [width, a.code.length].max }
     format_string = "%-#{code_width}s  %-10.10s  %s\n"
-    accounts.each { |a| result << (format_string % [a.code, a.type.to_s, a.name]) }
+    accounts.each { |a| result << sprintf(format_string, a.code, a.type.to_s, a.name) }
 
-    result
+    result.string
   end
 
 

data/lib/rock_books/documents/journal.rb

@@ -7,7 +7,7 @@ require_relative '../types/acct_amount'
 require_relative '../types/journal_entry_context'
 require_relative 'journal_entry'
 require_relative 'journal_entry_builder'
-require_relative '../reports/reporter'
+require_relative '../reports/helpers/reporter'
 
 module RockBooks
 
@@ -45,9 +45,7 @@ class Journal
   end
 
 
-
-
-    def self.acct_amounts_in_documents(documents, entries_filter = nil, acct_amounts_filter = nil)
+  def self.acct_amounts_in_documents(documents, entries_filter = nil, acct_amounts_filter = nil)
     entries = entries_in_documents(documents, entries_filter)
 
     acct_amounts = entries.each_with_object([]) do |entry, acct_amounts|
@@ -72,7 +70,6 @@ class Journal
     @short_name = short_name
     @entries = []
     @date_prefix = ''
-    @title = ''
     input_lines.each_with_index do |line, linenum|
       context = JournalEntryContext.new(self, linenum + 1, line)
       parse_line(context)
@@ -131,7 +128,6 @@ class Journal
   end
 
 
-
   def acct_amounts
     entries.each_with_object([]) { |entry, acct_amounts|  acct_amounts << entry.acct_amounts }.flatten
   end

data/lib/rock_books/documents/journal_entry.rb

@@ -32,12 +32,17 @@ class JournalEntry < Struct.new(:date, :acct_amounts, :doc_short_name, :descript
 
   def self.sort_entries_by_amount_descending!(entries)
     entries.sort_by! do |entry|
-      [entry.total_absolute_value, entry.doc_short_name]
+      [-entry.total_absolute_value, entry.doc_short_name]
       end
-    entries.reverse!
   end
 
 
+  def self.sort_entries_by_date!(entries)
+    entries.sort_by! { |entry| [entry.date, entry.doc_short_name] }
+  end
+
+
+
   def total_for_code(account_code)
     acct_amounts_with_code(account_code).map(&:amount).sum
   end

data/lib/rock_books/documents/journal_entry_builder.rb

@@ -14,6 +14,10 @@ class JournalEntryBuilder < Struct.new(:journal_entry_context)
   def chart_of_accounts; journal_entry_context.chart_of_accounts;  end
 
 
+  # A "token" in this context means an account name and account amount pair, e.g. "pnc.checking 1234.56".
+  # @param tokens the account name/amount pairs found in the source text
+  # @date transaction date
+  # @return array of AcctAmount instances
   def acct_amounts_from_tokens(tokens, date)
     acct_amounts = []
 

data/lib/rock_books/helpers/book_set_loader.rb

@@ -7,6 +7,7 @@ module RockBooks
 
     module_function
 
+    # @return a hash whose keys are the filespecs and values are the document types
     def get_files_with_types(directory)
       files = Dir[File.join(directory, '*.txt')]
       files.each_with_object({}) do |filespec, files_with_types|
@@ -42,7 +43,7 @@ module RockBooks
 
     # Uses all *.txt files in the specified directory; uses @doc_type to determine which
     # is the chart of accounts and which are journals.
-    # To exclude a file, make the extension other than .rdt.
+    # To exclude a file, make the extension something other than .txt.
     def load(run_options)
 
       files_with_types = get_files_with_types(run_options.input_dir)
@@ -54,9 +55,8 @@ module RockBooks
       validate_journal_file_count(journal_files)
 
       chart_of_accounts = ChartOfAccounts.from_file(chart_of_account_files.first)
-      journals = journal_files.map { |fs| Journal.from_file(chart_of_accounts, fs) }
+      journals = journal_files.map { |filespec| Journal.from_file(chart_of_accounts, filespec) }
       BookSet.new(run_options, chart_of_accounts, journals)
     end
-
   end
 end

data/lib/rock_books/helpers/html_helper.rb

@@ -1,29 +1,35 @@
 module RockBooks
 module HtmlHelper
 
-  module_function
+  def self.convert_receipts_to_hyperlinks(original_html_string, html_filespec)
+    html_lines = original_html_string.split("\n")
+    content_changed = false
 
-  def self.convert_receipts_to_hyperlinks(html_text)
-    html_lines = html_text.split("\n")
-    replacements_made = false
+    # If the HTML file being created is in DATA_DIR/rockbooks-reports/html/single-account, then
+    #   the processed link should be '../../../receipts/[receipt_filespec]'
+    # else it's in DATA_DIR/rockbooks-reports/html, and
+    #   the processed link should be '../../receipts/[receipt_filespec]'
+    processed_receipt_filespec = ->(listed_receipt_filespec) do
+      num_dirs_up = html_filespec.include?('/single-account/') ? 3 : 2
+      File.join(('../' * num_dirs_up), 'receipts', listed_receipt_filespec)
+    end
+
+    receipt_anchor_line = ->(line, listed_receipt_filespec) do
+      line.gsub( \
+            /Receipt:\s*#{listed_receipt_filespec}/, \
+            %Q{Receipt: <a href="#{processed_receipt_filespec.(listed_receipt_filespec)}">#{listed_receipt_filespec}</a>})
+    end
 
     html_lines.each_with_index do |line, index|
-      matches = /Receipt:\s*(.*?)</.match(line)
+      matches = /Receipt:\s*(\S*)/.match(line)
       if matches
-        receipt_filespec = matches[1]
-        line_with_hyperlink = line.gsub( \
-            /Receipt:\s*#{receipt_filespec}/, \
-            %Q{Receipt: <a href="../../../receipts/#{receipt_filespec}">#{receipt_filespec}</a>})
-        html_lines[index] = line_with_hyperlink
-        replacements_made = true
+        listed_receipt_filespec = matches[1]
+        html_lines[index] = receipt_anchor_line.(line, listed_receipt_filespec)
+        content_changed = true
       end
     end
 
-    if replacements_made
-      html_text = html_lines.join("\n")
-    end
-
-    [html_text, replacements_made]
+    content_changed ? html_lines.join("\n") : original_html_string
   end
 end
 end

data/lib/rock_books/reports/balance_sheet.rb

@@ -1,6 +1,5 @@
-require_relative '../filters/journal_entry_filters'
-require_relative '../documents/journal'
-require_relative 'report_context'
+require_relative 'helpers/erb_helper'
+require_relative 'helpers/reporter'
 
 module RockBooks
 
@@ -10,51 +9,18 @@ module RockBooks
 class BalanceSheet
 
   include Reporter
+  include ErbHelper
 
-  attr_accessor :context
+  attr_accessor :context, :data
 
-  def initialize(report_context)
+  def initialize(report_context, data)
     @context = report_context
+    @data = data
   end
 
 
-  def end_date
-    context.chart_of_accounts.end_date
+  def generate
+    ErbHelper.render_hashes('text/balance_sheet.txt.erb', data, template_presentation_context)
   end
-
-
-  def generate_header
-    lines = [banner_line]
-    lines << center(context.entity || 'Unspecified Entity')
-    lines << center("Balance Sheet for Period Ending #{end_date}")
-    lines << banner_line
-    lines << ''
-    lines << ''
-    lines << ''
-    lines.join("\n")
-  end
-
-
- def generate_report
-    filter = RockBooks::JournalEntryFilters.date_on_or_before(end_date)
-    acct_amounts = Journal.acct_amounts_in_documents(context.journals, filter)
-    totals = AcctAmount.aggregate_amounts_by_account(acct_amounts)
-    output = generate_header
-
-    asset_output,  asset_total  = generate_account_type_section('Assets',      totals, :asset,     false)
-    liab_output,   liab_total   = generate_account_type_section('Liabilities', totals, :liability, true)
-    equity_output, equity_total = generate_account_type_section('Equity',      totals, :equity,    true)
-
-    output << [asset_output, liab_output, equity_output].join("\n\n")
-
-    grand_total = asset_total - (liab_total + equity_total)
-
-    output << "\n#{"%12.2f    Assets - (Liabilities + Equity)" % grand_total}\n============\n"
-    output
-  end
-
-  alias_method :to_s, :generate_report
-  alias_method :call, :generate_report
-
 end
 end

data/lib/rock_books/reports/book_set_reporter.rb

@@ -1,13 +1,19 @@
 require_relative '../documents/book_set'
 
 require_relative 'balance_sheet'
+require_relative 'data/bs_is_data'
+require_relative 'data/receipts_report_data'
 require_relative 'income_statement'
-require_relative 'multidoc_transaction_report'
+require_relative 'multidoc_txn_report'
 require_relative 'receipts_report'
 require_relative 'report_context'
-require_relative 'transaction_report'
-require_relative 'tx_by_account'
+require_relative 'journal_report'
+require_relative 'multidoc_txn_by_account_report'
 require_relative 'tx_one_account'
+require_relative 'helpers/erb_helper'
+require_relative 'helpers/reporter'
+
+require 'prawn'
 
 module RockBooks
 class BookSetReporter
@@ -21,6 +27,8 @@ class BookSetReporter
   def_delegator :book_set, :chart_of_accounts
   def_delegator :book_set, :run_options
 
+  FONT_FILESPEC = File.absolute_path(File.join(File.dirname(__FILE__), '..', '..', '..', 'assets', 'fonts', 'JetBrainsMono-Medium.ttf'))
+
 
   def initialize(book_set, output_dir, filter = nil)
     @book_set = book_set
@@ -30,72 +38,84 @@ class BookSetReporter
   end
 
 
-  def call
-    check_prequisite_executables
-    reports = all_reports(filter)
+  def generate
     create_directories
     create_index_html
-    write_reports(reports)
+
+    do_statements
+    do_journals
+    do_transaction_reports
+    do_single_account_reports
+    do_receipts_report
   end
 
 
   # All methods after this point are private.
 
-  private def all_reports(filter = nil)
+  private def do_statements
+    bs_is_data = BsIsData.new(context)
+
+    bal_sheet_text_report = BalanceSheet.new(context, bs_is_data.bal_sheet_data).generate
+    write_report(:balance_sheet, bal_sheet_text_report)
 
-    report_hash = journals.each_with_object({}) do |journal, report_hash|
-      key = journal.short_name.to_sym
-      report_hash[key] = TransactionReport.new(journal, context).call(filter)
+    inc_stat_text_report = IncomeStatement.new(context, bs_is_data.inc_stat_data).generate
+    write_report(:income_statement, inc_stat_text_report)
+  end
+
+
+  private def do_journals
+    journals.each do |journal|
+      report_data = JournalData.new(journal, context, filter).fetch
+      text_report = JournalReport.new(report_data, context, filter).generate
+      write_report(journal.short_name, text_report)
     end
+  end
 
-    report_hash[:all_txns_by_date] = MultidocTransactionReport.new(context).call(filter)
-    report_hash[:all_txns_by_amount] = MultidocTransactionReport.new(context).call(filter, :amount)
-    report_hash[:all_txns_by_acct] = TxByAccount.new(context).call
-    report_hash[:balance_sheet] = BalanceSheet.new(context).call
-    report_hash[:income_statement] = IncomeStatement.new(context).call
 
-    if run_options.do_receipts
-      report_hash[:receipts] = ReceiptsReport.new(context, *missing_existing_unused_receipts).call
+  private def do_transaction_reports
+
+    do_date_or_amount_report = ->(sort_field, short_name) do
+      data = MultidocTxnReportData.new(context, sort_field, filter).fetch
+      text_report = MultidocTransactionReport.new(data, context).generate
+      write_report(short_name, text_report)
     end
 
-    chart_of_accounts.accounts.each do |account|
-      key = ('acct_' + account.code).to_sym
-      report = TxOneAccount.new(context, account.code).call
-      report_hash[key] = report
+    do_acct_report = -> do
+      data = MultidocTxnByAccountData.new(context).fetch
+      text_report = MultidocTransactionByAccountReport.new(data, context).generate
+      write_report(:all_txns_by_acct, text_report)
     end
 
-    report_hash
+    do_date_or_amount_report.(:date,   :all_txns_by_date)
+    do_date_or_amount_report.(:amount, :all_txns_by_amount)
+    do_acct_report.()
   end
 
 
-  private def run_command(command)
-    puts "\n----\nRunning command: #{command}"
-    stdout, stderr, status = Open3.capture3(command)
-    puts "Status was #{status}."
-    unless stdout.size == 0
-      puts "\nStdout was:\n\n#{stdout}"
-    end
-    unless stderr.size == 0
-      puts "\nStderr was:\n\n#{stderr}"
+  private def do_single_account_reports
+    chart_of_accounts.accounts.each do |account|
+      short_name = ('acct_' + account.code).to_sym
+      data = TxOneAccountData.new(context, account.code).fetch
+      text_report = TxOneAccount.new(data, context).generate
+      write_report(short_name, text_report)
     end
-    puts
-    stdout
   end
 
 
-  private def executable_exists?(name)
-    `which #{name}`
-    $?.success?
+  private def do_receipts_report
+    data = ReceiptsReportData.new(book_set.all_entries, run_options.receipt_dir).fetch
+    text_report = ReceiptsReport.new(context, data).generate
+    write_report(:receipts, text_report)
   end
 
 
-  private def check_prequisite_executables
-    raise "Report generation is not currently supported in Windows." if OS.windows?
-    required_exes = OS.mac? ? %w(textutil cupsfilter) : %w(txt2html wkhtmltopdf)
-    missing_exes = required_exes.reject { |exe| executable_exists?(exe) }
-    if missing_exes.any?
-      raise "Missing required report generation executable(s): #{missing_exes.join(', ')}. Please install them with your system's package manager."
-    end
+  private def run_command(command)
+    puts "\n----\nRunning command: #{command}"
+    stdout, stderr, status = Open3.capture3(command)
+    puts "Exit code was #{status.exitstatus}."
+    puts "\nStdout was:\n\n#{stdout}" unless stdout.size == 0
+    puts "\nStderr was:\n\n#{stderr}" unless stderr.size == 0
+    puts
   end
 
 
@@ -125,71 +145,69 @@ class BookSetReporter
   end
 
 
-  private def write_reports(reports)
+  private def prawn_create_document(pdf_filespec, text)
+    Prawn::Document.generate(pdf_filespec) do
+      font(FONT_FILESPEC, size: 10)
 
-    reports.each do |short_name, report_text|
-      txt_filespec = build_filespec(output_dir, short_name, 'txt')
-      html_filespec = build_filespec(output_dir, short_name, 'html')
-      pdf_filespec = build_filespec(output_dir, short_name, 'pdf')
+      utf8_nonbreaking_space = "\uC2A0"
+      unicode_nonbreaking_space = "\u00A0"
+      text(text.gsub(' ', unicode_nonbreaking_space))
+    end
+    puts "Finished generating #{pdf_filespec} with prawn."
+  end
 
-      File.write(txt_filespec, report_text)
 
-      # Mac OS
-      textutil = ->(font_size) do
-        run_command("textutil -convert html -font 'Courier New Bold' -fontsize #{font_size} #{txt_filespec} -output #{html_filespec}")
-      end
-      cupsfilter = -> { run_command("cupsfilter #{txt_filespec} > #{pdf_filespec}") }
-
-      # Linux
-      txt2html = -> { run_command("txt2html --preformat_trigger_lines 0 #{txt_filespec} > #{html_filespec}") }
-      html2pdf = -> { run_command("wkhtmltopdf #{html_filespec} #{pdf_filespec}") }
-
-      # Use smaller size for the PDF but larger size for the web pages:
-      if OS.mac?
-        textutil.(11)
-        cupsfilter.()
-        textutil.(14)
-      else
-        txt2html.()
-        html2pdf.()
-      end
+  private def write_report(short_name, text_report)
 
-      hyperlinkized_text, replacements_made = HtmlHelper.convert_receipts_to_hyperlinks(File.read(html_filespec))
-      if replacements_made
-        File.write(html_filespec, hyperlinkized_text)
-      end
+    txt_filespec = build_filespec(output_dir, short_name, 'txt')
+    html_filespec = build_filespec(output_dir, short_name, 'html')
+    pdf_filespec = build_filespec(output_dir, short_name, 'pdf')
+
+    create_text_report = -> { File.write(txt_filespec, text_report) }
+
+    create_pdf_report = -> { prawn_create_document(pdf_filespec, text_report) }
 
-      puts "Created reports in txt, html, and pdf for #{"%-20s" % short_name} at #{File.dirname(txt_filespec)}.\n\n\n"
+    create_html_report = -> do
+      data = { report_body: text_report }
+      html_raw_report = ErbHelper.render_hashes("html/report_page.html.erb", data, {})
+      html_report = HtmlHelper.convert_receipts_to_hyperlinks(html_raw_report, html_filespec)
+      File.write(html_filespec, html_report)
     end
-  end
+
+    create_text_report.()
+    create_pdf_report.()
+    create_html_report.()
 
 
-  private def receipt_full_filespec(receipt_filespec)
-    File.join(run_options.receipt_dir, receipt_filespec)
+    puts "Created reports in txt, html, and pdf for #{"%-20s" % short_name} at #{File.dirname(txt_filespec)}.\n\n\n"
   end
 
 
   private def missing_existing_unused_receipts
-    missing = []
-    existing = []
-    unused = Dir['receipts/**/*'].select { |s| File.file?(s) }.sort # Remove files as they are found
-    unused.map! { |s| "./" +  s }  # Prepend './' to match the data
+    missing_receipts = []
+    existing_receipts = []
+    receipt_full_filespec = ->(receipt_filespec) { File.join(run_options.receipt_dir, receipt_filespec) }
+
+    # We will start out putting all filespecs in the unused array, and delete them as they are found in the transactions.
+    unused_receipt_filespecs = Dir['receipts/**/*'].select { |s| File.file?(s) } \
+        .sort \
+        .map { |s| "./" +  s }  # Prepend './' to match the data
 
     all_entries.each do |entry|
       entry.receipts.each do |receipt|
-        filespec = receipt_full_filespec(receipt)
-        unused.delete(filespec)
+        filespec = receipt_full_filespec.(receipt)
+        unused_receipt_filespecs.delete(filespec)
         file_exists = File.file?(filespec)
-        list = (file_exists ? existing : missing)
+        list = (file_exists ? existing_receipts : missing_receipts)
         list << { receipt: receipt, journal: entry.doc_short_name }
       end
     end
-    [missing, existing, unused]
+    [missing_receipts, existing_receipts, unused_receipt_filespecs]
   end
 
 
   private def index_html_content
-    erb_filespec = File.join(File.dirname(__FILE__), 'index.html.erb')
+    erb_filespec = File.join(File.dirname(__FILE__), 'templates', 'html', 'index.html.erb')
     erb = ERB.new(File.read(erb_filespec))
     erb.result_with_hash(
         journals: journals,