Ubuntu

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

Reported by Daniel Ellis on 2009-10-08
162
This bug affects 35 people
Affects Status Importance Assigned to Milestone
Ubuntu Server papercuts
Undecided
Unassigned
phpLDAPadmin
Confirmed
Undecided
Unassigned
Declined for 1.1.x by Deon George
phpldapadmin (Debian)
Fix Released
Unknown
phpldapadmin (Ubuntu)
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.

Daniel Ellis (danellisuk) wrote :
Brian Murray (brian-murray) wrote :

Formatted patch.

Changed in phpldapadmin (Ubuntu):
status: New → Triaged
importance: Undecided → High

still persisting in final karmic

Changed in phpldapadmin (Debian):
importance: Undecided → Unknown
status: New → Unknown

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
KAMI (kami911) wrote :

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

KAMI

KAMI (kami911) wrote :

It seems okay.

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

This bug has had a fix released!

Changed in phpldapadmin:
importance: Unknown → Undecided
status: Unknown → New
status: New → Fix Released
C de-Avillez (hggdh2) on 2010-02-12
Changed in phpldapadmin:
status: Fix Released → Confirmed
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

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

Tomasz Czapiewski (xeros) wrote :

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

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) {

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.

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

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.

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.

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
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 ?

Thierry Carrez (ttx) wrote :

Already fixed

Changed in server-papercuts:
status: New → Fix Released
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)

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) on 2010-09-02
Changed in phpldapadmin (Ubuntu):
status: Fix Released → Confirmed
status: Confirmed → Fix Released
Sergey Svishchev (svs) wrote :

Will it be fixed in 9.10?

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  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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