brisk-bills 0.7.0 → 0.8.1

data/db/migrate/019_create_settings.rb

@@ -1,6 +1,6 @@
 class CreateSettings < ActiveRecord::Migration
   def self.up
-    create_table( :settings, :options => 'TYPE=InnoDB') do |t|
+    create_table( :settings) do |t|
       t.string :keyname, :label
       t.text :keyval, :description
 

data/db/migrate/023_create_credentials_migrate_representatives.rb

@@ -12,7 +12,7 @@ class CreateCredentialsMigrateRepresentatives < ActiveRecord::Migration
     # Phase 1: Since we're here - habtm the client_representatives
     say_with_time "Migrating Client Representatives" do 
       # Create the Representatives to clients habtm relationship
-      create_table( :client_representatives_clients, :options => 'TYPE=InnoDB', :id => false ) do |t|
+      create_table( :client_representatives_clients, :id => false ) do |t|
         t.integer :client_id, :client_representative_id
       end
       
@@ -52,7 +52,7 @@ class CreateCredentialsMigrateRepresentatives < ActiveRecord::Migration
     say_with_time "Migrating Credentials" do 
       
       # First we create the credentials table:
-      create_table( :credentials, :options => 'TYPE=InnoDB') do |t|
+      create_table( :credentials) do |t|
         t.string :email_address, :password_hash
                 
         t.integer :failed_login_count, :null => false, :default => 0

data/db/migrate/028_money_to_cents.rb

@@ -107,8 +107,8 @@ class MoneyToCents < ActiveRecord::Migration
 
   def self.up
     CONVERT_FIELDS.each_pair do |klass, cols|
-      cols.to_a.each do |col|
-        klass.update_all('%s = %s * 100' % (col.to_s.to_a * 2) )
+      [cols].flatten.each do |col|
+        klass.update_all('%s = %s * 100' % ([col.to_s] * 2) )
         change_column klass.table_name, col, :integer
         rename_column klass.table_name, col, ('%s_in_cents' % col.to_s ).to_sym 
       end

data/db/schema.rb

@@ -72,17 +72,19 @@ ActiveRecord::Schema.define(:version => 29) do
     t.datetime "created_at"
   end
 
-  create_table "client_finance_transactions", :force => true do |t|
+  create_table "client_finance_transactions", :id => false, :force => true do |t|
+    t.string   "id",              :limit => 18,                                      :default => "", :null => false
     t.integer  "client_id"
     t.datetime "date"
-    t.binary   "description",     :limit => 16777215
+    t.text     "description",     :limit => 16777215
     t.integer  "amount_in_cents", :limit => 34,       :precision => 34, :scale => 0
   end
 
-  create_table "client_finance_transactions_union", :force => true do |t|
+  create_table "client_finance_transactions_union", :id => false, :force => true do |t|
+    t.string   "id",              :limit => 18,                                      :default => "", :null => false
     t.integer  "client_id"
     t.datetime "date"
-    t.binary   "description",     :limit => 16777215
+    t.text     "description",     :limit => 16777215
     t.integer  "amount_in_cents", :limit => 34,       :precision => 34, :scale => 0
   end
 
@@ -120,7 +122,8 @@ ActiveRecord::Schema.define(:version => 29) do
     t.boolean  "is_active",    :default => true, :null => false
   end
 
-  create_table "clients_with_balances", :force => true do |t|
+  create_table "clients_with_balances", :id => false, :force => true do |t|
+    t.integer  "id",                                                                                  :default => 0,    :null => false
     t.string   "company_name"
     t.string   "address1"
     t.string   "address2"
@@ -220,7 +223,8 @@ ActiveRecord::Schema.define(:version => 29) do
     t.integer "amount_paid_in_cents", :limit => 32, :precision => 32, :scale => 0
   end
 
-  create_table "invoices_with_totals", :force => true do |t|
+  create_table "invoices_with_totals", :id => false, :force => true do |t|
+    t.integer  "id",                                                                :default => 0,     :null => false
     t.integer  "client_id"
     t.text     "comments"
     t.datetime "issued_on"
@@ -251,10 +255,6 @@ ActiveRecord::Schema.define(:version => 29) do
   add_index "payments", ["client_id"], :name => "index_payments_on_client_id"
   add_index "payments", ["payment_method_id"], :name => "index_payments_on_payment_method_id"
 
-  create_table "schema_info", :id => false, :force => true do |t|
-    t.integer "version"
-  end
-
   create_table "sessions", :force => true do |t|
     t.string   "session_id", :null => false
     t.text     "data"

data/lib/brisk-bills.rb

@@ -3,8 +3,8 @@ BRISKBILLS_ROOT = File.expand_path(File.join(File.dirname(__FILE__), "..")) unle
 unless defined? BriskBills::Version
   module BriskBills::Version
     Major = '0'
-    Minor = '7'
-    Tiny  = '0'
+    Minor = '8'
+    Tiny  = '1'
 
     class << self
       def to_s

data/lib/brisk-bills/initializer.rb

@@ -1,6 +1,6 @@
 require 'rubygems'
-gem 'rails', '= 2.3.2'
-require 'initializer' # This is the rails-2.3.2/lib/initializer
+gem 'rails', '= 2.3.16'
+require 'initializer' # This is the rails-2.3.8/lib/initializer
 
 module BriskBills
 

data/lib/generators/instance/templates/instance_environment.rb

@@ -44,7 +44,7 @@ BriskBills::Initializer.run do |config|
   # Make sure the secret is at least 30 characters and all random, 
   # no regular words or you'll be exposed to dictionary attacks.
   config.action_controller.session = {
-    :session_key => '_brisk-bills_session',
+    :key => '_brisk-bills_session',
     :secret      => '<%= ([nil]*54).collect{character_pool[rand(character_pool.length)]}.join%>'
   }
   

