PES-2296, Check phpcs sniff ValidVariableName #625
Conversation
mzk
changed the title
mzk
force-pushed
the
PES-2296-phpcs-ValidVariableName
branch
from
c911f11
to
e1c76fe
Compare
| if ( isset( $wp->query_vars['order-pay'] ) && is_numeric( $wp->query_vars['order-pay'] ) ) { | ||
| $order = $this->orderRepository->getById( (int) $wp->query_vars['order-pay'], true ); | ||
| if ( isset( $wp->query_vars['order-pay'] ) && is_numeric( $wp->query_vars['order-pay'] ) ) { // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps | ||
| $order = $this->orderRepository->getById( (int) $wp->query_vars['order-pay'], true ); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps |
There was a problem hiding this comment.
proč tady to ignorujeme? Takhle to nikdo nikdy neopraví. nejde to přesunou t do adapteru?
There was a problem hiding this comment.
To přichází z vnějšku, takže ignorování je ok. Ale bylo by asi přehlednější $wp->query_vars['order-pay'] vytáhnout ternárem do proměnné, pak tam i to ignorování bude jen jednou.
| <arg name="encoding" value="utf-8"/> | ||
| <arg value="p"/> | ||
| <arg value="s"/> | ||
| <description>Packetery WP plugin coding standards ruleset.</description> |
| @@ -25,42 +25,42 @@ class WpdbAdapter { | |||
| * | |||
| * @var string | |||
| */ | |||
| public $packetery_carrier; | |||
| public $packeteryCarrier; | |||
There was a problem hiding this comment.
Ve wpdb je konvence, že název property odpovídá názvu tabulky. Raději bych s týmem ověřil, že se na camelCase pro toto použití shodneme.
| @@ -181,8 +181,8 @@ public function check(): void { | |||
| } | |||
|
|
|||
| if ( $oldVersion && version_compare( $oldVersion, '1.4.2', '<' ) ) { | |||
| $version_1_4_2 = new Version_1_4_2( $this->wpdbAdapter ); | |||
| $version_1_4_2->run(); | |||
| $version142 = new Version_1_4_2( $this->wpdbAdapter ); | |||
There was a problem hiding this comment.
To možná raději bez čísla verze. Stejně se tento způsob používá jen jednou.
| if ( isset( $wp->query_vars['order-pay'] ) && is_numeric( $wp->query_vars['order-pay'] ) ) { | ||
| $order = $this->orderRepository->getById( (int) $wp->query_vars['order-pay'], true ); | ||
| if ( isset( $wp->query_vars['order-pay'] ) && is_numeric( $wp->query_vars['order-pay'] ) ) { // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps | ||
| $order = $this->orderRepository->getById( (int) $wp->query_vars['order-pay'], true ); // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps |
There was a problem hiding this comment.
To přichází z vnějšku, takže ignorování je ok. Ale bylo by asi přehlednější $wp->query_vars['order-pay'] vytáhnout ternárem do proměnné, pak tam i to ignorování bude jen jednou.
| @@ -237,7 +237,7 @@ private function formatVariable( $variable, int $level = 0, bool $addSeparator = | |||
| } elseif ( $variable instanceof \WC_Shipping_Method ) { | |||
| $methodInfo = [ | |||
| 'id' => $variable->id, | |||
| 'method_title' => $variable->method_title, | |||
| 'method_title' => $variable->method_title, // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps | |||
There was a problem hiding this comment.
Ignorovací komentáře bych uváděl vždy nad řádek, usnadnilo by to porovnávání revizí.
No description provided.