Html5Module::wrapImgInFigure() - Code Metrics - Inspection of "refactor wrapImgInFigure | added tests" - gwa/zero-module-html5 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d6ea78...1300b3 )

by Daniel

created 2016-02-10 15:12 UTC

Html5Module::wrapImgInFigure() A

↳ Parent: Html5Module

Complexity

Conditions	2
Paths	1

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	2

Importance

Changes	2
Bugs	0	Features	2

Metric	Value
c	2
b	0
f	2
dl	0
loc	22
ccs	13
cts	13
cp	1
rs	9.2
cc	2
eloc	12
nc	1
nop	1
crap	2

<?php
namespace Gwa\Wordpress\Zero\Module;

use Gwa\Wordpress\WpBridge\Traits\WpBridgeTrait;
use Gwa\Wordpress\Zero\Module\AbstractThemeModule;

class Html5Module extends AbstractThemeModule
{
    use WpBridgeTrait;

    /**
     * Wrap images with figure tag.
     * Courtesy of Interconnectit http://interconnectit.com/2175/how-to-remove-p-tags-from-images-in-wordpress/
     *
     * @param string $content
     *
     * @return string
     */
    public function wrapImgInFigure($content)
    {
        $callback = function ($matches) {
            $img     = $matches[1];
            $pattern = '/class="([^"]+)"/';

            preg_match($pattern, $img, $matches);

            $classes = $matches[1];
            $class   = '';

            if (isset($classes)) {
                $class = ' class="' . $classes . '"';
            }

            return '<figure' . $class . '>' . $img . '</figure>';
        };

        $content = preg_replace_callback('/(<a .*?><img.*?><\/a>|<img.*?>)/s', $callback, $content);

        return $content;
    }

    protected function doInit()
    {
        $this->getWpBridge()->addThemeSupport('html5', [
            'search-form',
            'comment-form',
            'comment-list',
            'gallery',
            'caption',
        ]);
    }

    /**
     * Override in concrete subclass.
     *
     * @return array

     */
    protected function getFilterMap()
    {
        return [
            [
                'hooks'  => 'the_content',
                'class'  => $this,
                'method' => 'wrapImgInFigure',
                'prio'   => 30,
                'args'   => 1,
            ],
        ];
    }
}


1		<?php
2		namespace Gwa\Wordpress\Zero\Module;
3
4		use Gwa\Wordpress\WpBridge\Traits\WpBridgeTrait;
5		use Gwa\Wordpress\Zero\Module\AbstractThemeModule;
6
7		class Html5Module extends AbstractThemeModule
8		{
9		use WpBridgeTrait;
10
11		/**
12		* Wrap images with figure tag.
13		* Courtesy of Interconnectit http://interconnectit.com/2175/how-to-remove-p-tags-from-images-in-wordpress/
14		*
15		* @param string $content
16		*
17		* @return string
18		*/
19		public function wrapImgInFigure($content)
20		{
21	1	$callback = function ($matches) {
22	1	$img = $matches[1];
23	1	$pattern = '/class="([^"]+)"/';
24
25	1	preg_match($pattern, $img, $matches);
26
27	1	$classes = $matches[1];
28	1	$class = '';
29
30	1	if (isset($classes)) {
31	1	$class = ' class="' . $classes . '"';
32	1	}
33
34	1	return '<figure' . $class . '>' . $img . '</figure>';
35	1	};
36
37	1	$content = preg_replace_callback('/(<a .?><img.?><\/a>\|<img.*?>)/s', $callback, $content);
38
39	1	return $content;
40		}
41
42	1	protected function doInit()
43		{
44	1	$this->getWpBridge()->addThemeSupport('html5', [
45	1	'search-form',
46	1	'comment-form',
47	1	'comment-list',
48	1	'gallery',
49	1	'caption',
50	1	]);
51	1	}
52
53		/**
54		* Override in concrete subclass.
55		*
56		* @return array
		0 ignored issues – show Documentation introduced 2016-01-28 13:55 UTC by Report Bug Copy Issue Report Consider making the return type a bit more specific; maybe use `array<string,string\|Html5Module\|integer>[]`. This check looks for the generic type `array` as a return type and suggests a more specific type. This type is inferred from the actual code. Loading history...
57		*/
58	1	protected function getFilterMap()
59		{
60		return [
61		[
62	1	'hooks' => 'the_content',
63	1	'class' => $this,
64	1	'method' => 'wrapImgInFigure',
65	1	'prio' => 30,
66	1	'args' => 1,
67	1	],
68	1	];
69		}
70		}
71

gwa / zero-module-html5

Push — master ( d6ea78...1300b3 )

Html5Module::wrapImgInFigure() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like