@arcgis/charts-components 5.2.0-next.91 → 5.2.0-next.94

package/dist/components/arcgis-charts-config-pie-chart-data/customElement.js

@@ -21,28 +21,28 @@ class E extends b {
     super(...arguments), this._messages = C(), this.pieChartPopover = null, this.sortOrderElement = n(), this.categoryElement = n(), this.numericFieldsElement = n(), this.onPieChartDataChange = (e) => {
       try {
         if (e.detail !== null && e.detail?.value !== void 0) {
-          const { contentKind: t } = e.detail, i = [...e.detail.value];
-          switch (t) {
+          const { contentKind: i } = e.detail, t = [...e.detail.value];
+          switch (i) {
             case a.aggregation:
-              this.model.aggregationType = i[0] !== P.NoAggregation ? i[0] : "";
+              this.model.aggregationType = t[0] !== P.NoAggregation ? t[0] : "";
               break;
             case a.category: {
-              const r = i[0]?.trim() || "", d = this.model.numericFields;
+              const r = t[0]?.trim() || "", d = this.model.numericFields;
               d.length > 1 && r !== g && (this.model.numericFields = [d[0]]), this.model.category = r, this.model.colorMatch = !0;
               break;
             }
             case a.numericFields:
-              i.length > 1 && (this.model.category = g), this.model.numericFields = i;
+              (t.length > 1 || t.length > 0 && this.model.category.trim() === "") && (this.model.category = g), this.model.numericFields = t;
               break;
             case a.sortOrder:
-              this.model.setSortOrder(i[0]);
+              this.model.setSortOrder(t[0]);
               break;
             default:
               break;
           }
         }
-      } catch (t) {
-        throw t instanceof Error && this.arcgisChartsConfigDataChangeError.emit({ error: t, model: this.model }), t;
+      } catch (i) {
+        throw i instanceof Error && this.arcgisChartsConfigDataChangeError.emit({ error: i, model: this.model }), i;
       }
       this.requestUpdate();
     }, this.onModelConfigChange = () => {
@@ -66,8 +66,8 @@ class E extends b {
   disconnectedCallback() {
     super.disconnectedCallback(), this.removePieChartDataPopover(), this.model.removeEventListener("arcgisChartsModelConfigChange", this.onModelConfigChange);
   }
-  modelChange(e, t) {
-    t?.removeEventListener("arcgisChartsModelConfigChange", this.onModelConfigChange), e?.addEventListener("arcgisChartsModelConfigChange", this.onModelConfigChange);
+  modelChange(e, i) {
+    i?.removeEventListener("arcgisChartsModelConfigChange", this.onModelConfigChange), e?.addEventListener("arcgisChartsModelConfigChange", this.onModelConfigChange);
   }
   buildNumericFieldItem(e) {
     return f(e, s`<calcite-list-item .label=${m(this.layerFieldsInfo, e)} .value=${e} closable @calciteListItemClose=${this.onRemoveNumericField}></calcite-list-item>`);
@@ -82,27 +82,27 @@ class E extends b {
     this.contentKind = a.numericFields, this.setupPopover(e);
   }
   onRemoveNumericField(e) {
-    const t = e.target.value;
-    this.model.numericFields = this.model.numericFields.filter((i) => i !== t);
+    const i = e.target.value;
+    this.model.numericFields = this.model.numericFields.filter((t) => t !== i);
   }
   onShowDataLabelsChange(e) {
     this.model.dataLabelsVisibility = e.target.checked;
   }
   setupPopover(e) {
     e.stopImmediatePropagation(), this.removePieChartDataPopover();
-    let t, i, r;
+    let i, t, r;
     switch (this.contentKind) {
       case a.category:
-        t = this.categoryElement.value, i = this._messages.categoryHeader, r = this.model.category;
+        i = this.categoryElement.value, t = this._messages.categoryHeader, r = this.model.category;
         break;
       case a.numericFields:
-        t = this.numericFieldsElement.value, i = this._messages.numericFieldsMultipleHeader, r = this.model.numericFields;
+        i = this.numericFieldsElement.value, t = this._messages.numericFieldsMultipleHeader, r = this.model.numericFields;
         break;
       case a.sortOrder:
-        t = this.sortOrderElement.value, i = this._messages.sortOrderHeader, r = this.model.getSortOrder();
+        i = this.sortOrderElement.value, t = this._messages.sortOrderHeader, r = this.model.getSortOrder();
         break;
     }
-    this.pieChartPopover = v(s`<arcgis-charts-config-pie-chart-popover .headingTitle=${i} .referenceElement=${t} open .contentKind=${this.contentKind} .layerFieldsInfo=${this.layerFieldsInfo} .selectedContent=${r} @arcgisChartsConfigPieChartDataPopoverChange=${this.onPieChartDataChange} @arcgisChartsConfigPopoverClose=${this.popoverClose}></arcgis-charts-config-pie-chart-popover>`), !document.body.contains(this.pieChartPopover) && this.pieChartPopover !== null && document.body.appendChild(this.pieChartPopover);
+    this.pieChartPopover = v(s`<arcgis-charts-config-pie-chart-popover .headingTitle=${t} .referenceElement=${i} open .contentKind=${this.contentKind} .layerFieldsInfo=${this.layerFieldsInfo} .selectedContent=${r} @arcgisChartsConfigPieChartDataPopoverChange=${this.onPieChartDataChange} @arcgisChartsConfigPopoverClose=${this.popoverClose}></arcgis-charts-config-pie-chart-popover>`), !document.body.contains(this.pieChartPopover) && this.pieChartPopover !== null && document.body.appendChild(this.pieChartPopover);
   }
   getCorrectedCategoryValue() {
     return y({
@@ -121,17 +121,17 @@ class E extends b {
     return s`<arcgis-charts-config-field-picker .label=${e} @click=${this.onCategorySelect} ${c(this.categoryElement)}></arcgis-charts-config-field-picker>`;
   }
   renderNumericFields(e) {
-    const t = F(this.model.layer).idField, i = [];
+    const i = F(this.model.layer).idField, t = [];
     for (const r of e)
-      r !== t && i.push(this.buildNumericFieldItem(r));
-    return s`<calcite-list label class=${o(l.fieldList)} ${c(this.numericFieldsElement)}>${i}</calcite-list>`;
+      r !== i && t.push(this.buildNumericFieldItem(r));
+    return s`<calcite-list label class=${o(l.fieldList)} ${c(this.numericFieldsElement)}>${t}</calcite-list>`;
   }
   render() {
-    const e = this.renderCategory(), t = this.renderSorting(), i = this.renderNumericFields([...this.model.numericFields]);
+    const e = this.renderCategory(), i = this.renderSorting(), t = this.renderNumericFields([...this.model.numericFields]);
     return s`<calcite-label class=${o(l.label)}>${this._messages.category}${e}</calcite-label>${[
-      s`<calcite-label class=${o(l.label)}>${this._messages.numericFields}${i}</calcite-label>`,
+      s`<calcite-label class=${o(l.label)}>${this._messages.numericFields}${t}</calcite-label>`,
       s`<div class=${o(l.fab)}><calcite-fab appearance=outline-fill kind=neutral scale=m .label=${this._messages.selectNumericFields} text-enabled .text=${this._messages.selectNumericFields} icon=plus @click=${this.onAddNumericFields}></calcite-fab></div>`
-    ]}<calcite-label class=${o(l.label)} layout=inline-space-between>${this._messages.showDataLabels}<calcite-switch .checked=${this.model.dataLabelsVisibility} @calciteSwitchChange=${this.onShowDataLabelsChange}></calcite-switch></calcite-label><calcite-label class=${o(l.label)}>${this._messages.sortOrder}${t}</calcite-label>`;
+    ]}<calcite-label class=${o(l.label)} layout=inline-space-between>${this._messages.showDataLabels}<calcite-switch .checked=${this.model.dataLabelsVisibility} @calciteSwitchChange=${this.onShowDataLabelsChange}></calcite-switch></calcite-label><calcite-label class=${o(l.label)}>${this._messages.sortOrder}${i}</calcite-label>`;
   }
 }
 p("arcgis-charts-config-pie-chart-data", E);