App\Controller\OrganizationsController

Complexity

Total Complexity

10

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	view()	0	7	1
A	add()	0	14	3
A	delete()	0	11	2
A	edit()	0	16	3
A	index()	0	8	1

<?php

Filename "OrganizationsController.php" doesn't match the expected filename "organizationscontroller.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

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

Missing @package tag in class comment

Missing @link tag in class comment

Missing @license tag in class comment

Missing @author tag in class comment

Missing @category tag in class comment

class OrganizationsController extends AppController
{

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

Opening brace should be on the same line as the declaration

    /**
     * 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

        $this->paginate = [
            'contain' => ['Users']

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

        ];
        $organizations = $this->paginate($this->Organizations);

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

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

    /**
     * View method
     *
     * @param string|null $id Organization 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

        $organization = $this->Organizations->get($id, [
            'contain' => ['Users']

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

        ]);

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

    /**
     * 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

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

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

No blank line found after control structure

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

No blank line found after control structure

        $users = $this->Organizations->Users->find('list', ['limit' => 200]);
        $this->set(compact('organization', 'users'));
    }

    /**
     * Edit method
     *
     * @param string|null $id Organization 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

        $organization = $this->Organizations->get($id, [
            'contain' => []

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

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

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

No blank line found after control structure

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

No blank line found after control structure

        $users = $this->Organizations->Users->find('list', ['limit' => 200]);
        $this->set(compact('organization', 'users'));
    }

    /**
     * Delete method
     *
     * @param string|null $id Organization 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.

@throws comment must be on the next line

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

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

Opening brace should be on the same line as the declaration

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

Expected newline after closing brace

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

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

Expected 1 blank line after function; 0 found

}