Ah, you’re absolutely right. It’s just unnecessary, and actually plain wrong, to include all the parameters within a POST request.

I’ve updated the code and released a new fix version: DOMAssistant 2.5.6 released - AJAX module post method has been fixed.

The current way still works for short params, but you will essentially be sending over GET as well as POST data in one request. For requests with extremely long params (which is the reason why POST is being used), it might break.

No problem, I should have done it before.

chenghong,

Thanks!
I appreciate you letting me know! It was ok in my code repository, but in the offered download, it was initially wrong. When I tried to fix that, Google code mangled the end of the file again.

Therefore, I changed the namestandard (something I’ve wanted to do anyway) so that there’s a dot (.) instead of a hyphen (.) in the file name, i.e. DOMAssistantComplete.2.2.5.js instead of DOMAssistantComplete-2.2.5.js.

For future downloads, this should never be a problem anyway. Lesson learned: never try to update a file with the same filename; instead, use new names for modified files.

And, you’re absolutely right about what’s posted in the post request, and I’ve seen to that.

Thanks for your help!

You may want to check the ‘complete’ file. It is missing the DOMAssistant.Load module.

Also, shouldn’t the last line in the makeCall function be
XMLHttp.send((method==="GET")? null : params[1]);
?

Thanks! Sloppy me.

Yes, the POST method would indeed be just a little bit more code. When you mentioned it, I think I really should’ve added it in this version. Therefore, I just coded it together and uploaded a new version where every element and the DOMAssistant.AJAX object also has a post method.

This post (above) is updated with more info.

I like the AJAX header you send with AJAX calls! I’ve already found it useful.

Still wish there was a POST method in AJAX for unsafe interactions, though. It could even be an extra parameter like .get(URL, callBack, {method : "POST"}). That wouldn’t increase the code TOO much, would it?