strimeapp /
Slackify
| Conditions | 46 |
| Paths | 21 |
| Total Lines | 112 |
| Code Lines | 70 |
| Lines | 4 |
| Ratio | 3.57 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 46 | public function block_page_content($context, array $blocks = array()) |
||
| 47 | { |
||
| 48 | // line 7 |
||
| 49 | echo " |
||
| 50 | <div class=\"page-header\"> |
||
| 51 | <h1>Index</h1> |
||
| 52 | </div> |
||
| 53 | |||
| 54 | <ul class=\"pagination\"> |
||
| 55 | "; |
||
| 56 | // line 13 |
||
| 57 | $context['_parent'] = $context; |
||
| 58 | $context['_seq'] = twig_ensure_traversable(range("A", "Z")); |
||
| 59 | foreach ($context['_seq'] as $context["_key"] => $context["letter"]) { |
||
| 60 | // line 14 |
||
| 61 | echo " "; |
||
| 62 | if ((twig_get_attribute($this->env, $this->getSourceContext(), ($context["items"] ?? null), $context["letter"], array(), "array", true, true) && (twig_length_filter($this->env, twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["items"]) || array_key_exists("items", $context) ? $context["items"] : (function () { throw new Twig_Error_Runtime('Variable "items" does not exist.', 14, $this->getSourceContext()); })()), $context["letter"], array(), "array")) > 1))) { |
||
| 63 | // line 15 |
||
| 64 | echo " <li><a href=\"#letter"; |
||
| 65 | echo $context["letter"]; |
||
| 66 | echo "\">"; |
||
| 67 | echo $context["letter"]; |
||
| 68 | echo "</a></li> |
||
| 69 | "; |
||
| 70 | } else { |
||
| 71 | // line 17 |
||
| 72 | echo " <li class=\"disabled\"><a href=\"#letter"; |
||
| 73 | echo $context["letter"]; |
||
| 74 | echo "\">"; |
||
| 75 | echo $context["letter"]; |
||
| 76 | echo "</a></li> |
||
| 77 | "; |
||
| 78 | } |
||
| 79 | // line 19 |
||
| 80 | echo " "; |
||
| 81 | } |
||
| 82 | $_parent = $context['_parent']; |
||
| 83 | unset($context['_seq'], $context['_iterated'], $context['_key'], $context['letter'], $context['_parent'], $context['loop']); |
||
| 84 | $context = array_intersect_key($context, $_parent) + $_parent; |
||
| 85 | // line 20 |
||
| 86 | echo " </ul> |
||
| 87 | |||
| 88 | "; |
||
| 89 | // line 22 |
||
| 90 | $context['_parent'] = $context; |
||
| 91 | $context['_seq'] = twig_ensure_traversable((isset($context["items"]) || array_key_exists("items", $context) ? $context["items"] : (function () { throw new Twig_Error_Runtime('Variable "items" does not exist.', 22, $this->getSourceContext()); })())); |
||
| 92 | foreach ($context['_seq'] as $context["letter"] => $context["elements"]) { |
||
| 93 | // line 23 |
||
| 94 | echo "<h2 id=\"letter"; |
||
| 95 | echo $context["letter"]; |
||
| 96 | echo "\">"; |
||
| 97 | echo $context["letter"]; |
||
| 98 | echo "</h2> |
||
| 99 | <dl id=\"index\">"; |
||
| 100 | // line 25 |
||
| 101 | $context['_parent'] = $context; |
||
| 102 | $context['_seq'] = twig_ensure_traversable($context["elements"]); |
||
| 103 | foreach ($context['_seq'] as $context["_key"] => $context["element"]) { |
||
| 104 | // line 26 |
||
| 105 | $context["type"] = twig_get_attribute($this->env, $this->getSourceContext(), $context["element"], 0, array(), "array"); |
||
| 106 | // line 27 |
||
| 107 | $context["value"] = twig_get_attribute($this->env, $this->getSourceContext(), $context["element"], 1, array(), "array"); |
||
| 108 | // line 28 |
||
| 109 | if (("class" == (isset($context["type"]) || array_key_exists("type", $context) ? $context["type"] : (function () { throw new Twig_Error_Runtime('Variable "type" does not exist.', 28, $this->getSourceContext()); })()))) { |
||
| 110 | // line 29 |
||
| 111 | echo "<dt>"; |
||
| 112 | echo $context["__internal_351d30495b226f3f6aab1ed2b80d7044af729b76f061877fc02bc5b20e6949bc"]->macro_class_link((isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 29, $this->getSourceContext()); })())); |
||
| 113 | if ((isset($context["has_namespaces"]) || array_key_exists("has_namespaces", $context) ? $context["has_namespaces"] : (function () { throw new Twig_Error_Runtime('Variable "has_namespaces" does not exist.', 29, $this->getSourceContext()); })())) { |
||
| 114 | echo " — <em>Class in namespace "; |
||
| 115 | echo $context["__internal_351d30495b226f3f6aab1ed2b80d7044af729b76f061877fc02bc5b20e6949bc"]->macro_namespace_link(twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 29, $this->getSourceContext()); })()), "namespace", array())); |
||
| 116 | } |
||
| 117 | echo "</em></dt> |
||
| 118 | <dd>"; |
||
| 119 | // line 30 |
||
| 120 | echo $this->env->getExtension('Sami\Renderer\TwigExtension')->parseDesc($context, twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 30, $this->getSourceContext()); })()), "shortdesc", array()), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 30, $this->getSourceContext()); })())); |
||
| 121 | echo "</dd>"; |
||
| 122 | } elseif (("method" == // line 31 |
||
| 123 | (isset($context["type"]) || array_key_exists("type", $context) ? $context["type"] : (function () { throw new Twig_Error_Runtime('Variable "type" does not exist.', 31, $this->getSourceContext()); })()))) { |
||
| 124 | // line 32 |
||
| 125 | echo "<dt>"; |
||
| 126 | echo $context["__internal_351d30495b226f3f6aab1ed2b80d7044af729b76f061877fc02bc5b20e6949bc"]->macro_method_link((isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 32, $this->getSourceContext()); })())); |
||
| 127 | echo "() — <em>Method in class "; |
||
| 128 | echo $context["__internal_351d30495b226f3f6aab1ed2b80d7044af729b76f061877fc02bc5b20e6949bc"]->macro_class_link(twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 32, $this->getSourceContext()); })()), "class", array())); |
||
| 129 | echo "</em></dt> |
||
| 130 | <dd>"; |
||
| 131 | // line 33 |
||
| 132 | echo $this->env->getExtension('Sami\Renderer\TwigExtension')->parseDesc($context, twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 33, $this->getSourceContext()); })()), "shortdesc", array()), twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 33, $this->getSourceContext()); })()), "class", array())); |
||
| 133 | echo "</dd>"; |
||
| 134 | } elseif (("property" == // line 34 |
||
| 135 | (isset($context["type"]) || array_key_exists("type", $context) ? $context["type"] : (function () { throw new Twig_Error_Runtime('Variable "type" does not exist.', 34, $this->getSourceContext()); })()))) { |
||
| 136 | // line 35 |
||
| 137 | echo "<dt>\$"; |
||
| 138 | echo $context["__internal_351d30495b226f3f6aab1ed2b80d7044af729b76f061877fc02bc5b20e6949bc"]->macro_property_link((isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 35, $this->getSourceContext()); })())); |
||
| 139 | echo " — <em>Property in class "; |
||
| 140 | echo $context["__internal_351d30495b226f3f6aab1ed2b80d7044af729b76f061877fc02bc5b20e6949bc"]->macro_class_link(twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 35, $this->getSourceContext()); })()), "class", array())); |
||
| 141 | echo "</em></dt> |
||
| 142 | <dd>"; |
||
| 143 | // line 36 |
||
| 144 | echo $this->env->getExtension('Sami\Renderer\TwigExtension')->parseDesc($context, twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 36, $this->getSourceContext()); })()), "shortdesc", array()), twig_get_attribute($this->env, $this->getSourceContext(), (isset($context["value"]) || array_key_exists("value", $context) ? $context["value"] : (function () { throw new Twig_Error_Runtime('Variable "value" does not exist.', 36, $this->getSourceContext()); })()), "class", array())); |
||
| 145 | echo "</dd>"; |
||
| 146 | } |
||
| 147 | } |
||
| 148 | $_parent = $context['_parent']; |
||
| 149 | unset($context['_seq'], $context['_iterated'], $context['_key'], $context['element'], $context['_parent'], $context['loop']); |
||
| 150 | $context = array_intersect_key($context, $_parent) + $_parent; |
||
| 151 | // line 39 |
||
| 152 | echo " </dl>"; |
||
| 153 | } |
||
| 154 | $_parent = $context['_parent']; |
||
| 155 | unset($context['_seq'], $context['_iterated'], $context['letter'], $context['elements'], $context['_parent'], $context['loop']); |
||
| 156 | $context = array_intersect_key($context, $_parent) + $_parent; |
||
| 157 | } |
||
| 158 | |||
| 220 |
