chore(angularjs): Remove all 'ngInject'; in favor of explicit DI anno…

…tation
spinnaker · Feb 21, 2019 · cc52bee · cc52bee
1 parent b6bab1e
commit cc52bee
Show file tree

Hide file tree

Showing 184 changed files with 84 additions and 384 deletions.
diff --git a/app/scripts/modules/amazon/src/instance/amazon.instance.write.service.ts b/app/scripts/modules/amazon/src/instance/amazon.instance.write.service.ts
@@ -25,7 +25,6 @@ export interface IAmazonMultiInstanceJob extends IMultiInstanceJob {
 export class AmazonInstanceWriter extends InstanceWriter {
   public static $inject = ['providerServiceDelegate'];
   public constructor(protected providerServiceDelegate: ProviderServiceDelegate) {
-    'ngInject';
     super(providerServiceDelegate);
   }
 

diff --git a/app/scripts/modules/amazon/src/loadBalancer/details/loadBalancerDetails.controller.ts b/app/scripts/modules/amazon/src/loadBalancer/details/loadBalancerDetails.controller.ts
@@ -70,7 +70,6 @@ export class AwsLoadBalancerDetailsController implements IController {
     private securityGroupReader: SecurityGroupReader,
     private loadBalancerReader: LoadBalancerReader,
   ) {
-    'ngInject';
     this.application = app;
     this.loadBalancerFromParams = loadBalancer;
 

diff --git a/app/scripts/modules/amazon/src/loadBalancer/details/targetGroupDetails.controller.ts b/app/scripts/modules/amazon/src/loadBalancer/details/targetGroupDetails.controller.ts
@@ -29,7 +29,6 @@ export class AwsTargetGroupDetailsController implements IController {
     targetGroup: ITargetGroupFromStateParams,
     private app: Application,
   ) {
-    'ngInject';
     this.application = app;
     this.targetGroupFromParams = targetGroup;
 

diff --git a/app/scripts/modules/amazon/src/serverGroup/configure/serverGroupConfiguration.service.ts b/app/scripts/modules/amazon/src/serverGroup/configure/serverGroupConfiguration.service.ts
@@ -135,9 +135,7 @@ export class AwsServerGroupConfigurationService {
     private cacheInitializer: CacheInitializerService,
     private loadBalancerReader: LoadBalancerReader,
     private serverGroupCommandRegistry: ServerGroupCommandRegistry,
-  ) {
-    'ngInject';
-  }
+  ) {}
 
   public configureUpdateCommand(command: IAmazonServerGroupCommand): void {
     command.backingData = {

diff --git a/...n/src/serverGroup/details/scalingPolicy/targetTracking/targetTrackingSummary.component.ts b/...n/src/serverGroup/details/scalingPolicy/targetTracking/targetTrackingSummary.component.ts
@@ -15,9 +15,7 @@ class TargetTrackingSummaryController implements IComponentController {
   public popoverTemplate = require('./targetTrackingPopover.html');
 
   public static $inject = ['$uibModal', 'confirmationModalService'];
-  constructor(private $uibModal: IModalService, private confirmationModalService: ConfirmationModalService) {
-    'ngInject';
-  }
+  constructor(private $uibModal: IModalService, private confirmationModalService: ConfirmationModalService) {}
 
   public $onInit() {
     this.config = this.policy.targetTrackingConfiguration;

diff --git a/...n/src/serverGroup/details/scalingPolicy/targetTracking/upsertTargetTracking.controller.ts b/...n/src/serverGroup/details/scalingPolicy/targetTracking/upsertTargetTracking.controller.ts
@@ -37,9 +37,7 @@ export class UpsertTargetTrackingController implements IComponentController {
     public policy: ITargetTrackingPolicy,
     public serverGroup: IServerGroup,
     public application: Application,
-  ) {
-    'ngInject';
-  }
+  ) {}
 
   public $onInit() {
     const metricType = this.policy.targetTrackingConfiguration.customizedMetricSpecification ? 'custom' : 'predefined';

diff --git a/app/scripts/modules/appengine/src/cache/cacheConfigurer.service.ts b/app/scripts/modules/appengine/src/cache/cacheConfigurer.service.ts
@@ -16,9 +16,7 @@ class AppengineCacheConfigurer {
   };
 
   public static $inject = ['loadBalancerReader'];
-  constructor(private loadBalancerReader: any) {
-    'ngInject';
-  }
+  constructor(private loadBalancerReader: any) {}
 }
 
 export const APPENGINE_CACHE_CONFIGURER = 'spinnaker.appengine.cacheConfigurer.service';

diff --git a/app/scripts/modules/appengine/src/common/conditionalDescriptionListItem.component.ts b/app/scripts/modules/appengine/src/common/conditionalDescriptionListItem.component.ts
@@ -6,9 +6,7 @@ class AppengineConditionalDescriptionListItemCtrl implements IController {
   public component: any;
 
   public static $inject = ['$filter'];
-  constructor(private $filter: IFilterService) {
-    'ngInject';
-  }
+  constructor(private $filter: IFilterService) {}
 
   public $onInit(): void {
     if (!this.label) {

diff --git a/app/scripts/modules/appengine/src/instance/details/details.controller.ts b/app/scripts/modules/appengine/src/instance/details/details.controller.ts
@@ -41,8 +41,6 @@ class AppengineInstanceDetailsController implements IController {
     private confirmationModalService: ConfirmationModalService,
     instance: InstanceFromStateParams,
   ) {
-    'ngInject';
-
     this.app
       .ready()
       .then(() => this.retrieveInstance(instance))

diff --git a/app/scripts/modules/appengine/src/loadBalancer/configure/wizard/wizard.controller.ts b/app/scripts/modules/appengine/src/loadBalancer/configure/wizard/wizard.controller.ts
@@ -42,7 +42,6 @@ class AppengineLoadBalancerWizardController implements IController {
     private appengineLoadBalancerTransformer: AppengineLoadBalancerTransformer,
     private wizardSubFormValidation: any,
   ) {
-    'ngInject';
     this.submitButtonLabel = this.forPipelineConfig ? 'Done' : 'Update';
 
     if (this.isNew) {

diff --git a/app/scripts/modules/appengine/src/loadBalancer/details/details.controller.ts b/app/scripts/modules/appengine/src/loadBalancer/details/details.controller.ts
@@ -35,7 +35,6 @@ class AppengineLoadBalancerDetailsController implements IController {
     private app: Application,
     private confirmationModalService: ConfirmationModalService,
   ) {
-    'ngInject';
     this.loadBalancerFromParams = loadBalancer;
     this.app
       .getDataSource('loadBalancers')

diff --git a/app/scripts/modules/appengine/src/loadBalancer/transformer.ts b/app/scripts/modules/appengine/src/loadBalancer/transformer.ts
@@ -78,9 +78,7 @@ export class AppengineLoadBalancerUpsertDescription implements ILoadBalancerUpse
 
 export class AppengineLoadBalancerTransformer {
   public static $inject = ['$q'];
-  constructor(private $q: ng.IQService) {
-    'ngInject';
-  }
+  constructor(private $q: ng.IQService) {}
 
   public normalizeLoadBalancer(loadBalancer: ILoadBalancer): ng.IPromise<ILoadBalancer> {
     loadBalancer.provider = loadBalancer.type;

diff --git a/app/scripts/modules/appengine/src/pipeline/stages/destroyAsg/appengineDestroyAsgStage.ts b/app/scripts/modules/appengine/src/pipeline/stages/destroyAsg/appengineDestroyAsgStage.ts
@@ -8,7 +8,6 @@ import { AppengineStageCtrl } from '../appengineStage.controller';
 class AppengineDestroyAsgStageCtrl extends AppengineStageCtrl {
   public static $inject = ['$scope'];
   constructor(protected $scope: IAppengineStageScope) {
-    'ngInject';
     super($scope);
 
     super.setAccounts().then(() => {

diff --git a/app/scripts/modules/appengine/src/pipeline/stages/disableAsg/appengineDisableAsgStage.ts b/app/scripts/modules/appengine/src/pipeline/stages/disableAsg/appengineDisableAsgStage.ts
@@ -9,7 +9,6 @@ import { AppengineStageCtrl } from '../appengineStage.controller';
 class AppengineDisableAsgStageCtrl extends AppengineStageCtrl {
   public static $inject = ['$scope'];
   constructor(protected $scope: IAppengineStageScope) {
-    'ngInject';
     super($scope);
 
     super.setAccounts().then(() => {

diff --git a/.../modules/appengine/src/pipeline/stages/editLoadBalancer/appengineEditLoadBalancerStage.ts b/.../modules/appengine/src/pipeline/stages/editLoadBalancer/appengineEditLoadBalancerStage.ts
@@ -9,7 +9,6 @@ import { APPENGINE_LOAD_BALANCER_CHOICE_MODAL_CTRL } from './loadBalancerChoice.
 class AppengineEditLoadBalancerStageCtrl implements IController {
   public static $inject = ['$scope', '$uibModal'];
   constructor(public $scope: any, private $uibModal: IModalService) {
-    'ngInject';
     $scope.stage.loadBalancers = $scope.stage.loadBalancers || [];
     $scope.stage.cloudProvider = 'appengine';
   }

diff --git a/...les/appengine/src/pipeline/stages/editLoadBalancer/loadBalancerChoice.modal.controller.ts b/...les/appengine/src/pipeline/stages/editLoadBalancer/loadBalancerChoice.modal.controller.ts
@@ -15,7 +15,6 @@ class AppengineLoadBalancerChoiceModalCtrl implements IController {
     private $uibModalInstance: IModalServiceInstance,
     private application: Application,
   ) {
-    'ngInject';
     this.initialize();
   }
 

diff --git a/app/scripts/modules/appengine/src/pipeline/stages/enableAsg/appengineEnableAsgStage.ts b/app/scripts/modules/appengine/src/pipeline/stages/enableAsg/appengineEnableAsgStage.ts
@@ -9,7 +9,6 @@ import { AppengineStageCtrl } from '../appengineStage.controller';
 class AppengineEnableAsgStageCtrl extends AppengineStageCtrl {
   public static $inject = ['$scope'];
   constructor(protected $scope: IAppengineStageScope) {
-    'ngInject';
     super($scope);
 
     super.setAccounts().then(() => {

diff --git a/...cripts/modules/appengine/src/pipeline/stages/shrinkCluster/appengineShrinkClusterStage.ts b/...cripts/modules/appengine/src/pipeline/stages/shrinkCluster/appengineShrinkClusterStage.ts
@@ -14,7 +14,6 @@ interface IAppengineShrinkClusterStage extends IAppengineStage {
 class AppengineShrinkClusterStageCtrl extends AppengineStageCtrl {
   public static $inject = ['$scope'];
   constructor(public $scope: IAppengineStageScope) {
-    'ngInject';
     super($scope);
 
     super.setAccounts().then(() => {

diff --git a/.../modules/appengine/src/pipeline/stages/startServerGroup/appengineStartServerGroupStage.ts b/.../modules/appengine/src/pipeline/stages/startServerGroup/appengineStartServerGroupStage.ts
@@ -9,7 +9,6 @@ import { AppengineStageCtrl } from '../appengineStage.controller';
 class AppengineStartServerGroupStageCtrl extends AppengineStageCtrl {
   public static $inject = ['$scope'];
   constructor(public $scope: IAppengineStageScope) {
-    'ngInject';
     super($scope);
 
     super.setAccounts().then(() => {

diff --git a/...ts/modules/appengine/src/pipeline/stages/stopServerGroup/appengineStopServerGroupStage.ts b/...ts/modules/appengine/src/pipeline/stages/stopServerGroup/appengineStopServerGroupStage.ts
@@ -9,7 +9,6 @@ import { AppengineStageCtrl } from '../appengineStage.controller';
 class AppengineStopServerGroupStageCtrl extends AppengineStageCtrl {
   public static $inject = ['$scope'];
   constructor(public $scope: IAppengineStageScope) {
-    'ngInject';
     super($scope);
 
     super.setAccounts().then(() => {

diff --git a/app/scripts/modules/appengine/src/serverGroup/configure/serverGroupCommandBuilder.service.ts b/app/scripts/modules/appengine/src/serverGroup/configure/serverGroupCommandBuilder.service.ts
@@ -94,9 +94,7 @@ export class AppengineServerGroupCommandBuilder {
   }
 
   public static $inject = ['$q'];
-  constructor(private $q: IQService) {
-    'ngInject';
-  }
+  constructor(private $q: IQService) {}
 
   public buildNewServerGroupCommand(
     app: Application,

diff --git a/app/scripts/modules/appengine/src/serverGroup/configure/wizard/basicSettings.controller.ts b/app/scripts/modules/appengine/src/serverGroup/configure/wizard/basicSettings.controller.ts
@@ -23,8 +23,6 @@ class AppengineServerGroupBasicSettingsCtrl implements IController {
     $controller: IControllerService,
     $uibModalStack: any,
   ) {
-    'ngInject';
-
     extend(
       this,
       $controller('BasicSettingsMixin', {

diff --git a/...scripts/modules/appengine/src/serverGroup/configure/wizard/cloneServerGroup.controller.ts b/...scripts/modules/appengine/src/serverGroup/configure/wizard/cloneServerGroup.controller.ts
@@ -37,7 +37,6 @@ class AppengineCloneServerGroupCtrl implements IController {
     private serverGroupWriter: ServerGroupWriter,
     appengineServerGroupCommandBuilder: AppengineServerGroupCommandBuilder,
   ) {
-    'ngInject';
     if (['create', 'clone', 'editPipeline'].includes(get<string>(serverGroupCommand, 'viewState.mode'))) {
       this.$scope.command = serverGroupCommand;
       this.state.loading = false;

diff --git a/app/scripts/modules/appengine/src/serverGroup/details/details.controller.ts b/app/scripts/modules/appengine/src/serverGroup/details/details.controller.ts
@@ -79,8 +79,6 @@ class AppengineServerGroupDetailsController implements IController {
     private appengineServerGroupWriter: AppengineServerGroupWriter,
     private appengineServerGroupCommandBuilder: AppengineServerGroupCommandBuilder,
   ) {
-    'ngInject';
-
     this.app
       .ready()
       .then(() => this.extractServerGroup(serverGroup))

diff --git a/app/scripts/modules/appengine/src/serverGroup/transformer.ts b/app/scripts/modules/appengine/src/serverGroup/transformer.ts
@@ -69,9 +69,7 @@ export class AppengineDeployDescription {
 
 class AppengineServerGroupTransformer {
   public static $inject = ['$q'];
-  constructor(private $q: ng.IQService) {
-    'ngInject';
-  }
+  constructor(private $q: ng.IQService) {}
 
   public normalizeServerGroup(serverGroup: IServerGroup): ng.IPromise<IServerGroup> {
     return this.$q.resolve(serverGroup);

diff --git a/app/scripts/modules/canary/canary/canaryStage.js b/app/scripts/modules/canary/canary/canaryStage.js
@@ -188,8 +188,6 @@ module.exports = angular
     'serverGroupCommandBuilder',
     'awsServerGroupTransformer',
     function($scope, $uibModal, stage, providerSelectionService, serverGroupCommandBuilder, awsServerGroupTransformer) {
-      'ngInject';
-
       $scope.isExpression = function(value) {
         return isString(value) && value.includes('${');
       };

diff --git a/app/scripts/modules/cloudfoundry/src/instance/details/cloudfoundryInstanceDetails.module.ts b/app/scripts/modules/cloudfoundry/src/instance/details/cloudfoundryInstanceDetails.module.ts
@@ -34,7 +34,6 @@ class CloudFoundryInstanceDetailsCtrl implements IController {
     private confirmationModalService: ConfirmationModalService,
     private $q: IQService,
   ) {
-    'ngInject';
     this.$scope.application = this.app;
     this.$scope.instanceWriter = this.instanceWriter;
     this.$scope.confirmationModalService = this.confirmationModalService;

diff --git a/...s/modules/cloudfoundry/src/loadBalancer/details/cloudFoundryLoadBalancerDetails.module.ts b/...s/modules/cloudfoundry/src/loadBalancer/details/cloudFoundryLoadBalancerDetails.module.ts
@@ -24,7 +24,6 @@ class CloudFoundryLoadBalancerDetailsCtrl implements IController {
     loadBalancer: ILoadBalancerFromStateParams,
     private $q: IQService,
   ) {
-    'ngInject';
     this.$scope.application = this.app;
     this.$scope.confirmationModalService = this.confirmationModalService;
     this.$scope.loading = true;

diff --git a/app/scripts/modules/cloudfoundry/src/loadBalancer/loadBalancer.transformer.ts b/app/scripts/modules/cloudfoundry/src/loadBalancer/loadBalancer.transformer.ts
@@ -45,9 +45,7 @@ export class CloudFoundryLoadBalancerUpsertDescription
 
 export class CloudFoundryLoadBalancerTransformer {
   public static $inject = ['$q'];
-  constructor(private $q: ng.IQService) {
-    'ngInject';
-  }
+  constructor(private $q: ng.IQService) {}
 
   public normalizeLoadBalancer(loadBalancer: ILoadBalancer): ng.IPromise<ILoadBalancer> {
     loadBalancer.provider = loadBalancer.type;

diff --git a/...dfoundry/src/pipeline/stages/cloneServerGroup/cloudfoundryCloneServerGroupStage.module.ts b/...dfoundry/src/pipeline/stages/cloneServerGroup/cloudfoundryCloneServerGroupStage.module.ts
@@ -5,9 +5,7 @@ import { IStage, Registry } from '@spinnaker/core';
 
 class CloudfoundryCloneServerGroupStageCtrl implements IController {
   public static $inject = ['$scope'];
-  constructor(public $scope: IScope) {
-    'ngInject';
-  }
+  constructor(public $scope: IScope) {}
 }
 
 export const CLOUD_FOUNDRY_CLONE_SERVER_GROUP_STAGE = 'spinnaker.cloudfoundry.pipeline.stage.cloneServerGroupStage';

diff --git a/...s/cloudfoundry/src/pipeline/stages/deployService/cloudfoundryDeployServiceStage.module.ts b/...s/cloudfoundry/src/pipeline/stages/deployService/cloudfoundryDeployServiceStage.module.ts
@@ -8,9 +8,7 @@ import { IServiceFieldValidatorConfig } from 'cloudfoundry/pipeline/config/valid
 
 class CloudFoundryDeployServiceStageCtrl implements IController {
   public static $inject = ['$scope'];
-  constructor(public $scope: IScope) {
-    'ngInject';
-  }
+  constructor(public $scope: IScope) {}
 }
 
 const serviceInstanceNameValidatorConfig: IServiceFieldValidatorConfig = {

diff --git a/...modules/cloudfoundry/src/pipeline/stages/destroyAsg/cloudfoundryDestroyAsgStage.module.ts b/...modules/cloudfoundry/src/pipeline/stages/destroyAsg/cloudfoundryDestroyAsgStage.module.ts
@@ -7,7 +7,6 @@ import { Application, IStage, Registry } from '@spinnaker/core';
 class CloudFoundryDestroyAsgStageCtrl implements IController {
   public static $inject = ['$scope', 'application'];
   constructor(public $scope: IScope, private application: Application) {
-    'ngInject';
     this.$scope.application = this.application;
   }
 }

diff --git a/...cloudfoundry/src/pipeline/stages/destroyService/cloudfoundryDestroyServiceStage.module.ts b/...cloudfoundry/src/pipeline/stages/destroyService/cloudfoundryDestroyServiceStage.module.ts
@@ -7,9 +7,7 @@ import { CloudfoundryDestroyServiceExecutionDetails } from './CloudfoundryDestro
 
 class CloudFoundryDestroyServiceStageCtrl implements IController {
   public static $inject = ['$scope'];
-  constructor(public $scope: IScope) {
-    'ngInject';
-  }
+  constructor(public $scope: IScope) {}
 }
 
 export const CLOUD_FOUNDRY_DESTROY_SERVICE_STAGE = 'spinnaker.cloudfoundry.pipeline.stage.deleteServiceStage';

diff --git a/...modules/cloudfoundry/src/pipeline/stages/disableAsg/cloudfoundryDisableAsgStage.module.ts b/...modules/cloudfoundry/src/pipeline/stages/disableAsg/cloudfoundryDisableAsgStage.module.ts
@@ -7,7 +7,6 @@ import { Application, IStage, Registry } from '@spinnaker/core';
 class CloudFoundryDisableAsgStageCtrl implements IController {
   public static $inject = ['$scope', 'application'];
   constructor(public $scope: IScope, private application: Application) {
-    'ngInject';
     this.$scope.application = this.application;
   }
 }

diff --git a/...s/modules/cloudfoundry/src/pipeline/stages/enableAsg/cloudfoundryEnableAsgStage.module.ts b/...s/modules/cloudfoundry/src/pipeline/stages/enableAsg/cloudfoundryEnableAsgStage.module.ts
@@ -7,7 +7,6 @@ import { Application, IStage, Registry } from '@spinnaker/core';
 class CloudFoundryEnableAsgStageCtrl implements IController {
   public static $inject = ['$scope', 'application'];
   constructor(public $scope: IScope, private application: Application) {
-    'ngInject';
     this.$scope.application = this.application;
   }
 }

diff --git a/...dfoundry/src/pipeline/stages/mapLoadBalancers/cloudfoundryMapLoadBalancersStage.module.ts b/...dfoundry/src/pipeline/stages/mapLoadBalancers/cloudfoundryMapLoadBalancersStage.module.ts
@@ -8,7 +8,6 @@ import { CloudfoundryMapLoadBalancersExecutionDetails } from './CloudfoundryMapL
 class CloudFoundryMapLoadBalancersStageCtrl implements IController {
   public static $inject = ['$scope'];
   constructor(public $scope: IScope) {
-    'ngInject';
     $scope.accounts = [];
     AccountService.listAccounts('cloudfoundry').then(accounts => {
       $scope.accounts = accounts;

diff --git a/...s/modules/cloudfoundry/src/pipeline/stages/resizeAsg/cloudfoundryResizeAsgStage.module.ts b/...s/modules/cloudfoundry/src/pipeline/stages/resizeAsg/cloudfoundryResizeAsgStage.module.ts
@@ -8,7 +8,6 @@ import { IInstanceFieldSizeValidationConfig } from 'cloudfoundry/pipeline/config
 class CloudFoundryResizeAsgStageCtrl implements IController {
   public static $inject = ['$scope', 'application'];
   constructor(public $scope: IScope, private application: Application) {
-    'ngInject';
     this.$scope.application = this.application;
   }
 }

diff --git a/...oudfoundry/src/pipeline/stages/rollbackCluster/cloudfoundryRollbackClusterStage.module.ts b/...oudfoundry/src/pipeline/stages/rollbackCluster/cloudfoundryRollbackClusterStage.module.ts
@@ -7,7 +7,6 @@ import { Application, IStage, Registry } from '@spinnaker/core';
 class CloudFoundryRollbackClusterStageCtrl implements IController {
   public static $inject = ['$scope', 'application'];
   constructor(public $scope: IScope, private application: Application) {
-    'ngInject';
     this.$scope.application = this.application;
   }
 }

diff --git a/...ndry/src/pipeline/stages/unmapLoadBalancers/cloudfoundryUnmapLoadBalancersStage.module.ts b/...ndry/src/pipeline/stages/unmapLoadBalancers/cloudfoundryUnmapLoadBalancersStage.module.ts
@@ -8,7 +8,6 @@ import { CloudfoundryUnmapLoadBalancersExecutionDetails } from './CloudfoundryUn
 class CloudFoundryUnmapLoadBalancersStageCtrl implements IController {
   public static $inject = ['$scope'];
   constructor(public $scope: IScope) {
-    'ngInject';
     $scope.accounts = [];
     AccountService.listAccounts('cloudfoundry').then(accounts => {
       $scope.accounts = accounts;

diff --git a/...ts/modules/cloudfoundry/src/serverGroup/configure/serverGroupCommandBuilder.service.cf.ts b/...ts/modules/cloudfoundry/src/serverGroup/configure/serverGroupCommandBuilder.service.cf.ts
@@ -39,9 +39,7 @@ export class CloudFoundryServerGroupCommandBuilder {
   }
 
   public static $inject = ['$q'];
-  constructor(private $q: IQService) {
-    'ngInject';
-  }
+  constructor(private $q: IQService) {}
 
   public buildNewServerGroupCommand(app: Application, defaults: any): IPromise<ICloudFoundryCreateServerGroupCommand> {
     defaults = defaults || {};

diff --git a/app/scripts/modules/cloudfoundry/src/serverGroup/serverGroup.transformer.ts b/app/scripts/modules/cloudfoundry/src/serverGroup/serverGroup.transformer.ts
@@ -6,9 +6,7 @@ import { ICloudFoundryServerGroup, ICloudFoundryEnvVar } from 'cloudfoundry/doma
 
 export class CloudFoundryServerGroupTransformer {
   public static $inject = ['$q'];
-  public constructor(private $q: ng.IQService) {
-    'ngInject';
-  }
+  public constructor(private $q: ng.IQService) {}
 
   public normalizeServerGroupDetails(serverGroup: ICloudFoundryServerGroup): ICloudFoundryServerGroup {
     return serverGroup;

diff --git a/app/scripts/modules/core/src/api/network.interceptor.ts b/app/scripts/modules/core/src/api/network.interceptor.ts
@@ -30,7 +30,6 @@ export class NetworkInterceptor implements IHttpInterceptor {
     private $timeout: ITimeoutService,
     private $injector: any,
   ) {
-    'ngInject';
     this.$window.addEventListener('offline', this.handleOffline);
     this.$window.addEventListener('online', this.handleOnline);
     this.resetNetworkAvailable();

diff --git a/app/scripts/modules/core/src/application/application.state.provider.ts b/app/scripts/modules/core/src/application/application.state.provider.ts
@@ -27,7 +27,6 @@ export class ApplicationStateProvider implements IServiceProvider {
 
   public static $inject = ['stateConfigProvider'];
   constructor(private stateConfigProvider: StateConfigProvider) {
-    'ngInject';
     this.childStates.push(this.insightState);
   }