Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more

From: Date: Wed, 13 Oct 2010 08:27:06 +0000
Subject: Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-155137@lists.php.net to get a copy of this message 
Edit report at http://bugs.php.net/bug.php?id=42060&edit=1

 ID:                 42060
 Comment by:         jeanseb at au-fil-du dot net
 Reported by:        iarenuno at eteo dot mondragon dot edu
 Summary:            [PATCH] LDAP: Add pagedResults support and more
 Status:             Feedback
 Type:               Feature/Change Request
 Package:            *General Issues
 Operating System:   *
 PHP Version:        5CVS, 6CVS (2008-11-01)
 Assigned To:        pajoye
 Block user comment: N

 New Comment:

I have build the ldap extension for PHP 5.2 on WinXP and i can build it
for PHP 5.2 or 5.3 on linux if you want.

But it's without any support.


Previous Comments:
------------------------------------------------------------------------
[2010-10-13 10:24:02] jeanseb at au-fil-du dot net

I'm trying to do some lobbying arround this, without success for the
moment.

------------------------------------------------------------------------
[2010-10-06 10:47:55] jonas dot ranerfors at gmail dot com

Chance for this to be in next release? 5.3.4?

------------------------------------------------------------------------
[2010-08-17 07:48:51] qmt9z3 at yahoo dot com

Hello,
Any plans to fold this into the main stable releases?
Appreciate update on when this is going to become
main stream.

Thanks

------------------------------------------------------------------------
[2010-08-04 17:59:30] jeanseb at au-fil-du dot net

I've attached a patch solving this.

In ldap_ctrl_paged_results_resp the 3rd and 4th args are string & int
passed by reference. 

I've defined those args as zval ("zz"), should I change this to string &
int "si" too ?

------------------------------------------------------------------------
[2010-06-16 20:29:05] johannes@php.net

Quick review of the patch shows this part:

	switch (myargcount) {
+		case 4:
+			convert_to_string_ex(&cookie);
+			lcookie.bv_val = Z_STRVAL_PP(&cookie);
+			lcookie.bv_len = Z_STRLEN_PP(&cookie);
+			/* fallthru */

Why is it done this way? Shouldn't "rl|bz" in zend_parse_parameters be
changed to "rl|bs"? I assume the current form will cause trouble with
references and stuff which can't be converted to string (object without
__toString() method)

Didn't do a deeper review.

As we're rolling 5.3.3 very soon i'd keep it out there for now.

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


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/bug.php?id=42060


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

Thread (75 messages)

« previous php.bugs (#155137) next »