jenkins-bot has submitted this change and it was merged.

Change subject: Watch and unwatch had the no-js experience
......................................................................


Watch and unwatch had the no-js experience

Due to a small javascript error, the watch and unwatch
were following the anchor tag and providing the no-js
experience.

Bug: T100406
Change-Id: I000395606aa1a90934f443d44a6613bad6113ac1
---
M modules/engine/misc/flow-api.js
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Matthias Mullie: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/engine/misc/flow-api.js b/modules/engine/misc/flow-api.js
index 1256d80..39ce2f3 100644
--- a/modules/engine/misc/flow-api.js
+++ b/modules/engine/misc/flow-api.js
@@ -240,7 +240,9 @@
                // We will do a separate view request afterwards, so we don't 
want view data
                // to be returned from POST.
                // Can be removed when 
Iddd999536d1c37eedf5c61938e0c2b2f4bc66827 is merged.
-               queryMap[apiTransformMap[queryMap.submodule][0] + 
'metadataonly'] = true;
+               if ( apiTransformMap[queryMap.submodule] ) {
+                       queryMap[apiTransformMap[queryMap.submodule][0] + 
'metadataonly'] = true;
+               }
 
                // Default action is flow
                queryMap.action = queryMap.action || 'flow';

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I000395606aa1a90934f443d44a6613bad6113ac1
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Sbisson <[email protected]>
Gerrit-Reviewer: Matthias Mullie <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to