Edit report at http://bugs.php.net/bug.php?id=42060&edit=1
ID: 42060 Comment by: sriram dot natarajan at gmail dot com 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 Private report: N New Comment: I don't think this patch will be applied against 5.2 branch. since 5.2 release train is in security bug fix only. however, we can ask if this patch can be applied against 5.3. thanks for the patch. Previous Comments: ------------------------------------------------------------------------ [2010-11-18 22:34:54] jeanseb at au-fil-du dot net Pierre ? PS : I have build the patch on PHP 5.2 : http://www.au-fil-du.net/public/php/php_ldap.PHP5.2.14-withPaginationSupport.Win32-VC6-x86.zip ------------------------------------------------------------------------ [2010-11-04 22:20:35] jeanseb at au-fil-du dot net That's it. You can apply ext-ldap-review.patch and api-rename.patch on trunk. For api-rename.patch, I should do some svn add & svn del on the phpt file. I have rename the 3 phpt but the svn diff seems to fail to create a good patch. ------------------------------------------------------------------------ [2010-11-03 23:04:53] jeanseb at au-fil-du dot net I have updated the initial patch (php-trunk_ldap-pagination.patch) to reflect this api change and i have integrated the commit of felipe @303159. Tomorrow I'll integrate my review patch (ext-ldap-review.patch) to the initial one and I'll rebuild one, based on trunk. ------------------------------------------------------------------------ [2010-11-03 13:20:46] paj...@php.net yes, please do, then I can apply the patch to trunk ------------------------------------------------------------------------ [2010-11-03 12:47:59] jeanseb at au-fil-du dot net "resp" means "response" We can rename the API in ldap_control_paged_result & ldap_control_paged_result_response ------------------------------------------------------------------------ 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
