ｐｈｐでのエラーメッセージの解釈

Question

前提・実現したいこと

$product->get_dollPrice_html()の値を1.2で割りたい。そこでwc_get_price_to_displayの連想配列に'price'を1.2で割った'dollPrice'　=>　$dollPriceを追加して解決しようと試みた。

発生している問題・エラーメッセージ

自分で作った$product->get_dollPrice_html()（詳細は該当のコードエリアにて）で出力した結果が、既存の$product->get_price_html()で出力した結果と変わらない。

試したこと

get_price_html()を見てみると以下の記述があった。

public function get_price_html( $deprecated = '' ) {
		if ( '' === $this->get_price() ) {
			$price = apply_filters( 'woocommerce_empty_price_html', '', $this );
		} elseif ( $this->is_on_sale() ) {
			$price = wc_format_sale_price( wc_get_price_to_display( $this, array( 'price' => $this->get_regular_price() ) ), wc_get_price_to_display( $this ) ) . $this->get_price_suffix();
		} else {
			$price = wc_price( wc_get_price_to_display( $this ) ) . $this->get_price_suffix();
		}

		return apply_filters( 'woocommerce_get_price_html', $price, $this );
	}

自分の状況はelseの場合のみしかないのでwc_price()とwc_get_price_to_display()のみを考え、それらの内容は以下の通りだった。

function wc_price( $price, $args = array() ) {
  $args = apply_filters(
    'wc_price_args',
    wp_parse_args(
      $args,
      array(
        'ex_tax_label'       => false,
        'currency'           => '',
        'decimal_separator'  => wc_get_price_decimal_separator(),
        'thousand_separator' => wc_get_price_thousand_separator(),
        'decimals'           => wc_get_price_decimals(),
        'price_format'       => get_woocommerce_price_format(),
      )
    )
  );

  $unformatted_price = $price;
  $negative          = $price < 0;
  $price             = apply_filters( 'raw_woocommerce_price', floatval( $negative ? $price * -1 : $price ) );
  $price             = apply_filters( 'formatted_woocommerce_price', number_format( $price, $args['decimals'], $args['decimal_separator'], $args['thousand_separator'] ), $price, $args['decimals'], $args['decimal_separator'], $args['thousand_separator'] );

  if ( apply_filters( 'woocommerce_price_trim_zeros', false ) && $args['decimals'] > 0 ) {
    $price = wc_trim_zeros( $price );
  }

  $formatted_price = ( $negative ? '-' : '' ) . sprintf( $args['price_format'], '<span class="woocommerce-Price-currencySymbol">' . get_woocommerce_currency_symbol( $args['currency'] ) . '</span>', $price );
  $return          = '<span class="woocommerce-Price-amount amount">' . $formatted_price . '</span>';

  if ( $args['ex_tax_label'] && wc_tax_enabled() ) {
    $return .= ' <small class="woocommerce-Price-taxLabel tax_label">' . WC()->countries->ex_tax_or_vat() . '</small>';
  }

  
/**
 * Filters the string of price markup.
 *
 * @param string $return            Price HTML markup.
 * @param string $price             Formatted price.
 * @param array  $args              Pass on the args.
 * @param float  $unformatted_price Price as float to allow plugins custom formatting. Since 3.2.0.
 */
  return apply_filters( 'wc_price', $return, $price, $args, $unformatted_price );
}

function wc_get_price_to_display( $product, $args = array() ) {
	$args = wp_parse_args(
		$args,
		array(
			'qty'   => 1,
			'price' => $product->get_price(),
			
		)
	);
	$price = $args['price'];
	$qty   = $args['qty'];

	return 'incl' === get_option( 'woocommerce_tax_display_shop' ) ?
		wc_get_price_including_tax(
			$product,
			array(
				'qty'   => $qty,
				'price' => $price,
				
			)
		) :
		wc_get_price_excluding_tax(
			$product,
			array(
				'qty'   => $qty,
				'price' => $price,
				
			)
		);
}

該当のソースコード

新たに自分で作ったget_dollPrice_html()のコード