data/lib/libpptable.rb

@@ -0,0 +1,48 @@
+# I ended up using this in a couple tools - so, I gave it its own library
+
+# This got a little complicated. But, it works well ansd should be reuseable.
+# I didn't put a check in here to ensure that every data row is of the same length as the header.
+# But they should be - or else weirdness will ensue
+def pp_table(header,data)
+    row_prefix    = "| "
+    row_suffix    = " |"
+    col_join      = " | "
+    header_colsep = "-+-"
+    colsep_prefix = "+-"
+    colsep_suffix = "-+"
+    table_corner  = "+"
+
+    # This is a quick way to calculate each column of the data's max string-width:
+    array_col_widths = ([header]+data).inject(Array.new(header.length,0)){|ret,row|
+       row.each_with_index{|f,i| ret[i] = f.length if f.to_s.length > ret[i] }
+       ret
+    }
+
+    col_sprintf = row_prefix+array_col_widths.collect{|w| "%-#{w}s" }.join(col_join)+row_suffix
+
+    table_width = (
+      array_col_widths+[
+        (array_col_widths.length-1)*col_join.length,
+        row_prefix.length,
+        row_suffix.length
+      ]
+    ).reduce(:+)
+
+    table_ends = [table_corner,"-" * (table_width-table_corner.length*2),table_corner].join
+
+    puts [
+      table_ends,                        # Table-top
+      col_sprintf % header,              # Header row
+      [                                  # Header/data Separator
+        colsep_prefix,
+        array_col_widths.collect{|w| "-" * w}.join(header_colsep),
+        colsep_suffix
+      ].join,
+      data.collect{|a| col_sprintf % a}, # Data rows
+      table_ends,                        # Table-bottom
+    ].flatten.join("\n")
+end
+
+def humanize_bool(v)
+  v ? "Yes" : "No"
+end

data/lib/tasks/create_last_months_invoices.rake

@@ -4,20 +4,7 @@ namespace :brisk_bills do
   task :create_last_months_invoices => :environment do
     end_of_last_month = Time.utc(*Time.now.to_a).last_month.end_of_month
  
-    invoiceable_client_ids = Activity.find(
-      :all, 
-      :select => 'DISTINCT client_id', 
-      :conditions => [
-        [
-        'is_published = ?',
-        'invoice_id IS NULL',
-        'client_id IS NOT NULL', 
-        'occurred_on <= ?'
-        ].join(' AND '),
-        true,
-        end_of_last_month
-      ]
-    ).collect{|a| a.client.id}
+    invoiceable_client_ids = Client.find_invoiceable_clients_at(end_of_last_month)
     
     if invoiceable_client_ids.length > 0
       all_activity_types = ActivityType.find(:all)

data/lib/tasks/first_time_setup.rake

@@ -38,7 +38,7 @@ namespace :brisk_bills do
           "CREATE DATABASE `%s`;" % envconf[:database],
           "USE `%s`;" % envconf[:database],
           "GRANT ALL PRIVILEGES ON `%s`.* TO `%s`@`%s` IDENTIFIED BY '%s';" % [envconf[:database], envconf[:username],envconf[:host],envconf[:password]]
-        ].join ("\n  ")
+        ].join("\n  ")
       ) if db_version.nil?
       
       # Let's run migrations?
@@ -93,4 +93,4 @@ namespace :brisk_bills do
         exit
     end
   end
-end
+end

data/lib/tasks/package.rake

@@ -40,7 +40,8 @@ namespace 'package' do
     s.add_dependency 'slimtimer4r', '>= 0.2.4'
     s.add_dependency 'money',       '>= 2.2.0'
     s.add_dependency 'mysql',       '>= 2.7'
