Building an array in nested foreach loops in Laravel controller

Question

I am trying to build an array of user ids from a long eloquent relationship, using nested foreach loops in a Laravel controller, but can't get it to work.

Users can have publishers, those publishers can have teams and each team has members. Users can be in multiple teams so I also need to remove duplicate IDs.

I want to end up with a count to see how many team members are associated with a user.

In my user model

public function publishers()
{
    return $this->belongsToMany('App\Publisher')->withTimestamps();
}

In my publisher model

public function teams()
{
  return $this->belongsToMany('App\Team')->withTimestamps();
}

and in my team model

public function members()
{
  return $this->belongsToMany('App\User')->withPivot('status', 'title', 'team_role_ids')->withTimestamps();
}

and in my profile controller

foreach ($user->publishers as $userPublisher) {
  foreach ($userPublisher->teams as $publisherTeam) {
    $teamUserIds[] = $publisherTeam->members->pluck('id')->toarray();
  }
}
$deDupedIds = array_unique($teamUserIds, SORT_NUMERIC);
$idsCount = count($deDupedIds);

But I'm getting multiple arrays and not just one compiled array and the count isn't working. Any idea what I'm doing wrong?

bdtiger · Accepted Answer · 2020-05-19 18:18:20Z

1

You are assigning a new array into $teamUserIds each iteration. That's why you are getting multiple arrays.

$teamUserIds = [];
foreach ($user->publishers as $userPublisher) {
  foreach ($userPublisher->teams as $publisherTeam) {
    $teamUserIds = array_merge($teamUserIds, $publisherTeam->members->pluck('id')->toarray());
  }
}
$deDupedIds = array_unique($teamUserIds, SORT_NUMERIC);
$idsCount = count($deDupedIds);

answered May 19, 2020 at 18:18

bdtiger

5214 silver badges10 bronze badges

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

Comments

Hamidreza · Accepted Answer · 2020-05-19 18:17:05Z

0

you are adding arrays of id $publisherTeam->members->pluck('id')->toarray(); as a new index in $teamUserIds . but what you want to do is to merge the array of ids

so your code would be like this :

foreach ($user->publishers as $userPublisher) {
  foreach ($userPublisher->teams as $publisherTeam) {
    $teamUserIds = array_merge($teamUserIds , $publisherTeam->members->pluck('id')->toarray());
  }
}
$deDupedIds = array_unique($teamUserIds, SORT_NUMERIC);
$idsCount = count($deDupedIds);

answered May 19, 2020 at 18:17

Hamidreza

7346 silver badges22 bronze badges

Collectives™ on Stack Overflow

Building an array in nested foreach loops in Laravel controller

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related