Laravel: even though im submitting an input. Im still getting the "input fields must be filled up" message

Question

0.00/5 (No votes)

See more:

PHP

public function evalstudap(Request $request) {
        
        $start = $request->start;
        $end = $request->end;
        $date = $request->date;
        $message = $request->message;
        $id = $request->input('id');
        $user = DB::table('studapforms')->where('transno', $id)->first();

        if(empty($message)&&empty($start)&&empty($end)&&empty($date)) {
            return redirect()->back()->with('message', "Input fields must be filled up.");
        }else {

            if($user) {
                MailController::notifeval($user->email,$request->start, $request->end, $request->date, $request->message);

                return redirect()->back()->with('emessage', "Student has been notified");
            }else {
                return redirect()->back()->with('message', "Refresh the page and try again");
            }
        }

        return view('/studap/admin/evalstudap ', ['list' => $user]);
    }

What I have tried:

i still havent thought of an alternative for this one. ive been stuck here for days now please help me

Posted 9-Jul-21 1:01am

My Jinji

Updated 9-Jul-21 2:13am

Chris Copeland

v2

Add a Solution

Comments

Richard MacCutchan 9-Jul-21 7:11am

You need to check exactly what values are in those fields at the time this function is called. The fact that all are empty is suspicious as it suggests that $request has not been set correctly. Also, if all fields must be filled in then your test should really be using the or (||) operator, rather than &&.

My Jinji 10-Jul-21 2:10am

ive echoed all of the variable and all of them displays the correct value that the user inputs. I've done the || but its still the same.

Richard MacCutchan 10-Jul-21 4:02am

Looking at your new code I cannot see how that could happen. I am afraid I have nothing further that I can suggest.

My Jinji 10-Jul-21 5:27am

but still thank you so much

Chris Copeland 9-Jul-21 8:14am

How come you're using $request->input('id') for one field, yet using dynamic properties ($request->message) for the others? Have you tried replacing the dynamic properties with the input() method to see whether you get different results? $request->input('message');

My Jinji 10-Jul-21 2:11am

i have changed it into like that but its still the same

My Jinji 10-Jul-21 2:14am

also, the input ('id') came from a get method and I've called it to my post method. here is the full code including the get method.

public function evaluation(Request $request) {
$transno = request('id');
$user = DB::table('studapforms')->where('transno', $transno)->get();
$this->evalstudap($request, $transno);
return view('/studap/admin/evalstudap', ['list' => $user]);

}

public function evalstudap(Request $request) {

echo $start = $request->input('start');
echo $end = $request->input('end');
echo $date = $request->input('date');
echo $message = $request->input('message');
$id = $request->input('id');
$user = DB::table('studapforms')->where('transno', $id)->first();

if(empty($message)||empty($start)||empty($end)||empty($date)) {
return redirect()->back()->with('message', "Input fields must be filled up.");
}else {

if($user) {
MailController::notifeval($user->email,$start, $end, $date, $message);

return redirect()->back()->with('emessage', "Student has been notified");
}else {
return redirect()->back()->with('message', "Refresh the page and try again");
}
}

return view('/studap/admin/evalstudap ', ['list' => $user]);
}

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)