refactor: simplify provider setup

json-derulo · json-derulo · commit e25fbd3c8c0a · 2024-11-14T20:39:46.000+01:00
diff --git a/projects/material-css-vars/src/lib/material-css-vars.module.ts b/projects/material-css-vars/src/lib/material-css-vars.module.ts
@@ -20,13 +20,9 @@ export class MaterialCssVarsModule {
   ): ModuleWithProviders<MaterialCssVarsModule> {
     return {
       ngModule: MaterialCssVarsModule,
-      providers: [{ provide: MATERIAL_CSS_VARS_CFG, useValue: config }],
+      providers: [provideMaterialCssVars(config)],
     };
   }
-
-  // This is necessary, so the service is constructed, even if the service is never injected
-  // ToDo: change to environment initializer, like in the provideMaterialCssVars() function below
-  constructor(private materialCssVarsService: MaterialCssVarsService) {}
 }
 
 export function provideMaterialCssVars(

Original file line number	Diff line number	Diff line change
`@@ -20,13 +20,9 @@ export class MaterialCssVarsModule {`
`20`	`20`	`): ModuleWithProviders<MaterialCssVarsModule> {`
`21`	`21`	`return {`
`22`	`22`	`ngModule: MaterialCssVarsModule,`
`23`		`- providers: [{ provide: MATERIAL_CSS_VARS_CFG, useValue: config }],`
	`23`	`+ providers: [provideMaterialCssVars(config)],`
`24`	`24`	`};`
`25`	`25`	`}`
`26`		`-`
`27`		`- // This is necessary, so the service is constructed, even if the service is never injected`
`28`		`- // ToDo: change to environment initializer, like in the provideMaterialCssVars() function below`
`29`		`- constructor(private materialCssVarsService: MaterialCssVarsService) {}`
`30`	`26`	`}`
`31`	`27`
`32`	`28`	`export function provideMaterialCssVars(`