giumar / fortunecookies

src/Controller/AssetsController.php

<?php

Filename "AssetsController.php" doesn't match the expected filename "assetscontroller.php"

The PHP open tag does not have a corresponding PHP close tag

Header blocks must be separated by a single blank line

This file is missing a doc comment.

Class found in ".php" file; use ".inc" extension instead

namespace App\Controller;

Missing file doc comment

use App\Controller\AppController;

Unused use statement

/**
 * Assets Controller
 *
 * @property \App\Model\Table\AssetsTable $Assets
 *
 * @method \App\Model\Entity\Asset[]|\Cake\Datasource\ResultSetInterface paginate($object = null, array $settings = [])
 */

Missing @link tag in class comment

Missing @license tag in class comment

Missing @author tag in class comment

Missing @package tag in class comment

Missing @category tag in class comment

class AssetsController extends AppController
{

Opening brace should be on the same line as the declaration

Opening brace should be on the same line as the declaration for class AssetsController

    /**
     * Index method
     *
     * @return \Cake\Http\Response|void

@return doc comment specified, but function has no return statement

     */
    public function index()
    {

Opening brace should be on the same line as the declaration

        $assets = $this->paginate($this->Assets);

        $this->set(compact('assets'));
    }

    /**
     * View method
     *
     * @param string|null $id Asset id.

Parameter comment must be on the next line

     * @return \Cake\Http\Response|void

Separate the @param and @return sections by a blank line.

@return doc comment specified, but function has no return statement

     * @throws \Cake\Datasource\Exception\RecordNotFoundException When record not found.

@throws comment must be on the next line

Separate the @return and @throws sections by a blank line.

     */
    public function view($id = null)
    {

Opening brace should be on the same line as the declaration

        $asset = $this->Assets->get($id, [
            'contain' => []

A comma should follow the last multiline array item. Found: ]

        ]);

        $this->set('asset', $asset);
    }

    /**
     * Add method
     *
     * @return \Cake\Http\Response|null Redirects on successful add, renders view otherwise.
     */
    public function add()
    {

Opening brace should be on the same line as the declaration

        $asset = $this->Assets->newEntity();
        if ($this->request->is('post')) {
            $asset = $this->Assets->patchEntity($asset, $this->request->getData());
            if ($this->Assets->save($asset)) {
                $this->Flash->success(__('The asset has been saved.'));

                return $this->redirect(['action' => 'index']);
            }

No blank line found after control structure

            $this->Flash->error(__('The asset could not be saved. Please, try again.'));
        }

No blank line found after control structure

        $this->set(compact('asset'));
    }

    /**
     * Edit method
     *
     * @param string|null $id Asset id.

Parameter comment must be on the next line

     * @return \Cake\Http\Response|null Redirects on successful edit, renders view otherwise.

Separate the @param and @return sections by a blank line.

     * @throws \Cake\Network\Exception\NotFoundException When record not found.

Separate the @return and @throws sections by a blank line.

@throws comment must be on the next line

     */
    public function edit($id = null)
    {

Opening brace should be on the same line as the declaration

        $asset = $this->Assets->get($id, [
            'contain' => []

A comma should follow the last multiline array item. Found: ]

        ]);
        if ($this->request->is(['patch', 'post', 'put'])) {
            $asset = $this->Assets->patchEntity($asset, $this->request->getData());
            if ($this->Assets->save($asset)) {
                $this->Flash->success(__('The asset has been saved.'));

                return $this->redirect(['action' => 'index']);
            }

No blank line found after control structure

            $this->Flash->error(__('The asset could not be saved. Please, try again.'));
        }

No blank line found after control structure

        $this->set(compact('asset'));
    }

    /**
     * Delete method
     *
     * @param string|null $id Asset id.

Parameter comment must be on the next line

     * @return \Cake\Http\Response|null Redirects to index.

Separate the @param and @return sections by a blank line.

     * @throws \Cake\Datasource\Exception\RecordNotFoundException When record not found.

Separate the @return and @throws sections by a blank line.

@throws comment must be on the next line

     */
    public function delete($id = null)
    {

Opening brace should be on the same line as the declaration

        $this->request->allowMethod(['post', 'delete']);
        $asset = $this->Assets->get($id);
        if ($this->Assets->delete($asset)) {
            $this->Flash->success(__('The asset has been deleted.'));
        } else {

Expected newline after closing brace

            $this->Flash->error(__('The asset could not be deleted. Please, try again.'));
        }

        return $this->redirect(['action' => 'index']);
    }

Expected 1 blank line after function; 0 found

}