インラインJavaScriptを正しく使用する

Question

私はOneSignalというサービスを使用して、デスクトップおよびモバイルデバイスにプッシュ通知を配信しています。ボタンのインラインonclick javascriptイベントを指定できるように、この要素に直接使用できるオプションのインラインJavaScriptをトリガーするためにボタン要素を使用しようとしています。私が開発チームによって与えられたjavascriptは以下の通りです：

var OneSignal = OneSignal || []; OneSignal.push(function() { OneSignal.registerForPushNotifications(); }); 

私は要素に追加しましたが、機能していないようです。彼らはまた、成功したサブスクリプションのリダイレクト機能を提供してくれまし：

var OneSignal = OneSignal || []; 
OneSignal.push(function() { 
OneSignal.on('subscriptionChange', function(isSubscribed) { 
if (isSubscribed) { 
    location.href = "http://redirecturlhere.com"; 
} 
}); 
}); 

ボタン要素は、Visual Composerのの一部です：CSS/HTMLの私の知識は良いですが、私のjavascriptの知識

View button element.

多くのことが望まれています。誰かが私を正しい方向に向けることができるのか、コード内の何かを指摘できるか、あるいはこの機能を妨害する可能性のあるコードを適用したかどうかは疑問だ。

Answer 1

OneSignal.on('subscriptionChange', function(isSubscribed) 

通常、subscriptionChangeは、要素のIDまたはクラスのいずれかを指します。

は、期間は通常クラスを示しているので、通常はそうでない.subscriptionChangeを試してみてくださいIDです意味#subscriptionChangeを試してみてください。

JSで

Answer 2

括弧またはインライン関数を閉じるとき、あなたはセミコロンを追加しないので、それはこのように終わるでしょう：

var OneSignal=OneSignal||[];OneSignal.push(function(){OneSignal.on("subscriptionChange",function(a){a&&(location.href="http://redirecturlhere.com")})});