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

Edit report at https://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 Private report: N New Comment: With Pierre we agreed to use singular form. Could someone with karma fix this ? https://bugs.php.net/bug.php?id=42060&edit=2#1288788479 Previous Comments: ------------------------------------------------------------------------ [2011-08-23 15:55:40] peng1can at gmail dot com Is this plural/singular typo the only thing holding this patch up? Can't a choice just be made and move on? ------------------------------------------------------------------------ [2011-06-02 11:48:21] salathe@php.net The implemented functions are currently called "ldap_control_paged_result[_response]" but the tests try to use the plural "results" names. Which are we going to keep, Scott is there any reason you chose the singular names? ------------------------------------------------------------------------ [2011-05-19 19:42:24] scottmac@php.net I applied the rename patch and tidied up the code a little. Anything else that needs done here? ------------------------------------------------------------------------ [2011-05-19 19:41:24] scottmac@php.net Automatic comment from SVN on behalf of scottmac Revision: http://svn.php.net/viewvc/?view=revision&revision=311264 Log: Tidy up ldap paging code and rename the API as discussed in #42060 ------------------------------------------------------------------------ [2011-05-18 14:33:32] jeanseb at au-fil-du dot net Can we expect to see ext-ldap-review.patch and api-rename.patch applied on trunk and PHP5.4 branch ? Thanks. ------------------------------------------------------------------------ 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 https://bugs.php.net/bug.php?id=42060 -- Edit this bug report at https://bugs.php.net/bug.php?id=42060&edit=1