Fixed #2398 - Missing custom object in database causes error on expor…

…t in time_accounting

(cherry picked from commit 3ca697d)

app/controllers/reports_controller.rb

@@ -219,6 +219,7 @@ def sheet(profile, title, result)
     # needs to be skipped
     objects = ObjectManager::Attribute.where(editable: true,
                                              active: true,
+                                             to_create: false,
                                              object_lookup_id: ObjectLookup.lookup(name: 'Ticket').id)
                                       .pluck(:name, :display, :data_type, :data_option)
                                       .map { |name, display, data_type, data_option| { name: name, display: display, data_type: data_type, data_option: data_option } }

app/controllers/time_accountings_controller.rb

@@ -165,6 +165,7 @@ def by_ticket
       ]
       objects = ObjectManager::Attribute.where(editable: true,
                                                active: true,
+                                               to_create: false,
                                                object_lookup_id: ObjectLookup.lookup(name: 'Ticket').id)
                                         .pluck(:name, :display, :data_type, :data_option)
                                         .map { |name, display, data_type, data_option| { name: name, display: display, data_type: data_type, data_option: data_option } }

spec/requests/time_accounting_spec.rb

@@ -39,5 +39,27 @@
         expect(response['Content-Type']).to eq('application/vnd.ms-excel')
       end
     end
+
+    # Regression test for issue #2398 - Missing custom object in database causes error on export in time_accounting
+    # This test is identical to the above one, except with the added step of a pending migration in the beginning
+    context 'with pending attribute migrations, requesting a log report download' do
+      it 'responds with an Excel spreadsheet' do
+        ObjectManager::Attribute.add attributes_for :object_manager_attribute_select
+
+        group   = create(:group)
+        ticket  = create(:ticket, state: Ticket::State.lookup(name: 'open'), customer: customer )
+        article = create(:ticket_article, ticket: ticket, type: Ticket::Article::Type.lookup(name: 'note') )
+
+        create(:ticket_time_accounting, ticket_id: ticket.id, ticket_article_id: article.id)
+
+        authenticated_as(admin)
+        get "/api/v1/time_accounting/log/by_ticket/#{year}/#{month}?download=true", params: {}
+
+        expect(response).to have_http_status(200)
+        expect(response['Content-Disposition']).to be_truthy
+        expect(response['Content-Disposition']).to eq("attachment; filename=\"by_ticket-#{year}-#{month}.xls\"")
+        expect(response['Content-Type']).to eq('application/vnd.ms-excel')
+      end
+    end
   end
 end