valkyrjaio /
valkyrja
| Conditions | 1 |
| Paths | 1 |
| Total Lines | 65 |
| Code Lines | 63 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 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 |
||
| 126 | public function asPhrase(): string |
||
| 127 | { |
||
| 128 | return match ($this) { |
||
| 129 | self::CONTINUE => StatusText::CONTINUE, |
||
| 130 | self::SWITCHING_PROTOCOLS => StatusText::SWITCHING_PROTOCOLS, |
||
| 131 | self::PROCESSING => StatusText::PROCESSING, |
||
| 132 | self::EARLY_HINTS => StatusText::EARLY_HINTS, |
||
| 133 | self::OK => StatusText::OK, |
||
| 134 | self::CREATED => StatusText::CREATED, |
||
| 135 | self::ACCEPTED => StatusText::ACCEPTED, |
||
| 136 | self::NON_AUTHORITATIVE_INFORMATION => StatusText::NON_AUTHORITATIVE_INFORMATION, |
||
| 137 | self::NO_CONTENT => StatusText::NO_CONTENT, |
||
| 138 | self::RESET_CONTENT => StatusText::RESET_CONTENT, |
||
| 139 | self::PARTIAL_CONTENT => StatusText::PARTIAL_CONTENT, |
||
| 140 | self::MULTI_STATUS => StatusText::MULTI_STATUS, |
||
| 141 | self::ALREADY_REPORTED => StatusText::ALREADY_REPORTED, |
||
| 142 | self::IM_USED => StatusText::IM_USED, |
||
| 143 | self::MULTIPLE_CHOICES => StatusText::MULTIPLE_CHOICES, |
||
| 144 | self::MOVED_PERMANENTLY => StatusText::MOVED_PERMANENTLY, |
||
| 145 | self::FOUND => StatusText::FOUND, |
||
| 146 | self::SEE_OTHER => StatusText::SEE_OTHER, |
||
| 147 | self::NOT_MODIFIED => StatusText::NOT_MODIFIED, |
||
| 148 | self::USE_PROXY => StatusText::USE_PROXY, |
||
| 149 | self::TEMPORARY_REDIRECT => StatusText::TEMPORARY_REDIRECT, |
||
| 150 | self::PERMANENT_REDIRECT => StatusText::PERMANENT_REDIRECT, |
||
| 151 | self::BAD_REQUEST => StatusText::BAD_REQUEST, |
||
| 152 | self::UNAUTHORIZED => StatusText::UNAUTHORIZED, |
||
| 153 | self::PAYMENT_REQUIRED => StatusText::PAYMENT_REQUIRED, |
||
| 154 | self::FORBIDDEN => StatusText::FORBIDDEN, |
||
| 155 | self::NOT_FOUND => StatusText::NOT_FOUND, |
||
| 156 | self::METHOD_NOT_ALLOWED => StatusText::METHOD_NOT_ALLOWED, |
||
| 157 | self::NOT_ACCEPTABLE => StatusText::NOT_ACCEPTABLE, |
||
| 158 | self::PROXY_AUTHENTICATION_REQUIRED => StatusText::PROXY_AUTHENTICATION_REQUIRED, |
||
| 159 | self::REQUEST_TIMEOUT => StatusText::REQUEST_TIMEOUT, |
||
| 160 | self::CONFLICT => StatusText::CONFLICT, |
||
| 161 | self::GONE => StatusText::GONE, |
||
| 162 | self::LENGTH_REQUIRED => StatusText::LENGTH_REQUIRED, |
||
| 163 | self::PRECONDITION_FAILED => StatusText::PRECONDITION_FAILED, |
||
| 164 | self::PAYLOAD_TOO_LARGE => StatusText::PAYLOAD_TOO_LARGE, |
||
| 165 | self::URI_TOO_LONG => StatusText::URI_TOO_LONG, |
||
| 166 | self::UNSUPPORTED_MEDIA_TYPE => StatusText::UNSUPPORTED_MEDIA_TYPE, |
||
| 167 | self::RANGE_NOT_SATISFIABLE => StatusText::RANGE_NOT_SATISFIABLE, |
||
| 168 | self::EXPECTATION_FAILED => StatusText::EXPECTATION_FAILED, |
||
| 169 | self::I_AM_A_TEAPOT => StatusText::I_AM_A_TEAPOT, |
||
| 170 | self::MISDIRECTED_REQUEST => StatusText::MISDIRECTED_REQUEST, |
||
| 171 | self::UNPROCESSABLE_ENTITY => StatusText::UNPROCESSABLE_ENTITY, |
||
| 172 | self::LOCKED => StatusText::LOCKED, |
||
| 173 | self::FAILED_DEPENDENCY => StatusText::FAILED_DEPENDENCY, |
||
| 174 | self::TOO_EARLY => StatusText::TOO_EARLY, |
||
| 175 | self::UPGRADE_REQUIRED => StatusText::UPGRADE_REQUIRED, |
||
| 176 | self::PRECONDITION_REQUIRED => StatusText::PRECONDITION_REQUIRED, |
||
| 177 | self::TOO_MANY_REQUESTS => StatusText::TOO_MANY_REQUESTS, |
||
| 178 | self::REQUEST_HEADER_FIELDS_TOO_LARGE => StatusText::REQUEST_HEADER_FIELDS_TOO_LARGE, |
||
| 179 | self::UNAVAILABLE_FOR_LEGAL_REASONS => StatusText::UNAVAILABLE_FOR_LEGAL_REASONS, |
||
| 180 | self::INTERNAL_SERVER_ERROR => StatusText::INTERNAL_SERVER_ERROR, |
||
| 181 | self::NOT_IMPLEMENTED => StatusText::NOT_IMPLEMENTED, |
||
| 182 | self::BAD_GATEWAY => StatusText::BAD_GATEWAY, |
||
| 183 | self::SERVICE_UNAVAILABLE => StatusText::SERVICE_UNAVAILABLE, |
||
| 184 | self::GATEWAY_TIMEOUT => StatusText::GATEWAY_TIMEOUT, |
||
| 185 | self::HTTP_VERSION_NOT_SUPPORTED => StatusText::HTTP_VERSION_NOT_SUPPORTED, |
||
| 186 | self::VARIANT_ALSO_NEGOTIATES => StatusText::VARIANT_ALSO_NEGOTIATES, |
||
| 187 | self::INSUFFICIENT_STORAGE => StatusText::INSUFFICIENT_STORAGE, |
||
| 188 | self::LOOP_DETECTED => StatusText::LOOP_DETECTED, |
||
| 189 | self::NOT_EXTENDED_OBSOLETED => StatusText::NOT_EXTENDED_OBSOLETED, |
||
| 190 | self::NETWORK_AUTHENTICATION_REQUIRED => StatusText::NETWORK_AUTHENTICATION_REQUIRED, |
||
| 191 | }; |
||
| 194 |
The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g.
excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths