YumlClient::makeScale() - Code Metrics - Inspection of "add yuml.me customization - fixes #6" - Nono1971/doctrine-yuml-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c87318...99f3de )

by Bruno

created 2018-02-22 15:55 UTC

YumlClient::makeScale() B

↳ Parent: YumlClient

Complexity

Conditions	5
Paths	5

Size

Total Lines	15
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	5

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	15
ccs	11
cts	11
cp	1
rs	8.8571
cc	5
eloc	12
nc	5
nop	1
crap	5

<?php

namespace Onurb\Bundle\YumlBundle\Yuml;

use Doctrine\Common\Persistence\Mapping\ClassMetadata;
use Doctrine\Common\Persistence\Mapping\ClassMetadataFactory as ClassMetadataFactoryInterface;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\Mapping\ClassMetadataFactory;
use Onurb\Bundle\YumlBundle\Curl\Curl;
use Onurb\Bundle\YumlBundle\Curl\CurlInterface;
use Onurb\Doctrine\ORMMetadataGrapher\YUMLMetadataGrapher as MetadataGrapher;
use Onurb\Doctrine\ORMMetadataGrapher\YUMLMetadataGrapherInterface as MetadataGrapherInterface;

/**
 * Utility to generate Yuml compatible strings from metadata graphs
 * Adaptation of DoctrineORMModule\Yuml\YumlController for ZendFramework-Zend-Developer-Tools
 *
 * @license MIT
 * @link    http://www.doctrine-project.org/
 * @author  Bruno Heron <[email protected]>
 * @author  Marco Pivetta <[email protected]>
 **/
class YumlClient implements YumlClientInterface
{
    const YUML_POST_URL = 'https://yuml.me/diagram/%STYLE%/class';
    const YUML_REDIRECT_URL = 'https://yuml.me/';

    protected $entityManager;

    protected $metadataFactory;

    protected $metadataGrapher;

    /**
     * @param EntityManagerInterface $entityManager
     * @param ClassMetadataFactoryInterface|null $classMetadataFactory
     * @param MetadataGrapherInterface|null $metadataGrapher
     */
    public function __construct(
        EntityManagerInterface          $entityManager,
        ClassMetadataFactoryInterface   $classMetadataFactory = null,
        MetadataGrapherInterface        $metadataGrapher = null
    ) {
        $this->entityManager = $entityManager;
        $this->metadataFactory = $classMetadataFactory ? $classMetadataFactory : new ClassMetadataFactory();
        $this->metadataFactory->setEntityManager($this->entityManager);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->metadataGrapher = $metadataGrapher ? $metadataGrapher : new MetadataGrapher();
    }


    /**
     * Get doctrine metadata as yuml.
     *
     * @param bool $showDetail
     * @param array $colors
     * @param array $notes
     * @return string
     */
    public function makeDslText($showDetail = false, $colors = array(), $notes = array())
    {
        return $this->metadataGrapher->generateFromMetadata(
            $this->getClasses(),
            $showDetail,
            $colors,
            $notes
        );
    }

    /**
     * Use yuml.me to generate an image from yuml.
     *
     * @param string $dsl_text
     * @param string $style the yuml style plain, boring or scruffy
     * @param string $extension the file extension to redirect to
     * @param string $direction the direction of the graph (LR,RL, TB)
     * @param string $scale the graph scale : huge, big, normal, small or tiny.
     *
     * @return string The url of the generated image.
     */
    public function getGraphUrl($dsl_text, $style, $extension, $direction, $scale)
    {
        $curl = new Curl($this->makePostUrl($style, $direction, $scale));
        $curl->setPosts(array('dsl_text' => $dsl_text));

        return self::YUML_REDIRECT_URL . $this->makeExtensionUrl($curl->getResponse(), $extension);
    }

    /**
     * @param string $graphUrl
     * @param string $filename
     * @return mixed
     */
    public function downloadImage($graphUrl, $filename, CurlInterface $curl = null)
    {
        $curl = $curl ? $curl : new Curl($graphUrl);
        $curl->setOutput($filename);

        return $curl->getResponse();
    }

    /**
     * @return ClassMetadata[]
     */
    private function getMetadata()
    {
        return $this->metadataFactory->getAllMetadata();
    }

    /**
     * @return array
     */
    private function getClasses()
    {
        $classes = array();
        /** @var ClassMetadata $class */
        foreach ($this->getMetadata() as $class) {
            $classes[$class->getName()] = $class;
        }
        ksort($classes);

        return $classes;
    }

