update transactions

app/controllers/api_controller.rb

@@ -1,7 +1,7 @@
 class ApiController < ::ApplicationController
   protect_from_forgery
 
-  before_filter :check_rights, :only => [:create, :update, :new, :delete]
+  # before_filter :check_rights, :only => [:create, :update, :new, :delete]
 
   private
 

db/migrate/20120508192547_create_transactions_transactions.refinery_transactions.rb

@@ -1,4 +1,3 @@
-# This migration comes from refinery_transactions (originally 1)
 class CreateTransactionsTransactions < ActiveRecord::Migration
 
   def up
@@ -15,11 +14,13 @@ def up
       t.text :message
       t.text :raw
 
-      t.string :stamp
+      t.string :stamp, :null => false
 
       t.timestamps
     end
 
+    add_index :refinery_transactions, :stamp, :unique => true
+
   end
 
   def down

vendor/extensions/transactions/app/controllers/refinery/transactions/transactions_controller.rb

@@ -16,18 +16,23 @@ def create
         response = {'status' => false}
         ptrans = params[:transaction]
 
-        trans = Transaction.new(ptrans)
-        if params[:help].present?
-          response['accessible_attributes'] = trans._accessible_attributes
-        else
-          if trans.valid?
-            response['status'] = trans.save!
+        begin
+          trans = Transaction.new(ptrans)
+          if params[:help].present?
+            response['accessible_attributes'] = trans._accessible_attributes
           else
-            response['errors'] = trans.errors
+            if trans.valid?
+              response['status'] = trans.save!
+            else
+              response['errors'] = trans.errors
+            end
           end
+        rescue
+          response = $!
         end
 
         render :text => response.to_json
+        # render :text => ptrans
       end
 
     end

vendor/extensions/transactions/app/models/refinery/transactions/transaction.rb

@@ -38,9 +38,9 @@ def custom_types
       def title
         case self.primary_type
         when INCOME_TRANSACTION_TYPE
-          "Income from #{self.from_account.truncate(10)}"
+          "Income from #{self.from_account.truncate(10) if self.from_account.present?}"
         when OUTCOME_TRANSACTION_TYPE
-          "Payment to #{self.to_account}"
+          "Payment to #{self.to_account if self.to_account.present?}"
         end
       end
 

vendor/extensions/transactions/app/views/refinery/transactions/admin/transactions/_transaction.html.erb

@@ -1,6 +1,7 @@
 <tr class=" record <%= cycle('on', 'on-hover') %> <%= "transaction-#{transaction.primary_type}" %>" id="<%= dom_id(transaction) -%>">
   <td class="title">
     <span>
+      <%= transaction.stamp %>-t-
       <%= transaction.title %>
       <%= ", #{transaction.message.truncate(32)} " unless transaction.message.nil? %>
     </span>
@@ -15,8 +16,7 @@
   <% if current_refinery_user.has_role?(:superuser)  %>
     <td style="text-align: center">
       <%#= link_to refinery_icon_tag("application_go.png"), refinery.transactions_transaction_path(transaction),
-        :title => t('.view_live_html'),
-        :target => "_blank" %>
+        :title => t('.view_live_html') %>
 
       <%= link_to refinery_icon_tag("application_edit.png"), refinery.edit_transactions_admin_transaction_path(transaction),
         :title => t('.edit') %>

vendor/extensions/transactions/config/routes.rb

@@ -3,7 +3,6 @@
   # Frontend routes
   get '/api/transaction/new' => 'transactions::transactions#create'
 
-
   # Admin routes
   namespace :transactions, :path => '' do
     namespace :admin, :path => 'refinery' do

vendor/extensions/transactions/db/migrate/1_create_transactions_transactions.rb

@@ -14,11 +14,13 @@ def up
       t.text :message
       t.text :raw
 
-      t.string :stamp
+      t.string :stamp, :null => false
 
       t.timestamps
     end
 
+    add_index :refinery_transactions, :stamp, :unique => true
+
   end
 
   def down