Troubleshooting (Script Authors): Difference between revisions

From GreaseSpot Wiki
Jump to navigationJump to search
Marti (talk | contribs)
m →‎Variables change before a GM_xmlhttpRequest or setTimeout callback runs: Marked as "Bad Example" since this is what it is showing not to do... code validated... also \n strippage discourages c/p
Marti (talk | contribs)
Line 18: Line 18:
Problem example:
Problem example:


{{Bad example|<nowiki>for (var i = 0; i < document.links.length; i++) {
{{Bad example|1=<pre style="border: none; margin: inherit;">for (var i = 0; i < document.links.length; i++) {
   var link = document.links[i];
   var link = document.links[i];
    
    
Line 28: Line 28:
     }
     }
   });
   });
}</nowiki>}}
}</pre>}}


The request is asynchronous, meaning the rest of the code doesn't wait for it to complete. The <code>for</code> keeps running as the request loads. When the request for the first link on the page completes and the <code>onload</code> callback function runs, the <code>link</code> variable might point to a different link altogether, typically the last one (since the <code>for</code> loop completes much quicker than the HTTP requests).
The request is asynchronous, meaning the rest of the code doesn't wait for it to complete. The <code>for</code> keeps running as the request loads. When the request for the first link on the page completes and the <code>onload</code> callback function runs, the <code>link</code> variable might point to a different link altogether, typically the last one (since the <code>for</code> loop completes much quicker than the HTTP requests).

Revision as of 09:36, 6 May 2009

The article Avoid Common Pitfalls in Greasemonkey covers some of the most common problems people come across when writing user scripts.

If your problem is not listed, please ask on the mailing list.

Changes don't take effect when editing a script

You may be editing your original copy of the script, not the installed copy. Once you create and install a script, you need to follow these steps to make sure you're editing the installed, active copy.

Also, changing the include and exclude rules in the metadata block of the installed script does not do anything, as the script metadata is only accessed during installation. The script must be re-installed (or config.xml edited manually) for these changes to take.

Note: You can change the include and exclude rules in the Manage User Scripts window and those changes will take effect.

Variables change before a GM_xmlhttpRequest or setTimeout callback runs

Problem example:

Template:Bad example

The request is asynchronous, meaning the rest of the code doesn't wait for it to complete. The for keeps running as the request loads. When the request for the first link on the page completes and the onload callback function runs, the link variable might point to a different link altogether, typically the last one (since the for loop completes much quicker than the HTTP requests).

One solution is to pass those values that you want unchanged into a function surrounding the request, as arguments. Then they will be in a different scope, and will no longer be changed from the outside.

Solution example with named function:

function do_it(link_inside) {
  GM_xmlhttpRequest({
    method:"GET",
    url:"http://example.com/lookup?url=" + link_inside.href,
    onload:function(result) {
      link_inside.href = result.responseText;
    }
  });
}
  
for (var i = 0; i < document.links.length; i++) {
  var link = document.links[i];
  
  do_it(link);
}

Solution example with anonymous function:

for (var i = 0; i < document.links.length; i++) {
  var link = document.links[i];
  
  (function (link_inside) {
  
    GM_xmlhttpRequest({
      method:"GET",
      url:"http://example.com/lookup?url=" + link_inside.href,
      onload:function(result) {
        link_inside.href = result.responseText;
      }
    });
  
  })(link);
}

Component not Available errors

These are usually caused by attempting to access an element's event handlers (onclick, onkeydown, etc.) directly, rather than through addEventListener. GreaseMonkey 0.5 and above require the use of addEventListener. For details, see XPCNativeWrapper.

Form POST data

When using GM_xmlhttpRequest to POST form data, remember to add the Content-Type parameter:

GM_xmlhttpRequest({
  method:"POST",
  url:myurl,
  headers:{
    "User-Agent":navigator.userAgent,
    "Content-Type":"application/x-www-form-urlencoded"
  },
  data:mydata,
  onload:function(res) { /* Some code */ }
});

Links