Skip to content

Commit 7ce362f

Browse files
committed
Side effect
1 parent 87c6144 commit 7ce362f

File tree

7 files changed

+14
-13
lines changed

7 files changed

+14
-13
lines changed

CHANGELOG.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,11 @@ All notable changes to this project will be documented in this file.
55
The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
66
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
77

8-
## Version 1.9.2 - 2025-12-xx
8+
## Version 1.9.2 - 2025-12-01
99

1010
### Fixed
1111

12+
- Fix side effect event name and source properties
1213
- Queue/outbox websocket services automatically when configured accordingly in `cds.requires`
1314
- Document how to handle websocket paths in different scenarios
1415

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1208,15 +1208,15 @@ Example:
12081208
@ws.pcp.sideEffect
12091209
@ws.pcp.channel: 'amc\://notification/notify'
12101210
event sideEffect {
1211-
sideEffectsSource: String;
1211+
sideEffectSource: String;
12121212
}
12131213
```
12141214
12151215
Emitting the event `sideEffect` via CDS emit, as follows:
12161216
12171217
```js
12181218
await srv.emit("sideEffect", {
1219-
sideEffectsSource: "/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')",
1219+
sideEffectSource: "/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')",
12201220
});
12211221
```
12221222
@@ -1225,8 +1225,8 @@ It results in the following PCP message sent via websocket protocol:
12251225
```
12261226
pcp-action:MESSAGE
12271227
pcp-channel:amc:\:notification/notify
1228-
sideEffectsSource:/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')
1229-
sideEffectsEventName:sideEffect
1228+
sideEffectSource:/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')
1229+
sideEffectEventName:sideEffect
12301230
serverAction:RaiseSideEffect
12311231

12321232

src/format/pcp.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,8 @@ class PCPFormat extends GenericFormat {
117117
}
118118
if (pcpSideEffect) {
119119
pcpFieldsFiltered = {
120-
sideEffectsSource: "",
121-
sideEffectsEventName: pcpEvent,
120+
sideEffectSource: "",
121+
sideEffectEventName: pcpEvent,
122122
serverAction: "RaiseSideEffect",
123123
...pcpFieldsFiltered,
124124
};

test/_env/srv/handlers/pcp.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ module.exports = (srv) => {
5454

5555
srv.on("triggerSideEffects", async () => {
5656
await srv.emit("sideEffect1", {
57-
sideEffectsSource: "/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')",
57+
sideEffectSource: "/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')",
5858
});
5959
return true;
6060
});

test/_env/srv/pcp.cds

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,6 @@ service PCPService {
5353
@ws.pcp.event: 'sideEffect1'
5454
@ws.pcp.channel: 'amc\://notification/notify'
5555
event sideEffect1 {
56-
sideEffectsSource: String;
56+
sideEffectSource: String;
5757
}
5858
}

test/socketio/pcp_socket.io.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ const pcpTriggerSideEffects = `pcp-action:triggerSideEffects
7171

7272
const pcpSideEffect1 = `pcp-action:MESSAGE
7373
pcp-channel:amc\\://notification/notify
74-
sideEffectsSource:/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')
75-
sideEffectsEventName:sideEffect1
74+
sideEffectSource:/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')
75+
sideEffectEventName:sideEffect1
7676
serverAction:RaiseSideEffect
7777
7878
`;

test/ws/pcp_ws.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,8 @@ const pcpTriggerSideEffects = `pcp-action:triggerSideEffects
6868

6969
const pcpSideEffect1 = `pcp-action:MESSAGE
7070
pcp-channel:amc\\://notification/notify
71-
sideEffectsSource:/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')
72-
sideEffectsEventName:sideEffect1
71+
sideEffectSource:/Header(ID='e0582b6a-6d93-46d9-bd28-98723a285d40')
72+
sideEffectEventName:sideEffect1
7373
serverAction:RaiseSideEffect
7474
7575
`;

0 commit comments

Comments
 (0)