#42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)

From: Date: Mon, 07 Jan 2008 15:51:34 +0000
Subject: #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-121108@lists.php.net to get a copy of this message 
 ID:               42060
 Comment by:       fkhan at emergen dot biz
 Reported By:      iarenuno at eteo dot mondragon dot edu
 Status:           No Feedback
 Bug Type:         LDAP related
 Operating System: Any
 PHP Version:      5.2.3
 Assigned To:      jani
 New Comment:

I cannot get the patches to compile on the latest CVS head. It
complains of a missing zstr typedef (which is actually not there). I
have checked from php 5.2.0 upwards and this is not defined- should i do
it manually in zend.h?

/php5/Zend    -I/usr/include -g -O2  -c
/usr/local/src/php5/ext/ldap/ldap.c -o ext/ldap/ldap.lo 
/usr/local/src/php5/ext/ldap/ldap.c: In function 'php_ldap_do_modify':
/usr/local/src/php5/ext/ldap/ldap.c:1731: error: 'zstr' undeclared
(first use in this function)
/usr/local/src/php5/ext/ldap/ldap.c:1731: error: (Each undeclared
identifier is reported only once
/usr/local/src/php5/ext/ldap/ldap.c:1731: error: for each function it
appears in.)
/usr/local/src/php5/ext/ldap/ldap.c:1731: error: expected ';' before
'attribute'
/usr/local/src/php5/ext/ldap/ldap.c:1790: error: 'attribute' undeclared
(first use in this function)
make: *** [ext/ldap/ldap.lo] Error 1


Previous Comments:
------------------------------------------------------------------------

[2007-11-21 11:33:02] avel+php at noc dot uoa dot gr

I intend to write a wrapper function for PEAR::Pager and LDAP, to make
paged results more accessible.

Without this patch, paged results from LDAP are not possible. [Well,
they _are_ possible, if one loads the entire result set in memory. :-| 
]

How come is this patch's status "No Feedback"? What is the PHP
developers' stand on this patch as of now?

Alexandros Vellis

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

[2007-09-24 12:58:06] avel+php at noc dot uoa dot gr

This is a _killer_ feature that we have been waiting for a long time. 
White pages apps written in PHP, with large directories behind them,
desparetely need the paging feature -- and not only them!

The API seems intuitive to me so far. I'm eager for this to be included
in PHP 5.3.

Thanks for the patch.

Alexandros Vellis

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

[2007-09-07 18:40:25] ando at sys-net dot it

A working patch with test examples lined up to HEAD as of 5 minutes ago
is available here

<http://www.sys-net.it/~ando/Download/#PHP>

I'm the original developer, and I'm willing to cooperate in integrating
it, if needed.  So you can re-open the bug now.

p.

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

[2007-09-07 14:59:29] ando at sys-net dot it

Yes, so many functions are needed to support generic stuff + dozens of
well-known, RFC/I.D. based LDAP controls and extended operations,
without code duplication.  Any developer would agree.

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

[2007-09-07 14:57:40] ando at sys-net dot it

Be serious: the patch applies back up to PHP 4 (the last time an update
occurred to LDAP support).

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

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/42060

-- 
Edit this bug report at http://bugs.php.net/?id=42060&edit=1

Thread (75 messages)

« previous php.bugs (#121108) next »