From e5d951c9a9b5222875edeb4100a4d3aad1d3fbbd Mon Sep 17 00:00:00 2001
From: David Taylor <david@taylorhq.com>
Date: Tue, 26 Nov 2024 19:10:23 +0000
Subject: [PATCH] PERF: Remove dynamic use of `<template>` (#340)

Context: https://github.com/discourse/discourse/pull/29942
---
 .../discourse/components/param-input-form.gjs | 58 ++++++++-----------
 1 file changed, 23 insertions(+), 35 deletions(-)

diff --git a/assets/javascripts/discourse/components/param-input-form.gjs b/assets/javascripts/discourse/components/param-input-form.gjs
index 4b28313d..fa09833d 100644
--- a/assets/javascripts/discourse/components/param-input-form.gjs
+++ b/assets/javascripts/discourse/components/param-input-form.gjs
@@ -100,46 +100,34 @@ function validationOf(info) {
   }
 }
 
+const components = {
+  int: <template>
+    <@field.Input @type="number" name={{@info.identifier}} />
+  </template>,
+  boolean: <template><@field.Checkbox name={{@info.identifier}} /></template>,
+  boolean_three: BooleanThree,
+  category_id: CategoryIdInput, // TODO
+  user_id: UserIdInput,
+  user_list: UserListInput,
+  group_list: GroupInput,
+  date: <template>
+    <@field.Input @type="date" name={{@info.identifier}} />
+  </template>,
+  time: <template>
+    <@field.Input @type="time" name={{@info.identifier}} />
+  </template>,
+  datetime: <template>
+    <@field.Input @type="datetime-local" name={{@info.identifier}} />
+  </template>,
+  default: <template><@field.Input name={{@info.identifier}} /></template>,
+};
+
 function componentOf(info) {
   let type = layoutMap[info.type] || "generic";
   if (info.nullable && type === "boolean") {
     type = "boolean_three";
   }
-  switch (type) {
-    case "int":
-      return <template>
-        <@field.Input @type="number" name={{@info.identifier}} />
-      </template>;
-    case "boolean":
-      return <template><@field.Checkbox name={{@info.identifier}} /></template>;
-    case "boolean_three":
-      return BooleanThree;
-    case "category_id":
-      // TODO
-      return CategoryIdInput;
-    case "user_id":
-      return UserIdInput;
-    case "user_list":
-      return UserListInput;
-    case "group_list":
-      return GroupInput;
-    case "date":
-      return <template>
-        <@field.Input @type="date" name={{@info.identifier}} />
-      </template>;
-    case "time":
-      return <template>
-        <@field.Input @type="time" name={{@info.identifier}} />
-      </template>;
-    case "datetime":
-      return <template>
-        <@field.Input @type="datetime-local" name={{@info.identifier}} />
-      </template>;
-    case "bigint":
-    case "string":
-    default:
-      return <template><@field.Input name={{@info.identifier}} /></template>;
-  }
+  return components[type] || components.default;
 }
 
 export default class ParamInputForm extends Component {