Hey !

We need some code to work with collections of objects. Sometimes we have gets and sets methods to work like ->name(), or get_name(), set_name(), getName()., getAge() , ..........

The underscore php need to be prepered to work with this methods. like: _::pluck($collection, $nameOfMethod, $arrayParams.)

It seems the list() function is being used incorrectly:

list($one, $two) = array('one')

While this produces what is usually the desired result in the code ($one is set to 'one' and $two is set to NULL), this isn't technically a correct use of list() and produces an undefined offset notice.

From http://php.net/manual/en/language.oop5.magic.php (and later http://us3.php.net/manual/en/userlandnaming.rules.php)

"PHP reserves all function names starting with __ as magical. It is recommended that you do not use function names with __ in PHP unless you want some documented magic functionality."

This does make me a bit reluctant about using the library. As far as I know there are no existing conflicts, but I worry about the potential down the road.

Provides the option of passing the first argument by reference, directly modifying the object.

Current syntax: $object = array('k1' => 'a2'); $object = __::extend($object, array('k1' => 'v1', 'k2' => 'v2')); // $object === array('k1'=>'v1', 'k2' => 'v2');

With changes: $object = array('k1' => 'a2'); __::extend(&$object, array('k1' => 'v1', 'k2' => 'v2')); OR $object = __::extend($object, array('k1' => 'v1', 'k2' => 'v2')); // $object === array('k1'=>'v1', 'k2' => 'v2');

I saw that isRegExp is not implemented yet. I have once before needed to know during run time whether or not a string is a regular expression string or not, and after a lot of research, I failed quite epicly. So I turned to stackoverflow where user ThiefMaster gave me some code that works:

ini_set('track_errors', 'on');
$php_errormsg = '';
@preg_match('/[blah/', '');
if($php_errormsg) echo 'regex is invalid';

Which works. Maybe this can be implemented as isRegExp? I'm not sure if it's a good idea though, hence why I'm asking.

Alternatively, maybe interoperability with a library like Flux?

Adds in support for 'bind' and 'bindAll', although 'bindAll' is not as fancy as it is in underscore.js. It needs to be used with the list construct.

Ex:

<?php
require('underscore.php');

$f = function($passed=null) {
    echo $passed . "\n";
};

$f2 = function($passed=null) {
    var_dump($this);
};

$f =  __::bind($f, json_decode('{"property":"value"}'), 'passed by __', $f);
$f2 = __::bind($f2, json_decode('{"property":"value"}'), 'passed', $f);

$f('i passed this directly');
$f();
$f2();
$a = json_decode('{"property":"value"}');

$f = function() {
    $this->property = 'nope';
};

$f2 = function($passed=null) {
    var_dump($this);
};

list($f, $f2) = __::bindAll($a, $f, $f2);

$f();
$f2();
var_dump($a);

This seems to have fixed the issues with this ticket. I also added the isNull function which the docs claim this has, but it's actually missing.

Like the ticket says, I basically made a proxy class that handles the static and non static methods yet keeps the chaining functionality.

@brianhaveri since it seems you have abandoned this package, would you consider transferring the ownership to someone else to maintain? I think that @thephpleague would consider taking it over.

Although I understand that objects of type ArrayAccess don't and shouldn't behave the same way as arrays, it would be great if pluck would support objects of type ArrayAccess just like any other arrays or objects.

This commit supports that behavior and contains the additional unit tests for that.

In PHP it's possible to implement a custom array class, by implementing the ArrayAccess interface. That way you can use any instance of your class as be it an array. We had our own implementation of pluck which could handle elements that did this. But underscore's pluck doesn't seem to be able to handle these custom array types.

The line from our own implementation that checked for keys existence in elements, was:

if (isset($element[$key])) { // Must use isset instead of array_key_exists, because the latter does not work correctly on self implementing ArrayAccess classes.

Note the comment.

It would be great if underscore would support this too. We'd rather use your pluck version than our own.

The context of where the function is being called is lost. This is very unfortunate when I want to do some nested 'each' functions. Is there a way to overcome this?