Laravel: concat multiple model queries

Question

I have two method's in the User class that return different queries. I would like to use both like this

$users = User::fromCountry('DE')
                    ->isMaster()
                    ->paginate(20);

but it is not working, since isMaster() is a method from User and not from the query builder:

Call to undefined method Illuminate\Database\Query\Builder::isMaster()

I also tried

$users = User::fromCountry('DE')
                    ->(User::isMaster())
                    ->paginate(20);

but this is also not working. Is it possible to easily combine my queries from the User class?

Just in case its necessary, here are the two methods from the User class:

public static function isMaster()
    {
      switch (orga) {
        case 1:
          return static::where('modeIN','=','MT');

        case 2:
          return static::where('modeICI','=','CMT');

        case 3:
          return static::where('modeWHO','=','HMT');
      }
    }


    public static function fromCountry($country)
    {
      return static::where(
              function($query) use($country){
                $query->whereHas('addresses',
                          function($q) use($country){
                            $q->where('idCountry','=',$country);
                          }
                        )
                      ->orWhereHas('institutes',
                        function($q) use($country){
                          $q->whereHas('addresses',
                            function($q) use($country){
                              $q->where('idCountry','=',$country);
                            });
                        }
                      );
             });
    }

madalinivascu · Accepted Answer · 2017-10-13 08:39:26Z

1

Try changing your modal function to:

public function scopeIsMaster($query)
    {
      switch (orga) {
        case 1:
          return $query->where('modeIN','=','MT');

        case 2:
          return $query->where('modeICI','=','CMT');

        case 3:
          return $query->where('modeWHO','=','HMT');
      }
    }

public function scopeFromCountry($query,$country)
    {
      return $query->where(
              function($query) use($country){
                $query->whereHas('addresses',
                          function($q) use($country){
                            $q->where('idCountry','=',$country);
                          }
                        )
                      ->orWhereHas('institutes',
                        function($q) use($country){
                          $q->whereHas('addresses',
                            function($q) use($country){
                              $q->where('idCountry','=',$country);
                            });
                        }
                      );
             });
    }

edited Oct 13, 2017 at 8:39

answered Oct 13, 2017 at 8:20

madalinivascu

32.4k4 gold badges41 silver badges59 bronze badges

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

2 Comments

Adam Over a year ago

DId you not make it static on purpose? Do I understand it right that I have to split my query call into two calls: $users = User::fromCountry('DE'); and $users = User::queryIsMaster($users)->paginate(20); ?

madalinivascu Over a year ago

no you just call $users = User::fromCountry('DE') ->isMaster() ->paginate(20);

Samuel James · Accepted Answer · 2017-10-13 08:22:11Z

1

This is what you can do. Laravel uses scope for stuff like this

User {

   public function scopeOfType($query, $orga)
    {
        switch ($orga) {
        case 1:
          return $query->where('modeIN','=','MT');

        case 2:
          return $query->where('modeICI','=','CMT');

        case 3:
          return $query->where('modeWHO','=','HMT');
      }
    }


}

Then call it like this.

User::ofType(1);
User::ofType(2);
User::ofType(3);

For more info check https://laravel.com/docs/5.3/eloquent#local-scopes

answered Oct 13, 2017 at 8:22

Samuel James

1,53616 silver badges17 bronze badges

Comments

beauraF · Accepted Answer · 2017-10-13 08:21:16Z

0

Have a look at Laravel Scope :)

answered Oct 13, 2017 at 8:21

beauraF

1008 bronze badges

Collectives™ on Stack Overflow

Laravel: concat multiple model queries

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related