Laravel 5.4 - Wie man mehrere Fehlermeldungen für denselben benutzerdefinierten Gültigkeitsregel

Schlechten Umgang mit der benutzerdefinierte Validierungsregeln ist der Grund, warum ich ditched laravel (gut, es war einer der vielen Gründe, aber es war der Tropfen, der das Kamel brach ' zurück, um so zu sprechen). Aber egal, ich habe eine dreiteilige Antwort für Sie: ein Grund, warum Sie nicht wollen, dies zu tun, in diesem konkreten Fall, einen schnellen überblick über die Sauerei, die Sie zu bewältigen haben, und dann die Antwort auf Ihre Frage in den Fall, Sie wollen immer noch, es zu tun.

Wichtig Sicherheit Sorge

Best security practices für die Verwaltung von Anmeldungen zu diktieren, sollten Sie immer eine generische Fehlermeldung für login-Probleme. Die Quintessenz der Zähler-Beispiel wäre, wenn Sie zurückgegeben ", Dass E-Mail nicht registriert ist mit unserem system" für eine E-Mail-nicht-gefunden und "Falsches Passwort" für eine korrekte E-Mail mit dem falschen Passwort. In dem Fall, wo Sie geben separate Validierung der Nachrichten, die Sie geben potentiellen Angreifer zusätzliche Informationen darüber, wie Sie effektiver, richten Ihre Angriffe. Als Ergebnis werden alle login-Problemen zurückkehren sollte, eine generische Validierung Nachricht, unabhängig von der zugrunde liegende Ursache, die etwas zum Effekt von "Invalid E-Mail/Passwort Kombination". Das gleiche gilt für die Passwort-Wiederherstellung Formen, die oft etwas sagen wie, "Passwort-Wiederherstellung Anweisungen erhalten haben, dass E-Mail, wenn Sie vorhanden ist in unserem system". Ansonsten geben Sie Angreifern (und andere) eine Möglichkeit zu wissen, was E-Mail-Adressen registriert sind, mit Ihrem system, und können, setzen zusätzliche Angriffsvektoren. Also in diesem insbesondere Fall einer Validierung der Nachricht ist, was Sie wollen.

Die Mühe mit laravel

Das Problem, das Sie laufen in ist, dass laravel Prüfungen einfach true oder false zurück, um zu Kennzeichnen, ob oder nicht die Regel ist erfüllt. Fehlermeldungen werden separat behandelt. Sie insbesondere angeben können, der validator die Fehlermeldung aus der eigenen validator Logik. Ich weiß. Es ist lächerlich und schlecht geplant. Alles, was Sie tun können, ist true oder false zurück. Sie haben keinen Zugriff auf irgendetwas anderes, Ihnen zu helfen, damit Ihre pseudo-code der nicht gehen, es zu tun.

Die (unschöne) Antwort

Der einfachste Weg, um erstellen Sie Ihr eigenes Validierung von Nachrichten ist erstellen Sie Ihre eigenen validator. Das sieht ungefähr so aus (im inneren des Controllers):

$validator = Validator::make($input, $rules, $messages);

Müssten Sie noch schaffen die eigene Prüfung auf boot (Ihr Valiator::Extend nennen. Dann können Sie die $rules normalerweise, indem man Sie in Ihrer benutzerdefinierten validator. Schließlich können Sie angeben, Ihre Nachrichten. So etwas wie dieses, insgesamt (im inneren des Controllers):

public function login( Request $request )
{
    $rules = [
        'email' => 'bail|required|checkEmailPresenceAndValidity'
    ]

    $messages = [
        'checkEmailPresenceAndValidity' => 'Invalid email.',
    ];

    $validator = Validator::make($request->all(), $rules, $messages);
}

(Ich erinnere mich nicht, wenn Sie angeben müssen, die jede Regel in Ihrem $messages array. Ich glaube nicht, dass es so ist). Natürlich, auch das ist nicht sehr genial, was du da pass für $messages ist einfach ein array von strings (und das ist alles was es sein darf). Als Ergebnis, werden Sie noch nicht haben, können diese Fehlermeldung leicht ändern, je nach Benutzereingabe. Dies alles geschieht vor Ihren validator läuft auch. Ihr Ziel ist es, die Validierung der Nachricht ändern sich je nach den Ergebnissen der Validierung, aber laravel Kräfte, die Sie zum erstellen der Nachrichten zunächst. Als ein Ergebnis, um wirklich zu tun, was Sie tun möchten, Sie müssen, um die tatsächliche Strömung des Systems, das ist nicht sehr genial.

Eine Lösung wäre es, eine Methode im controller, der berechnet, ob oder nicht Ihre benutzerdefinierte Gültigkeitsregel erfüllt ist. Es würde dies tun, bevor Sie Ihre Bestätigung, so dass Sie können senden Sie eine entsprechende Nachricht an den validator, die Sie bauen. Dann, wenn Sie erstellen die Validierung Regel, können Sie auch binden Sie es auf die Ergebnisse Ihrer Validierungs-Rechner, so lange wie Sie sich bewegen, Ihre Regel-definition im inneren des Controllers. Sie müssen nur sicherstellen, dass und nicht zufällig nennen die Dinge nicht in Ordnung. Sie haben auch zu beachten, dass dies erfordert, bewegen Sie Ihre Validierungslogik außerhalb der Prüfungen, die ziemlich hacky. Leider, ich bin 95% sicher, dass es wirklich keinen anderen Weg, dies zu tun.

Habe ich einige Beispiel-code unten. Es hat auf jeden Fall einige draw-Rücken: die Regel ist nicht mehr global (ist definiert in der Steuerung), die Validierungs-Logik bewegt sich der validator (das verstößt gegen das Prinzip der geringsten Verwunderung), und Sie haben zu kommen mit einer in-Objekt-caching-System (das ist nicht schwer), um sicherzustellen, dass Sie nicht führen Sie Ihre Abfrage zweimal, da die Validierung Logik ist zweimal genannt. Zu wiederholen, es ist definitiv hacky, aber ich bin mir ziemlich sicher, dass dies der einzige Weg, das zu tun, was Sie tun möchten, mit laravel. Möglicherweise gibt es bessere Wege das zu organisieren, aber dies sollte zumindest geben Ihnen eine Idee von, was Sie brauchen, um passieren.

<?php
namespace App\Http\Controllers;

use User;
use Validator;
use Illuminate\Http\Request;
use App\Http\Controllers\Controller;

class LoginController extends Controller
{
    public function __construct() {
        Validator::extend('checkEmailPresenceAndValidity', function ($attribute, $value, $parameters, $validator) {

            return $this->checkLogin( $value ) === true ? true : false;

        });
    }

    public function checkLogin( $email ) {
        $user = User::where('email', $email)->first();

        //Email has not been found
        if (! $user) {
            return 'not found';
        }

        //Email has not been validated
        if (! $user->valid_email) {
            return 'invalid';
        }

        return true;
    }

    public function login( Request $request ) {

        $rules = [
            'email' => 'bail|required|checkEmailPresenceAndValidity'
        ]

        $hasError = $this->checkLogin( $request->email );
        if ( $hasError === 'not found' )
            $message = "That email wasn't found";
        elseif ( $hasError === 'invalid' )
            $message = "That is an invalid email";
        else
            $message = "Something was wrong with your request";


        $messages = [
            'checkEmailPresenceAndValidity' => $message,
        ];

        $validator = Validator::make($request->all(), $rules, $messages);

        if ($validator->fails()) {
            //do something and redirect/exit
        }

        //process successful form here
    }
}

Ist es auch, lohnt sich ein kurzer Hinweis, dass diese Implementierung setzt auf $this Unterstützung für closures, die (glaube ich) wurde Hinzugefügt in PHP 5.4. Wenn Sie auf einer alten version von PHP du wirst $this den Verschluss mit use.

Bearbeiten zu schimpfen

Was es wirklich hinausläuft, ist, dass die laravel validation system wurde entwickelt, um sehr fein abgestimmt werden. Jede Validierungsregel wird speziell eigentlich nur bestätigen, was. Als Ergebnis der Validierung Nachricht für eine angegebene Prüfung sollte nie geändert werden müssen, also warum $messages (wenn Sie bauen Sie Ihre eigenen validator) akzeptiert nur die plain-strings.

Im Allgemeinen Granularität ist eine gute Sache, application design, und etwas, dass die ordnungsgemäße Durchführung von FESTEN Prinzipien Streben. Jedoch, diese Art der Implementierung macht mich verrückt. Meine Allgemeine Programmier-Philosophie ist, dass eine gute Umsetzung sollten die häufigsten nutzt-Fällen sehr einfach, und dann raus aus dem Weg, für die weniger häufige Anwendungsfälle. In diesen Fällen ist die Architektur von laravel die häufigsten Anwendungsfälle einfach, aber die weniger häufige Anwendungsfälle fast unmöglich. Ich bin nicht in Ordnung mit, dass der Handel aus. Mein allgemeiner Eindruck von Laravel war, dass es wunderbar funktioniert, so lange, wie Sie benötigen, Dinge zu tun, die laravel Weg, aber wenn Sie haben, um Schritt aus der box aus irgendeinem Grund, es geht um aktiv Ihr Leben erschweren. In Ihrem Fall die beste Antwort ist wahrscheinlich nur Schritt rechts zurück in die box, d.h. zwei Prüfern, auch wenn es bedeutet, dass eine redundante Abfrage. Die tatsächlichen Auswirkungen auf die Leistung Ihrer Anwendung wird wahrscheinlich nicht egal, aber die Treffer werden Sie ergreifen, um Ihre langfristige Wartbarkeit zu bekommen laravel zu Verhalten, wie Sie wollen, es wird ziemlich groß sein.

Wo haben Sie den Fehler gefunden-Nachrichten für die Größe Validierung?

Ich schaute auf die Validierungsregeln in der
Illuminate\Validation\ConcernsValidatesAttributes Merkmal und alle Funktionen geben einen booleschen Wert (auch von der Größe Validierung).

protected function validateSize($attribute, $value, $parameters)
{
    $this->requireParameterCount(1, $parameters, 'size');

    return $this->getSize($attribute, $value) == $parameters[0];
}

Was Sie gefunden haben, gehört zu diesem Teil:

$keys = ["{$attribute}.{$lowerRule}", $lowerRule];

In diesem Fall ist es nur für die Formatierung der Ausgabe, indem ein lowerRule Wert, dass laravel Griffe in besonderen Fällen, wie die Größe Validierung:

    //If the rule being validated is a "size" rule, we will need to gather the
    //specific error message for the type of attribute being validated such
    //as a number, file or string which all have different message types.
    elseif (in_array($rule, $this->sizeRules)) {
        return $this->getSizeMessage($attribute, $rule);
    }

So lange, wie Validierungsregeln sind zur Rückgabe ein boolescher Wert, es gibt keine Möglichkeit zur Rückkehr mehr als eine Fehlermeldung. Ansonsten müssen Sie neu schreiben einige Partei der Validierungs-Regeln.

Einen Ansatz für dein problem mit der Validierung konnte man mit dem exists-Validierung:

public function rules()
{
    return [
        'email' => ['bail', 'required',  Rule::exists('users')->where(function($query) {
        return $query->where('valid_email', 1);
    })]
    ];
}   

So müssten Sie 2 existiert Validierungs-Regeln. Ich würde empfehlen, das bestehende von laravel zu überprüfen, ob die E-Mail festgelegt ist, und einer eigenen, um zu überprüfen, ob das Konto überprüft wird.