ClickStatistic::__construct() - Code Metrics - Inspection of "4.9" - imagecms/ImageCMS - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — development ( eb9524...db4517 )

by Andrij

created 2016-02-25 14:36 UTC

ClickStatistic::__construct() A

↳ Parent: ClickStatistic

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	9
rs	9.6666
cc	2
eloc	6
nc	2
nop	2

<?php

namespace xbanners\src\Statistic;

use xbanners\models\BannerImageQuery;

/**
 * Click Statistic class
 */
class ClickStatistic
{


    /**
     * Banner image object
     * @var array|\Banners\Models\BannerImage|mixed
     */
    private $bannerImage;

    public function __construct($imageIdMd5, $locale = NULL) {

        $locale = $locale ? $locale : \MY_Controller::getCurrentLocale();

        $this->bannerImage = BannerImageQuery::create()
            ->joinWithI18n($locale)
            ->where('md5(BannerImage.Id) = ?', $imageIdMd5)
            ->findOne();
    }

    /**
     * Run click statistic
     */
    public function run() {

        if ($this->bannerImage->getUrl()) {
            $this->increaseClicks();
            $this->goToUrl();
        } else {
            \CI::$APP->core->error_404();
        }
    }

    /**
     * Increase banner image clicks value
     * @throws \Propel\Runtime\Exception\PropelException

     */
    public function increaseClicks() {

        $this->bannerImage->setClicks($this->bannerImage->getClicks() + 1);
        $this->bannerImage->save();
    }

    /**
     * Go to banner image url
     */
    private function goToUrl() {

        redirect($this->bannerImage->getUrl(), 'location', '301');
    }

    /**
     * Get click statistic
     * @param $imageId - banner image id

     * @return string
     */
    public static function getUrl($imageId) {

        return site_url('xbanners/go') . '/' . md5($imageId);
    }

}

1			<?php
2
3			namespace xbanners\src\Statistic;
4
5			use xbanners\models\BannerImageQuery;
6
7			/**
8			* Click Statistic class
9			*/
10			class ClickStatistic
11			{
			0 ignored issues – show introduced 2016-02-25 15:02 UTC by Report Bug Copy Issue Report Opening brace of a class must be on the same line as the definition Loading history...
12
13			/**
14			* Banner image object
15			* @var array\|\Banners\Models\BannerImage\|mixed
16			*/
17			private $bannerImage;
18
19			public function __construct($imageIdMd5, $locale = NULL) {
20
21			$locale = $locale ? $locale : \MY_Controller::getCurrentLocale();
22
23			$this->bannerImage = BannerImageQuery::create()
24			->joinWithI18n($locale)
25			->where('md5(BannerImage.Id) = ?', $imageIdMd5)
26			->findOne();
27			}
28
29			/**
30			* Run click statistic
31			*/
32			public function run() {
33
34			if ($this->bannerImage->getUrl()) {
35			$this->increaseClicks();
36			$this->goToUrl();
37			} else {
38			\CI::$APP->core->error_404();
39			}
40			}
41
42			/**
43			* Increase banner image clicks value
44			* @throws \Propel\Runtime\Exception\PropelException
			0 ignored issues – show introduced 2015-11-28 17:40 UTC by Report Bug Copy Issue Report Comment missing or not on the next line for @throws tag in function comment Loading history...
45			*/
46			public function increaseClicks() {
47
48			$this->bannerImage->setClicks($this->bannerImage->getClicks() + 1);
49			$this->bannerImage->save();
50			}
51
52			/**
53			* Go to banner image url
54			*/
55			private function goToUrl() {
56
57			redirect($this->bannerImage->getUrl(), 'location', '301');
58			}
59
60			/**
61			* Get click statistic
62			* @param $imageId - banner image id
			0 ignored issues – show introduced 2015-11-28 17:40 UTC by Report Bug Copy Issue Report Missing parameter type Loading history...
63			* @return string
64			*/
65			public static function getUrl($imageId) {
66
67			return site_url('xbanners/go') . '/' . md5($imageId);
68			}
69
70			}

imagecms / ImageCMS

Push — development ( eb9524...db4517 )

ClickStatistic::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like