【確認待ち】WP5.8で 外観 ＞ カスタマイズ ＞ ウィジェット でブロックが表示されなかったのを修正

[drill-lancer]

WP5.8で 外観 ＞ カスタマイズ ＞ ウィジェット でブロックが表示されなかったのを修正しました。
WP5.8の 外観 ＞ カスタマイズ ＞ ウィジェット で VK Blocks Pro のブロックが提示されるか確認をお願いします。

[kurudrive]

@drill-lancer Cc: @miminari
おつかれさまです。
さっき見たらmasterブランチのままでも表示されてたんだけど、
このブランチでも表示されました。

WordPress 自体の更新か何かの都合で master ブランチのままでも表示されるようになったのかな？

この変更は必要な変更という理解で良いのよね？

[kurudrive]

@drill-lancer @miminari
※ 返信は月曜で問題ありません。

[drill-lancer]

@kurudrive
5.8 Beta-1 の時点では必要な変更でした。
（npm run build:dev を行ったところこのあたりでコンソールエラーが発生したため）
あと、Requires at least を 5.7 に設定してるので 5.5 以下のための条件分岐は不要かと。

[miminari]

@kurudrive すみません、遅くともRCリリースまでには確認しますので、お待ちいただけたら嬉しいかもです。

[miminari]

@drill-lancer すみません、もしできたら、どうして必要なのかの説明をもう少しいただけたらとても助かります！

[drill-lancer]

npm run build:dev を実行したときの 外観 ＞ カスタマイズ ＞ ウィジェット でののコンソールエラー

index.js:32 Uncaught TypeError: Invalid argument expected string
    at validate (index.js:32)
    at Array.forEach (<anonymous>)
    at compareVersions (index.js:40)
    at registerBlock (index.js_+_667_modules:35437)
    at Array.forEach (<anonymous>)
    at registerVKBlocks (index.js_+_667_modules:35457)
    at eval (index.js_+_667_modules:35461)
    at Module../src/blocks/index.js (block-build.js?ver=1.8.1:1585)
    at __webpack_require__ (block-build.js?ver=1.8.1:20)
    at block-build.js?ver=1.8.1:84

これを解消したものになります。
あと、Requires at least を 5.7 に設定してるので 5.5 以下のための条件分岐は不要かと。

[kurudrive]

@drill-lancer

コンソールエラーを解消したものになります。

これは既に前に記載いただいている通りなので、説明が欲しいというのは

どういう理由のエラーが出ていたので、
今回何を変更した事によってエラーが出なくなったのかの補足が欲しいという意味だと思います。

[drill-lancer]

コンソールエラーを分析したところ

	//WP5.5未満の場合
	if (compareVersions(window.wpVersion, '5.5') < 0) {
		//nameを削除
		delete metadata.name;
		//カテゴリ等を追加
		settings = {
			...settings,
			...metadata,
		};
	} else if (metadata) {
		unstable__bootstrapServerSideBlockDefinitions({ [name]: metadata }); // eslint-disable-line camelcase
	}

上記部分でエラーが発生していました。
Requires at least を 5.7 に設定してるので 5.5 以下のための条件分岐は不要ということもあり
下記のように不要コードを削除したらエラーが解消されたということです。

unstable__bootstrapServerSideBlockDefinitions({ [name]: metadata }); // eslint-disable-line camelcase

[miminari]

あ、いま気が付きましたー！これ後方互換がないのと、descriptionの対応（i18nの対応）で別PR走っているので、すみません、クローズします。

#571