[phpldapadmin] [ phpldapadmin-Bugs-3370546 ] AjaxEnabled create and delete entry fails on IE9

Bugs item #3370546, was opened at 2011-07-19 05:02
Message generated for change (Comment added) made by wurley
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=3370546&group_id=61828

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Doug Campbell (chinadoug)
Assigned to: Nobody/Anonymous (nobody)
Summary: AjaxEnabled create and delete entry fails on IE9

Initial Comment:
After installing phpLDAPadmin 1.2.1.1 today, I tried creating/deleting a directory entry in IE9 and FF4.  IE9 would never perform the task, it just would say Creating or Deleting object but never finish.  FF4 worked flawlessly.

After a little research I found the problem to be in the ajSUBMIT function of htdocs/js/ajax_functions.js

The line:

makeHttpRequest('cmd.php',getParameters(obj.parentNode)+'meth=ajax','POST','alertAJ','cancelAJ',div);

should be moved before the section that displays the progress message.  I think this is needed because the progress message destroys the information that the getParameters(obj.parentNode) function needs.

The avSUBMIT() function should looks as follows:

function ajSUBMIT(div,obj,display) {
        var pageDiv = getDiv(div);

        window.scrollTo(0,95);

        makeHttpRequest('cmd.php',getParameters(obj.parentNode)+'meth=ajax','POST','alertAJ','cancelAJ',div);

        if (pageDiv)
                includeHTML(pageDiv,'<img src="images/ajax-progress.gif"><br><small>'+display+'...</small>');
        else
                return true;

        return false;
}


I have tested this fix on IE9 and now everything works as it should.

----------------------------------------------------------------------

>Comment By: Deon George (wurley)
Date: 2011-10-06 09:13

Message:
Thanks for identifying this and providing a solution :D

Committed to GIT as #d5744b0

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=3370546&group_id=61828

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
______________________________________
phpLDAPadmin development mailing list.
To unsbuscribe: https://lists.sourceforge.net/lists/listinfo/phpldapadmin-devel
http://phpldapadmin.sourceforge.net/