2026.3.0

app/controllers/vehicles_controller.rb

@@ -60,10 +60,15 @@ class VehiclesController < ApplicationController
   # display a specific vehicle
   def show
     begin
-      @vehicle = Vehicle.find_by_id(params[:id])
+      @vehicle = Vehicle.includes(issues: [:estimate, :invoices]).find(params[:id])
       @vin = @vehicle.vin.scan(/.{1,9}/) if @vehicle.vin
-      @issues = @vehicle.issues.order(id: :desc)
+      @issues = @vehicle.issues
-      @closed_issues = (@issues - @issues.open)
+        .joins(:status)
+        .includes(:estimate, :invoices, :status, :project, :tracker, :priority)
+        .order(id: :desc)
+      @open_issues = @issues.select { |i| !i.status.is_closed }
+      @closed_issues = @issues.select { |i| i.status.is_closed }
+
       flash[:error] = t :alert_no_customer if @vehicle.customer.nil?
     rescue 
       flash[:error] = t :alert_vehicle_not_found

app/models/vehicle.rb

@@ -92,14 +92,11 @@ class Vehicle < ActiveRecord::Base
     scope = self.all
 
     tokens.each do |token|
-      q = "%#{sanitize_sql_like(token)}%"
+      scope = scope.search(token)
-      scope = where("vin LIKE ? OR make LIKE ? OR model LIKE ? OR year LIKE ?", "%#{q}%", "%#{q}%", "%#{q}%", "%#{q}%") 
     end
 
     ids = scope.distinct.limit(options[:limit] || 100).pluck(:id)
-
+    ids.index_with { |id| id }
-    # rank by id
-    ids.each_with_object({}) { |id, h| h[id] = id }
   end
 
   # returns a human readable string

app/views/vehicles/show.html.erb

@@ -14,9 +14,9 @@
     </div>
 </div>
 
-<h3><%=@issues.open.count%> <%=t(:label_open_issues)%></h3>
+<h3><%=@open_issues.count%> <%=t(:label_open_issues)%></h3>
 
-<%= render partial: 'issues/list_simple', locals: {issues: @issues.open} %>
+<%= render partial: 'issues/list_simple', locals: {issues: @open_issues} %>
 
 <h3><%=@closed_issues.count%> <%=t(:label_closed_issues)%></h3>
 

assets/javascripts/copy.js

@@ -12,6 +12,11 @@ async function handleCopy(event) {
     link = event.target;
   }
 
+  // If the text is already "Copied!", don't do anything
+  if (text == "Copied!") {
+    return;
+  }
+
   try {
     // Write to clipboard
     await navigator.clipboard.writeText(text);

init.rb

@@ -14,7 +14,7 @@ Redmine::Plugin.register :redmine_qbo_vehicles do
   name 'Redmine QBO Vehicles plugin'
   author 'Rick Barrette'
   description 'This is a plugin for Redmine to intergrate with the redmine_qbo plugin to provide vehicle data tracking'
-  version '2026.2.6'
+  version '2026.3.0'
   url 'https://github.com/rickbarrette/redmine_qbo_vehicles'
   author_url 'https://barrettefabrication.com'
   requires_redmine version_or_higher: '6.1.0'

lib/vehicles/hooks/invoice_hook_listener.rb

@@ -17,13 +17,13 @@ module Vehicles
 
       # Called by Redmine QBO Invoice
       def process_invoice_custom_fields(context={})
-        Rails.logger.info "redmine_qbo_vehicles.process_invoice_custom_fields"
+        log "Processing invoice custom fields for invoice ##{context[:invoice].id}"
         issue = context[:issue]
 
         # update the invoive custom fields with infomation from the issue if available
         context[:invoice].custom_fields.each do |cf|
 
-          Rails.logger.info "Checking invoice.custom field: #{cf.name}"
+          log "Checking invoice custom field: #{cf.name}"
           
           # VIN from the attached vehicle 
           begin
@@ -32,13 +32,13 @@ module Vehicles
               # TODO check cf_sync_confict flag once implemented
               if cf.string_value.to_s.blank?
 
-                Rails.logger.info "VIN was blank, updating the invoice vin in quickbooks"
+                log "VIN was blank, updating the invoice vin in quickbooks"
                 vin = context[:issue].vehicle.vin
                 break if vin.nil?
                 
                 if not cf.string_value.to_s.eql? vin
                   cf.string_value = vin.to_s
-                  Rails.logger.info "VIN has changed"
+                  log "VIN has changed"
                   context[:is_changed] = true
                 end
 
@@ -47,7 +47,7 @@ module Vehicles
             end
           rescue
             #do nothing
-            Rails.logger.info "redmine_qbo_vehicles.process_invoice_custom_fields failed, skipping"
+            log "redmine_qbo_vehicles.process_invoice_custom_fields failed, skipping"
             return nil
           end
         end
@@ -56,6 +56,12 @@ module Vehicles
         
         return nil
       end
+
+      private
+
+      def log(msg)
+        Rails.logger.info "[InvoiceHookListener] #{msg}"
+      end
     end
 
   end