saurabh
/
adbanner-test
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Replace inline onclick with data-track-click-url attribute (CSP compatibility, requires orbit-kernel core update)

This commit is contained in:
rulingcom 2026-07-31 15:44:16 +08:00
parent 06c6a100ce
commit 63d9214082
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/controllers/ad_banners_controller.rb
View File

@ -43,7 +43,7 @@ class AdBannersController < ApplicationController
target = ""
end
if !b.out_link.blank?
image_link += "\" style=\"cursor:pointer;\" onclick=\"if(navigator.sendBeacon){navigator.sendBeacon('/ad_banners/track_click/#{b.id}');}\" onload=\"var el=this;(function(){if((el.parentElement==undefined||el.parentElement.tagName!='A')&&$('<p>'+$(el).parents('.w-ba-banner__slide').eq(0).attr('data-overlay-template')+'</p>').find('a').length==0){el.outerHTML=('<a href=#{b.out_link} target=#{target}>'+el.outerHTML+'</a>');}})()\""
image_link += "\" style=\"cursor:pointer;\" data-track-click-url=\"/ad_banners/track_click/#{b.id}\" onload=\"var el=this;(function(){if((el.parentElement==undefined||el.parentElement.tagName!='A')&&$('<p>'+$(el).parents('.w-ba-banner__slide').eq(0).attr('data-overlay-template')+'</p>').find('a').length==0){el.outerHTML=('<a href=#{b.out_link} target=#{target}>'+el.outerHTML+'</a>');}})()\""
end
context = (b.context.tr('"',"'") rescue "")
images << {