【问题标题】:Laravel 5 on failed request return back with model dataLaravel 5 在请求失败时返回模型数据
【发布时间】:2015-08-27 15:34:30
【问题描述】:

编辑
这个问题已经解决了,但我不明白为什么答案有效。
我想知道为什么它不起作用。 有没有人可以解释一下?


原始问题
我卡在设置表单上,我的问题是在设置表单中您可以输入一些电子邮件设置,但您也可以更改密码。

电子邮件设置和密码重置有效,我的表单使用当前用户的数据自行填写。 但是当表单验证失败时,它会将我重定向回没有表单数据的表单。

我不确定我是否说清楚了,但下面的代码会解释它。

ChangeUserSettingRequest.php

class ChangeUserSettingsRequest extends Request {

    /**
     * Determine if the user is authorized to make this request.
     *
     * @return bool
     */
    public function authorize()
    {
        if (\Auth::check()) {
            return true;
        }
        return false;
    }

    /**
     * Get the validation rules that apply to the request.
     *
     * WHEN THIS VALIDATION FAILS IT GOES BACK TO SETTINGS.BLADE.PHP 
     * BUT IT DOES NOT KEEP THE SETTINGS DATA IN THE FORM
     */
    public function rules()
    {
        return [
            'current_password' => 'sometimes|required_with:password',
            'password' => 'required_with:current_password|min:8|confirmed',
            'password_confirmation' => 'required_with:password',
        ];
    }

}

settings.blade.php

{!! Form::model($settings, array('url' => route('store.settings'), 'class' => 'col s12')) !!}

    <p>@lang('forms.info.settings')</p>

    <div class="input-field col s12 m6 l6">
        {!! Form::checkbox('info_mail', 0, false, ['id' => 'info_mail']) !!}
        {!! Form::label('info_mail', Lang::get('forms.newsletter'), ['for' => 'info_mail']) !!}
    </div>
    <div class="input-field col s12 m6 l6">
        {!! Form::checkbox('message_notification', 0, false, ['id' => 'message_notification']) !!}
        {!! Form::label('message_notification', Lang::get('forms.messages'), ['for' => 'message_notification']) !!}
    </div>
    <div class="input-field col s12 m6 l6">
        {!! Form::checkbox('friend_notification', 0, false, ['id' => 'friend_notification']) !!}
        {!! Form::label('friend_notification', Lang::get('forms.friendrequest'), ['for' => 'friend_notification']) !!}
    </div>
    <div class="input-field col s12 m6 l6">
        {!! Form::checkbox('item_notification', 0, false, ['id' => 'item_notification']) !!}
        {!! Form::label('item_notification', Lang::get('forms.reactiononitem'), ['for' => 'item_notification']) !!}
    </div>

    @if ($settings and $settings->google_maps !== null)
        <div class="settings-explain">
            <p class="margin-top-20">@lang('forms.info.companysettings')</p>
        </div>
        <div class="input-field col s12 m6 l6">
            {!! Form::checkbox('type', 0, false, ['id' => 'type']) !!}
            {!! Form::label('type', Lang::get('forms.companytype'), ['for' => 'type']) !!}
        </div>
        <div class="input-field col s12 m6 l6">
            {!! Form::checkbox('google_maps', 0, false, ['id' => 'google_maps']) !!}
            {!! Form::label('google_maps', Lang::get('forms.companymap'), ['for' => 'google_maps']) !!}
        </div>
    @endif

    <div class="settings-explain">
        <p class="margin-top-20">@lang('forms.info.changepassword')</p>
    </div>
    <div class="input-field col s12">
        {!! Form::label('current_password', $errors->has('current_password') ? $errors->first('current_password') : Lang::get('forms.currentpassword'), ['for' => 'current_password']) !!}
        {!! Form::password('current_password', ['class' => $errors->has('current_password') ? 'invalid' : '']) !!}
    </div>
    <div class="input-field col s12">
        {!! Form::label('password', $errors->has('password') ? $errors->first('password') : Lang::get('forms.newpassword'), ['for' => 'password']) !!}
        {!! Form::password('password', ['class' => $errors->has('password') ? 'invalid' : '']) !!}
    </div>  
    <div class="input-field col s12">
        {!! Form::label('password_confirmation', $errors->has('password_confirmation') ? $errors->first('password_confirmation') : Lang::get('forms.repeatpassword'), ['for' => 'password_confirmation']) !!}
        {!! Form::password('password_confirmation', ['class' => $errors->has('password_confirmation') ? 'invalid' : '']) !!}
    </div>
    <div class="input-field col s12">
        {!! Form::button(Lang::get('forms.save'), ['class' => 'btn waves-effect waves-light', 'type' => 'submit', 'name' => 'Save']) !!}
    </div>
{!! Form::close() !!}

