Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

4.1.1-4.1.2-p1のプラグイン更新。 #25

Merged
merged 10 commits into from
Sep 16, 2022

Conversation

sai-gillingham
Copy link
Contributor

@sai-gillingham sai-gillingham commented Sep 14, 2022

概要(Overview・Refs Issue)

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


優先目的

1)
プラグインソースコードを

4.1.1-4.1.2
から
4.1.1-4.1.2-p1
に更新しました。

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

を更新しました。


スクリプト更新は2つ

1)MAC OSとLINUX OS(Ubuntu/Deblianディストロ)両方で

bin/replace_version.sh
bin/upgrade_file_hash.sh

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

2)p{x}バージョンの対応

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

@chihiro-adachi
Copy link
Contributor

@sai-gillingham
ご対応ありがとうございます。

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

すみません、こちらの変更は行う必要はありません。

p1対応は、以下のpull requestをご参考いただければと思います。
https://github.com/EC-CUBE/eccube-update-plugin/pull/18/files

@sai-gillingham
Copy link
Contributor Author

@chihiro-adachi
承知いたしました!

https://github.com/EC-CUBE/eccube-update-plugin/pull/18/files
で更新された箇所だけを更新して行きます。

ルーティング名
コマンド名
ダウンロードパッケージ名
の更新は不要のため、元の状態に戻す。
@sai-gillingham
Copy link
Contributor Author

sai-gillingham commented Sep 15, 2022

@chihiro-adachi

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

上記のの部分を修正前の状態に戻しました。

確認したいことがありますが、
こちらの判定を追加しましたが、よろしいでしょうか?

-       $this->supported = version_compare(Constant::VERSION, UpdaterConstant::FROM_VERSION, '=');
+       $this->supported = version_compare(Constant::VERSION, UpdaterConstant::FROM_VERSION, '=') ||
+         version_compare(Constant::VERSION, '4.1.2', '=');
        ;

@chihiro-adachi
Copy link
Contributor

@sai-gillingham

確認したいことがありますが、
こちらの判定を追加しましたが、よろしいでしょうか?
確認ありがとうございます。
こちらは対応不要です。

@sai-gillingham
Copy link
Contributor Author

@chihiro-adachi
確認してくれてありがとうございます。
バージョンチェックのところを元の状態に戻しました。

@chihiro-adachi
Copy link
Contributor

@sai-gillingham
pull requestしました。マージいただいてよいでしょうか。
sai-gillingham#1

@sai-gillingham
Copy link
Contributor Author

@chihiro-adachi

プルリクエストありがとうございます。そこの分を修正出来なくてすみません。
sai-gillingham#1
プルリクエストを承認しました。

@chihiro-adachi chihiro-adachi merged commit 8694381 into EC-CUBE:4.1.1-4.1.2 Sep 16, 2022
@chihiro-adachi
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants