Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
From: scottmac@php.net Date: Wed, 24 Aug 2011 06:08:15 +0000 Subject: Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more References: 1 Groups: php.bugs Request: Send a blank email to php-bugs+get-162893@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=42060&edit=1
ID: 42060
Updated by: scottmac@php.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:
I fixed the tests, singular made more sense.
Previous Comments:
------------------------------------------------------------------------
[2011-08-24 05:54:58] jeanseb at au-fil-du dot net
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
------------------------------------------------------------------------
[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
------------------------------------------------------------------------
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
Thread (75 messages)
- #42060 [Opn->Asn]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Asn->Fbk]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Fbk->Opn]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Opn->Fbk]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [NoF->Opn]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Opn]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Opn->Fbk]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [NoF->Asn]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Asn]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Com]: [PATCH] Add pagedResults support and more (PAT18 and PAT19 updated for 5.2.3)
- #42060 [Asn]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Asn]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Asn->Csd]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Csd]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Csd->Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Com]: [PATCH] LDAP: Add pagedResults support and more
- Req #42060 [Fbk->Csd]: [PATCH] LDAP: Add pagedResults support and more
| « previous | php.bugs (#162893) | next » |
|---|