From 7a3d154447f3dc14f1ab60d1aa20d697d22d7e7b Mon Sep 17 00:00:00 2001 From: Florianschmidtwelzow Date: Sat, 10 Jan 2015 03:09:41 +0100 Subject: [PATCH] Mobile: Conform to new LoggingSchema MobileFrontend recently introduced a new loggingSchema structure. Match here, too. Bug: T77929 Change-Id: Ie863937566fb894860d705f51a04ab30de76140d --- modules/ext.thanks.mobilediff.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/modules/ext.thanks.mobilediff.js b/modules/ext.thanks.mobilediff.js index d19d3153..7e3e3c8a 100644 --- a/modules/ext.thanks.mobilediff.js +++ b/modules/ext.thanks.mobilediff.js @@ -1,7 +1,8 @@ ( function( M, $ ) { var api = M.require( 'api' ), popup = M.require( 'toast' ), - schema = M.require( 'loggingSchemas/MobileWebClickTracking' ); + SchemaMobileWebClickTracking = M.require( 'loggingSchemas/SchemaMobileWebClickTracking' ), + schema = new SchemaMobileWebClickTracking( {}, 'MobileWebDiffClickTracking' ); function thankUser( name, revision, gender ) { var d = $.Deferred(); @@ -57,7 +58,10 @@ ) .on( 'click', function() { var $this = $( this ); - schema.log( 'Diff', 'thank', name ); + schema.log( { + name: 'thank', + destination: name + } ); if ( !$this.hasClass( 'thanked' ) ) { thankUser( name, rev, gender ).done( function() { $this.addClass( 'thanked' ).attr( 'disabled', true )