-    s.add_dependency 'rails',       '>= 2.3.2'
+    s.add_dependency 'rails',       '= 2.3.16'
+    s.add_dependency 'i18n',        '= 0.4.2'
 
     s.has_rdoc = true
     s.rdoc_options = [

data/lib/tasks/payment_assignment_consistency_check.rake

@@ -0,0 +1,110 @@
+require 'libpptable'
+namespace :brisk_bills do
+  desc "Check for inconsistencies in the invoice & payment assignments"
+  
+  task :payment_assignment_consistency_check => :environment do
+    
+    # NOTE: One way to 'fix' any problems with your invoice assignments is to: 
+    #   InvoicePayment.find(:all).each{|ip| ip.destroy}
+    #   Payment.find(:all).each{|pmt| if pmt.amount > Money.new(0) then pmt.invoice_assignments = pmt.client.recommend_invoice_assignments_for(pmt.amount); pmt.save! end }
+    # But, keep in mind that this basically destroys all the associations you may have worked so hard on...
+
+    # We use this as a helper for each test below 
+    def db_check(name, &block)
+      print ' - %s...' % name
+      records = block.call
+
+      puts (records.length > 0) ? 
+        ("ERROR: %d records Found" % records.length) : 
+        "PASS"
+    end
+    
+    pp_table(
+      ["Table", "Record Count"],      
+      [Invoice,Payment,InvoicePayment,Activity,Client].collect{|klass|
+        [klass.table_name, klass.count]
+      }
+    )
+    
+    puts 
+    puts "Tests: "
+    
+    # Test : Check for InvoicePayments <= 0
+    db_check 'Testing for assignments with an invalid amount' do
+      InvoicePayment.find(:all, :conditions => ['amount_in_cents <= 0'])
+    end
+    
+    # Test : Check for InvoicePayments which are orphaned from an invoice or payment
+    #        (This means NULL or an id to a payment/invoice that's not in the db
+    db_check 'Testing for assignments with an NULL payment_id or invoice_id' do    
+      InvoicePayment.find(
+        :all, 
+        :conditions => ['invoice_id IS NULL OR payment_id IS NULL']
+      )
+    end
+    
+    db_check 'Testing for assignments with a missing payment or invoice' do
+      InvoicePayment.find(
+        :all, 
+        :select => [
+          'invoice_payments.id',
+          'payments.id AS payment_id',
+          'invoices.id AS invoice_id'
+          ].join(','),
+        :joins => [
+          'LEFT JOIN payments ON invoice_payments.payment_id = payments.id',
+          'LEFT JOIN invoices ON invoice_payments.invoice_id = invoices.id'
+          ].join(' '),
+        :conditions => [
+          'payments.id IS NULL OR invoices.id IS NULL'
+        ]
+      )
+    end
+
+    # Test : Ensure that the InvoicePayments for each invoice don't exceed its total amount  
+    db_check "Testing for payments whose assignments exceed the payment total" do
+      Payment.find(
+        :all,
+        :select => [
+          'payments.id', 
+          'payments.amount_in_cents',
+          'ip.allocated_in_cents'
+        ].join(','),
+        :joins => 'LEFT JOIN (
+          SELECT 
+            invoice_payments.payment_id, 
+            SUM(invoice_payments.amount_in_cents) AS allocated_in_cents 
+          FROM invoice_payments 
+          GROUP BY invoice_payments.payment_id
+        ) AS ip ON ip.payment_id = payments.id ',
+        :conditions => 'ip.allocated_in_cents > payments.amount_in_cents'
+      )
+    end
+
+    # Test : Ensure that the InvoicePayments for each payment don't exceed its total amount
+    db_check "Testing for invoices whose assignments exceed the invoice total" do
+      # This one gets a little complicated since the invoice totals query is complicated
+      # and I don't want to make this query completely ridiculous.
+      
+      # Generate a map that tracks allocations:
+      invoice_allocations = InvoicePayment.find(
+        :all, 
+        :select => [
+          'id',
+          'invoice_id',
+          'SUM(amount_in_cents) AS allocated_in_cents'
+        ].join(' ,'), 
+        :group => 'invoice_id'
+      ).inject({}){ |ret,ip| ret.merge({ip.invoice_id => ip.allocated_in_cents.to_i}) }
+      
+      
+      Invoice.find_with_totals( :all, :conditions => ['is_published = ?', true] ).find_all{|inv|
+        ( 
+        invoice_allocations.has_key? inv.id and 
+        invoice_allocations[inv.id] > inv.amount_in_cents.to_i
+        )
+      }
+    end
+    
+  end
+end

data/lib/utilities.rb

@@ -7,12 +7,12 @@ class String
     
     options.each_char do |c| 
       mods |= case c
-        when 'i': Regexp::IGNORECASE
-        when 'x': Regexp::EXTENDED
-        when 'm': Regexp::MULTILINE
+        when 'i' then Regexp::IGNORECASE
+        when 'x' then Regexp::EXTENDED
+        when 'm' then Regexp::MULTILINE
       end
     end unless options.nil? or options.empty?
         
     Regexp.new source, mods
   end
-end
+end

data/public/javascripts/active_scaffold/default/active_scaffold.js

