E_STRICT: Declaration of AJAXTree::draw_dn() should be compatible with that of PLMTree::draw_dn()

Bug #446669 reported by Daniel Ellis
162
This bug affects 35 people
Affects Status Importance Assigned to Milestone
Ubuntu Server papercuts
Fix Released
Undecided
Unassigned
phpLDAPadmin
Confirmed
Undecided
Unassigned
Declined for 1.1.x by Deon George
phpldapadmin (Debian)
Fix Released
Unknown
phpldapadmin (Ubuntu)
Fix Released
High
Unassigned
Nominated for Jaunty by Peter M
Nominated for Karmic by grimko
Nominated for Lucid by Gerry C.

Bug Description

Binary package hint: phpldapadmin

After installing phpldapadmin on Ubuntu Karmic Server Beta (x86). The web interface is unusable due to the error:-

E_STRICT: Declaration of AJAXTree::draw_dn() should be compatible with that of PLMTree::draw_dn()

Searching for this error found an answer at http://sourceforge.net/tracker/?func=detail&atid=498546&aid=2809461&group_id=61828

I managed to get my installation to work with the very simple change to lib/AJAXTree.php. The diff is attached.

Revision history for this message
Daniel Ellis (danellisuk) wrote :
Revision history for this message
Brian Murray (brian-murray) wrote :

Formatted patch.

Changed in phpldapadmin (Ubuntu):
status: New → Triaged
importance: Undecided → High
Revision history for this message
waldmeister (6-launchpad-computerdichter-de) wrote :

still persisting in final karmic

Changed in phpldapadmin (Debian):
importance: Undecided → Unknown
status: New → Unknown
Revision history for this message
michael perigard (overprescribed) wrote :

confirmed still happening in karmic final. this is a show stopper for me, possibly severe or critical importance? this bug renders phpldapadmin nonfunctional with the latest php and phpldapadmin packages in karmic. I'm not sure what the proper procedure is from here; suggest another 1.1.x release from the debian maintainer? create another ubuntu package with the patch applied and submit it as a necessary update to karmic since phpldapadmin is unusable with latest stable package 1.1.0.7-1ubuntu1 ?

Changed in phpldapadmin (Debian):
status: Unknown → Confirmed
Revision history for this message
KAMI (kami911) wrote :

Same problem here. I will try the provided patch an come back with the results.

KAMI

Revision history for this message
KAMI (kami911) wrote :

It seems okay.

Revision history for this message
GoofY (goofy-dse) wrote :

Bug is stll happening in LUCID.
Solution though is quite simple.
As stated above change in /usr/share/phpldapadmin/lib/AJAXTree.php line 16 ->

