Renderer::addMessages() - Code Metrics - bramstroker/zf2-form - Measure and Improve Code Quality continuously with Scrutinizer

Renderer::addMessages() A
last analyzed 2017-06-01 07:23 UTC

↳ Parent: Renderer

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php
/**
 * Renderer for the jquery.validate plugin
 *
 * @category  StrokerForm
 * @package   StrokerForm\Renderer
 * @copyright 2012 Bram Gerritsen
 * @version   SVN: $Id$
 */

namespace StrokerForm\Renderer\JqueryValidate;

use StrokerForm\Renderer\AbstractValidateRenderer;
use StrokerForm\Renderer\JqueryValidate\Rule\RuleInterface;
use StrokerForm\Renderer\JqueryValidate\Rule\RulePluginManager;
use Zend\Form\Element\Email;
use Zend\Form\ElementInterface;
use Zend\Form\FormInterface;
use Zend\Json\Json;
use Zend\Validator\EmailAddress;
use Zend\Validator\Regex;
use Zend\Validator\ValidatorInterface;
use Zend\View\Renderer\PhpRenderer as View;

class Renderer extends AbstractValidateRenderer
{
    /**
     * @var array
     */
    protected $skipValidators = [
        'Explode',
        'Upload'
    ];

    /**
     * @var array
     */
    protected $rules = [];

    /**
     * @var array
     */
    protected $messages = [];

    /**
     * @var RulePluginManager
     */
    protected $rulePluginManager;

    /**
     * @param RulePluginManager $rulePluginManager
     */
    public function setRulePluginManager(RulePluginManager $rulePluginManager)
    {
        $this->rulePluginManager = $rulePluginManager;
    }

    /**
     * @return RulePluginManager
     */
    public function getRulePluginManager()
    {
        return $this->rulePluginManager;
    }

    /**
     * Executed before the ZF2 view helper renders the element
     *
     * @param string                          $formAlias
     * @param \Zend\View\Renderer\PhpRenderer $view
     * @param \Zend\Form\FormInterface        $form
     * @param array                           $options
     *
     * @return FormInterface
     */
    public function preRenderForm($formAlias, View $view, FormInterface $form = null, array $options = [])
    {
        $form = parent::preRenderForm($formAlias, $view, $form, $options);

        /** @var $options Options */
        $options = $this->getOptions();

        $inlineScript = $view->plugin('inlineScript');
        $inlineScript->appendScript($this->buildInlineJavascript($form, $options));

        if ($options->getIncludeAssets()) {
            $assetBaseUri = $this->getHttpRouter()->assemble([], ['name' => 'strokerform-asset']);
            $inlineScript->appendFile($assetBaseUri . '/jquery_validate/js/jquery.validate.js');
            $inlineScript->appendFile($assetBaseUri . '/jquery_validate/js/custom_rules.js');
            if ($options->isUseTwitterBootstrap() === true) {
                $inlineScript->appendFile($assetBaseUri . '/jquery_validate/js/jquery.validate.bootstrap.js');
            }
        }

        $this->reset();
        return $form;
    }

    /**
     * @param  \Zend\Form\FormInterface $form
     * @param Options                   $options
     *
     * @return string
     */
    protected function buildInlineJavascript(FormInterface $form, Options $options)
    {
        $validateOptions = [];
        foreach ($options->getValidateOptions() as $key => $value) {
            $value = (is_string($value)) ? $value : var_export($value, true);
            $validateOptions[] = '"' . $key . '": ' . $value;
        }

        return sprintf(
            $options->getInitializeTrigger(),
            sprintf(
                '$(\'form[name="%s"]\').each(function() { $(this).validate({%s"rules":%s,"messages":%s}); });',
                $form->getName(),
                count($validateOptions) > 0 ? implode(',', $validateOptions) . ',' : '',
                Json::encode($this->rules),
                Json::encode($this->messages)
            )
        );
    }

