ValidateActionException errorMessage wird nicht übergeben

Warly · May 17, 2017 at 9:48 AM

Hallo,

in der AJAXInvokeAction wird sowohl bei der gewrappten UserInputException als auch bei der ValidateActionException die errorMessage nicht übergeben:

PHP: \wcf\lib\action\AjaxInvokeAction.class.php

else if ($e instanceof UserInputException) {
    // repackage as ValidationActionException
    $exception = new ValidateActionException($e->getField(), $e->getType(), $e->getVariables());
    throw new AJAXException($exception->getMessage(), AJAXException::BAD_PARAMETERS, $e->getTraceAsString(), [
        'errorMessage' => $exception->getMessage(), // $exception->getErrorMessage()?
        'errorType' => $e->getType(),
        'fieldName' => $exception->getFieldName()
    ]);
}
else if ($e instanceof ValidateActionException) {
    throw new AJAXException($e->getMessage(), AJAXException::BAD_PARAMETERS, $e->getTraceAsString(), [
        'errorMessage' => $e->getMessage(), // $e->getErrorMessage()?
        'fieldName' => $e->getFieldName()
    ]);
}

Display More

Da die eigentliche $e->getMessage() bereits übergeben wird, vemute ich mal, dass die errorMessage als Parameter übergeben werden sollte, da man ansonsten keinen Zugriff im Javascript darauf hat.

**Alexander Ebert** · May 17, 2017 at 10:06 AM Official Post

Warly Der Fehler ist Tim Düsterhus bereits vor einigen Wochen aufgefallen, allerdings existiert dieser bereits seit WCF 2.0. Eine Korrektur würde so ziemlich die meisten JS-Implementierungen mit Fehlerbehandlung an die Wand fahren, da praktisch jeder bereits um das aktuelle Verhalten drumherum programmiert hat.

Sinnvoller wäre es da eher, die bestehende Implementierung um ein weiteres Feld, etwa realErrorMessage oder actualErrorMessage, zu erweitern und so abwärtskompatibel zur Verfügung zu stellen.

Warly · May 17, 2017 at 11:26 AM

Ah ok, ich kannte die ValidateActionException bis eben noch nicht und dachte, dass sie erst mit dem WSC dazu gekommen war

Ja gut, dann nutze ich den errorType und pflege die Übersetzungen per JS.

Danke für die Info!

Edit:

so wirklich gehen tut das ja gar nicht, da die Variablen nicht übergeben werden Gibt es hier eine best practice?

Participate now!