@@ -1,430 +1,532 @@
-if (typeof Prototype == 'undefined')
-{
-  warning = "ActiveScaffold Error: Prototype could not be found. Please make sure that your application's layout includes prototype.js (e.g. <%= javascript_include_tag :defaults %>) *before* it includes active_scaffold.js (e.g. <%= active_scaffold_includes %>).";
-  alert(warning);
-}
-if (Prototype.Version.substring(0, 3) != '1.6')
-{
-  warning = "ActiveScaffold Error: Prototype version 1.6.x is required. Please update prototype.js (rake rails:update:javascripts).";
-  alert(warning);
-}
-
-/*
- * Simple utility methods
- */
-
-var ActiveScaffold = {
-  records_for: function(tbody_id) {
-    var rows = [];
-    var child = $(tbody_id).down('.record');
-    while (child) {
-      rows.push(child);
-      child = child.next('.record');
-    }
-    return rows;
-  },
-  stripe: function(tbody_id) {
-    var even = false;
-    var rows = this.records_for(tbody_id);
-    for (var i = 0; i < rows.length; i++) {
-      var child = rows[i];
-      //Make sure to skip rows that are create or edit rows or messages
-      if (child.tagName != 'SCRIPT'
-        && !child.hasClassName("create")
-        && !child.hasClassName("update")
-        && !child.hasClassName("inline-adapter")
-        && !child.hasClassName("active-scaffold-calculations")) {
-
-        if (even) child.addClassName("even-record");
-        else child.removeClassName("even-record");
-
-        even = !even;
-      }
-    }
-  },
-  hide_empty_message: function(tbody, empty_message_id) {
-    if (this.records_for(tbody).length != 0) {
-      $(empty_message_id).hide();
-    }
-  },
-  reload_if_empty: function(tbody, url) {
-    var content_container_id = tbody.replace('tbody', 'content');
-    if (this.records_for(tbody).length == 0) {
-      new Ajax.Updater($(content_container_id), url, {
-        method: 'get',
-        asynchronous: true,
-        evalScripts: true
-      });
-    }
-  },
-  removeSortClasses: function(scaffold_id) {
-    $$('#' + scaffold_id + ' td.sorted').each(function(element) {
-      element.removeClassName("sorted");
-    });
-    $$('#' + scaffold_id + ' th.sorted').each(function(element) {
-      element.removeClassName("sorted");
-      element.removeClassName("asc");
-      element.removeClassName("desc");
-    });
-  },
-  decrement_record_count: function(scaffold_id) {
-    // decrement the last record count, firsts record count are in nested lists
-    count = $$('#' + scaffold_id + ' span.active-scaffold-records').last();
-    count.innerHTML = parseInt(count.innerHTML) - 1;
-  },
-  increment_record_count: function(scaffold_id) {
-    // increment the last record count, firsts record count are in nested lists
-    count = $$('#' + scaffold_id + ' span.active-scaffold-records').last();
-    count.innerHTML = parseInt(count.innerHTML) + 1;
-  },
-
-  server_error_response: '',
-  report_500_response: function(active_scaffold_id) {
-    messages_container = $(active_scaffold_id).down('td.messages-container');
-    new Insertion.Top(messages_container, this.server_error_response);
-  }
-}
-
-/*
- * DHTML history tie-in
- */
-function addActiveScaffoldPageToHistory(url, active_scaffold_id) {
-  if (typeof dhtmlHistory == 'undefined') return; // it may not be loaded
-
-  var array = url.split('?');
-  var qs = new Querystring(array[1]);
-  var sort = qs.get('sort')
-  var dir = qs.get('sort_direction')
-  var page = qs.get('page')
-  if (sort || dir || page) dhtmlHistory.add(active_scaffold_id+":"+page+":"+sort+":"+dir, url);
-}
-
-/*
- * Add-ons/Patches to Prototype
- */
-
-/* patch to support replacing TR/TD/TBODY in Internet Explorer, courtesy of http://dev.rubyonrails.org/ticket/4273 */
-Element.replace = function(element, html) {
-  element = $(element);
-  if (element.outerHTML) {
-    try {
-    element.outerHTML = html.stripScripts();
-    } catch (e) {
-      var tn = element.tagName;
-      if(tn=='TBODY' || tn=='TR' || tn=='TD')
-      {
-              var tempDiv = document.createElement("div");
-              tempDiv.innerHTML = '<table id="tempTable" style="display: none">' + html.stripScripts() + '</table>';
-              element.parentNode.replaceChild(tempDiv.getElementsByTagName(tn).item(0), element);
-      }
-      else throw e;
-    }
-  } else {
-    var range = element.ownerDocument.createRange();
-    /* patch to fix <form> replaces in Firefox. see http://dev.rubyonrails.org/ticket/8010 */
-    range.selectNodeContents(element.parentNode);
-    element.parentNode.replaceChild(range.createContextualFragment(html.stripScripts()), element);
-  }
-  setTimeout(function() {html.evalScripts()}, 10);
-  return element;
-};
-
-/*
- * URL modification support. Incomplete functionality.
- */
-Object.extend(String.prototype, {
-  append_params: function(params) {
-    url = this;
-    if (url.indexOf('?') == -1) url += '?';
-    else if (url.lastIndexOf('&') != url.length) url += '&';
-
-    url += $H(params).collect(function(item) {
-      return item.key + '=' + item.value;
-    }).join('&');
-
-    return url;
-  }
-});
-
-/*
- * Prototype's implementation was throwing an error instead of false
- */
-Element.Methods.Simulated = {
-  hasAttribute: function(element, attribute) {
-    var t = Element._attributeTranslations;
-    attribute = (t.names && t.names[attribute]) || attribute;
-    // Return false if we get an error here
-    try {
-      return $(element).getAttributeNode(attribute).specified;
-    } catch (e) {
-      return false;
-    }
-  }
-};
-
-/**
- * A set of links. As a set, they can be controlled such that only one is "open" at a time, etc.
- */
-ActiveScaffold.Actions = new Object();
-ActiveScaffold.Actions.Abstract = function(){}
-ActiveScaffold.Actions.Abstract.prototype = {
-  initialize: function(links, target, loading_indicator, options) {
-    this.target = $(target);
-    this.loading_indicator = $(loading_indicator);
-    this.options = options;
-    this.links = links.collect(function(link) {
-      return this.instantiate_link(link);
-    }.bind(this));
-  },
-
-  instantiate_link: function(link) {
-    throw 'unimplemented'
-  }
-}
-
-/**
- * A DataStructures::ActionLink, represented in JavaScript.
- * Concerned with AJAX-enabling a link and adapting the result for insertion into the table.
- */
-ActiveScaffold.ActionLink = new Object();
-ActiveScaffold.ActionLink.Abstract = function(){}
-ActiveScaffold.ActionLink.Abstract.prototype = {
-  initialize: function(a, target, loading_indicator) {
-    this.tag = $(a);
-    this.url = this.tag.href;
-    this.method = 'get';
-    if(this.url.match('_method=delete')){
-      this.method = 'delete';
-    } else if(this.url.match('_method=post')){
-      this.method = 'post';
-    }
-    this.target = target;
-    this.loading_indicator = loading_indicator;
-    this.hide_target = false;
-    this.position = this.tag.getAttribute('position');
-		this.page_link = this.tag.getAttribute('page_link');
-
-    this.onclick = this.tag.onclick;
-    this.tag.onclick = null;
-    this.tag.observe('click', function(event) {
-      this.open();
-      Event.stop(event);
-    }.bind(this));
-
-    this.tag.action_link = this;
-  },
-
-  open: function() {
-    if (this.is_disabled()) return;
-
-    if (this.tag.hasAttribute( "dhtml_confirm")) {
-      if (this.onclick) this.onclick();
-      return;
-    } else {
-      if (this.onclick && !this.onclick()) return;//e.g. confirmation messages
-      this.open_action();
-    }
-  },
-
-	open_action: function() {
-		if (this.position) this.disable();
-
-		if (this.page_link) {
-			window.location = this.url;
-		} else {
-			if (this.loading_indicator) this.loading_indicator.style.visibility = 'visible';
-	    new Ajax.Request(this.url, {
-	      asynchronous: true,
-	      evalScripts: true,
-              method: this.method,
-	      onSuccess: function(request) {
-	        if (this.position) {
-	          this.insert(request.responseText);
-	          if (this.hide_target) this.target.hide();
-	        } else {
-	          request.evalResponse();
-	        }
-	      }.bind(this),
-
-	      onFailure: function(request) {
-	        ActiveScaffold.report_500_response(this.scaffold_id());
-	        if (this.position) this.enable()
-	      }.bind(this),
-
-	      onComplete: function(request) {
-	        if (this.loading_indicator) this.loading_indicator.style.visibility = 'hidden';
-	      }.bind(this)
-			});
-		}
-	},
-
-  insert: function(content) {
-    throw 'unimplemented'
-  },
-
-  close: function() {
-    this.enable();
-    this.adapter.remove();
-    if (this.hide_target) this.target.show();
-  },
-
-  register_cancel_hooks: function() {
-    // anything in the insert with a class of cancel gets the closer method, and a reference to this object for good measure
-    var self = this;
-    this.adapter.select('.cancel').each(function(elem) {
-      elem.observe('click', this.close_handler.bind(this));
-      elem.link = self;
-    }.bind(this))
-  },
-
-  reload: function() {
-    this.close();
-    this.open();
-  },
-
-  get_new_adapter_id: function() {
-    var id = 'adapter_';
-    var i = 0;
-    while ($(id + i)) i++;
-    return id + i;
-  },
-
-  enable: function() {
-    return this.tag.removeClassName('disabled');
-  },
-
-  disable: function() {
-    return this.tag.addClassName('disabled');
-  },
-
-  is_disabled: function() {
-    return this.tag.hasClassName('disabled');
-  },
-
-  scaffold_id: function() {
-    return this.tag.up('div.active-scaffold').id;
-  }
-}
-
-/**
- * Concrete classes for record actions
- */
-ActiveScaffold.Actions.Record = Class.create();
-ActiveScaffold.Actions.Record.prototype = Object.extend(new ActiveScaffold.Actions.Abstract(), {
-  instantiate_link: function(link) {
-    var l = new ActiveScaffold.ActionLink.Record(link, this.target, this.loading_indicator);
-    l.refresh_url = this.options.refresh_url;
-    if (l.position) l.url = l.url.append_params({adapter: '_list_inline_adapter'});
-    l.set = this;
-    return l;
-  }
-});
-
-ActiveScaffold.ActionLink.Record = Class.create();
-ActiveScaffold.ActionLink.Record.prototype = Object.extend(new ActiveScaffold.ActionLink.Abstract(), {
-  close_previous_adapter: function() {
-    this.set.links.each(function(item) {
-      if (item.url != this.url && item.is_disabled() && item.adapter) item.close();
-    }.bind(this));
-  },
-
-  insert: function(content) {
-    this.close_previous_adapter();
-
-    if (this.position == 'replace') {
-      this.position = 'after';
-      this.hide_target = true;
-    }
-
-    if (this.position == 'after') {
-      new Insertion.After(this.target, content);
-      this.adapter = this.target.next();
-    }
-    else if (this.position == 'before') {
-      new Insertion.Before(this.target, content);
-      this.adapter = this.target.previous();
-    }
-    else {
-      return false;
-    }
-
-    this.adapter.down('a.inline-adapter-close').observe('click', this.close_handler.bind(this));
-    this.register_cancel_hooks();
-
-    new Effect.Highlight(this.adapter.down('td'));
-  },
-
-  close_handler: function(event) {
-    this.close_with_refresh();
-    if (event) Event.stop(event);
-  },
-
-  /* it might simplify things to just override the close function. then the Record and Table links could share more code ... wouldn't need custom close_handler functions, for instance */
-  close_with_refresh: function() {
-    new Ajax.Request(this.refresh_url, {
-      asynchronous: true,
-      evalScripts: true,
-      method: this.method,
-      onSuccess: function(request) {
-        Element.replace(this.target, request.responseText);
-        var new_target = $(this.target.id);
-        if (this.target.hasClassName('even-record')) new_target.addClassName('even-record');
-        this.target = new_target;
-        this.close();
-      }.bind(this),
-
-      onFailure: function(request) {
-        ActiveScaffold.report_500_response(this.scaffold_id());
-      }
-    });
-  },
-
-  enable: function() {
-    this.set.links.each(function(item) {
-      if (item.url != this.url) return;
-      item.tag.removeClassName('disabled');
-    }.bind(this));
-  },
-
-  disable: function() {
-    this.set.links.each(function(item) {
-      if (item.url != this.url) return;
-      item.tag.addClassName('disabled');
-    }.bind(this));
-  }
-});
-
-/**
- * Concrete classes for table actions
- */
-ActiveScaffold.Actions.Table = Class.create();
-ActiveScaffold.Actions.Table.prototype = Object.extend(new ActiveScaffold.Actions.Abstract(), {
-  instantiate_link: function(link) {
-    var l = new ActiveScaffold.ActionLink.Table(link, this.target, this.loading_indicator);
-    if (l.position) l.url = l.url.append_params({adapter: '_list_inline_adapter'});
-    return l;
-  }
-});
-
-ActiveScaffold.ActionLink.Table = Class.create();
-ActiveScaffold.ActionLink.Table.prototype = Object.extend(new ActiveScaffold.ActionLink.Abstract(), {
-  insert: function(content) {
-    if (this.position == 'top') {
-      new Insertion.Top(this.target, content);
-      this.adapter = this.target.immediateDescendants().first();
-    }
-    else {
-      throw 'Unknown position "' + this.position + '"'
-    }
-
-    this.adapter.down('a.inline-adapter-close').observe('click', this.close_handler.bind(this));
-    this.register_cancel_hooks();
-
-    new Effect.Highlight(this.adapter.down('td'));
-  },
-
-  close_handler: function(event) {
-    this.close();
-    if (event) Event.stop(event);
-  }
-});
+if (typeof Prototype == 'undefined')
+{
+  warning = "ActiveScaffold Error: Prototype could not be found. Please make sure that your application's layout includes prototype.js (e.g. <%= javascript_include_tag :defaults %>) *before* it includes active_scaffold.js (e.g. <%= active_scaffold_includes %>).";
+  alert(warning);
+}
+if (Prototype.Version.substring(0, 3) < '1.6')
+{
+  warning = "ActiveScaffold Error: Prototype version 1.6.x or higher is required. Please update prototype.js (rake rails:update:javascripts).";
+  alert(warning);
+}
+if (!Element.Methods.highlight) Element.addMethods({highlight: Prototype.emptyFunction});
+
+
+/*
+ * Simple utility methods
+ */
+
+var ActiveScaffold = {
+  records_for: function(tbody_id) {
+    var rows = [];
+    var child = $(tbody_id).down('.record');
+    while (child) {
+      rows.push(child);
+      child = child.next('.record');
+    }
+    return rows;
+  },
+  stripe: function(tbody_id) {
+    var even = false;
+    var rows = this.records_for(tbody_id);
+    for (var i = 0; i < rows.length; i++) {
+      var child = rows[i];
+      //Make sure to skip rows that are create or edit rows or messages
+      if (child.tagName != 'SCRIPT'
+        && !child.hasClassName("create")
+        && !child.hasClassName("update")
+        && !child.hasClassName("inline-adapter")
+        && !child.hasClassName("active-scaffold-calculations")) {
+
+        if (even) child.addClassName("even-record");
+        else child.removeClassName("even-record");
+
+        even = !even;
+      }
+    }
+  },
+  hide_empty_message: function(tbody, empty_message_id) {
+    if (this.records_for(tbody).length != 0) {
+      $(empty_message_id).hide();
+    }
+  },
+  reload_if_empty: function(tbody, url) {
+    if (this.records_for(tbody).length == 0) {
+      new Ajax.Request(url, {
+        method: 'get',
+        asynchronous: true,
+        evalScripts: true
+      });
+    }
+  },
+  removeSortClasses: function(scaffold_id) {
+    $$('#' + scaffold_id + ' td.sorted').each(function(element) {
+      element.removeClassName("sorted");
+    });
+    $$('#' + scaffold_id + ' th.sorted').each(function(element) {
+      element.removeClassName("sorted");
+      element.removeClassName("asc");
+      element.removeClassName("desc");
+    });
+  },
+  decrement_record_count: function(scaffold_id) {
+    // decrement the last record count, firsts record count are in nested lists
+    count = $$('#' + scaffold_id + ' span.active-scaffold-records').last();
+    if (count) count.update(parseInt(count.innerHTML, 10) - 1);
+  },
+  increment_record_count: function(scaffold_id) {
+    // increment the last record count, firsts record count are in nested lists
+    count = $$('#' + scaffold_id + ' span.active-scaffold-records').last();
+    if (count) count.update(parseInt(count.innerHTML, 10) + 1);
+  },
+  update_row: function(row, html) {
+    row = $(row);
+    Element.replace(row, html);
+    var new_row = $(row.id);
+    if (row.hasClassName('even-record')) new_row.addClassName('even-record');
+    new_row.highlight();
+  },
+
+  server_error_response: '',
+  report_500_response: function(active_scaffold_id) {
+    messages_container = $(active_scaffold_id).down('td.messages-container');
+    new Insertion.Top(messages_container, this.server_error_response);
+  }
+}
+
+/*
+ * DHTML history tie-in
+ */
+function addActiveScaffoldPageToHistory(url, active_scaffold_id) {
+  if (typeof dhtmlHistory == 'undefined') return; // it may not be loaded
+
+  var array = url.split('?');
+  var qs = new Querystring(array[1]);
+  var sort = qs.get('sort')
+  var dir = qs.get('sort_direction')
+  var page = qs.get('page')
+  if (sort || dir || page) dhtmlHistory.add(active_scaffold_id+":"+page+":"+sort+":"+dir, url);
+}
+
+/*
+ * Add-ons/Patches to Prototype
+ */
+
+/* patch to support replacing TR/TD/TBODY in Internet Explorer, courtesy of http://dev.rubyonrails.org/ticket/4273 */
+Element.replace = function(element, html) {
+  element = $(element);
+  if (element.outerHTML) {
+    try {
+    element.outerHTML = html.stripScripts();
+    } catch (e) {
+      var tn = element.tagName;
+      if(tn=='TBODY' || tn=='TR' || tn=='TD')
+      {
+              var tempDiv = document.createElement("div");
+              tempDiv.innerHTML = '<table id="tempTable" style="display: none">' + html.stripScripts() + '</table>';
+              element.parentNode.replaceChild(tempDiv.getElementsByTagName(tn).item(0), element);
+      }
+      else throw e;
+    }
+  } else {
+    var range = element.ownerDocument.createRange();
+    /* patch to fix <form> replaces in Firefox. see http://dev.rubyonrails.org/ticket/8010 */
+    range.selectNodeContents(element.parentNode);
+    element.parentNode.replaceChild(range.createContextualFragment(html.stripScripts()), element);
+  }
+  setTimeout(function() {html.evalScripts()}, 10);
+  return element;
+};
+
+/*
+ * URL modification support. Incomplete functionality.
+ */
+Object.extend(String.prototype, {
+  append_params: function(params) {
+    url = this;
+    if (url.indexOf('?') == -1) url += '?';
+    else if (url.lastIndexOf('&') != url.length) url += '&';
+
+    url += $H(params).collect(function(item) {
+      return item.key + '=' + item.value;
+    }).join('&');
+
+    return url;
+  }
+});
+
+/*
+ * Prototype's implementation was throwing an error instead of false
+ */
+Element.Methods.Simulated = {
+  hasAttribute: function(element, attribute) {
+    var t = Element._attributeTranslations;
+    attribute = (t.names && t.names[attribute]) || attribute;
+    // Return false if we get an error here
+    try {
+      return $(element).getAttributeNode(attribute).specified;
+    } catch (e) {
+      return false;
+    }
+  }
+};
+
+/**
+ * A set of links. As a set, they can be controlled such that only one is "open" at a time, etc.
+ */
+ActiveScaffold.Actions = new Object();
+ActiveScaffold.Actions.Abstract = Class.create({
+  initialize: function(links, target, loading_indicator, options) {
+    this.target = $(target);
+    this.loading_indicator = $(loading_indicator);
+    this.options = options;
+    this.links = links.collect(function(link) {
+      return this.instantiate_link(link);
+    }.bind(this));
+  },
+
+  instantiate_link: function(link) {
+    throw 'unimplemented'
+  }
+});
+
+/**
+ * A DataStructures::ActionLink, represented in JavaScript.
+ * Concerned with AJAX-enabling a link and adapting the result for insertion into the table.
+ */
+ActiveScaffold.ActionLink = new Object();
+ActiveScaffold.ActionLink.Abstract = Class.create({
+  initialize: function(a, target, loading_indicator) {
+    this.tag = $(a);
+    this.url = this.tag.href;
+    this.method = 'get';
+    if(this.url.match('_method=delete')){
+      this.method = 'delete';
+    } else if(this.url.match('_method=post')){
+      this.method = 'post';
+    } else if(this.url.match('_method=put')){
+      this.method = 'put';
+    }
+    this.target = target;
+    this.loading_indicator = loading_indicator;
+    this.hide_target = false;
+    this.position = this.tag.getAttribute('position');
+		this.page_link = this.tag.getAttribute('page_link');
+
+    this.onclick = this.tag.onclick;
+    this.tag.onclick = null;
+    this.tag.observe('click', function(event) {
+      this.open();
+      Event.stop(event);
+    }.bind(this));
+
+    this.tag.action_link = this;
+  },
+
+  open: function() {
+    if (this.is_disabled()) return;
+
+    if (this.tag.hasAttribute( "dhtml_confirm")) {
+      if (this.onclick) this.onclick();
+      return;
+    } else {
+      if (this.onclick && !this.onclick()) return;//e.g. confirmation messages
+      this.open_action();
+    }
+  },
+
+  open_action: function() {
+    if (this.position) this.disable();
+
+    if (this.page_link) {
+      window.location = this.url;
+    } else {
+      if (this.loading_indicator) this.loading_indicator.style.visibility = 'visible';
+      new Ajax.Request(this.url, {
+        asynchronous: true,
+        evalScripts: true,
+        method: this.method,
+        onSuccess: function(request) {
+          if (this.position) {
+            this.insert(request.responseText);
+            if (this.hide_target) this.target.hide();
+          } else {
+            request.evalResponse();
+          }
+        }.bind(this),
+
+        onFailure: function(request) {
+          ActiveScaffold.report_500_response(this.scaffold_id());
+          if (this.position) this.enable()
+        }.bind(this),
+
+        onComplete: function(request) {
+          if (this.loading_indicator) this.loading_indicator.style.visibility = 'hidden';
+        }.bind(this)
+      });
+    }
+  },
+
+  insert: function(content) {
+    throw 'unimplemented'
+  },
+
+  close: function() {
+    this.enable();
+    this.adapter.remove();
+    if (this.hide_target) this.target.show();
+  },
+
+  close_handler: function(event) {
+    this.close();
+    if (event) Event.stop(event);
+  },
+
+  register_cancel_hooks: function() {
+    // anything in the insert with a class of cancel gets the closer method, and a reference to this object for good measure
+    var self = this;
+    this.adapter.select('.cancel').each(function(elem) {
+      elem.observe('click', this.close_handler.bind(this));
+      elem.link = self;
+    }.bind(this))
+  },
+
+  reload: function() {
+    this.close();
+    this.open();
+  },
+
+  get_new_adapter_id: function() {
+    var id = 'adapter_';
+    var i = 0;
+    while ($(id + i)) i++;
+    return id + i;
+  },
+
+  enable: function() {
+    return this.tag.removeClassName('disabled');
+  },
+
+  disable: function() {
+    return this.tag.addClassName('disabled');
+  },
+
+  is_disabled: function() {
+    return this.tag.hasClassName('disabled');
+  },
+
+  scaffold_id: function() {
+    return this.tag.up('div.active-scaffold').id;
+  }
+});
+
+/**
+ * Concrete classes for record actions
+ */
+ActiveScaffold.Actions.Record = Class.create(ActiveScaffold.Actions.Abstract, {
+  instantiate_link: function(link) {
+    var l = new ActiveScaffold.ActionLink.Record(link, this.target, this.loading_indicator);
+    l.refresh_url = this.options.refresh_url;
+    if (link.hasClassName('delete')) {
+      l.url = l.url.replace(/\/delete(\?.*)?$/, '$1');
+      l.url = l.url.replace(/\/delete\/(.*)/, '/destroy/$1');
+    }
+    if (l.position) l.url = l.url.append_params({adapter: '_list_inline_adapter'});
+    l.set = this;
+    return l;
+  }
+});
+
+ActiveScaffold.ActionLink.Record = Class.create(ActiveScaffold.ActionLink.Abstract, {
+  close_previous_adapter: function() {
+    this.set.links.each(function(item) {
+      if (item.url != this.url && item.is_disabled() && item.adapter) {
+        item.enable();
+        item.adapter.remove();
+      }
+    }.bind(this));
+  },
+
+  insert: function(content) {
+    this.close_previous_adapter();
+
+    if (this.position == 'replace') {
+      this.position = 'after';
+      this.hide_target = true;
+    }
+
+    if (this.position == 'after') {
+      new Insertion.After(this.target, content);
+      this.adapter = this.target.next();
+    }
+    else if (this.position == 'before') {
+      new Insertion.Before(this.target, content);
+      this.adapter = this.target.previous();
+    }
+    else {
+      return false;
+    }
+
+    this.adapter.down('a.inline-adapter-close').observe('click', this.close_handler.bind(this));
+    this.register_cancel_hooks();
+
+    this.adapter.down('td').down().highlight();
+  },
+
+  close: function($super, updatedRow) {
+    if (updatedRow) {
+      ActiveScaffold.update_row(this.target, updatedRow);
+      $super();
+    } else {
+      new Ajax.Request(this.refresh_url, {
+        asynchronous: true,
+        evalScripts: true,
+        method: this.method,
+        onSuccess: function(request) {
+          ActiveScaffold.update_row(this.target, request.responseText);
+          $super();
+        }.bind(this),
+  
+        onFailure: function(request) {
+          ActiveScaffold.report_500_response(this.scaffold_id());
+        }
+      });
+    }
+  },
+
+  enable: function() {
+    this.set.links.each(function(item) {
+      if (item.url != this.url) return;
+      item.tag.removeClassName('disabled');
+    }.bind(this));
+  },
+
+  disable: function() {
+    this.set.links.each(function(item) {
+      if (item.url != this.url) return;
+      item.tag.addClassName('disabled');
+    }.bind(this));
+  }
+});
+
+/**
+ * Concrete classes for table actions
+ */
+ActiveScaffold.Actions.Table = Class.create(ActiveScaffold.Actions.Abstract, {
+  instantiate_link: function(link) {
+    var l = new ActiveScaffold.ActionLink.Table(link, this.target, this.loading_indicator);
+    if (l.position) l.url = l.url.append_params({adapter: '_list_inline_adapter'});
+    return l;
+  }
+});
+
+ActiveScaffold.ActionLink.Table = Class.create(ActiveScaffold.ActionLink.Abstract, {
+  insert: function(content) {
+    if (this.position == 'top') {
+      new Insertion.Top(this.target, content);
+      this.adapter = this.target.immediateDescendants().first();
+    }
+    else {
+      throw 'Unknown position "' + this.position + '"'
+    }
+
+    this.adapter.down('a.inline-adapter-close').observe('click', this.close_handler.bind(this));
+    this.register_cancel_hooks();
+
+    this.adapter.down('td').down().highlight();
+  }
+});
+
+if (Ajax.InPlaceEditor) {
+ActiveScaffold.InPlaceEditor = Class.create(Ajax.InPlaceEditor, {
+  setFieldFromAjax: function(url, options) {
+    var ipe = this;
+    $(ipe._controls.editor).remove();
+    new Ajax.Request(url, {
+      method: 'get',
+      onComplete: function(response) {
+        ipe._form.insert({top: response.responseText});
+        if (options.plural) {
+          ipe._form.getElements().each(function(el) {
+            if (el.type != "submit" && el.type != "image") {
+              el.name = ipe.options.paramName + '[]';
+              el.className = 'editor_field';
+            }
+          });
+        } else {
+          var fld = ipe._form.findFirstElement();
+          fld.name = ipe.options.paramName;
+          fld.className = 'editor_field';
+          if (ipe.options.submitOnBlur)
+            fld.onblur = ipe._boundSubmitHandler;
+          ipe._controls.editor = fld;
+        }
+      }
+    });
+  },
+
+  clonePatternField: function() {
+    var patternNodes = this.getPatternNodes(this.options.inplacePatternSelector);
+    if (patternNodes.editNode == null) {
+      alert('did not find any matching node for ' + this.options.editFieldSelector);
+      return;
+    }
+
+    var fld = patternNodes.editNode.cloneNode(true);
+    if (fld.id.length > 0) fld.id += this.options.nodeIdSuffix;
+    fld.name = this.options.paramName;
+    fld.className = 'editor_field';
+    this.setValue(fld, this._controls.editor.value);
+    if (this.options.submitOnBlur)
+      fld.onblur = this._boundSubmitHandler;
+    $(this._controls.editor).remove();
+    this._controls.editor = fld;
+    this._form.appendChild(this._controls.editor);
+
+    $A(patternNodes.additionalNodes).each(function(node) {
+      var patternNode = node.cloneNode(true);
+      if (patternNode.id.length > 0) {
+        patternNode.id = patternNode.id + this.options.nodeIdSuffix;
+      }
+      this._form.appendChild(patternNode);
+    }.bind(this));
+  },
+  
+  getPatternNodes: function(inplacePatternSelector) {
+    var nodes = {editNode: null, additionalNodes: []};
+    var selectedNodes = $$(inplacePatternSelector);
+    var firstNode = selectedNodes.first();
+    
+    if (typeof(firstNode) !== 'undefined') {
+      // AS inplace_edit_control_container -> we have to select all child nodes
+      // Workaround for ie which does not support css > selector
+      if (firstNode.className.indexOf('as_inplace_pattern') !== -1) {
+        selectedNodes = firstNode.childElements();
+      }
+      nodes.editNode = selectedNodes.first();
+      selectedNodes.shift();
+      nodes.additionalNodes = selectedNodes;
+    }
+    return nodes;
+  },
+  
+  setValue: function(editField, textValue) {
+    var function_name = 'setValueFor' + editField.nodeName.toLowerCase();
+    if (typeof(this[function_name]) == 'function') {
+      this[function_name](editField, textValue);
+    } else {
+      editField.value = textValue;
+    }
+  },
+  
+  setValueForselect: function(editField, textValue) {
+    var len = editField.options.length;
+    var i = 0;
+    while (i < len && editField.options[i].text != textValue) {
+      i++;
+    }
+    if (i < len) {
+      editField.value = editField.options[i].value
+    }
+  }
+});
+}