Database Query not working as expected using eloquent commands for laravel

Question

I am trying to query my database that is connected to my model "Address" to find any addresses that are 'LIKE' any input into my form in the address input field. My model code is:

public function CheckForPrevious($verify)

{
  $matches = \App\Address::where('address', 'like', '%'.$verify.'%')

  ->get();

  foreach($matches as $match)
  {
    echo "$match";
  }



}

My controller code is:

public function show()
    {
      $address = new Address;
      $check = $_POST['address'];

      $address->CheckForPrevious($check);

      return $address;

    }

Any help would be greatly appreciated

You are not using scope correctly, take a look on the documentation laravel.com/docs/5.4/eloquent#local-scopes — jogesh_pi
– jogesh_pi, Commented May 20, 2017 at 5:26
@jogesh_pi I thought I did use scope right I have the instance used at the top of my model and I used the word scope before the name of my method. I don't know what I am missing — watkins1179
– watkins1179, Commented May 20, 2017 at 5:31
Here is my latest code: public function CheckForPrevious($verify) { $matches = \App\Address::where('address', 'like', '%'.$verify.'%') ->get(); ` foreach($matches as $match) { echo "$match"; } }I get a return of an empty array — watkins1179
– watkins1179, Commented May 20, 2017 at 5:32

Exprator · Accepted Answer · 2017-05-20 05:39:12Z

1

public function show()
{
  $address = new Address();
  $check = $_POST['address'];

  $address->scopeCheckForPrevious($check);

}

public function scopeCheckForPrevious($verify)

    {
      $query = App\Address::where('address', 'like', '%'.$verify.'%')

      ->get();

      foreach($query as $match)
      {
        echo "$match";
      }



    }

use this

edited May 20, 2017 at 5:39

answered May 20, 2017 at 5:26

Exprator

27.8k6 gold badges54 silver badges64 bronze badges

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

Comments

Sapnesh Naik · Accepted Answer · 2017-05-20 05:24:37Z

0

You don't have to prefix scope for query scope functions

Change

  `$address->scopeCheckForPrevious($check)`

To

  `$address->CheckForPrevious($check)`

edited May 20, 2017 at 5:24

answered May 20, 2017 at 5:20

Sapnesh Naik

11.7k9 gold badges70 silver badges103 bronze badges

Comments

jogesh_pi · Accepted Answer · 2017-05-20 05:42:18Z

0

There is no need to access model unnecessary on the scope
$query = App\Address::where('address', 'like', '%'.$verify.'%')

Try like this:

public function scopeCheckForPrevious($query, $verify)
{
    return $query->where('address', 'like', '%'.$verify.'%');
}

And call it with the following code:

$address = new Address;
$check = $_POST['address'];
dd( $address->checkForPrevious($check)->get() );

answered May 20, 2017 at 5:42

jogesh_pi

9,7745 gold badges40 silver badges65 bronze badges

1 Comment

watkins1179 Over a year ago

There are two parameters in the first Model portion of the code and there is only one in the controller

Collectives™ on Stack Overflow

Database Query not working as expected using eloquent commands for laravel

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related