protected function draw_dn($dn,$level=0,$first_child=true,$last_child=true) {

in

protected function draw_dn($dn,$level,$first_child=true,$last_child=true) {

Save & you're good to go.

Changed in phpldapadmin (Ubuntu):
status: Triaged → Confirmed
Revision history for this message
Gerry C. (ubuntunerd-nospammail) wrote :

This bug has had a fix released!

Changed in phpldapadmin:
importance: Unknown → Undecided
status: Unknown → New
status: New → Fix Released
C de-Avillez (hggdh2)
Changed in phpldapadmin:
status: Fix Released → Confirmed
Revision history for this message
bernhard (bernhardredl) wrote :

this bug is still open in LUCID alpha2 10.04
but the level=0 fix works.

#lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu lucid (development branch)
Release: 10.04
Codename: lucid

#apt-cache policy phpldapadmin
phpldapadmin:
  Installed: 1.1.0.7-1ubuntu1
  Candidate: 1.1.0.7-1ubuntu1
  Version table:
 *** 1.1.0.7-1ubuntu1 0
        500 http://at.archive.ubuntu.com lucid/universe Packages
        100 /var/lib/dpkg/status

Revision history for this message
bernhard (bernhardredl) wrote :

i checked the upstream repo:
it was been fixed in
ea17aadef46fd29850160987fe7740ceed1381ad
http://phpldapadmin.git.sourceforge.net/git/gitweb.cgi?p=phpldapadmin/phpldapadmin;a=commit;h=ea17aadef46fd29850160987fe7740ceed1381ad

this had been released in RELEASE-1.2.0-rc2

Revision history for this message
Tomasz Czapiewski (xeros) wrote :

This bug still exists on up-to-date karmic.

Revision history for this message
Marek (mlb) wrote :

Yes, the bug still exists in karmic.

Manual workaroud is to edit:

/usr/share/phpldapadmin/lib/AJAXTree.php

and change

 protected function draw_dn($dn,$level=0,$first_child=true,$last_child=true) {

to

 protected function draw_dn($dn,$level,$first_child=true,$last_child=true) {

Revision history for this message
Ro (robert-markula) wrote :

Confirmed that the patch works.

Come on, it's just a single line to edit that finally fixes the whole package.

Revision history for this message
Samuli Seppänen (samuli-seppanen) wrote :

Problem still present on up-to-date Ubuntu 9.10 32-bit desktop edition. The patch fixes this issue.

Revision history for this message
michael perigard (overprescribed) wrote :

As far as I can tell, the fix for this bug was committed after the 1.1.0.7 branch was released. Upstream has already fixed this in the 1.2 branch. I see that 1.1.0 is still being used in Lucid, so... this bug will persist for the next six months unless someone creates a patch at the debian/ubuntu level? Just hoping this gets in front of someone who can do something about it before lucid is released.

Revision history for this message
Igor Wawrzyniak (igor-tumus) wrote :

I nominated this bug for server papercuts. The fix is trivial and the issue is annoying to everyone trying to use this package.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package phpldapadmin - 1.1.0.7-1.2ubuntu2

---------------
phpldapadmin (1.1.0.7-1.2ubuntu2) lucid; urgency=low

  * debian/patches/fix-ajax-estrict-declaration.dpatch: Fix E_STRICT
    warning when loading phpldapadmin interface. (LP: #446669)
 -- Chuck Short <email address hidden> Thu, 01 Apr 2010 10:28:09 -0400

Changed in phpldapadmin (Ubuntu):
status: Confirmed → Fix Released
description: updated
Revision history for this message
larsmith217 (larsmith217) wrote :

How do I access / download / install the fix for the "E_STRICT: Declaration of AJAXTree::draw_dn() should be compatible with that of PLMTree::draw_dn()" error with phpLDAPAdmin ?

Revision history for this message
Thierry Carrez (ttx) wrote :

Already fixed

Changed in server-papercuts:
status: New → Fix Released
Revision history for this message
Bandicoot (andrew-beals) wrote :

"Already fixed" but still exists in fully-patched as of about 90 minutes ago 32-bit Lucid Lynx (9.10).

Yes, the patch works. Still, users shouldn't have to apply it in order to get this package to work.
(Or just nuke the durn package)

Revision history for this message
Thierry Carrez (ttx) wrote :

In Lucid Lynx (10.04) or Karmic Koala (9.10) ?

The bug is supposed to have been fixed in the lucid (10.04) packages, released at version 1.1.0.7-1.2ubuntu2. It's not fixed in 9.10.

anand (nith-anandan)
Changed in phpldapadmin (Ubuntu):
status: Fix Released → Confirmed
status: Confirmed → Fix Released
Revision history for this message
Sergey Svishchev (svs) wrote :

Will it be fixed in 9.10?

Revision history for this message
Clint Byrum (clint-fewbar) wrote : Re: [Bug 446669] Re: E_STRICT: Declaration of AJAXTree::draw_dn() should be compatible with that of PLMTree::draw_dn()

Excerpts from Sergey Svishchev's message of Tue Apr 26 15:26:28 UTC 2011:
> Will it be fixed in 9.10?
>

9.10 will reach EOL in just 4 days, so no.

https://lists.ubuntu.com/archives/ubuntu-release/2011-March/000183.html

Changed in phpldapadmin (Debian):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.