extern/egroupware
1
0
Fork 1
mirror of https://github.com/EGroupware/egroupware.git synced 2024-11-09 01:25:22 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix infinite loop in missingOption in country/state due to not waiting for results

Browse Source 
Add class method to search through select options
This commit is contained in:
nathan 2023-12-05 11:51:24 -07:00
parent fd4bdd4997
commit 5ef349f057
5 changed files with 50 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
api/js/etemplate/Et2Select/Et2WidgetWithSelectMixin.ts
View File

@ -167,6 +167,27 @@ export const Et2WidgetWithSelectMixin = <T extends Constructor<LitElement>>(supe
return [this.value];
}
/**
* Search options for a given value, returning the first matching option
*
* @return SelectOption | null
*/
public optionSearch(value : string, options : SelectOption[] = null) : SelectOption | null
{
let search = function(options, value)
{
return options.find((option) =>
{
if(Array.isArray(option.value))
{
return search(option.value, value);
}
return option.value == value;
});
}
return search(options ?? this.select_options, value);
}
/**
* Render select_options as child DOM Nodes
* @protected

13
api/js/etemplate/Et2Select/SearchMixin.ts
View File

@ -377,21 +377,10 @@ export const Et2WithSearchMixin = dedupeMixin(<T extends Constructor<LitElement>
}
if(this.searchEnabled)
{
let search = function(options, value)
{
return options.some((option) =>
{
if(Array.isArray(option.value))
{
return search(option.value, value);
}
return option.value == value;
});
};
// Check to see if value is for an option we do not have
for(const newValueElement of this.getValueAsArray())
{
if(search(this.select_options, newValueElement))
if(this.optionSearch(newValueElement))
{
continue;
}

2
api/js/etemplate/Et2Select/Select/Et2SelectCountry.ts
View File

@ -37,7 +37,7 @@ export class Et2SelectCountry extends Et2StaticSelectMixin(Et2Select)
this.search = true;
(<Promise<SelectOption[]>>so.country(this, {}, true)).then(options =>
this.fetchComplete = (<Promise<SelectOption[]>>so.country(this, {}, true)).then(options =>
{
this._static_options = options
this.requestUpdate("select_options");

11
api/js/etemplate/Et2Select/Select/Et2SelectState.ts
View File

@ -1,6 +1,6 @@
import {Et2Select} from "../Et2Select";
import {Et2StaticSelectMixin, StaticOptions} from "../StaticOptions";
import {SelectOption} from "../FindSelectOptions";
import {Et2StaticSelectMixin, StaticOptions as so} from "../StaticOptions";
import {cleanSelectOptions, SelectOption} from "../FindSelectOptions";
export class Et2SelectState extends Et2StaticSelectMixin(Et2Select)
{
@ -32,8 +32,11 @@ export class Et2SelectState extends Et2StaticSelectMixin(Et2Select)
set countryCode(code : string)
{
this.__countryCode = code;
this._static_options = <SelectOption[]>StaticOptions.state(this, {country_code: code});
this.requestUpdate("select_options");
this.fetchComplete = so.state(this, {country_code: code}).then((options : SelectOption[]) =>
{
this._static_options = cleanSelectOptions(options);
this.requestUpdate();
});
}
set_country_code(code)

22
api/js/etemplate/Et2Select/StaticOptions.ts
View File

@ -90,6 +90,24 @@ export const Et2StaticSelectMixin = <T extends Constructor<Et2WidgetWithSelect>>
}
}
/**
* Override the parent _missingOption to wait for server-side options
* to come back before we check to see if the value is not there.
*
* @param {string} newValueElement
* @protected
*/
protected _missingOption(newValueElement : string)
{
this.fetchComplete.then(() =>
{
if(this.optionSearch(newValueElement) == null)
{
super._missingOption(newValueElement);
}
})
}
/**
* Override the parent fix_bad_value() to wait for server-side options
* to come back before we check to see if the value is not there.
@ -381,10 +399,10 @@ export const StaticOptions = new class StaticOptionsType
return this.cached_server_side(widget, 'select-country', options, return_promise);
}
state(widget : Et2SelectWidgets, attrs) : SelectOption[] | Promise<SelectOption[]>
state(widget : Et2SelectWidgets, attrs) : Promise<SelectOption[]>
{
var options = attrs.country_code ? attrs.country_code : 'de';
return this.cached_server_side(widget, 'select-state', options);
return <Promise<SelectOption[]>>this.cached_server_side(widget, 'select-state', options, true);
}
dow(widget : Et2SelectWidgets, attrs) : Promise<SelectOption[]>