[phpldapadmin] [ phpldapadmin-Bugs-2954027 ] "Create one?" button not working

Bugs item #2954027, was opened at 2010-02-18 17:04
Message generated for change (Comment added) made by wurley
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=2954027&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: Open
Resolution: None
Priority: 5
Private: No
Submitted By: https://www.google.com/accounts ()
Assigned to: Nobody/Anonymous (nobody)
Summary: "Create one?" button not working

Initial Comment:
Brand new install of phpLDAPadmin 1.2.0.5 with new OpenLDAP server. I log in with the web interface and see this: "This base entry does not exist.Create it?" I click on Create it and nothing happens, and I receive a javascript error in Firebug. 

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

Comment By: Deon George (wurley)
Date: 2010-02-18 20:43

Message:
This artifact has been marked as a duplicate of artifact 2887656 with
reason:
No explanation provided.

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

Comment By: kipiop (kipiop)
Date: 2010-02-18 20:24

Message:
Hum... Sorry for my early enthusiasm, this patch create another error .. I
continu working on it and I'll post a solution later.

-- 
Sorry, no sig.


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

Comment By: kipiop (kipiop)
Date: 2010-02-18 20:05

Message:
Hi there,

I need this functionnality so I sought a solution and found it.
This is maybe not the best solution but it works...

Here is the patch :

-------------- PATCH BEGIN -----------------
@@ -105,6 +105,12 @@
                                                echo '</div></td></tr>';

                                        echo '</table>';
+                                       static $js_drawn = false;
+
+                                       if (! $js_drawn) {
+                                               $this->draw_javascript();
+                                               $js_drawn = true;
+                                       }
                                        return;

                                } else {
@@ -133,7 +139,6 @@
                                        die(sprintf('Error: %s hasnt been
configured for auth_type %s',__METHOD__,$server->getAuthType()));
                        }
                }
-
                # Tree Footer.
                echo '</table>';
                echo "\n\n";
@@ -532,7 +537,7 @@
        /**
         * If there is javascript, draw it
         */
-       protected function draw_javascript() {
+       private function draw_javascript() {
                if (DEBUG_ENABLED &&
(($fargs=func_get_args())||$fargs='NOARGS'))
                        debug_log('Entered
(%%)',33,0,__FILE__,__LINE__,__METHOD__,$fargs);
-------------- PATCH END -----------------


-- 
Sorry, no sig.

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

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

------------------------------------------------------------------------------
Download Intel&reg; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs 
proactively, and fine-tune applications for parallel performance. 
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
______________________________________
phpLDAPadmin development mailing list.
To unsbuscribe: https://lists.sourceforge.net/lists/listinfo/phpldapadmin-devel
http://phpldapadmin.sourceforge.net/