    /**
     * @param string                             $formAlias
     * @param \Zend\Form\ElementInterface        $element
     * @param \Zend\Validator\ValidatorInterface $validator
     *
     * @return mixed|void
     */
    protected function addValidationAttributesForElement($formAlias, ElementInterface $element, ValidatorInterface $validator = null)
    {
        if (in_array($this->getValidatorClassName($validator), $this->skipValidators)) {
            return;
        }
        if ($element instanceof Email && $validator instanceof Regex) {
            $validator = new EmailAddress();
        }

        $rule = $this->getRule($validator);
        $rules = [];
        if ($rule !== null) {
            $rules = $rule->getRules($validator, $element);
            $messages = $rule->getMessages($validator);
        } elseif (!$this->getOptions()->isDisableAjaxFallback()) {
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends 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.
    }
}
            //fallback ajax
            $ajaxUri = $this->getHttpRouter()->assemble(['form' => $formAlias], ['name' => 'strokerform-ajax-validate']);
            $rules = [
                'remote' => [
                    'url'  => $ajaxUri,
                    'type' => 'POST'
                ]
            ];
            $messages = [];
        }

        if (count($rules) > 0) {
            $elementName = $this->getElementName($element);
            $this->addRules($elementName, $rules);
            $this->addMessages($elementName, $messages);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        }
    }

    /**
     * @param  \Zend\Validator\ValidatorInterface $validator
     *
     * @return null|RuleInterface
     */
    public function getRule(ValidatorInterface $validator = null)
    {
        foreach ($this->getRulePluginManager()->getRegisteredServices() as $rules) {
            foreach ($rules as $rule) {
                $ruleInstance = $this->getRulePluginManager()->get($rule);
                if ($ruleInstance->canHandle($validator)) {
                    return $ruleInstance;
                }
            }
        }

        return;
    }

    /**
     * @param string $elementName
     * @param array  $rules
     */
    protected function addRules($elementName, array $rules = [])
    {
        if (!isset($this->rules[$elementName])) {
            $this->rules[$elementName] = [];
        }
        $this->rules[$elementName] = array_merge($this->rules[$elementName], $rules);
    }

    /**
     * @param string $elementName
     * @param array  $messages
     */
    protected function addMessages($elementName, array $messages = [])
    {
        if (!isset($this->messages[$elementName])) {
            $this->messages[$elementName] = [];
        }
        $this->messages[$elementName] = array_merge($this->messages[$elementName], $messages);
    }

    /**
     * Resets previously set rules and messages, if you have multiple forms on one request
     */
    protected function reset()
    {
        $this->rules = [];
        $this->messages = [];
    }
}


GitHub Access Token became invalid

Renderer::addMessages() A
last analyzed 2017-06-01 07:23 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

