Skip to content

Commit fdee7ee

Browse files
committed
fix: based on code comment and feedback
1 parent 7cdf9fc commit fdee7ee

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

packages/pluggableWidgets/combobox-web/src/Combobox.editorConfig.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,10 @@ const ASSOCIATION_SOURCE_CONFIG: Array<keyof ComboboxPreviewProps> = [
3535
"attributeAssociation"
3636
];
3737

38-
export function getProperties(values: ComboboxPreviewProps, defaultProperties: Properties): Properties {
38+
export function getProperties(
39+
values: ComboboxPreviewProps & { Editability?: unknown },
40+
defaultProperties: Properties
41+
): Properties {
3942
if (values.source !== "database") {
4043
hidePropertiesIn(defaultProperties, values, ["customEditability", "customEditabilityExpression"]);
4144
}
@@ -133,7 +136,6 @@ export function getProperties(values: ComboboxPreviewProps, defaultProperties: P
133136
}
134137
if (values.databaseAttributeString.length === 0) {
135138
hidePropertiesIn(defaultProperties, values, ["optionsSourceDatabaseValueAttribute"]);
136-
// @ts-expect-error ignore error due to the Editability is part of system properties
137139
hidePropertiesIn(defaultProperties, values, ["Editability"]);
138140
if (values.customEditability !== "conditionally") {
139141
hidePropertiesIn(defaultProperties, values, ["customEditabilityExpression"]);

packages/pluggableWidgets/combobox-web/src/helpers/Database/DatabaseSingleSelectionSelector.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,6 @@ export class DatabaseSingleSelectionSelector<T extends string | Big, R extends E
6060
}
6161
this._attr = targetAttribute as R;
6262
this.readOnly = getReadonly(targetAttribute, props.customEditability, props.customEditabilityExpression);
63-
console.log("readOnly", this.readOnly);
6463
this.lazyLoader.updateProps(ds);
6564
this.lazyLoader.setLimit(
6665
this.lazyLoader.getLimit(ds.limit, this.readOnly, targetAttribute?.status ?? ds.status, lazyLoading)

packages/pluggableWidgets/combobox-web/src/helpers/Database/utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ export function getReadonly(
9999
return true;
100100
}
101101
if (customEditability === "conditionally") {
102-
return customEditabilityExpression.value ?? false;
102+
return customEditabilityExpression.value ?? true;
103103
}
104104
return false;
105105
}

0 commit comments

Comments
 (0)