Adapt to grouped WP attributes

app/assets/javascripts/angular/openproject-costs-app.js

@@ -29,9 +29,12 @@
 // main app
 var openprojectCostsApp = angular.module('openproject');
 
-openprojectCostsApp.run(['HookService', function(HookService) {
+openprojectCostsApp.run(['HookService',
-  var setupCostsAttributes = function(attributes) {
+                         'ConfigurationService',
-    var spentTimeIndex = attributes.indexOf('spentTime');
+                         'WorkPackagesOverviewService',
+                         function(HookService, ConfigurationService, WorkPackagesOverviewService) {
+  var setupCostsAttributes = function() {
+    var position = WorkPackagesOverviewService.getGroupedWorkPackageOverviewAttributes().length - 1;
     var costsAttributes = {
       overallCosts: null,
       spentHours: 'spentHoursLinked',
@@ -39,22 +42,17 @@ openprojectCostsApp.run(['HookService', function(HookService) {
       summarizedCostEntries: 'spentUnits'
     };
 
+    WorkPackagesOverviewService.removeAttribute('spentTime');
+    WorkPackagesOverviewService.addGroup('costs', position);
+
     angular.forEach(costsAttributes, function(id, costAttribute) {
-      attributes.push(id || costAttribute);
+      WorkPackagesOverviewService.addAttributeToGroup('costs', id || costAttribute);
     });
-
-    if (spentTimeIndex >= 0) {
-      attributes.splice(spentTimeIndex, 1);
-    }
   }
 
-  HookService.register('workPackagePluginAttributes', function(params) {
+  if (ConfigurationService.isModuleEnabled('costs_module')) {
-    var costsActivted = params.enabledModules.indexOf('costs_module') >= 0;
+    setupCostsAttributes();
-
-    if (costsActivted) {
-      setupCostsAttributes(params.attributes)
   }
-  });
 
   HookService.register('workPackageOverviewAttributes', function(params) {
     var directive;

config/locales/js-de.yml

@@ -36,6 +36,8 @@
 de:
   js:
     work_packages:
+      property_groups:
+        costs: "Kosten"
       properties:
         costObject: "Budget"
         spentHoursLinked: "Aufgewendete Zeit"

config/locales/js-en.yml

@@ -36,6 +36,8 @@
 en:
   js:
     work_packages:
+      property_groups:
+        costs: "Costs"
       properties:
         costObject: "Budget"
         spentHoursLinked: "Spent time"