Laravel: Rücksendung der Namespace Besitzer eine polymorphe Beziehung

Finde ich eine Reihe von Diskussionen zu diesem aber keine eindeutige Lösung. Hier sind zwei links, obwohl ich werde alles in meiner Frage hier.

Github Issues

Laravel.io-Diskussion

Einfache Erklärung

Dies ist eine einfache Erklärung für mein problem für jemanden bereits vertraut mit Laravel ist polymorphe Beziehungen.

Bei der Verwendung von $morphClass, der Inhalt von $morphClass, die in der Datenbank gespeichert wird, wie die morph - "Typ" wird für den Klassennamen, wenn Sie versuchen, den Besitzer zu finden, der eine polymorphe Beziehung. Dies führt zu einem Fehler, da die ganze Stelle von $morphClass ist, dass es sich nicht um ein vollständig Namespace name der Klasse.

Wie definieren Sie den Klassennamen, dass die polymorphe Beziehung verwenden sollte?

Ausführlichere Erklärung

Dies ist eine ausführlichere Erklärung, die erklären, genau das, was ich versuche zu tun, und warum bin ich versucht, es zu tun mit Beispielen.

Bei der Verwendung von Polymorphen Beziehungen in Laravel, was wird gespeichert, wie die "morph_type" - Typ in der Datenbank wird davon ausgegangen, dass die Klasse.

So, in diesem Beispiel:

class Photo extends Eloquent {

    public function imageable()
    {
        return $this->morphTo();
    }

}

class Staff extends Eloquent {

    public function photos()
    {
        return $this->morphOne('Photo', 'imageable');
    }

}

class Order extends Eloquent {

    public function photos()
    {
        return $this->morphOne('Photo', 'imageable');
    }

}

Die Datenbank würde so Aussehen:

staff

 - id - integer
 - name - string

orders

 - id - integer
 - price - integer

photos

 - id - integer
 - path - string
 - imageable_id - integer
 - imageable_type - string

Nun die erste Reihe von Fotos, die könnte wie folgt Aussehen:

id,Pfad,imageable_id,imageable_type

1,Bild.png -, 1 -, Personal -

Nun kann ich entweder den Zugriff auf die Foto aus einem Stab-Modell oder ein Mitarbeiter ein Foto-Modell.

//Find a staff member and dump their photos
$staff = Staff::find(1);

var_dump($staff->photos);

//Find a photo and dump the related staff member
$photo = Photo::find(1);

var_dump($photo->imageable);

So weit So gut. Allerdings, wenn ich namespace-ich laufen in ein problem.

namespace App/Store;
class Order {}

namespace App/Users;
class Staff {}

namespace App/Photos;
class Photo {}

Nun was ist gesichert in meiner Datenbank ist diese:

id,Pfad,imageable_id,imageable_type

1,Bild.png,1,App/Benutzer/Mitarbeiter

Aber ich will nicht, dass. Das ist eine schreckliche Idee, um die volle Klasse in einem Namespace befinden, Namen in der Datenbank gespeichert, so!

Glücklicherweise Laravel hat eine option, um ein $morphClass variable. Etwa so:

class Staff extends Eloquent {

    protected $morphClass = 'staff';

    public function photos()
    {
        return $this->morphOne('Photo', 'imageable');
    }

}

Nun die Zeile in meiner Datenbank wird so Aussehen, das ist genial!

id,Pfad,imageable_id,imageable_type

1,Bild.png -, 1 -, Personal -

Und immer die Fotos von einem Mitarbeiter funktioniert sehr gut.

//Find a staff member and dump their photos
$staff = Staff::find(1);

//This works!
var_dump($staff->photos);

Jedoch die polymorph-Zauber zu finden, der Besitzer des Fotos nicht funktioniert:

//Find a photo and dump the related staff member
$photo = Photo::find(1);

//This doesn't work!
var_dump($photo->imageable);

//Error: Class 'staff' not found

Vermutlich muss es eine Möglichkeit zu informieren, die polymorphe Beziehung, was classname zu verwenden, wenn $morphClass aber ich kann nicht finden jede Bezugnahme auf, wie diese Arbeit sollte in den docs, in den source-code oder über Google.

Hilfe?

Hast du versucht, pass, $this->morphOne () - Modell-Namen mit namespace?
Welches argument würde es sein?
zuerst return $this->morphOne('App/Fotos/Foto', 'druckbaren');
Ah ja, das habe ich getan, sorry kopieren und einfügen die ganze Sache wieder schien sich nicht lohnt, aber das macht keinen Unterschied. Es ist die morphTo() das ist das problem, nicht das morphOne()

InformationsquelleAutor John Mellor | 2015-01-09

45

