4.0.5~4.0.6-p2

[sai-gillingham]

概要(Overview・Refs Issue)

3点の更新がありました。

---

優先目的

１）
プラグインソースコードを

4.0.5-4.0.6
から
4.0.5-4.0.6-p2
に更新しました。

• 名前空間名
• ルーティング名
• コマンド名
• ダウンロードパッケージ名

を更新しました。

---

スクリプト更新は２つ

１）MAC OSとLINUX OS（Ubuntu/Deblianディストロ）両方で

bin/replace_version.sh
bin/upgrade_file_hash.sh

上記のスクリプトを実行できるようになりました。

２）p{x}バージョンの対応

今までreplace_versionシェルスクリプトでバージョンは数字しか入力出来なかったけど、
p1/p2等のバージョンの更新を対応しました。

[sai-gillingham]

名前空間名
ルーティング名
コマンド名
ダウンロードパッケージ名

の更新は不要になっているため、その更新を元の状態に戻します。

[sai-gillingham]

名前空間名
ルーティング名
コマンド名
ダウンロードパッケージ名

更新を元の状態に戻しました。

[chihiro-adachi]

@sai-gillingham
以下バージョンで実行結果ご確認いただいてよいでしょうか。

• 4.0.5
• 4.0.5-p1
• 4.0.6
• 4.0.6-p1

パッケージはこちらお使いください。
https://github.com/chihiro-adachi/eccube-update-plugin/releases/download/405406-2022-09-16/eccube-update-plugin-405406-2022-09-16.tar.gz

[chihiro-adachi]

@sai-gillingham
4.0.5-p1/4.0.6で実行すると、4.0.6-p1に更新されるようです。
※期待値は4.0.6-p2に更新されてほしい

[chihiro-adachi]

@sai-gillingham
4.0.6-p1で実行すると、以下のように差分エラーが表示されました。

[sai-gillingham]

@chihiro-adachi
ご確認ありがとうございます。
今から上記の課題を修正して行きます。

[sai-gillingham]

@chihiro-adachi

#26 (comment)

4.0.5-p1/4.0.6で実行すると、4.0.6-p1に更新されるようです。
※期待値は4.0.6-p2に更新されてほしい

上記の原因を調査出来て、
修正しました。

すみません、

その原因は

405p1
406
406p1

それぞれ
md5ハッシュファイルとupdate_file.tar.gzの更新を忘れましたからです。

なので、

405_file_hash.yaml
405_file_hash_crlf.yaml
406p1_file_hash.yaml
406p1_file_hash_crlf.yaml

上記のハッシュファイルを更新しました。

それと405/406p1新規ハッシュファイルを作成しました。

405_file_hash.yaml
405_file_hash_crlf.yaml
406p1_file_hash.yaml
406p1_file_hash_crlf.yaml

---

バグの発見(?)

今は4.0.5バージョン環境でテストしていますが、問題を発見しました。

それはアップデートプラグインのの上書きログの表示です。更新されているファイルが上記添付した画像の画面に表示されていないです。

例えば：4.0.6で更新された

src/Eccube/Controller/Admin/Content/FileController.php
src/Eccube/Controller/Admin/Setting/Shop/PaymentController.php

ファイルです。

Linux(Deblianディストロ） で
update_file_hashコマンドを実行出来て、

405_file_hash.yamlの中に

src/Eccube/Controller/Admin/Content/FileController.php
src/Eccube/Controller/Admin/Setting/Shop/PaymentController.php

405->406-p2の差分でファイルの存在がありますが、

4.0.5環境のFileController.phpのmd5ハッシュと
アップデートプラグインの405_file_hash.yamlのFileController.phpのmd5ハッシュは一致しているようです。

つまり、md5ハッシュが変わっていないようです。

今はmd5ハッシュの更新動きを確認しています。

すみません、もし可能であれば私のブランチでMac OSでupdate_hash_file.shを実行して頂いて、md5 ハッシュが変わるかどうかを確認していただければ助かります。

[chihiro-adachi]

4.0.5環境のFileController.phpのmd5ハッシュと
アップデートプラグインの405_file_hash.yamlのFileController.phpのmd5ハッシュは一致しているようです。
つまり、md5ハッシュが変わっていないようです。

実行してみました。ハッシュ値は同じ値でした。

4.0.5環境のFileController.php：3bc77ffe5b17805c421e81d49cb42523
405_file_hash.yaml：3bc77ffe5b17805c421e81d49cb42523

が、これは正しい挙動かと思います。
file_hash.yamlには、FROM_VERSIONのソースコードのmd5値が記録されます。

プラグインを実行する環境でファイルが変更されているかどうかを確認するためのものです。

[sai-gillingham]

@chihiro-adachi
ご確認ありがとうございます、助かりました。

4.0.5バージョンの環境ですが、上記の添付した画像の画面のファイル一覧は間違っていないという理解でよろしいでしょうか？

つまり、ECCUBEを更新する前に、開発者が更新されるファイルを以前に触ったら、そのファイルパスが表示されるという理解で正しいでしょうか？

[chihiro-adachi]

@sai-gillingham

つまり、ECCUBEを更新する前に、開発者が更新されるファイルを以前に触ったら、そのファイルパスが表示されるという理解で正しいでしょうか？

はい、その理解で正しいです。

[sai-gillingham]

@chihiro-adachi
ご返信ありがとうございます。

4.0.5環境でアップデートプラグインの実行結果が問題なく動けましたので、

4.0.5p1
4.0.6
4.0.6p1

のそれぞれのバージョンで実行結果を確認して行きます。

[chihiro-adachi]

@sai-gillingham
対応ありがとうございます。
各バージョンの実行結果、お手数ですがご確認お願いします。

一部軽微な修正していますので、以下マージいただけますと幸いです。
sai-gillingham#2

[sai-gillingham]

@chihiro-adachi

了解いたしました。

sai-gillingham#2
プルリクエストについて、
マージしました。

[sai-gillingham]

@chihiro-adachi

お待たせしました。

4.0.5
4.0.5-p1
4.0.6
4.0.6-p1

上記のそれぞれの環境でアップデートプラグインを実行しました。

その結果は

上記の課題を再現確認できず、
期待したファイルが更新されて、アップデートプラグインの完了画面をみて、
デバッグバーでECCUBEのバージョン番号が更新されました。

[chihiro-adachi]

@sai-gillingham
ご対応ありがとうございます。
マージしました。