FileUploadController::validateAndUpload()

Complexity

Conditions	6
Paths	9

Size

Total Lines	41
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
 function validateAndUpload($folderName,$dbSave) {

It is generally recommended to explicitly declare the visibility for methods.

        $successStatus;

The variable $successStatus seems to be never defined.

        $fileErr;

The variable $fileErr seems to be never defined.

        // Build the input for our validation
        $input      = Input::all();

The type Input was not found. Maybe you did not declare it correctly or list all dependencies?

        // Within the ruleset, make sure we let the validator know that this
        // file should be an image
        $rules      = array(
             'file' => 'max:100000000',
        );
        $validation = Validator::make($input, $rules);

        if ($validation->fails()) {
                return Response::make($validation->errors->first(), 400);
        }

        $file = Input::file('file');
        foreach ($file as $files) {

                $upload_success = $files->move('fusionmate/public/plugins/' . $folderName . '', $files->getClientOriginalName());

                $imgPath = $files->getClientOriginalName();

                if ($upload_success) {
                        if($dbSave){
                             $this->taskRepository->saveTaskFiles($imgPath);   
                        }
                        $fileErr = false;
                }
                else {
                        $fileErr = true;
                }
        }
        if (!$fileErr) {

The variable $fileErr seems to be defined by a foreach iteration on line 42. Are you sure the iterator is never empty, otherwise this variable is not defined?

                $successStatus = Response::json('success', 200);
        }
        else {
                $successStatus = Response::json('error', 400);
        }
        return $successStatus;
}
}