User::updatePhoto() - Code Metrics - Inspection of "Now session_start() is in AppController" - henrystivens/upload-image-and-data - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 024767...e7bc73 )

by Henry Stivens

created 2017-03-13 01:36 UTC

User::updatePhoto() A

↳ Parent: User

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	4
nc	2
nop	0
dl	0
loc	7
rs	9.4285
c	0
b	0
f	0

<?php

/**
 * Clase para manejar los datos del usuario, tabla 'user'
 */
class User extends ActiveRecord {

    /**
     * Guarda un usuario y sube la foto de un usuario.
     * 
     * @param array $data Arreglo con los datos de usuario
     * @return boolean
     * @throws Exception
     */
    public function saveWithPhoto($data) {
        //Inicia la transacción
        $this->begin();
        if ($this->create($data)) {
            //Intenta actualizar la foto
            if($this->updatePhoto()){
                //Se confirma la transacción
                $this->commit();
                return true;
            }           
        } 
        
        //Si alga falla se regresa la transacción
        $this->rollback();
        return false;        
    }

    /**
     * Sube y actualiza la foto del usuario.
     * 
     * @return boolean|null
     */
    public function updatePhoto() {
        if ($photo = $this->uploadPhoto('photo')) {
            //Actualiza el campo photo
            $this->photo = $photo;
            return $this->update();
        }
    }

    /**
     * Sube la foto y retorna el nombre del archivo generado.
     * 
     * @param string $imageField
     * @return string|false
     */
    public function uploadPhoto($imageField) {
        //Usamos el adapter 'image'
        $file = Upload::factory($imageField, 'image');
        $fileName = false;
        //le asignamos las extensiones a permitir
        $file->setExtensions(array('jpg', 'png', 'gif'));
        //Intenta subir el arhivo
        if ($file->isUploaded()) {
            //Lo guarda usando un nombre de archivo aleatorio
            $fileName = $file->saveRandom();
        }
        
        return $fileName;
    }

}


1			<?php
2
3			/**
4			* Clase para manejar los datos del usuario, tabla 'user'
5			*/
6			class User extends ActiveRecord {
7
8			/**
9			* Guarda un usuario y sube la foto de un usuario.
10			*
11			* @param array $data Arreglo con los datos de usuario
12			* @return boolean
13			* @throws Exception
14			*/
15			public function saveWithPhoto($data) {
16			//Inicia la transacción
17			$this->begin();
18			if ($this->create($data)) {
19			//Intenta actualizar la foto
20			if($this->updatePhoto()){
21			//Se confirma la transacción
22			$this->commit();
23			return true;
24			}
25			}
26
27			//Si alga falla se regresa la transacción
28			$this->rollback();
29			return false;
30			}
31
32			/**
33			* Sube y actualiza la foto del usuario.
34			*
35			* @return boolean\|null
36			*/
37			public function updatePhoto() {
38			if ($photo = $this->uploadPhoto('photo')) {
39			//Actualiza el campo photo
40			$this->photo = $photo;
41			return $this->update();
42			}
43			}
44
45			/**
46			* Sube la foto y retorna el nombre del archivo generado.
47			*
48			* @param string $imageField
49			* @return string\|false
50			*/
51			public function uploadPhoto($imageField) {
52			//Usamos el adapter 'image'
53			$file = Upload::factory($imageField, 'image');
54			$fileName = false;
55			//le asignamos las extensiones a permitir
56			$file->setExtensions(array('jpg', 'png', 'gif'));
57			//Intenta subir el arhivo
58			if ($file->isUploaded()) {
59			//Lo guarda usando un nombre de archivo aleatorio
60			$fileName = $file->saveRandom();
61			}
62
63			return $fileName;
64			}
65
66			}
67

henrystivens / upload-image-and-data

Push — master ( 024767...e7bc73 )

User::updatePhoto() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like