Fixed partial billing and added flash messages

app/controllers/qbo_controller.rb

@@ -64,11 +64,16 @@ class QboController < ApplicationController
   def bill
     i = Issue.find_by_id params[:id]
     if i.customer
-      i.bill_time
+      billed = i.bill_time
-      redirect_to i, flash: { notice: I18n.t(:label_billed_success) + i.customer.name }
+
+      if i.bill_time
+        redirect_to i, flash: { notice: I18n.t( :label_billed_success ) + i.customer.name }
       else
         redirect_to i, flash: { error: I18n.t(:label_billing_error) }
       end
+    else
+      redirect_to i, flash: { error: I18n.t(:label_billing_error_no_customer) }
+    end
   end
   
   # Quickbooks Webhook Callback

config/locales/en.yml

@@ -74,7 +74,8 @@ en:
   label_connected: "Successfully connected to Quickbooks"
   label_error: "Error"
   label_billed_success: "Successfully Billed "
-  label_billing_error: "Cannot bill without a customer assigned"
+  label_billing_error: "Customer could not be billed, check for Customer or Assignee and try again"
+  label_billing_error_no_customer: "Cannot bill without a customer assigned"
   label_qbo_sync_success: "Successfully synced to Quickbooks"
   label_hours: "Hours"
   label_oauth2_refresh_token_expires_at: "Refresh Token Expires At"

lib/redmine_qbo/patches/issue_patch.rb

@@ -44,10 +44,9 @@ module RedmineQbo
         # Create billable time entries
         def bill_time
           logger.debug "QBO: Billing time for issue ##{id}"
-          return unless  status.is_closed?
+          return false if assigned_to.nil?
-          return if assigned_to.nil?
+          return false unless Qbo.first 
-          return unless Qbo.first 
+          return false unless customer 
-          return unless customer 
 
           Thread.new do
             spent_time = time_entries.where(billed: [false, nil])
@@ -102,6 +101,7 @@ module RedmineQbo
               end
             end
           end
+          return true
         end
       end