Gibt es 2 einfache Möglichkeiten - der eine unten, der andere in @lukasgeiter Antwort, wie vorgeschlagen, von Taylor Otwell, die ich auf jeden Fall vorschlagen, prüfen, wie gut:
```
//app/config/app.php or anywhere you like
'aliases' => [
    ...
    'MorphOrder' => 'Some\Namespace\Order',
    'MorphStaff' => 'Maybe\Another\Namespace\Staff',
    ...
]

//Staff model
protected $morphClass = 'MorphStaff';

//Order model
protected $morphClass = 'MorphOrder';
```
getan:
```
$photo = Photo::find(5);
$photo->imageable_type; //MorphOrder
$photo->imageable; //Some\Namespace\Order

$anotherPhoto = Photo::find(10);
$anotherPhoto->imageable_type; //MorphStaff
$anotherPhoto->imageable; //Maybe\Another\Namespace\Staff
```
Ich würde es nicht verwenden echt Klasse Namen (Order und Staff) mögliche Duplikate zu vermeiden. Es gibt sehr wenig chance, dass etwas genannt würde MorphXxxx so ist es ziemlich sicher.

Dies ist besser als das speichern von namespaces (ich bin nicht dagegen, das sieht in der db, jedoch wäre es ungünstig, falls Sie etwas ändern - sagen statt App\Models\User verwenden Cartalyst\Sentinel\User etc), weil alles, was Sie brauchen, ist ein vertauschen der Umsetzung durch Aliase config.

Aber es gibt auch ein Nachteil - Sie nicht wissen, was das Modell ist, nur durch überprüfung der db - falls es Ihnen wichtig ist.
- Ah, dass ist eine sehr gute Lösung. Ich habe es nicht testen gerade Weg, aber es macht Sinn und sieht aus wie es funktionieren sollte. Ich habe nicht gefunden, eine solche Lösung beschrieben, sonst nirgends 🙂
- Ich glaube nicht, dass jemand mit ihm kam es vor.
- Zugewiesen Sie die bounty auf Grund der Einnahme der Zeit, um zu kommen mit einer einzigartigen Lösung. Obwohl zu jeder der Lektüre dieses später ist, ist es auch Wert, heraus zu überprüfen anderen Antwort von @lukasgeiter
- Ich habe immer MyClass::class bei der Festlegung der morphOne Beziehung, so dass ich nie vor diesem problem. Aber ich hätte nie gedacht, über die Auswirkungen der Umgestaltung auf die Datenbank, vielen Dank!
- Beachten Sie, dass Laravel 5.2 enthält MorpMaps: nicolaswidart.com/blog/laravel-52-morph-map
- Sie KO würde der Laravel Dokumentation, ich glaube nicht dass das in es. Ich Grüße Sie.
InformationsquelleAutor Jarek Tkaczyk
22

Ich wie @JarekTkaczyks Lösung und ich würde vorschlagen, Sie verwenden. Aber, der Vollständigkeit halber, es ist eine andere Art Taylor kurz erwähnt, auf github

Können Sie ein Attribut-accessor für die imageable_type - Attribut, und verwenden Sie dann eine "classmap" array, um die richtige Klasse.
```
class Photo extends Eloquent {

    protected $types = [
        'order' => 'App\Store\Order',
        'staff' => 'App\Users\Staff'
    ];

    public function imageable()
    {
        return $this->morphTo();
    }

    public function getImageableTypeAttribute($type) {
        //transform to lower case
        $type = strtolower($type);

        //to make sure this returns value from the array
        return array_get($this->types, $type, $type);

        //which is always safe, because new 'class'
        //will work just the same as new 'Class'
    }

}
```
Hinweis, dass Sie immer noch müssen die morphClass Attribut für die Abfrage von der anderen Seite der Beziehung obwohl.
- Cheers, ich mag deins auch. Deine (oder Taylor) ist vielleicht auch besser so, wie es ist alles beschränkt, die innerhalb der Beredte Modell eher als dass die config-Dateien. Es ist die Qual der Wahl auf, die zu geben, die Beute!
- Danke. Ich hätte das Kopfgeld @JarekTkaczyk die Antwort... Es ist noch nicht eine sehr schöne Lösung, obwohl dieser "bug" immer noch eine Art stört mich. Wenn ich etwas Zeit haben werde ich sehen, ob ich etwas herausfinden, um es zu beheben und machen einen pull-request. Ich werde auf jeden Fall hier einen Kommentar hinterlassen, wenn das passiert 😉
- Ja, das ist gut, noch besser mit diesem Ort. Nur die Korrektur der accessor, um sicherzustellen, gibt es keine Probleme mit Brief-Fall. Sie daran, dass $array['Type'] != $array['type'] ABER new 'Type' == new 'type' - willkommen auf PHP 😉
- Ich bin mir durchaus bewusst, dass. Aber ich sehe nicht, in welcher Art und Weise würde ich zu "reparieren" die accessor. Ich sehe die array-keys von $types wie Klassennamen, sondern eher zwischen groß-und Kleinschreibung von Bezeichnern. Wenn ich dich missverstanden, fühlen Sie sich frei, zu suggerieren, ein edit...
- Hast du die Bearbeiten.
- Oh ja, das ist sicherer 🙂
- Ich hatte, um '' => self:class zu den $types array oder sonst wäre es zu beschweren, wenn Sie zuerst das Modell speichern
InformationsquelleAutor lukasgeiter
19