1			<?php
2			/**
3			* Renderer for the jquery.validate plugin
4			*
5			* @category StrokerForm
6			* @package StrokerForm\Renderer
7			* @copyright 2012 Bram Gerritsen
8			* @version SVN: $Id$
9			*/
10
11			namespace StrokerForm\Renderer\JqueryValidate;
12
13			use StrokerForm\Renderer\AbstractValidateRenderer;
14			use StrokerForm\Renderer\JqueryValidate\Rule\RuleInterface;
15			use StrokerForm\Renderer\JqueryValidate\Rule\RulePluginManager;
16			use Zend\Form\Element\Email;
17			use Zend\Form\ElementInterface;
18			use Zend\Form\FormInterface;
19			use Zend\Json\Json;
20			use Zend\Validator\EmailAddress;
21			use Zend\Validator\Regex;
22			use Zend\Validator\ValidatorInterface;
23			use Zend\View\Renderer\PhpRenderer as View;
24
25			class Renderer extends AbstractValidateRenderer
26			{
27			/**
28			* @var array
29			*/
30			protected $skipValidators = [
31			'Explode',
32			'Upload'
33			];
34
35			/**
36			* @var array
37			*/
38			protected $rules = [];
39
40			/**
41			* @var array
42			*/
43			protected $messages = [];
44
45			/**
46			* @var RulePluginManager
47			*/
48			protected $rulePluginManager;
49
50			/**
51			* @param RulePluginManager $rulePluginManager
52			*/
53			public function setRulePluginManager(RulePluginManager $rulePluginManager)
54			{
55			$this->rulePluginManager = $rulePluginManager;
56			}
57
58			/**
59			* @return RulePluginManager
60			*/
61			public function getRulePluginManager()
62			{
63			return $this->rulePluginManager;
64			}
65
66			/**
67			* Executed before the ZF2 view helper renders the element
68			*
69			* @param string $formAlias
70			* @param \Zend\View\Renderer\PhpRenderer $view
71			* @param \Zend\Form\FormInterface $form
72			* @param array $options
73			*
74			* @return FormInterface
75			*/
76			public function preRenderForm($formAlias, View $view, FormInterface $form = null, array $options = [])
77			{
78			$form = parent::preRenderForm($formAlias, $view, $form, $options);
79
80			/** @var $options Options */
81			$options = $this->getOptions();
82
83			$inlineScript = $view->plugin('inlineScript');
84			$inlineScript->appendScript($this->buildInlineJavascript($form, $options));
85
86			if ($options->getIncludeAssets()) {
87			$assetBaseUri = $this->getHttpRouter()->assemble([], ['name' => 'strokerform-asset']);
88			$inlineScript->appendFile($assetBaseUri . '/jquery_validate/js/jquery.validate.js');
89			$inlineScript->appendFile($assetBaseUri . '/jquery_validate/js/custom_rules.js');
90			if ($options->isUseTwitterBootstrap() === true) {
91			$inlineScript->appendFile($assetBaseUri . '/jquery_validate/js/jquery.validate.bootstrap.js');
92			}
93			}
94
95			$this->reset();
96			return $form;
97			}
98
99			/**
100			* @param \Zend\Form\FormInterface $form
101			* @param Options $options
102			*
103			* @return string
104			*/
105			protected function buildInlineJavascript(FormInterface $form, Options $options)
106			{
107			$validateOptions = [];
108			foreach ($options->getValidateOptions() as $key => $value) {
109			$value = (is_string($value)) ? $value : var_export($value, true);
110			$validateOptions[] = '"' . $key . '": ' . $value;
111			}
112
113			return sprintf(
114			$options->getInitializeTrigger(),
115			sprintf(
116			'$(\'form[name="%s"]\').each(function() { $(this).validate({%s"rules":%s,"messages":%s}); });',
117			$form->getName(),
118			count($validateOptions) > 0 ? implode(',', $validateOptions) . ',' : '',
119			Json::encode($this->rules),
120			Json::encode($this->messages)
121			)
122			);
123			}
124
125			/**
126			* @param string $formAlias
127			* @param \Zend\Form\ElementInterface $element
128			* @param \Zend\Validator\ValidatorInterface $validator
129			*
130			* @return mixed\|void
131			*/
132			protected function addValidationAttributesForElement($formAlias, ElementInterface $element, ValidatorInterface $validator = null)
133			{
134			if (in_array($this->getValidatorClassName($validator), $this->skipValidators)) {
135			return;
136			}
137			if ($element instanceof Email && $validator instanceof Regex) {
138			$validator = new EmailAddress();
139			}
140
141			$rule = $this->getRule($validator);
142			$rules = [];
143			if ($rule !== null) {
144			$rules = $rule->getRules($validator, $element);
145			$messages = $rule->getMessages($validator);
146			} elseif (!$this->getOptions()->isDisableAjaxFallback()) {
			0 ignored issues – show Bug introduced 2016-09-12 15:00 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `StrokerForm\Renderer\Options` as the method `isDisableAjaxFallback()` does only exist in the following sub-classes of `StrokerForm\Renderer\Options`: `StrokerForm\Renderer\JqueryValidate\Options`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends 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 sub-classes 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 parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
147			//fallback ajax
148			$ajaxUri = $this->getHttpRouter()->assemble(['form' => $formAlias], ['name' => 'strokerform-ajax-validate']);
149			$rules = [
150			'remote' => [
151			'url' => $ajaxUri,
152			'type' => 'POST'
153			]
154			];
155			$messages = [];
156			}
157
158			if (count($rules) > 0) {
159			$elementName = $this->getElementName($element);
160			$this->addRules($elementName, $rules);
161			$this->addMessages($elementName, $messages);
			0 ignored issues – show Bug introduced 2016-09-12 15:00 UTC by Report Bug Copy Issue Report The variable `$messages` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
162			}
163			}
164
165			/**
166			* @param \Zend\Validator\ValidatorInterface $validator
167			*
168			* @return null\|RuleInterface
169			*/
170			public function getRule(ValidatorInterface $validator = null)
171			{
172			foreach ($this->getRulePluginManager()->getRegisteredServices() as $rules) {
173			foreach ($rules as $rule) {
174			$ruleInstance = $this->getRulePluginManager()->get($rule);
175			if ($ruleInstance->canHandle($validator)) {
176			return $ruleInstance;
177			}
178			}
179			}
180
181			return;
182			}
183
184			/**
185			* @param string $elementName
186			* @param array $rules
187			*/
188			protected function addRules($elementName, array $rules = [])
189			{
190			if (!isset($this->rules[$elementName])) {
191			$this->rules[$elementName] = [];
192			}
193			$this->rules[$elementName] = array_merge($this->rules[$elementName], $rules);
194			}
195
196			/**
197			* @param string $elementName
198			* @param array $messages
199			*/
200			protected function addMessages($elementName, array $messages = [])
201			{
202			if (!isset($this->messages[$elementName])) {
203			$this->messages[$elementName] = [];
204			}
205			$this->messages[$elementName] = array_merge($this->messages[$elementName], $messages);
206			}
207
208			/**
209			* Resets previously set rules and messages, if you have multiple forms on one request
210			*/
211			protected function reset()
212			{
213			$this->rules = [];
214			$this->messages = [];
215			}
216			}
217

bramstroker / zf2-form

GitHub Access Token became invalid

Renderer::addMessages() A last analyzed 2017-06-01 07:23 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

Renderer::addMessages() A
last analyzed 2017-06-01 07:23 UTC