Hi Rouslan,<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I personally wouldn't call that refactoring, but yeah, the clean-up<br>
looks pretty OK. Just one thing: when you are calling<br>
PMA_ajaxShowMessage() to display an error, you need to pass a second<br>
parameter with a false value. For example:<br>
<br>
PMA_ajaxShowMessage(data.error, false);<br>
<br></blockquote><div><br></div><div>okay. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
More info at [1].<br>
<div class="im"><br></div>I never said that you should not remove the dialog, you're just doing it<br>
wrong. The correct way to do this is by defining a callback that gets<br>
fired when the dialog is closed (this may happen in more than one way)<br>
and remove the dialog there. See example below.<br><br>That because the buttons are not where they belong. They should go into<br>
the bottom pane of the dialog. For example:<br>
<br>
// Define some buttons that will go into the bottom pane<br>
// of the dialog and bind some callbacks to them<br>
var buttonOptions = {};<br>
buttonOptions[PMA_messages['strGo']] = function () {<br>
    // do something useful here<br>
};<br>
buttonOptions[PMA_messages['strClose']] = function () {<br>
    $(this).dialog("close");<br>
};<br>
// Create the dialog<br>
$someDiv.dialog({<br>
    buttons: buttonOptions,<br>
    close: function () {<br>
        // This is a callback that gets fired when the user<br>
        // or the system try to close the dialog<br>
        $(this).remove();<br>
    }<br>
});<br>
<div class="im"><br></div></blockquote><div><br></div><div>Thanks for the clarification. Now I got the point and will update the patch. </div><div><br></div><div>Regards,</div><div>Thilanka. </div><div><br></div><div><br>
</div></div>-- <br><div><span style="border-collapse:collapse"><div style="text-align:left"><b><font><span style="color:rgb(255,102,0);font-family:arial;font-size:small;letter-spacing:normal">Thilanka Kaushalya,</span><br>
</font></b><span style="color:rgb(80,0,80);font-family:arial,sans-serif;font-size:medium;white-space:nowrap">                          </span><b><font><br></font></b></div><font face="arial, sans-serif">Connect with me at,</font><br>
<font face="arial, sans-serif">      Gmail        : </font><a href="mailto:lgtkaushalya@gmail.com" style="font-family:arial,sans-serif;font-size:13px;color:rgb(0,0,204)" target="_blank">lgtkaushalya@gmail.com</a><br><font face="arial, sans-serif">      Twitter       : @thilanka_k</font><br>
<font face="arial, sans-serif">      Facebook  : lgtkaushalya</font><br><font face="arial, sans-serif">      Yahoo       : </font><a href="mailto:lgtkaushalya@yahoo.com" style="font-family:arial,sans-serif;font-size:13px;color:rgb(0,0,204)" target="_blank">lgtkaushalya@yahoo.com</a></span></div>
<div>      Blog          : <a href="http://coders-view.blogspot.com/" target="_blank">http://coders-view.blogspot.com/</a></div><br><br>