ProdutoEstoqueController

Complexity

Total Complexity

18

Size/Duplication

Total Lines	98
Duplicated Lines	19.39 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	validar_estoque()	8	23	5
A	getUserActive()	11	11	1
B	diminuir_estoque_produto()	0	27	6
B	diminuir_estoque_produto_variacao()	0	30	6

<?php

class ProdutoEstoqueController extends AppController {

	public function validar_estoque($produto, $quantidade) {
		if (empty($produto)) {
			return false;
		}

		$user_active = $this->getUserActive($produto[0]['Produto']['id_usuario']);

		if ($user_active[0]['Usuario']['sale_without_stock'])
			return true;

		if ($produto[0]['Produto']['estoque'] <= 0) {
			$this->Session->setFlash('O Produto ' . $produto[0]['Produto']['name'] . ' selecionado não possui estoque disponivel');
			return false;
		}


		if ($produto[0]['Produto']['estoque'] < $quantidade) {
			$this->Session->setFlash('A quantidade para o produto ' . $produto[0]['Produto']['name'] . ' escolhida é maior do que a disponivel');
			return false;
		}

		return true;
	}

	public function getUserActive($id) {
		$this->loadModel('Usuario');

		$user = $this->Usuario->find('all', 
			array('conditions' => 
				array('Usuario.id' => $id)
			)
		);
		
		return $user;
	}

	public function diminuir_estoque_produto($produto_id, $quantidade) {
		if (!isset($produto_id) || !isset($quantidade) || !is_numeric($quantidade)) {
			return false;
		}

		try {
			$this->loadModel('Produto');

			$produto = $this->Produto->find('first', array(
					'conditions' => array('Produto.id' => $produto_id),
					'order' => array('Produto.id' => 'desc')
				)
			);

			$novo_estoque['estoque'] = $produto['Produto']['estoque'] - $quantidade;

$novo_estoque was never initialized. Although not strictly required by PHP, it is generally a good practice to add $novo_estoque = array(); before regardless.

			$this->Produto->id = $produto_id;
			if (!$this->Produto->save($novo_estoque)) {
				return false;
			}

			return true;
		} catch (Exception $e) {
			print_r($e);
			exit();

The method diminuir_estoque_produto() contains an exit expression.

		}
 	}

 	public function diminuir_estoque_produto_variacao($produto_id, $quantidade, $variacao) {
 		if (!isset($produto_id) || !isset($quantidade) || !isset($variacao)) {
 			return false;
 		}

 		try {
 			$this->loadModel('Variacao');

 			$variacao = $this->Variacao->find('first', array(
 					'conditions' => array(
 						'Variacao.produto_id' => $produto_id,
 						'Variacao.ativo' => 1,
 						'Variacao.nome_variacao' => $variacao
 					)
 				)
 			);

			$novo_estoque['estoque'] = $variacao['Variacao']['estoque'] - $quantidade;

$novo_estoque was never initialized. Although not strictly required by PHP, it is generally a good practice to add $novo_estoque = array(); before regardless.

			$this->Variacao->id = $variacao['Variacao']['id'];
			if (!$this->Variacao->save($novo_estoque)) {
				return false;
			}

			return true;
 		} catch (Exception $e) {
 			print_r($e);
 			exit();

The method diminuir_estoque_produto_variacao() contains an exit expression.

 		}
 	}

}