Skip to content
  1. Oct 30, 2012
  2. Oct 29, 2012
  3. Oct 28, 2012
  4. Oct 27, 2012
    • Kohsuke Kawaguchi's avatar
      [FIXED JENKINS-15617] · 1c578ab5
      Kohsuke Kawaguchi authored
      execScript cannot return value [1], so I'm restricting its use to where
      we don't care about the return value. See ticket for more discussions
      
      [1] http://msdn.microsoft.com/en-us/library/ie/ms536420(v=vs.85).aspx
      1c578ab5
    • Jesse Glick's avatar
      49646bfb
    • Jesse Glick's avatar
      addLoadEvent failed to pass on the Event object. · 52274032
      Jesse Glick authored
      Does not matter for any handlers used in Jenkins, but can break third-party frameworks.
      In particular mixpanel-2.1.min.js seems to detect when this mistake is made and somehow disables some Jenkins handlers,
      causing e.g. “LOADING” to stay on job config pages and breaking behavior.js (but only on Firefox).
      
      Using addEventListener/attachEvent where available also works, but this fix is a bit simpler and more general.
      https://gist.github.com/1930440 would also work but again seems like overkill here.
      
      To reproduce:
      diff --git a/core/src/main/resources/lib/layout/layout.jelly b/core/src/main/resources/lib/layout/layout.jelly
      index 1fb3fec..c1337c4 100644
      --- a/core/src/main/resources/lib/layout/layout.jelly
      +++ b/core/src/main/resources/lib/layout/layout.jelly
      @@ -88,6 +88,19 @@ ${h.initPageVariables(context)}
       <x:doctype name="html" />
       <html>
         <head>
      +<script type="text/javascript"><![CDATA[
      +(function(c,a){window.mixpanel=a;var b,d,h,e;b=c.createElement("script");
      +    b.type="text/javascript";b.async=!0;b.src=("https:"===c.location.protocol?"https:":"http:")+
      +    '//cdn.mxpnl.com/libs/mixpanel-2.1.min.js';d=c.getElementsByTagName("script")[0];
      +    d.parentNode.insertBefore(b,d);a._i=[];a.init=function(b,c,f){function d(a,b){
      +    var c=b.split(".");2==c.length&&(a=a[c[0]],b=c[1]);a[b]=function(){a.push([b].concat(
      +    Array.prototype.slice.call(arguments,0)))}}var g=a;"undefined"!==typeof f?g=a[f]=[]:
      +    f="mixpanel";g.people=g.people||[];h=['disable','track','track_pageview','track_links',
      +    'track_forms','register','register_once','unregister','identify','name_tag',
      +    'set_config','people.identify','people.set','people.increment'];for(e=0;e<h.length;e++)d(g,h[e]);
      +    a._i.push([b,c,f])};a.__SV=1.1;})(document,window.mixpanel||[]);
      +    mixpanel.init("…something valid here…");
      +]]></script>
           ${h.checkPermission(it,permission)}
      
           <title>${h.appendIfNotNull(title, ' [Jenkins]', 'Jenkins')}</title>
      52274032
    • Kohsuke Kawaguchi's avatar
      [FIXED JENKINS-9615] · d56ba241
      Kohsuke Kawaguchi authored
      Integrated a fix made in Stapler.
      d56ba241
  5. Oct 26, 2012
  6. Oct 25, 2012
  7. Oct 24, 2012
Loading