public function get_dollPrice_html( $deprecated = '' ) {
 		if ( '' === $this->get_price() ) {
 			$dollPrice = apply_filters( 'woocommerce_empty_price_html', '', $this );
 		} elseif ( $this->is_on_sale() ) {
 			$dollPprice = wc_format_sale_price( wc_get_price_to_display( $this, array( 'price' => $this->get_regular_price() ) ), wc_get_price_to_display( $this ) ) . $this->get_price_suffix();
 		} else {
 			$dollPrice = wc_price( wc_get_price_to_display( $this ) ) . $this->get_price_suffix();
 		}

 		return apply_filters( 'woocommerce_get_price_html', $dollPrice, $this );
 	}

改変したwc_get_price_to_display()

function wc_get_price_to_display( $product, $args = array() ) {
	$args = wp_parse_args(
		$args,
		array(
			'qty'   => 1,
			'price' => $product->get_price(),
			'dollPrice' =>intval($product->get_price()) / 1.2,
		)
	);
	$price = $args['price'];
	$qty   = $args['qty'];
        $dollPrice = $arg['dollPrice'];

	return 'incl' === get_option( 'woocommerce_tax_display_shop' ) ?
		wc_get_price_including_tax(
			$product,
			array(
				'qty'   => $qty,
				'price' => $price,
				'dollPrice' => $dollPrice,
			)
		) :
		wc_get_price_excluding_tax(
			$product,
			array(
				'qty'   => $qty,
				'price' => $price,
				'dollPrice' => $dollPrice,
			)
		);
}

お手数ですがご教示いただけましたら幸いです。

Answer 1

この手のエラーは32行目じゃなければ31行目を疑うとよいのですが
複数行を突っ込んでいてわかりづらいです

<p class="<?php
echo esc_attr( apply_filters( 'woocommerce_product_price_class', 'price' ) );
?>"><?php
echo $product->get_price_html();
echo get($product->get_price_html());
?></p>

のような行を分けて見るとわかりやすくなります。
もしくは

printf('<p class="%s">%s%s</pa>',
esc_attr( apply_filters( 'woocommerce_product_price_class', 'price' ) ),
$product->get_price_html(),
get($product->get_price_html())
);

のような処理でもよいかもしれません。

Comments

[135]

回答ありがとうございます。
試しにコードをコピーして実行してみたところ、$product->get_price_html()は文字列として出力されているようでしたので以下のようにコードを書き換えました。
```php
<?php function get($hoge){
$num = intval($hoge);
$result = 1.2 * $num;
var_dump($result);
}
<p class="<?php echo esc_attr( apply_filters( 'woocommerce_product_price_class', 'price' ) );?>"><?php
$product->get_price_html(),
get($product->get_price_html());

</p>
```
すると値段の下にFLOAT(0)が出てきました。
なぜ0になってしまうのでしょうか？ご回答いただけましたら幸いです。

[yambejp]

> FLOAT(0)

と表示されるなら、デバッグ処理でvar_dump的なことをしているのでは？
get()がどういう挙動をするかわからないのでなんとも・・・

[135]

get()はただ$product->get_price_html(),に格納されている値に1.2をかけるための関数を想定しております。
またおっしゃる通り、var_dumpで出力しているのでFLOATがつくのは構わないのですが本当は例えば
$product->get_price_html()が100だとするとFLOAT(120)と表示させたかったです。

[yambejp]

get_price_htmlが何か引数をうけるんじゃないですか？

[135]

恥ずかしながら今引数を受けると0になるのを知りました。
只今ファイルを確認してまいります。

[135]

yambejp様のおっしゃる通り、引数を受けておりました。
public function get_price_html( $deprecated = '' ) {
if ( '' === $this->get_price() ) {
$price = apply_filters( 'woocommerce_empty_price_html', '', $this );
} elseif ( $this->is_on_sale() ) {
$price = wc_format_sale_price( wc_get_price_to_display( $this, array( 'price' => $this->get_regular_price() ) ), wc_get_price_to_display( $this ) ) . $this->get_price_suffix();
} else {
$price = wc_price( wc_get_price_to_display( $this ) ) . $this->get_price_suffix();
}

return apply_filters( 'woocommerce_get_price_html', $price, $this );
}
この場合どのようにすれば$product->get_price_html()で得た値に1.2をかけて出力するという関数を設定できますでしょうか？