From bfc4c3cf4372773b526ff285115e4a650f05466e Mon Sep 17 00:00:00 2001
From: Paul Gschwendtner <paulgschwendtner@gmail.com>
Date: Mon, 7 Jun 2021 21:05:34 +0200
Subject: [PATCH] refactor(localize): ensure compatibility with
 noImplicitOverride (#42512)

Adds the `override` keyword to the `localize` sources to ensure
compatibility with `noImplicitOverride`.

PR Close #42512
---
 .../message_serialization/message_serializer.ts           | 8 ++++----
 .../translation_parsers/xliff1_translation_parser.ts      | 4 ++--
 .../translation_parsers/xliff2_translation_parser.ts      | 2 +-
 .../translation_parsers/xtb_translation_parser.ts         | 2 +-
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/packages/localize/src/tools/src/translate/translation_files/message_serialization/message_serializer.ts b/packages/localize/src/tools/src/translate/translation_files/message_serialization/message_serializer.ts
index af0fd9c3a7..57487a7b35 100644
--- a/packages/localize/src/tools/src/translate/translation_files/message_serialization/message_serializer.ts
+++ b/packages/localize/src/tools/src/translate/translation_files/message_serialization/message_serializer.ts
@@ -36,7 +36,7 @@ export class MessageSerializer<T> extends BaseVisitor {
     return this.renderer.message;
   }
 
-  visitElement(element: Element): void {
+  override visitElement(element: Element): void {
     if (this.config.placeholder && element.name === this.config.placeholder.elementName) {
       const name = getAttrOrThrow(element, this.config.placeholder.nameAttribute);
       const body = this.config.placeholder.bodyAttribute &&
@@ -55,18 +55,18 @@ export class MessageSerializer<T> extends BaseVisitor {
     }
   }
 
-  visitText(text: Text): void {
+  override visitText(text: Text): void {
     this.renderer.text(text.value);
   }
 
-  visitExpansion(expansion: Expansion): void {
+  override visitExpansion(expansion: Expansion): void {
     this.renderer.startIcu();
     this.renderer.text(`${expansion.switchValue}, ${expansion.type},`);
     visitAll(this, expansion.cases);
     this.renderer.endIcu();
   }
 
-  visitExpansionCase(expansionCase: ExpansionCase): void {
+  override visitExpansionCase(expansionCase: ExpansionCase): void {
     this.renderer.text(` ${expansionCase.value} {`);
     this.renderer.startContainer();
     visitAll(this, expansionCase.expression);
diff --git a/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff1_translation_parser.ts b/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff1_translation_parser.ts
index 2dd951ffe4..c70a56578a 100644
--- a/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff1_translation_parser.ts
+++ b/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff1_translation_parser.ts
@@ -106,7 +106,7 @@ export class Xliff1TranslationParser implements TranslationParser<XmlTranslation
 }
 
 class XliffFileElementVisitor extends BaseVisitor {
-  visitElement(fileElement: Element): any {
+  override visitElement(fileElement: Element): any {
     if (fileElement.name === 'file') {
       return {fileElement, locale: getAttribute(fileElement, 'target-language')};
     }
@@ -114,7 +114,7 @@ class XliffFileElementVisitor extends BaseVisitor {
 }
 
 class XliffTranslationVisitor extends BaseVisitor {
-  visitElement(element: Element, bundle: ParsedTranslationBundle): void {
+  override visitElement(element: Element, bundle: ParsedTranslationBundle): void {
     if (element.name === 'trans-unit') {
       this.visitTransUnitElement(element, bundle);
     } else {
diff --git a/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff2_translation_parser.ts b/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff2_translation_parser.ts
index b47d9c1ff8..4eef66fd67 100644
--- a/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff2_translation_parser.ts
+++ b/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xliff2_translation_parser.ts
@@ -90,7 +90,7 @@ interface TranslationVisitorContext {
 }
 
 class Xliff2TranslationVisitor extends BaseVisitor {
-  visitElement(element: Element, {bundle, unit}: TranslationVisitorContext): any {
+  override visitElement(element: Element, {bundle, unit}: TranslationVisitorContext): any {
     if (element.name === 'unit') {
       this.visitUnitElement(element, bundle);
     } else if (element.name === 'segment') {
diff --git a/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xtb_translation_parser.ts b/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xtb_translation_parser.ts
index bb1370b62a..337be9ef5f 100644
--- a/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xtb_translation_parser.ts
+++ b/packages/localize/src/tools/src/translate/translation_files/translation_parsers/xtb_translation_parser.ts
@@ -82,7 +82,7 @@ export class XtbTranslationParser implements TranslationParser<XmlTranslationPar
 }
 
 class XtbVisitor extends BaseVisitor {
-  visitElement(element: Element, bundle: ParsedTranslationBundle): any {
+  override visitElement(element: Element, bundle: ParsedTranslationBundle): any {
     switch (element.name) {
       case 'translation':
         // Error if no `id` attribute