inc/vk-blocks内のブロック関連のコードの場所と役割が関連せず、わかりにくいのでリファクタリングしたい

[shimotmk]

自分のイメージとしてはsrc ≒ inc/vk-blocks

ブロック関連のコードはinc/vk-blocks/blocksディレクトリーを作ってそこで管理する
inc/にはvektor-wp-librariesなどの他のライブラリーのみ置く
inc/vk-blocks内を読み込むファイルはinc/vk-blocks/vk-blocks-functions.phpに書く

変更すること

１タスク１ブランチというわけではないです。
とりあえず目についたものだけザッと書いたので少しずつやろうと思っています

• helpers.phpをutilsフォルダに移動、変更する
  srcも_helperからutilsになったので合わせたい
  de293d8
• inc/vk-blocks-config.php をvk-blocks-load.phpに変更する
  inc/vk-blocks内のブロック関連のコードの場所と役割が関連せず、わかりにくいのでリファクタリングしたい #1382 (comment)
• inc/vk-blocks-config.phpをinc直下にある各ロードするファイルを読み込むだけにする
  おそらく13ファイル以下
  【確認済】vk-blocksの読み込むファイルの役割のコメントを追加 #1386 (comment)
  Vk_Blocks_EntryPointファイルの読み込み場所を適切な場所に修正 #1490
• inc/font-awesomeはvektor-wp-librariesではないのでinc/vk-blocksに移動
  Font Awesomeを読み込むファイルを適切な場所へ移動 #1441
• inc/vk-blocks/vk-blocks-functions.phpのロードする以外の記述を移動
  【２人目確認待ち】使用していない関数vk_blocks_margin_size_array()を削除 #1396
  cssをminifyするコードを関数化 #1405
  【2人目レビュー中】使用していない関数get_selected、the_selectedを削除 #1410
  不要な条件を削除、使用しない関数削除 #1418
  不要な処理を削除 #1433
  不要な処理を削除 vk_blocks_sanitize_checkbox #1479
• inc/vk-blocks内のstyle,viewを調整
  後世の人がどこのPHPファイルに書けば良いか迷いが生じるため
• src/spacer/index.phpに書いてあるオプション周りの記述をinc/vk-blocks内に移動する
• freeのユニットテストがproで実行されないので移行する
• src/faq/index.phpに書いてあるオプション周りの記述をinc/vk-blocks内に移動する
• src/faq2/index.phpに書いてあるオプション周りの記述をinc/vk-blocks内に移動する
• src/faq2/index.phpに書いてあるオプション周りの記述をinc/vk-blocks内に移動する
• src/staff/index.phpに書いてある余計なコードを削除
  スタッフ 編集画面では個別のスクリプトを削除 #1393
• src/post-list/index.phpに書いてある余計なコードを削除
  【2人目確認待ち】投稿リスト 不要な処理削除 #1464
• 各index.phpファイルをbundle.jsのようにまとめる
  理由
  • Gutenbergと同じようにしたいから
    ダイナミックブロック以外はindex.phpを作っていない
    https://github.com/WordPress/gutenberg/blob/trunk/lib/blocks.php
  • 現状、各index.phpファイルに同じような記述が書いてあるのでコアの仕様が変わったなどの時に全てのファイルに同じような変更をしないといけないため

何かご意見、ご質問などありましたらコメントいただければと思います。

[shimotmk]

helpers.phpをutilsフォルダに移動、変更する

srcも_helperからutilsになったので合わせたい
de293d8

[shimotmk]

inc/vk-blocks-config.php をvk-blocks-load.phpに変更する

こちらはファイルの内容と合っていない感じがします。
元々、汎用的に利用する関数をここで定義するつもりだったと思うんですね。
なので、本来ならば「共通関数の定義」となるのかなと。

なんてことを書いているうちに思ったのですが、
冒頭のサーバーサイドレンダリングスクリプトの読み込みもvk-blocks-config.php でやったらいいと思うんですね。そうすると、こちらは関数の定義だけになり、役割分担がすっきりします。

vk-blocks-config.php も vk-blocks-pro.php からしか読み込まないのであれば、vk-blocks-load.php という名前に変えてもいい気もします。話広がってすみません。ちょっと検討してみていただけますと幸いです。

Originally posted by @mthaichi in #1386 (comment)