php array_filter is filtering too much

Question

Here is the code :

<?php 
$a_campagnes = $this->campagne->get_campagnes_client();
foreach($a_campagnes as $o_camp){
    if($o_camp->groupes){
        foreach($o_camp->groupes as $o_groupe){
            if($o_groupe->IDGroupe == $this->session->o_user->IDGroupe){ echo 'ok';}
        }
    }
}
$a_campagnes = array_filter($a_campagnes, function($o_camp){
    if($o_camp->groupes){
        foreach($o_camp->groupes as $o_groupe){
            if($o_groupe->IDGroupe == $this->session->o_user->IDGroupe) return true;
        }
    }
    return false;
});

$a_campagnes contains at first 10 objects

The result of the first foreach is okokokok

The result of $a_campagnes after the array_filter (which is the same code as the first foreach) is null

Where are the four objects matching my first foreach?

EDIT

Just tried that piece of code:

$i_id_groupe_user = $this->session->o_user->IDGroupe;
        foreach($a_campagnes as $o_camp){
            if($o_camp->groupes){
                foreach($o_camp->groupes as $o_groupe){
                    if($o_groupe->IDGroupe == $i_id_groupe_user){ echo 'ok';}
                }
            }
        }
        $a_campagnes = array_filter($a_campagnes, function($o_camp) use ($i_id_groupe_user){
            if($o_camp->groupes){
                foreach($o_camp->groupes as $o_groupe){
                    if($o_groupe->IDGroupe == $i_id_groupe_user) return true;
                }
            }
            return false;
        });

It gives the same result as before

Please make sure that you are getting $o_camp->groupes in your if condition. If yes then Please check foreach. — jilesh
– jilesh, Commented Mar 7, 2017 at 10:59
Condition is good, $o_camp->groupes is an object of objects — VeZoul
– VeZoul, Commented Mar 7, 2017 at 11:05

Narf · Accepted Answer · 2017-03-07 11:07:11Z

1

$this doesn't exist inside anonymous functions, and you're trying to use it as if it was inside your class scope, which would be even less logical.

If you want to use whatever $this->session is inside your array_filter() callback, you'll have to either declare a class method specifically for that, or tell the anonymous function that it can use it, like this:

$session = $this->session;
$a_campagnes = array_filter($a_campagnes, function($o_camp) use ($session) {
    if ($o_camp->groupes) {
        foreach($o_camp->groupes as $o_groupe) {
            if ($o_groupe->IDGroupe == $session->o_user->IDGroupe) return true;
        }
    }

    return false;
});

answered Mar 7, 2017 at 11:07

Narf

14.8k3 gold badges40 silver badges68 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

VeZoul Over a year ago

Just thought about that. Unfortunately, it does not change anything (see my edit)

Narf Over a year ago

Well, that's what was wrong with your initial code ... Anything else would be a flaw in your program's logic, which only you know and therefore only you could debug. Also, you would've seen this if you had error_reporting, display_errors enabled ...

VeZoul Over a year ago

Well, you are right... Just found the problem... my variable $a_campagneswas an object of objects... array_filter only accepts arrays as input. Sometime, it's usefull to look at error logs indeed...

Collectives™ on Stack Overflow

php array_filter is filtering too much

EDIT

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

EDIT

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related