Skip to content

Commit a97118d

Browse files
TASK-1828883: cleaning simple-table.component.js
1 parent 3e6dac6 commit a97118d

File tree

4 files changed

+8
-78
lines changed

4 files changed

+8
-78
lines changed

core/src/commonMain/kotlin/com/pega/constellation/sdk/kmp/core/components/containers/ModalViewContainer.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import kotlinx.serialization.json.JsonObject
1515

1616
class ModalViewContainerComponent(context: ComponentContext) : ContainerComponent(context) {
1717
var visible by mutableStateOf(false)
18+
private set
1819
var title by mutableStateOf("")
1920
private set
2021
var cancelButtonLabel by mutableStateOf("")

scripts/dxcomponents/components/containers/templates/simple-table.component.js

Lines changed: 6 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -60,101 +60,31 @@ export class SimpleTableComponent extends BaseComponent {
6060
this.props.visible = this.utils.getBooleanValue(configProps.visibility);
6161
}
6262

63-
const {
64-
multiRecordDisplayAs,
65-
label: labelProp,
66-
propertyLabel,
67-
displayMode,
68-
fieldMetadata,
69-
hideLabel,
70-
parameters,
71-
isDataObject,
72-
type,
73-
ruleClass,
74-
authorContext,
75-
name
76-
} = configProps;
77-
const label = labelProp || propertyLabel;
63+
const { multiRecordDisplayAs, fieldMetadata } = configProps;
7864

79-
let {contextClass} = configProps;
65+
let { contextClass } = configProps;
8066
if (!contextClass) {
8167
let listName = this.pConn.getComponentConfig().referenceList;
8268
listName = PCore.getAnnotationUtils().getPropertyName(listName);
8369
contextClass = this.pConn.getFieldMetadata(listName)?.pageClass;
8470
}
85-
if (multiRecordDisplayAs === "fieldGroup") { // table, simpleTable
86-
const fieldGroupProps = {...configProps, contextClass};
71+
if (multiRecordDisplayAs === "fieldGroup") {
72+
const fieldGroupProps = { ...configProps, contextClass };
8773
this.childComponent = this.componentsManager.upsert(this.childComponent, "FieldGroupTemplate", [
8874
this.pConn,
8975
fieldGroupProps,
9076
]);
9177
this.#sendPropsUpdate();
9278
} else if (fieldMetadata && fieldMetadata.type === 'Page List' && fieldMetadata.dataRetrievalType === 'refer') {
93-
const {
94-
children: [{children: rawFields}],
95-
parameters: rawParams
96-
} = (this.pConn.getRawMetadata()).config;
97-
const isDisplayModeEnabled = displayMode === 'DISPLAY_ONLY';
98-
const propsToUse = {label, ...this.pConn.getInheritedProps()};
99-
if (isDisplayModeEnabled && hideLabel) {
100-
propsToUse.label = '';
101-
}
102-
103-
const metaForListView = buildMetaForListView(
104-
fieldMetadata,
105-
rawFields,
106-
type,
107-
ruleClass,
108-
name,
109-
propsToUse.label,
110-
isDataObject,
111-
parameters // resolved params
112-
);
113-
114-
const metaForPConnect = JSON.parse(JSON.stringify(metaForListView));
115-
// @ts-ignore - PCore.getMetadataUtils().getPropertyMetadata - An argument for 'currentClassID' was not provided.
116-
metaForPConnect.config.parameters = rawParams ?? PCore.getMetadataUtils().getPropertyMetadata(name)?.datasource?.parameters;
117-
118-
const {referenceListStr: referenceList} = getContext(this.pConn);
119-
let requiredContextForQueryInDisplayMode = {};
120-
if (isDisplayModeEnabled) {
121-
requiredContextForQueryInDisplayMode = {
122-
referenceList
123-
};
124-
}
125-
const options = {
126-
context: this.pConn.getContextName(),
127-
pageReference: this.pConn.getPageReference(),
128-
...requiredContextForQueryInDisplayMode
129-
};
130-
131-
const listViewPConn = PCore.createPConnect({meta: metaForPConnect, options}).getPConnect();
132-
133-
const listViewProps = {
134-
...metaForListView.config,
135-
displayMode,
136-
fieldName: authorContext
137-
};
138-
// if (multiRecordDisplayAs === "simpleTable") {
139-
//
140-
// }
141-
this.childComponent = this.componentsManager.upsert(this.childComponent, "ListView", [
142-
listViewPConn,
143-
listViewProps,
144-
]);
145-
this.#sendPropsUpdate();
79+
console.warn(TAG, 'Displaying ListView in SimpleTable is not supported yet.');
14680
} else {
147-
// console.warn(
148-
// TAG,
149-
// `Unsupported display mode: ${multiRecordDisplayAs}. ListView and SimpleTableManual are not supported yet.`
150-
// );
15181
this.childComponent = this.componentsManager.upsert(this.childComponent, "SimpleTableManual", [this.pConn]);
15282
this.#sendPropsUpdate();
15383
}
15484
}
15585

15686
onEvent(event) {
157-
// TODO: remove optional call when other modes are implemented so that child component is always defined
87+
// TODO: remove optional call when all other modes are implemented so that child component is always defined
15888
this.childComponent?.onEvent(event);
15989
}
16090

scripts/dxcomponents/components/containers/templates/template-utils.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
export const TABLE_CELL = 'SdkRenderer';
2-
export const DELETE_ICON = 'DeleteIcon';
32
const PRIMARY_FIELDS = 'pyPrimaryFields';
43

54
export function getReferenceList(pConn) {

ui-renderer-cmp/src/commonMain/kotlin/com/pega/constellation/sdk/kmp/ui/renderer/cmp/containers/ModalViewContainerRenderer.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ class ModalViewContainerRenderer : ComponentRenderer<ModalViewContainerComponent
3333
onDismissRequest = {
3434
onCancelClick()
3535
}
36-
) {
36+
) {
3737
Card(
3838
modifier = Modifier.fillMaxWidth(),
3939
) {

0 commit comments

Comments
 (0)