Skip to content

Commit

Permalink
Merge pull request #3501 from bettyblocks/acceptance
Browse files Browse the repository at this point in the history 
Release 2024-10-11
  • Loading branch information
@ewoutquax
ewoutquax authored Oct 11, 2024
2 parents 8dc21b0 + 323ed59 commit fa6bdbe
Show file tree
Hide file tree
Showing 4 changed files with 290 additions and 293 deletions.
26 changes: 25 additions & 1 deletion CHANGELOG.md
View file
Original file line number Diff line number Diff line change
@@ -1,9 +1,33 @@
## [2.193.1](https://github.com/bettyblocks/material-ui-component-set/compare/v2.193.0...v2.193.1) (2024-10-07)
## [2.193.2](https://github.com/bettyblocks/material-ui-component-set/compare/v2.193.1...v2.193.2) (2024-10-11)


### Bug Fixes

* add default value to current filter and show remaining records ([2a51f91](https://github.com/bettyblocks/material-ui-component-set/commit/2a51f919d97cc3c189b7259d3a6d9deda4b833bc))
* datepicker can interpret default meta-data data ([c56ecb3](https://github.com/bettyblocks/material-ui-component-set/commit/c56ecb39c78bf70050d2647cd3c6c0f71f8f0f1e))
* deepcopy optionFilter in autocomplete ([b777209](https://github.com/bettyblocks/material-ui-component-set/commit/b777209a4b011df97391741d2476ee82f7b99323))
* fixed filter in the autocomplete ([a6c71d3](https://github.com/bettyblocks/material-ui-component-set/commit/a6c71d313b0516e08a701c61920e6f359f15a160))
* force floatingLabel when a value is present in dataPicker ([d2997db](https://github.com/bettyblocks/material-ui-component-set/commit/d2997db89d5fe9a794186bfdc7ff205ce4873638))
* give datePickerField a default value of null ([028183e](https://github.com/bettyblocks/material-ui-component-set/commit/028183e2456b05dbebe9ca908c29f54988a8df06))
* parse min- and maxDate correctly ([4baa79e](https://github.com/bettyblocks/material-ui-component-set/commit/4baa79e6a2a64de49253f08d4193674c6025cd58))
* pipelines ([d72e9ec](https://github.com/bettyblocks/material-ui-component-set/commit/d72e9ec380e289999528d109b857505a841c9336))
* use empty string as fallback ([0a2d6a9](https://github.com/bettyblocks/material-ui-component-set/commit/0a2d6a9c22d855ee65c3f91913c9a78ac099ba33))

## [2.193.1](https://github.com/bettyblocks/material-ui-component-set/compare/v2.193.0...v2.193.1) (2024-10-11)


### Bug Fixes

* add default value to current filter and show remaining records ([2a51f91](https://github.com/bettyblocks/material-ui-component-set/commit/2a51f919d97cc3c189b7259d3a6d9deda4b833bc))
* datepicker can interpret default meta-data data ([c56ecb3](https://github.com/bettyblocks/material-ui-component-set/commit/c56ecb39c78bf70050d2647cd3c6c0f71f8f0f1e))
* deepcopy optionFilter in autocomplete ([b777209](https://github.com/bettyblocks/material-ui-component-set/commit/b777209a4b011df97391741d2476ee82f7b99323))
* fixed filter in the autocomplete ([a6c71d3](https://github.com/bettyblocks/material-ui-component-set/commit/a6c71d313b0516e08a701c61920e6f359f15a160))
* force floatingLabel when a value is present in dataPicker ([d2997db](https://github.com/bettyblocks/material-ui-component-set/commit/d2997db89d5fe9a794186bfdc7ff205ce4873638))
* give datePickerField a default value of null ([028183e](https://github.com/bettyblocks/material-ui-component-set/commit/028183e2456b05dbebe9ca908c29f54988a8df06))
* **PAGE-4710:** Remove unused zone from publish prod files. ([05e1bbf](https://github.com/bettyblocks/material-ui-component-set/commit/05e1bbfe41028b96f0960f51e61c5f8c860bc23e))
* parse min- and maxDate correctly ([4baa79e](https://github.com/bettyblocks/material-ui-component-set/commit/4baa79e6a2a64de49253f08d4193674c6025cd58))
* pipelines ([d72e9ec](https://github.com/bettyblocks/material-ui-component-set/commit/d72e9ec380e289999528d109b857505a841c9336))
* use empty string as fallback ([0a2d6a9](https://github.com/bettyblocks/material-ui-component-set/commit/0a2d6a9c22d855ee65c3f91913c9a78ac099ba33))

# [2.193.0](https://github.com/bettyblocks/material-ui-component-set/compare/v2.192.0...v2.193.0) (2024-10-04)

Expand Down
2 changes: 1 addition & 1 deletion package.json
View file
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "component-set",
"version": "2.193.1",
"version": "2.193.2",
"main": "dist/templates.json",
"license": "UNLICENSED",
"private": false,
Expand Down
166 changes: 81 additions & 85 deletions src/components/autocompleteInput.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -90,14 +90,14 @@
const required = defaultRequired;
const label = useText(labelRaw);
const defaultValue = useText(valueRaw, { rawValue: true });
const initalValue = defaultValue.replace(/\n/g, '');
const [value, setValue] = useState(initalValue);
const initialValue = defaultValue.replace(/\n/g, '');
const [value, setValue] = useState(initialValue);
const [debouncedInputValue, setDebouncedInputValue] = useState();
const [debouncedCurrentValue, setDebouncedCurrentValue] = useState();
const [interactionFilter, setInteractionFilter] = useState({});
const defaultValueEvaluatedRef = useRef(false);
const { current: labelControlRef } = useRef(generateUUID());
const isNumberType = type === 'number';
const EMPTY_LABEL = '-- empty --';

const validPattern = pattern || null;
const validMinlength = minlength || null;
Expand All @@ -112,19 +112,20 @@
const belowMinimumMessage = useText(validationBelowMinimum);
const helperTextResolved = useText(helperTextRaw);
const modelProperty = getProperty(property || '') || {};
const labelProperty = getProperty(labelPropertyId) || {};
const { modelId: propertyModelId, referenceModelId } = modelProperty;
const { contextModelId } = model;
const modelId =
contextModelId || referenceModelId || propertyModelId || model || '';
const idProperty = getIdProperty(modelId) || {};
const labelProperty = getProperty(labelPropertyId) || idProperty;
const propertyModel = getModel(modelId);
const defaultLabelProperty =
getProperty(
propertyModel && propertyModel.labelPropertyId
? propertyModel.labelPropertyId
: '',
) || {};
const idProperty = getIdProperty(modelId) || {};

const isListProperty =
modelProperty.kind === 'LIST' || modelProperty.kind === 'list';

Expand All @@ -144,11 +145,9 @@

useEffect(() => {
if (defaultValue) {
defaultValueEvaluatedRef.current = false;
setValue(initalValue);
setValue(initialValue);
setInputValue('');
} else {
defaultValueEvaluatedRef.current = true;
setValue('');
setInputValue('');
setDebouncedInputValue('');
Expand Down Expand Up @@ -291,7 +290,7 @@
const optionFilter = useFilter(filterRaw || {});

// We need to do this, because options.filter is not immutable
let filter = { ...optionFilter };
let filter = JSON.parse(JSON.stringify(optionFilter));

const searchPropIsNumber = numberPropTypes.includes(searchProp.kind);
const valuePropIsNumber = numberPropTypes.includes(valueProp.kind);
Expand All @@ -318,80 +317,92 @@
/*
* Build up array for relational label
*/
const idOrPathLabel =
typeof labelPropertyId.id !== 'undefined'
? labelPropertyId.id
: labelPropertyId;
const labelPropertyPath =
typeof idOrPathLabel === 'string' ? [idOrPathLabel] : idOrPathLabel;
const labelPropertyPath = labelPropertyId.id
? labelPropertyId.id
: [labelProperty.id];

/*
* We extend the option filter with the value of the `value` state and the value of the `inputValue` state.
*
* Those values always need to be returned in the results of the request
*/
/* eslint-disable no-underscore-dangle */
if (
debouncedInputValue &&
(searchPropIsNumber
? parseFloat(debouncedInputValue, 10)
: debouncedInputValue) ===
(typeof value === 'string' ? value : value[searchProp.name])
) {
filter._or = [
{
[searchProp.name]: {
[searchPropIsNumber ? 'eq' : 'matches']: searchPropIsNumber
? parseFloat(debouncedInputValue, 10)
: debouncedInputValue,
},

// After searching or setting a default value
if (debouncedInputValue) {
// ["relational_property_name", "property_name"]
const propertyNames = labelPropertyPath.map((u) => getProperty(u).name);

// get nested value like this:
// value["relational_property_name"]["property_name"]
const valueFromProperyNames = propertyNames.reduce(
(acc, propertyName) => acc && acc[propertyName],
value,
);

const numberDebouncedInputValue = parseFloat(debouncedInputValue, 10);
const parsedDebouncedValue = searchPropIsNumber
? numberDebouncedInputValue
: debouncedInputValue;

const searchPropertyValue =
typeof value === 'string' ? value : valueFromProperyNames;

// When you select a property with a null value, use an empty string as fallback to prevent doesNotMatch: null
const isEmptySearchValue =
searchPropertyValue === null || searchPropertyValue === undefined;

const currentSearchValue =
isEmptySearchValue && !searchPropIsNumber ? '' : searchPropertyValue;

// Create the opposite filter of the current value to fetch other records
const remainingRecordsFilter = {
[valueProp.name]: {
[valuePropIsNumber ? 'neq' : 'doesNotMatch']:
typeof value === 'string' ? value : value[valueProp.name],
},
};

// When no keys in filter, use remaining records filter to fetch other records
const currentFilter =
Object.keys(filter).length > 0 || value === ''
? filter
: remainingRecordsFilter;

// Create filter with property name(s) as key(s)
const inputFilter = propertyNames.reduceRight(
(acc, propertyName) => ({ [propertyName]: acc }),
{
[valueProp.name]: {
neq: valuePropIsNumber
? parseFloat(value[valueProp.name], 10)
: value[valueProp.name],
},
},
];
} else if (debouncedInputValue) {
// if a relation is selected for label option
if (labelPropertyPath.length > 1) {
const newFilter = {
[labelPropIsNumber ? 'eq' : 'matches']: labelPropIsNumber
? parseFloat(debouncedInputValue, 10)
: debouncedInputValue,
};
const resolvedUuids = labelPropertyPath.map((u) => getProperty(u).name);
const resolvedFilter = resolvedUuids.reduceRight((acc, q) => {
return { [q]: acc };
}, newFilter);
filter = { ...filter, ...resolvedFilter };
} else {
filter[searchProp.name] = {
[searchPropIsNumber ? 'eq' : 'matches']: searchPropIsNumber
? parseFloat(debouncedInputValue, 10)
? numberDebouncedInputValue
: debouncedInputValue,
};
}
} else if (value !== '') {
filter._or = [
{
[valueProp.name]: {
[valuePropIsNumber ? 'eq' : 'matches']:
typeof value === 'string' ? value : value[valueProp.name],
},
},
];
);

if (defaultValueEvaluatedRef.current) {
filter._or.push({
[valueProp.name]: {
neq: typeof value === 'string' ? value : value[valueProp.name],
// When searching, the search value is different from the value
const newSearchValue = parsedDebouncedValue !== currentSearchValue;
const operator = newSearchValue ? '_and' : '_or';

filter = {
[operator]:
debouncedInputValue === EMPTY_LABEL
? [currentFilter]
: [inputFilter, currentFilter],
};
} else if (value !== '') {
// Use default value in filter to show it on render
filter = {
_or: [
{
[valueProp.name]: {
[valuePropIsNumber ? 'eq' : 'matches']:
typeof value === 'string' ? value : value[valueProp.name],
},
},
});
}
],
};
}

/* eslint-enable no-underscore-dangle */

/*
Expand Down Expand Up @@ -513,28 +524,13 @@
message = 'Something went wrong while loading.';
}

// If the default value is a value that lives outside the take range of the query we should fetch the values before we continue.
if (!isDev && !defaultValueEvaluatedRef.current && value && results) {
setValue((prev) => {
return (
results.find(
(result) =>
result[valueProp.name] &&
prev[valueProp.name] === result[valueProp.name],
) || ''
);
});

defaultValueEvaluatedRef.current = true;
}

B.defineFunction('Clear', () => {
setValue('');
setInputValue('');
setDebouncedInputValue('');
});

B.defineFunction('Reset', () => setValue(initalValue));
B.defineFunction('Reset', () => setValue(initialValue));

B.defineFunction('Refetch', () => refetch());

Expand Down Expand Up @@ -703,7 +699,7 @@
}

return optionLabel === '' || optionLabel === null
? '-- empty --'
? EMPTY_LABEL
: optionLabel.toString();
};

Expand Down
Loading

0 comments on commit fa6bdbe

Please sign in to comment.