Bei der Verwendung von Laravel 5.2 (oder neuer) können Sie das neue feature morphMap um dieses Problem zu beheben. Nur fügen Sie diese an die boot Funktion in app/Providers/AppServiceProvider:
```
Relation::morphMap([
    'post' => \App\Models\Post::class,
    'video' => \App\Models\Video::class,
]);
```
Mehr dazu: https://nicolaswidart.com/blog/laravel-52-morph-map
- wo Sie diese ?
- In der boot Funktion in app/Providers/AppServiceProvider.
- Dies funktioniert für mich, aber ich hatte Probleme unsing die MorphMap. Vielleicht hilft dies ja anderen. Wenn Sie mit der morphmap in Ihrer Beziehung, z.B. Fotos, können Sie es wie folgt hinzufügen: hinzufügen: $relation = Relation::morphMap(); vor der Rückkehr zu die Karte Holen und dann return $this->morphOne($relation['photo'], 'imagable');. Nicht sicher, ob es einen einfacheren Weg, aber so ist es die Arbeit mit-heraus sagen Foto nicht gefunden.
- Eine weitere nützliche Methode, um den setup-morph. Hinzufügen $table->morphs('imageable'); um Ihre Fotos migration. Erstellen imageable_type und imageable_id für Sie, das ist einfacher zu schreiben. Gefunden im test setup - hier
InformationsquelleAutor dersvenhesse
3

Lassen laravel setzen Sie es in die db - namespace und alle. Wenn Sie die kurze classname für etwas, neben Ihrer Datenbank hübscher aus, dann definieren Sie einen accessor für Sie so etwas wie :
```
<?php namespace App\Users;

class Staff extends Eloquent {

    //you may or may not want this attribute added to all model instances
    //protected $appends = ['morph_object'];

    public function photos()
    {
        return $this->morphOne('App\Photos\Photo', 'imageable');
    }

    public function getMorphObjectAttribute()
    {
        return (new \ReflectionClass($this))->getShortName();
    }

}
```
Die Argumentation, die ich immer wieder kommen, um in Fällen wie diesem ist, dass Laravel ist ziemlich gut getestet, und funktioniert wie erwartet für die meisten Teil. Warum kämpfen die Rahmen, wenn Sie nicht haben, um - vor allem, wenn Sie einfach nur genervt von der namespace wird in der db. Ich Zustimmen, dass es keine gute Idee ist, so zu tun, aber ich fühle auch, dass ich meine Zeit sinnvoller, über sich und die Arbeit auf der Domäne-code.
- Sie sind sich bewusst, dass es ein bug in der gut getestet Laravel? Es ist eine Methode, die für diese, aber Beredt und können nicht damit umgehen, am Ende 🙂
- Es ist nicht nur 'ärger', dass es in der db. Eines der wichtigsten features von Laravel ist, dass Sie können leicht ändern, welche Klassen, die Sie verwenden, und fallen verschiedenen Klassen in und out. Mit diesem Szenario, wenn ich eine änderung der Struktur des Codes brauche ich um die Datenbank zu aktualisieren und nicht nur die Struktur der Datenbank (wie üblich bei änderungen), aber die eigentlichen Daten in den Zeilen selbst. Das ist eine schreckliche Strömung in workflow/design der app.
- Gut, nehmen Sie es wie Sie wird. Eine schnelle update-Aufruf @ die db Ende löst der geänderte Pfad-problem - sobald Sie bereitstellen, die Wahrscheinlichkeit der namespace ändert sich für alle diese Modelle sollten relativ niedrig und das Problem, diese zu wechseln ist lösbar mit etwa 30ms der query-Zeit von der Befehlszeile aus. Mein Punkt war mehr, dass - selbst wenn es ein Fehler ist, sich die Zeit nehmen, Sie zu bekämpfen, wenn es wirkt, wenn Sie nicht eine Verschwendung von Mühe im moment.
- Auch, wenn Sie das Gefühl es ist ein bug (was es scheint, ist es!) dann Datei einen pull-request mit einem fix @ github - oder zumindest das öffnen ein Problem und die Bestätigung bekommen, dass dies nicht wie erwartet funktioniert.
InformationsquelleAutor Abba Bryant

Damit eager loading polymorpher Beziehungen zB Photo::with('imageable')->get(); es notwendig null zurück, wenn der Typ leer ist.

class Photo extends Eloquent {

    protected $types = [
        'order' => 'App\Store\Order',
        'staff' => 'App\Users\Staff'
    ];

    public function imageable()
    {
        return $this->morphTo();
    }

    public function getImageableTypeAttribute($type) {
        //Illuminate/Database/Eloquent/Model::morphTo checks for null in order
        //to handle eager-loading relationships
        if(!$type) {
            return null;
        }

        //transform to lower case
        $type = strtolower($type);

        //to make sure this returns value from the array
        return array_get($this->types, $type, $type);

        //which is always safe, because new 'class'
        //will work just the same as new 'Class'
    }

}

Mit dieser Methode fand ich die Variable $Typ hat extra die tab-Zeichen am Ende "\t".

InformationsquelleAutor DTownsend

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.