Page クラスのショートコードの例外補足が適切でなさそう

[ikaruga777]

Product_Api クラスの fetch は ColorMeShop\Swagger\ApiException を送出します。

colormeshop-wp-plugin/src/api/class-product-api.php

Lines 41 to 49 in 81434e0

	/**
	* 商品データの取得
	*
	* @param int $product_id 商品ID (required)
	* @throws \ColorMeShop\Swagger\ApiException on non-2xx response
	* @throws \InvalidArgumentException
	* @return \ColorMeShop\Swagger\Model\InlineResponse2007
	*/
	public function fetch( $product_id ) {

一方、この関数を呼び出す Page クラスのショートコードでは RuntimeException でキャッチしています。この例外は ColorMeShop\Swagger\ApiException を継承していないので、 ColorMeShop\Swagger\ApiException 例外時に突き抜けてしまうように見えます。

colormeshop-wp-plugin/src/shortcodes/product/class-page.php

Lines 45 to 52 in 81434e0

	try {
	$product = $container['api.product_api']->fetch( $filtered_atts['product_id'] )['product'];
	} catch ( \RuntimeException $e ) {
	if ( $container['WP_DEBUG_LOG'] ) {
	error_log( $e );
	}
	return '';
	}