Make static

cognitedata · Aug 17, 2024 · 3f66667 · 3f66667
1 parent d4a122e
commit 3f66667
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 20 deletions.
diff --git a/react-components/src/architecture/base/commands/BaseFilterCommand.ts b/react-components/src/architecture/base/commands/BaseFilterCommand.ts
@@ -91,40 +91,40 @@ export abstract class BaseFilterCommand extends RenderTargetCommand {
 
   public getSelectedLabel(translate: TranslateDelegate): string {
     if (this._children === undefined) {
-      return getNoneLabel(translate);
+      return BaseFilterCommand.getNoneString(translate);
     }
     const selected = this._children.filter((child) => child.isChecked);
     const counter = selected.length;
     if (counter === 0) {
-      return getNoneLabel(translate);
+      return BaseFilterCommand.getNoneString(translate);
     }
     if (counter === this._children.length) {
-      return getAllLabel(translate);
+      return BaseFilterCommand.getAllString(translate);
     }
     if (counter === 1) {
       return selected[0].getLabel(translate);
     }
-    return counter.toString() + ' ' + getSelectedLabel(translate);
+    return counter.toString() + ' ' + BaseFilterCommand.getSelectedString(translate);
   }
-}
 
-export abstract class BaseFilterItemCommand extends RenderTargetCommand {
-  public abstract get color(): Color | undefined;
-  public abstract setChecked(value: boolean): void;
-}
+  // ==================================================
+  // STATIC METHODS
+  // ==================================================
 
-// ==================================================
-// PRIVATE FUNCTIONS
-// ==================================================
+  public static getAllString(translate: TranslateDelegate): string {
+    return translate('ALL', 'All');
+  }
 
-function getAllLabel(translate: TranslateDelegate): string {
-  return translate('ALL', 'All');
-}
+  private static getNoneString(translate: TranslateDelegate): string {
+    return translate('NONE', 'None');
+  }
 
-function getNoneLabel(translate: TranslateDelegate): string {
-  return translate('NONE', 'None');
+  private static getSelectedString(translate: TranslateDelegate): string {
+    return translate('SELECTED', 'Selected');
+  }
 }
 
-function getSelectedLabel(translate: TranslateDelegate): string {
-  return translate('SELECTED', 'Selected');
+export abstract class BaseFilterItemCommand extends RenderTargetCommand {
+  public abstract get color(): Color | undefined;
+  public abstract setChecked(value: boolean): void;
 }
diff --git a/react-components/src/components/Architecture/FilterButton.tsx b/react-components/src/components/Architecture/FilterButton.tsx
@@ -125,7 +125,7 @@ export const FilterButton = ({
                 onClick={() => {
                   command.toggleAllChecked();
                 }}>
-                {command.getAllLabel(t)}
+                {BaseFilterCommand.getAllString(t)}
               </Menu.Item>
               <StyledMenuItems>
                 {children.map((child, _index): ReactElement => {