From 326e54e45aa05031936adb446b11c0d6400d25d2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Thu, 2 Jun 2022 18:29:03 +0200 Subject: [PATCH] Web: fix job & task subscriptions Making the UUID optional (rF221ed17fa27193086ec5c9201d48f7cd826e75a4) made the generated JavaScript code ignore the 3rd parameter. Apparently the constructor parameters are only for the mandatory properties, and the rest has to be set on a per-property basis. --- web/app/src/components/UpdateListener.vue | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/web/app/src/components/UpdateListener.vue b/web/app/src/components/UpdateListener.vue index e8bfe7e4..fde55c15 100644 --- a/web/app/src/components/UpdateListener.vue +++ b/web/app/src/components/UpdateListener.vue @@ -193,7 +193,8 @@ export default { * @param {string} jobID */ _updateJobSubscription(operation, jobID) { - const payload = new API.SocketIOSubscription(operation, "job", jobID); + const payload = new API.SocketIOSubscription(operation, "job"); + payload.uuid = jobID; console.log(`sending job ${operation}:`, payload); this.socket.emit("/subscription", payload); }, @@ -204,7 +205,8 @@ export default { * @param {string} jobID */ _updateTaskLogSubscription(operation, taskID) { - const payload = new API.SocketIOSubscription(operation, "tasklog", taskID); + const payload = new API.SocketIOSubscription(operation, "tasklog"); + payload.uuid = taskID; console.log(`sending tasklog ${operation}:`, payload); this.socket.emit("/subscription", payload); },