UserInfoController.php

/**
 *  Function shows the settings form
 */
public function showSettings() 
{
    $title = Lang::get('titles.settings');
    $user_id = Auth::User()->id;

    $settings = $this->settings->getUserSettings($user_id);
    $companySettings = $this->companySettings->getSettings($user_id);

    if ($companySettings) {
        $settings->type = $companySettings->type;
        $settings->google_maps = $companySettings->google_maps;
    }

    return view('pages.users.settings', ['title' => $title])->with(compact('settings'));
}

/**
 *  Function stores the setting changes
 *
 *  ChangeUserSettingsRequest makes sure that the request is valid
 */
public function storeSettings(ChangeUserSettingsRequest $request)
{
    $id = Auth::User()->id;
    $success = $this->settings->handleStoreSettingsRequest($request);

    // Checks if user has company settings
    $hasCompanySettings = $this->companySettings->checkForSettings($id);

    // If user has company settings
    if ($hasCompanySettings === true) {
        // Update company settings
        $this->companySettings->updateSettings($request);
    }

    if ($success === true) {

        /* Get translated message */
        $message = Lang::get('toast.settingsstored');
        return Redirect::route('user.profile', array(Auth::User()->permalink))->withMessage($message);
    }

    $settings = $this->settings->getUserSettings($id);

    /* Get translated message */
    $message = Lang::get('forms.error.wrongpassword');

    /* This works and the form is filled with the correct data after it  redirects me back */
    return Redirect::back()->withErrors(array('current_password' => $message))->withSettings($settings);
}

Request.php

use Illuminate\Foundation\Http\FormRequest;
abstract class Request extends FormRequest {

    //

}

所以我的问题是,在我的 UserInfoController 中,我重定向回来并且它具有良好的数据,但是当我的 ChangeUserSettingsRequest 将我重定向回来时,表单是空的。

有谁知道为什么 ChangeUserSettingsRequest 不发回数据?

  • 我说的不是密码数据,而是电子邮件设置。
  • 我不想制作服装验证器,因为我认为这应该可行。

请注意:当验证失败时,函数storeSettings根本不会被执行

【问题讨论】:

  • 尝试替换'->withSettings($settings);'用'->withInputs();'在控制器中
  • @pinkalvansia 控制器中的部分工作正常。只是 ChangeUserSettingRequest 将首先执行。如果失败,请求永远不会到达控制器。因此,当ChangeUserSettingRequest 失败时,它应该将我发送回带有表单数据的表单。
  • 您能否在ChangeUserSettingsRequest 失败后使用showSettings() 方法中dump($request-&gt;input()) 的输出更新您的问题?
  • @Björn 在 Laravel 5 中,当 ChangeUserSettingsRequest 失败时,它永远不会到达控制器。它会自动将您发送回表单。
  • 是的,我知道,showSettings 方法是表单呈现的地方吗?如果你 dump($request-&gt;input()) 那里你可以看到所有输入数据 laravel 从 ValidationRequest 发回。

标签: php validation laravel laravel-5


【解决方案1】:

失败时将它们返回到带有输入数据的页面(withInput())

return Redirect::back()->withErrors(['current_password' => $message])->withInput($request->except('password'));

有关更多信息,请参阅与请求有关的 Laravel 文档...http://laravel.com/docs/5.1/requests

【讨论】:

  • 正如您在文件底部看到的 UserInfoController 我已经在做类似的事情,但不完全一样,但它的工作原理是一样的。我的问题是,当ChangeUserSettingsRequest 失败时,它会直接将我重定向回来,因此它永远不会到达 controller 如果它确实到达了控制器,我就不会遇到这个问题。
  • 我已经奖励了你,因为你给了我某种答案。它没有直接解决我的问题,但我认为这个答案将来会帮助其他人。
【解决方案2】:

我找到了!

formRequest 应该正确处理它,但不知道为什么没有。
我在查看文件 formRequest.php 时找到了解决方案 https://github.com/laravel/framework/blob/master/src/Illuminate/Foundation/Http/FormRequest.php

我必须覆盖 响应函数,它现在可以工作了!
我必须添加到我的ChangeUserSettingsRequest 的唯一内容就是这个

public function response(array $errors)
{
    return $this->redirector->to($this->getRedirectUrl())->withErrors($errors, $this->errorBag);
}

唯一的区别是我没有将输入字段发回,这有点奇怪,但我很高兴现在解决了。

【讨论】:

  • 你能不能只扩展 FormRequest 而不是 Request?这也能解决问题吗?
  • @JonasHartmann 我更新了这个问题。但这并没有帮助导致类 Request 扩展 formRequest。
猜你喜欢
  • 2021-03-24
  • 2019-03-27
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2016-09-04
  • 2016-12-24
  • 2013-03-21
相关资源
最近更新 更多