Level 88
Well you can create a join with Eloquent! Try the documentation: http://laravel.com/docs/5.1/queries#joins
1 like
Aug 7, 2015
7
Level 1
Retrieving data from database
I have 3 tables:
professor
id
name
discipline
id
name
and one to link them
disc_prof
prof_id
disc_id
How do I retrieve data as an Inner Join ?
Something like:
SELECT prof.name, prof.id, disc.id, disc.name
FROM disc_prof AS dc INNER JOIN professor as prof ON prof.id = dc.prof_id
INNER JOIN discipline AS disc ON dc.disc_id = disc.id
What I did so far:
Professor Model
class Professor extends Model
{
protected $table = "professor";
public function disc()
{
return $this->belongsToMany('App\Discipline');
}
}
Discipline Model
class Disciplina extends Model
{
protected $table = "discipline";
}
But I'm getting the following error:
Base table or view not found: 1146 Table 'database.discipline_professor' doesn't exist
It's not database.discipline_professor, it's database.disc_prof.
What is the correct way to change it ?
Level 13
It's not database.discipline_professor, it's database.disc_prof. What is the correct way to change it ?
Try this :
public function disc()
{
return $this->belongsToMany('App\Discipline', 'disc_prof');
}
Level 80
@Ghaleon Your problem is you’re not using the table naming convention Eloquent expects for your pivot table.
From http://laravel.com/docs/master/eloquent-relationships#many-to-many:
The
role_usertable is derived from the alphabetical order of the related model names, and contains theuser_idandrole_idcolumns.
In your case, your table should be named discipline_professor. If you name it as such, your relationship will Just Work™.
Of course, you can supply additional parameters to override table and columns names in your relationships, but it’s just easier to use conventional names. In this case, your relationship method should be called disciplines() instead of disc(). The following is much more readable:
$disciplines = $professor->disciplines;
1 like
Level 1
Well, here is the solution I got from Josh Janusch on stackoverflow.
ps: As @martinbean, Josh also highly recommended me to follow the Eloquent names. So that's what I'm going to do. As it's just a teste, I'll rename the database.
The work around for those who might need it:
Model::belongsToMany(), you will see the arguments are:
string $related, string $table = null, string $foreignKey = null, string $otherKey = null, string $relation = null
class Professor extends Model
{
protected $table = "professor";
public function disc()
{
return $this->belongsToMany('App\Discipline', 'disc_prof', 'prof_id', 'disc_id');
}
}
Level 80
@Ghaleon Good of you to award yourself the answer.
Level 1
@martinbean Sorry if its illegal, but here is the original answer: http://stackoverflow.com/a/31881710/2054434
What was the right thing to do ? Just wanted to show the answer to everyone. Should I Delete it ?
Level 80
@Ghaleon It wasn’t illegal, but just think it’s a bit disingenuous to award yourself the answer when you arrived to it from others’ input (including mine as you so acknowledged).
Please or to participate in this conversation.