Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:64941 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 26085 invoked from network); 14 Jan 2013 18:12:03 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 14 Jan 2013 18:12:03 -0000 Authentication-Results: pb1.pair.com header.from=scott@macvicar.net; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=scott@macvicar.net; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain macvicar.net from 209.85.220.44 cause and error) X-PHP-List-Original-Sender: scott@macvicar.net X-Host-Fingerprint: 209.85.220.44 mail-pa0-f44.google.com Received: from [209.85.220.44] ([209.85.220.44:55040] helo=mail-pa0-f44.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A5/36-22727-27A44F05 for ; Mon, 14 Jan 2013 13:12:03 -0500 Received: by mail-pa0-f44.google.com with SMTP id hz11so2360845pad.31 for ; Mon, 14 Jan 2013 10:12:00 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:content-type:mime-version:subject:from:in-reply-to:date :cc:content-transfer-encoding:message-id:references:to:x-mailer :x-gm-message-state; bh=qOj10Ow4PRtDq67dn43EnhLDvgekulGGPQ2Ly1ynbVI=; b=kcPA3e2C80XcNq4ju19gUDUyldMt8IbYJd7rLTWJd5XhWK880myzdRX1HZemiI7dz8 z5PkTYJ3NwrNaZIAkMNX1WY1Qd1Fk3E1cqT/+DNbb9gCLFe5Urng0i9SMEvfq0PiqbOJ OxcqBBXUlBlQ5ziN/Lxh+cC585DWL+Z582sr/manRenO+2nLX6n51i49UBrpaoMYwAEM l+ObW8zylwTbhG0LDqL9FleAZB9bkH4Ylhekz15/PVat456GDJimlREeXvQAV1Qv425a SaYGreQcWol27xFeEBxh97L3xWA22FGdBrtvKftI0Ev6U5tlrF3MfNmJ+GwjhWjTe6YA wm5g== X-Received: by 10.68.222.232 with SMTP id qp8mr77613503pbc.99.1358187120077; Mon, 14 Jan 2013 10:12:00 -0800 (PST) Received: from ?IPv6:2620:0:1cfe:99::8? ([2620:0:1cfe:99::8]) by mx.google.com with ESMTPS id vx2sm8584364pbc.33.2013.01.14.10.11.58 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 14 Jan 2013 10:11:59 -0800 (PST) Content-Type: text/plain; charset=iso-8859-1 Mime-Version: 1.0 (Mac OS X Mail 6.2 \(1499\)) In-Reply-To: <50F44726.3030706@php.net> Date: Mon, 14 Jan 2013 13:11:54 -0500 Cc: internals@lists.php.net Content-Transfer-Encoding: quoted-printable Message-ID: <9BAEFC3A-4440-45FB-92CD-16CB2B13D1F0@macvicar.net> References: <42E4F513-0412-440D-BBBB-589F2A2463BD@macvicar.net> <50F44726.3030706@php.net> To: Ben Ramsey X-Mailer: Apple Mail (2.1499) X-Gm-Message-State: ALoCoQkbbzJRb6gzmNjv9slcWTRbcPtbVPxqvuysV/Ir3yDiGyt9iazO/8S+buDbuktdm0SNar3s Subject: Re: [PHP-DEV] Re: [VOTE] array_column() function From: scott@macvicar.net (Scott MacVicar) On 14 Jan 2013, at 12:57, Ben Ramsey wrote: > On 1/14/13 11:54 AM, Scott MacVicar wrote: >> On 14 Jan 2013, at 11:37, Ben Ramsey wrote: >>=20 >>> On 1/11/13 6:17 PM, Ben Ramsey wrote: >>>> I've opened voting for the array_column() function RFC. >>>>=20 >>>> You can vote at https://wiki.php.net/rfc/array_column#voting >>>=20 >>> I have updated the pull request by removing the array_pluck() alias. = I favor the array_column() alias, as it is more in keeping with other = function names like array_keys() and array_values(), which do not have = verbs in them. >>>=20 >>> If the RFC is accepted, I will submit a new pull request against the = 5.5 branch and close the current pull request against the 5.3 branch. If = I should do that now, please let me know, and I'll go ahead and take = care of it. >>=20 >> I've talked about the Facebook version of this when the key bit got = added, but we have 2 other methods to go with it. >>=20 >> mpull() which does methods >> = https://github.com/facebook/libphutil/blob/master/src/utils/utils.php#L110= >>=20 >> ppull() which does properties >> = https://github.com/facebook/libphutil/blob/master/src/utils/utils.php#L184= >>=20 >> Not sure if these should be added separately? The property version is = good for PDO when using FETCH_CLASS. >>=20 >> - S >>=20 >=20 > I did review these, and I like them, but I think they should be added = separately, especially since they deal more with objects than arrays. As long as the work with objects that implement ArrayAccess then it = should be fine? - S=