Minor fix to RuleContext.getParameter (#131)

* Minor fix to RuleContext.getParameter

* Minor cleanup

* Minor cleanup

Author: rayokota

schemaregistry/serde/avro.ts

@@ -314,7 +314,7 @@ async function transformField(
       fullName,
       field.name,
       getType(field.type),
-      ctx.getInlineTags(fullName),
+      null
     )
     const newVal = await transform(ctx, field.type, record[field.name], fieldTransform)
     if (ctx.rule.kind === 'CONDITION') {

schemaregistry/serde/serde.ts

@@ -424,7 +424,7 @@ export abstract class Deserializer extends Serde {
       previous = version
     }
     if (migrationMode === RuleMode.DOWNGRADE) {
-      migrations = migrations.map(x => x).reverse()
+      migrations = migrations.reverse()
     }
     return migrations
   }
@@ -512,16 +512,15 @@ export class RuleContext {
 
   getParameter(name: string): string | null {
     const params = this.rule.params
-    if (params == null) {
-      return null
-    }
-    let value = params[name]
-    if (value != null) {
-      return value
+    if (params != null) {
+      let value = params[name]
+      if (value != null) {
+        return value
+      }
     }
     let metadata = this.target.metadata
     if (metadata != null && metadata.properties != null) {
-      value = metadata.properties[name]
+      let value = metadata.properties[name]
       if (value != null) {
         return value
       }
@@ -545,8 +544,9 @@ export class RuleContext {
     return this.fieldContexts[size - 1]
   }
 
-  enterField(containingMessage: any, fullName: string, name: string, fieldType: FieldType, tags: Set<string>): FieldContext {
-    let allTags = new Set<string>(tags)
+  enterField(containingMessage: any, fullName: string, name: string, fieldType: FieldType,
+             tags: Set<string> | null): FieldContext {
+    let allTags = new Set<string>(tags ?? this.getInlineTags(fullName))
     for (let v of this.getTags(fullName)) {
       allTags.add(v)
     }