    /**
     * @param string $style
     * @return string
     */
    private function makePostUrl($style, $direction, $scale)
    {
        return str_replace('%STYLE%', $this->makeStyle($style, $direction, $scale), self::YUML_POST_URL);
    }

    /**
     * @param string $return
     * @param string $extension
     * @return string
     */
    private function makeExtensionUrl($return, $extension)
    {
        return explode('.', $return)[0] . '.' . $this->checkExtension($extension);
    }

    private function makeStyle($style, $direction, $scale)
    {
        return $this->checkStyle($style) . $this->makeDirection($direction) . $this->makeScale($scale);
    }

    /**
     * @param string $direction
     * @return string
     */
    private function makeDirection($direction)
    {
        switch ($direction) {
            case 'LR':
            case 'RL':
                return ';dir:' . $direction;
            default:
                return '';
        }
    }

    /**
     * @param string $scale
     * @return string
     */
    private function makeScale($scale)
    {
        switch ($scale) {
            case 'huge':
                return ';scale:180';
            case 'big':
                return ';scale:120';
            case 'small':
                return ';scale:80';
            case 'tiny':
                return ';scale:60';
            default:
                return '';
        }
    }

    /**
     * @param string $extension
     * @return string
     */
    private function checkExtension($extension)
    {
        switch ($extension) {
            case 'jpg':
            case 'svg':
            case 'pdf':
            case 'json':
                return $extension;
            default:
                return 'png';
        }
    }

    private function checkStyle($style)
    {
        switch ($style) {
            case 'boring':
            case 'scruffy':
                return $style;
            default:
                return 'plain';
        }
    }
}


GitHub Access Token became invalid

Push — master ( c87318...99f3de )

YumlClient::makeScale() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

