Rename getArrayValue

Author: Lustmored

src/LiveComponent/assets/src/live_controller.ts

@@ -6,7 +6,7 @@ import { setDeepData, doesDeepPropertyExist, normalizeModelName, parseDeepData }
 import { haveRenderedValuesChanged } from './have_rendered_values_changed';
 import { normalizeAttributesForComparison } from './normalize_attributes_for_comparison';
 import { cloneHTMLElement } from './clone_html_element';
-import { getArrayValue } from "./get_array_value";
+import { updateArrayDataFromChangedElement } from "./update_array_data";
 
 interface ElementLoadingDirectives {
     element: HTMLElement|SVGElement,
@@ -211,7 +211,7 @@ export default class extends Controller {
             const { currentLevelData, finalKey } = parseDeepData(this.dataValue, normalizeModelName(model))
             const currentValue = currentLevelData[finalKey];
 
-            value = getArrayValue(element, value, currentValue);
+            value = updateArrayDataFromChangedElement(element, value, currentValue);
         } else if (
             element instanceof HTMLInputElement
             && element.type === 'checkbox'

src/LiveComponent/assets/src/get_array_value.ts renamed to src/LiveComponent/assets/src/update_array_data.ts

@@ -5,7 +5,7 @@
  * @param value         Resolved value of a single HTML element (.value or [data-value])
  * @param currentValue  Current data value
  */
-export function getArrayValue(
+export function updateArrayDataFromChangedElement(
     element: HTMLElement,
     value: string|null,
     currentValue: any

src/LiveComponent/assets/test/get_array_value.test.ts renamed to src/LiveComponent/assets/test/update_array_data.test.ts

@@ -1,4 +1,4 @@
-import {getArrayValue} from "../src/get_array_value";
+import {updateArrayDataFromChangedElement} from "../src/update_array_data";
 
 
 describe('getArrayValue', () => {
@@ -7,11 +7,11 @@ describe('getArrayValue', () => {
         input.type = 'checkbox';
         input.checked = true;
 
-        expect(getArrayValue(input, 'foo', null))
+        expect(updateArrayDataFromChangedElement(input, 'foo', null))
             .toEqual(['foo']);
-        expect(getArrayValue(input, 'foo', []))
+        expect(updateArrayDataFromChangedElement(input, 'foo', []))
             .toEqual(['foo']);
-        expect(getArrayValue(input, 'foo', ['bar']))
+        expect(updateArrayDataFromChangedElement(input, 'foo', ['bar']))
             .toEqual(['bar', 'foo']);
     })
 
@@ -20,13 +20,13 @@ describe('getArrayValue', () => {
         input.type = 'checkbox';
         input.checked = false;
 
-        expect(getArrayValue(input, 'foo', null))
+        expect(updateArrayDataFromChangedElement(input, 'foo', null))
             .toEqual([]);
-        expect(getArrayValue(input, 'foo', ['foo']))
+        expect(updateArrayDataFromChangedElement(input, 'foo', ['foo']))
             .toEqual([]);
-        expect(getArrayValue(input, 'foo', ['bar']))
+        expect(updateArrayDataFromChangedElement(input, 'foo', ['bar']))
             .toEqual(['bar']);
-        expect(getArrayValue(input, 'foo', ['foo', 'bar']))
+        expect(updateArrayDataFromChangedElement(input, 'foo', ['foo', 'bar']))
             .toEqual(['bar']);
     })
 
@@ -40,15 +40,15 @@ describe('getArrayValue', () => {
         barOption.value = 'bar';
         select.add(barOption);
 
-        expect(getArrayValue(select, '', null))
+        expect(updateArrayDataFromChangedElement(select, '', null))
             .toEqual([]);
 
         fooOption.selected = true;
-        expect(getArrayValue(select, '', null))
+        expect(updateArrayDataFromChangedElement(select, '', null))
             .toEqual(['foo']);
 
         barOption.selected = true;
-        expect(getArrayValue(select, '', null))
+        expect(updateArrayDataFromChangedElement(select, '', null))
             .toEqual(['foo', 'bar']);
     })
 });