EBernhardson has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/179983

Change subject: Dont pass along schema as an eventlogging argument
......................................................................

Dont pass along schema as an eventlogging argument

Flow's eventlogging tracking code needs to hold onto this schema
value for internal processing, but not pass it along to EL.  Simple
change should be easy to review.

Fixes T78577

Bug: T78577
Change-Id: I5b6c7c40be61a536b9c8b18ed978e6ce25303244
---
M modules/engine/components/common/flow-component-events.js
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow 
refs/changes/83/179983/1

diff --git a/modules/engine/components/common/flow-component-events.js 
b/modules/engine/components/common/flow-component-events.js
index f5edc41..66f1569 100644
--- a/modules/engine/components/common/flow-component-events.js
+++ b/modules/engine/components/common/flow-component-events.js
@@ -558,8 +558,9 @@
                        }
                }
 
-               // Forward is not loggable data!
+               // Forward and schema are not loggable data!
                delete eventInstance.forward;
+               delete eventInstance.schema;
 
                // Log the event
                eventInstance = component.logEvent( data.flowEventlogSchema, 
eventInstance );

-- 
To view, visit https://gerrit.wikimedia.org/r/179983
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b6c7c40be61a536b9c8b18ed978e6ce25303244
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: EBernhardson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to