1		<?php
2
3		namespace Onurb\Bundle\YumlBundle\Yuml;
4
5		use Doctrine\Common\Persistence\Mapping\ClassMetadata;
6		use Doctrine\Common\Persistence\Mapping\ClassMetadataFactory as ClassMetadataFactoryInterface;
7		use Doctrine\ORM\EntityManagerInterface;
8		use Doctrine\ORM\Mapping\ClassMetadataFactory;
9		use Onurb\Bundle\YumlBundle\Curl\Curl;
10		use Onurb\Bundle\YumlBundle\Curl\CurlInterface;
11		use Onurb\Doctrine\ORMMetadataGrapher\YUMLMetadataGrapher as MetadataGrapher;
12		use Onurb\Doctrine\ORMMetadataGrapher\YUMLMetadataGrapherInterface as MetadataGrapherInterface;
13
14		/**
15		* Utility to generate Yuml compatible strings from metadata graphs
16		* Adaptation of DoctrineORMModule\Yuml\YumlController for ZendFramework-Zend-Developer-Tools
17		*
18		* @license MIT
19		* @link http://www.doctrine-project.org/
20		* @author Bruno Heron <[email protected]>
21		* @author Marco Pivetta <[email protected]>
22		**/
23		class YumlClient implements YumlClientInterface
24		{
25		const YUML_POST_URL = 'https://yuml.me/diagram/%STYLE%/class';
26		const YUML_REDIRECT_URL = 'https://yuml.me/';
27
28		protected $entityManager;
29
30		protected $metadataFactory;
31
32		protected $metadataGrapher;
33
34		/**
35		* @param EntityManagerInterface $entityManager
36		* @param ClassMetadataFactoryInterface\|null $classMetadataFactory
37		* @param MetadataGrapherInterface\|null $metadataGrapher
38		*/
39	4	public function __construct(
40		EntityManagerInterface $entityManager,
41		ClassMetadataFactoryInterface $classMetadataFactory = null,
42		MetadataGrapherInterface $metadataGrapher = null
43		) {
44	4	$this->entityManager = $entityManager;
45	4	$this->metadataFactory = $classMetadataFactory ? $classMetadataFactory : new ClassMetadataFactory();
46	4	$this->metadataFactory->setEntityManager($this->entityManager);
		0 ignored issues – show Bug introduced 2016-07-17 19:45 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Doctrine\Common\Persiste...ng\ClassMetadataFactory` as the method `setEntityManager()` does only exist in the following implementations of said interface: `Doctrine\ORM\Mapping\ClassMetadataFactory`, `Doctrine\ORM\Tools\Disco...tedClassMetadataFactory`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
47	4	$this->metadataGrapher = $metadataGrapher ? $metadataGrapher : new MetadataGrapher();
48	4	}
49
50
51		/**
52		* Get doctrine metadata as yuml.
53		*
54		* @param bool $showDetail
55		* @param array $colors
56		* @param array $notes
57		* @return string
58		*/
59	1	public function makeDslText($showDetail = false, $colors = array(), $notes = array())
60		{
61	1	return $this->metadataGrapher->generateFromMetadata(
62	1	$this->getClasses(),
63	1	$showDetail,
64	1	$colors,
65	1	$notes
66		);
67		}
68
69		/**
70		* Use yuml.me to generate an image from yuml.
71		*
72		* @param string $dsl_text
73		* @param string $style the yuml style plain, boring or scruffy
74		* @param string $extension the file extension to redirect to
75		* @param string $direction the direction of the graph (LR,RL, TB)
76		* @param string $scale the graph scale : huge, big, normal, small or tiny.
77		*
78		* @return string The url of the generated image.
79		*/
80	1	public function getGraphUrl($dsl_text, $style, $extension, $direction, $scale)
81		{
82	1	$curl = new Curl($this->makePostUrl($style, $direction, $scale));
83	1	$curl->setPosts(array('dsl_text' => $dsl_text));
84
85	1	return self::YUML_REDIRECT_URL . $this->makeExtensionUrl($curl->getResponse(), $extension);
86		}
87
88		/**
89		* @param string $graphUrl
90		* @param string $filename
91		* @return mixed
92		*/
93	1	public function downloadImage($graphUrl, $filename, CurlInterface $curl = null)
94		{
95	1	$curl = $curl ? $curl : new Curl($graphUrl);
96	1	$curl->setOutput($filename);
97
98	1	return $curl->getResponse();
99		}
100
101		/**
102		* @return ClassMetadata[]
103		*/
104	1	private function getMetadata()
105		{
106	1	return $this->metadataFactory->getAllMetadata();
107		}
108
109		/**
110		* @return array
111		*/
112	1	private function getClasses()
113		{
114	1	$classes = array();
115		/** @var ClassMetadata $class */
116	1	foreach ($this->getMetadata() as $class) {
117	1	$classes[$class->getName()] = $class;
118		}
119	1	ksort($classes);
120
121	1	return $classes;
122		}
123
124		/**
125		* @param string $style
126		* @return string
127		*/
128	1	private function makePostUrl($style, $direction, $scale)
129		{
130	1	return str_replace('%STYLE%', $this->makeStyle($style, $direction, $scale), self::YUML_POST_URL);
131		}
132
133		/**
134		* @param string $return
135		* @param string $extension
136		* @return string
137		*/
138	1	private function makeExtensionUrl($return, $extension)
139		{
140	1	return explode('.', $return)[0] . '.' . $this->checkExtension($extension);
141		}
142
143	1	private function makeStyle($style, $direction, $scale)
144		{
145	1	return $this->checkStyle($style) . $this->makeDirection($direction) . $this->makeScale($scale);
146		}
147
148		/**
149		* @param string $direction
150		* @return string
151		*/
152	1	private function makeDirection($direction)
153		{
154	1	switch ($direction) {
155	1	case 'LR':
156	1	case 'RL':
157	1	return ';dir:' . $direction;
158		default:
159	1	return '';
160		}
161		}
162
163		/**
164		* @param string $scale
165		* @return string
166		*/
167	1	private function makeScale($scale)
168		{
169	1	switch ($scale) {
170	1	case 'huge':
171	1	return ';scale:180';
172	1	case 'big':
173	1	return ';scale:120';
174	1	case 'small':
175	1	return ';scale:80';
176	1	case 'tiny':
177	1	return ';scale:60';
178		default:
179	1	return '';
180		}
181		}
182
183		/**
184		* @param string $extension
185		* @return string
186		*/
187	1	private function checkExtension($extension)
188		{
189	1	switch ($extension) {
190	1	case 'jpg':
191	1	case 'svg':
192	1	case 'pdf':
193	1	case 'json':
194	1	return $extension;
195		default:
196	1	return 'png';
197		}
198		}
199
200	1	private function checkStyle($style)
201		{
202	1	switch ($style) {
203	1	case 'boring':
204	1	case 'scruffy':
205	1	return $style;
206		default:
207	1	return 'plain';
208		}
209		}
210		}
211

Nono1971 / doctrine-yuml-bundle

GitHub Access Token became invalid

Push — master ( c87318...99f3de )

YumlClient::makeScale() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like