profile
viewpoint

jenkinsci/grails-plugin 19

Jenkins grails plugin

kiy0taka/dbunit-runner 2

JUnit Runner implementation for DbUnit

kiy0taka/backend-crawler 1

tools crawler

kiy0taka/backend-update-center2 1

Update center backend

kiy0taka/boytwoman 1

grails application

kiy0taka/dbunit-runner-example 1

Example for DbUnit Runner

PR closed EC-CUBE/ec-cube

4.1.1上限下限テスト

<!-- 以下を参考にコメントを作成してください。 -->

概要(Overview・Refs Issue)

<!-- PullRequestの目的、関連するIssue番号など -->

システム要件の上限下限の組み合わせにより自動テストを実行

  • PHP
    • 7.3, 7.4
  • PostgreSQL
    • 9.6, 14
  • MySQL
    • 5.7

※PostgreSQL 14は4.1.1から対応

自動テストの対象は以下の通り

  • e2e-test
  • unit-test
  • plugin-test

テスト結果の確認用Pull Requestのため、マージ不要です。

方針(Policy)

<!-- このPullRequestを作るにあたって考慮したものや除外した内容  - 例)Symfony のXXにならって作成、3系で同様の仕様があったため など -->

実装に関する補足(Appendix)

<!-- コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと -->

テスト(Test)

<!-- テストを行っている範囲など、レビューアが安心できるような情報 -->

相談(Discussion)

<!-- 相談したいことや意見を求めたいこと -->

マイナーバージョン互換性保持のための制限事項チェックリスト

<!-- マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。 -->

  • [ ] 既存機能の仕様変更はありません
  • [ ] フックポイントの呼び出しタイミングの変更はありません
  • [ ] フックポイントのパラメータの削除・データ型の変更はありません
  • [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
+134 -133

2 comments

3 changed files

chihiro-adachi

pr closed time in 6 hours

pull request commentEC-CUBE/ec-cube

4.1.1上限下限テスト

上限下限での確認はできたので、このPRはクローズします。

chihiro-adachi

comment created time in 6 hours

PullRequestReviewEvent

created tagEC-CUBE/ec-cube

tagco/4.1-20211202

EC-CUBE is the most popular e-commerce solution in Japan

created time in a day

push eventEC-CUBE/ec-cube

Dakeyama

commit sha 31a7196d38e866347d2bcd518948a180bcaa97d5

商品一覧・登録画面の規格情報並び順をsort_noで制御

view details

Yuko Kajihara

commit sha d154543c3443b686ecb4f2615348cf19b34a7ab0

管理画面コンテンツ作成開始

view details

Yuko Kajihara

commit sha d90544ce940a82aa877ba1857c2249ead8c48476

定休日新規登録処理実装

view details

Yuko Kajihara

commit sha 9688c917134519c03e1a9d1efef288819fc8c769

更新・削除処理まで実装

view details

Yuko Kajihara

commit sha 3cfa0f36a874be5996b7fba1026198bfc9f241b5

フロント側処理実装中

view details

Yuko Kajihara

commit sha 4006a9c944c2df5f89ba7f325a1aa1877adf105d

Carbonでカレンダーデータ作成中

view details

Yuko Kajihara

commit sha e9c0405b485aa311eddb393f1ea1cd8cb1234bae

カレンダー取得処理など追加

view details

Yuko Kajihara

commit sha aa6b8a1dc9d7649557024e0174fc3a2e588ff8e7

2ヶ月分のカレンダー配列作成と前後の空白埋めを実装

view details

Yuko Kajihara

commit sha c4a8e79ef3ae4b160df2c820a4f04dee16d5e482

定休日マッチング実装中

view details

Yuko Kajihara

commit sha bb3181b78095be4fb7cec564aa951e05c92b3b09

定休日フラグ設定まではなんとかできた

view details

Yuko Kajihara

commit sha 28f69f236f83059c98ca10777caf3b16144447e0

カレンダーのスタイル以外は実装完了

view details

Yuko Kajihara

commit sha e1183e73d98e7eaeef0a2439911e2eb70749295d

重複チェック実装もれ発見。一旦コメントに記載。

view details

Yuko Kajihara

commit sha a69a6497b785ad58e06df086798ca7589dd5c244

休業日文言追加と今日を強調するフラグ追加と表示対応

view details

Yuko Kajihara

commit sha fff7dc4720631c997e97e7cc4a652b9aea02f066

定休日登録の重複チェックを実装

view details

Yuko Kajihara

commit sha 5a9aae8a6962e2eca45ac19f368e00896c1bb5b1

不要コメントと不要useを削除

view details

Yuko Kajihara

commit sha 5715867d46fbc9333c4669ce26de9105418d5674

メッセージファイル修正

view details

Yuko Kajihara

commit sha 8c51ed1000a94dc51cecad5d1ccca915de30be7a

dtb_blockのcsvデータを追加

view details

Yuko Kajihara

commit sha b3f9345670b48003179277e3fcfccd49648644fc

Migrationファイルを追加

view details

Yuko Kajihara

commit sha c15a3fa2c1a07b1999f9e746c4ea74aef3f7e54f

日曜はじまりの2月の考慮処理を追加

view details

Yuko Kajihara

commit sha 394badf114f984a0282c19eab1306d4d721a1ebd

インデント修正

view details

push time in a day

PR opened EC-CUBE/ec-cube

月末に定休日カレンダーのテストが失敗するのを修正

<!-- 以下を参考にコメントを作成してください。 -->

概要(Overview・Refs Issue)

<!-- PullRequestの目的、関連するIssue番号など --> 月末にテストを実行するとテストデータの定休日は翌月になる。

方針(Policy)

<!-- このPullRequestを作るにあたって考慮したものや除外した内容  - 例)Symfony のXXにならって作成、3系で同様の仕様があったため など -->

実装に関する補足(Appendix)

<!-- コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと -->

テスト(Test)

<!-- テストを行っている範囲など、レビューアが安心できるような情報 -->

相談(Discussion)

<!-- 相談したいことや意見を求めたいこと -->

マイナーバージョン互換性保持のための制限事項チェックリスト

<!-- マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。 -->

  • [ ] 既存機能の仕様変更はありません
  • [ ] フックポイントの呼び出しタイミングの変更はありません
  • [ ] フックポイントのパラメータの削除・データ型の変更はありません
  • [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
+1 -1

0 comment

1 changed file

pr created time in 2 days

create barnchkiy0taka/ec-cube

branch : fix/holiday-calendar-test

created branch time in 2 days

PullRequestReviewEvent

issue commentEC-CUBE/ec-cube

Let's EncryptのルートCA期限切れでプラグインのインストールが出来ない

package-api.ec-cube.net(4.0系)、package-api-c2.ec-cube.net(4.1系) の証明書をそれぞれLet's EncryptからAlphaSSLに変更しました。

nobuhiko

comment created time in 7 days

issue closedEC-CUBE/ec-cube

Let's EncryptのルートCA期限切れでプラグインのインストールが出来ない

概要(Overview)

オーナーズストアからプラグインのインストールに失敗する

期待する内容(Expect) or 要望 (Requirement)

ルート証明書方が期限切れが原因だが、サーバー環境をいじれないことも多いので、EC-CUBE側で対応出来るのであれば対応したい オーナーズストアの証明書をLet's Encryptじゃないものにしてくれればそれで解決する問題ではある

再現手順(Procedure)

オーナーズストアからプラグインのインストールをしようとする

環境 (environment)

  • EC-CUBE: 4.1.0
  • PHP: 7.3.29
  • DB:
    • PostgreSQL x.x.x
    • MySQL x.x.x

関連情報 (Ref)

https://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=26341&forum=16&post_id=104994#forumpost104994

closed time in 7 days

nobuhiko

created tagEC-CUBE/ec-cube

tagco/4.1-20211125

EC-CUBE is the most popular e-commerce solution in Japan

created time in 8 days

push eventEC-CUBE/ec-cube

hideki_okajima

commit sha 0a83160aca2ab9e24173b904bc227bac8fb7b4d7

Generator で Product 作成時に ProductTag もセットする

view details

hideki_okajima

commit sha 45ab00edc7a0001385777200f17523853618fdca

Product 削除で ProductTag も削除されるように DB のスキーマを変更

view details

hideki_okajima

commit sha d4a9df7cda0028aa225c490f40afd6f3bb0d9c27

flush() の引数は非推奨のため削除

view details

Kentaro Ohkouchi

commit sha a35e35698b6ae0b446ad97ea4ce89e808ce7227e

Bump version PHP7.4 and PostgreSQL13

view details

hideki_okajima

commit sha 441d6dda1b9f04f4af117978d49226e0fef658e1

利用されていないコードを削除

view details

Bell

commit sha 76a77f0171e887ad7e3020a3062710ff8257dda0

重複していたcompany_nameを削除

view details

Kentaro Ohkouchi

commit sha 5f3e9af95ea8580357ebe7ff4268b4e9b3cfa3f4

SiteKit, GMC プラグインも SQLite3 に未対応 - API プラグインに依存しているため

view details

Tao Sasaki

commit sha 1547e84879a96061f1c801f23cae94613989ca60

gulp-sass v5に対応

view details

Chihiro Adachi

commit sha 4cd1563b26814861997aa8a42b68782726f49555

Merge branch '4.1' into feat/generator-product-tag-add

view details

mrsekut

commit sha 4ebde4abed6bce10c4c91cf7d2a09f3da9c517cc

fix: cart.php内でthis返却の実装漏れを修正

view details

Kentaro Ohkouchi

commit sha d664c769950129e2bc8a54dfdb10c6f1072e87af

Merge remote-tracking branch 'upstream/4.1' into update-github-actions

view details

Tao Sasaki

commit sha 1cff8b027e0db2d431257cf407d99d17b6b0ff18

Merge branch '4.1' into patch-24

view details

Kazuaki Yabu

commit sha 997679cc40a4a3f56c46e4d99b01e9e0a0e226ee

Merge pull request #5182 from nanasess/unsupport-plugins SQLite3 使用時、SiteKit, GMC プラグインはインストーラの有効化ボタンを表示しないよう修正

view details

matsuoshi

commit sha aa1ad04a387e98160f754bc612e5ff26d3d6ab54

Revert "受注管理画面:表示項目「出荷ID」の追加"

view details

matsuoshi

commit sha bf76193e7753552b43b616e86b717c66564daaf4

Merge pull request #5217 from EC-CUBE/revert-5030-refine-kenshi/feature/order_list_admin Revert "受注管理画面:表示項目「出荷ID」の追加"

view details

Kentaro Ohkouchi

commit sha 5ccd7b8ba83a8493d3c78827cb12eea7526d72f2

Bump version PHP7.4 and PostgreSQL14

view details

Kentaro Ohkouchi

commit sha 5b8b6e1901e001675b82963643d47d8fbb247faa

Merge remote-tracking branch 'upstream/4.1' into update-github-actions

view details

Kentaro Ohkouchi

commit sha 0e2a6350038ae2875d3e82bf72c0a549f2134f7a

ユニットテストのPHPバージョンをシステム要件に合わせる PHP7.3, 7.4 に変更

view details

Kentaro Ohkouchi

commit sha ec477fb5deb41eb4149214d7a15d8bd1092d2007

Bump to php version

view details

Kentaro Ohkouchi

commit sha 7a0efa256fc95620686e110e6ef4369e801c3204

プラグインインストール後に config.platform.php を元に戻す

view details

push time in 8 days

created tagEC-CUBE/ec-cube

tagco/4.1-20211118

EC-CUBE is the most popular e-commerce solution in Japan

created time in 15 days

push eventEC-CUBE/ec-cube

Kentaro Ohkouchi

commit sha 36a92c1cbb3a3816e4e17e5f93435f75de611acc

有りの場合にチェックを入れるのか、無しの場合にチェックを入れるのかわかりづらいのを修正

view details

kenshiro.sawamura

commit sha ca14d1ac4ab82a6f873a0236110f2a9ef65d8c77

受注管理画面:表示項目「出荷ID」の追加

view details

Kazuyoshi Kakihara

commit sha 27a0bfdc51699d888812f5dd07612f2c6223003b

重複したステートメントを削除

view details

dependabot[bot]

commit sha bf39fa6227d69a08014748adafa89b538c915e28

Bump path-parse from 1.0.6 to 1.0.7 Bumps [path-parse](https://github.com/jbgutierrez/path-parse) from 1.0.6 to 1.0.7. - [Release notes](https://github.com/jbgutierrez/path-parse/releases) - [Commits](https://github.com/jbgutierrez/path-parse/commits/v1.0.7) --- updated-dependencies: - dependency-name: path-parse dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>

view details

dependabot[bot]

commit sha dae2be73cd111762e9487e51df7be3709147ea90

Bump codeception/codeception from 4.1.21 to 4.1.22 Bumps [codeception/codeception](https://github.com/Codeception/Codeception) from 4.1.21 to 4.1.22. - [Release notes](https://github.com/Codeception/Codeception/releases) - [Changelog](https://github.com/Codeception/Codeception/blob/4.1/CHANGELOG-4.x.md) - [Commits](https://github.com/Codeception/Codeception/compare/4.1.21...4.1.22) --- updated-dependencies: - dependency-name: codeception/codeception dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com>

view details

Kentaro Ohkouchi

commit sha ae8cbf2b624056c6528e7a324d969e65f4596c81

Remove docker-composer.owaspzap.daemon.yml

view details

Kentaro Ohkouchi

commit sha b24f05cda2b0fd4bd3925385822e94d22d077601

OWASP ZAP 自動化手順の README 追加

view details

dependabot[bot]

commit sha 8e2ebfb2c7c260d90eceee3368f01cc238f0c413

Bump composer/composer from 2.1.3 to 2.1.9 Bumps [composer/composer](https://github.com/composer/composer) from 2.1.3 to 2.1.9. - [Release notes](https://github.com/composer/composer/releases) - [Changelog](https://github.com/composer/composer/blob/master/CHANGELOG.md) - [Commits](https://github.com/composer/composer/compare/2.1.3...2.1.9) --- updated-dependencies: - dependency-name: composer/composer dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>

view details

Kazuaki Yabu

commit sha a3616d2cf89b3b192bfda29c980b18510a49d4bc

fixed タグ管理の項目追加の自動表示位置が誤り #4981

view details

Kazuaki Yabu

commit sha 0cf844491f22e2aa0004b68a538167d36b44c468

受注登録でお届け先に氏名が表示されない #4980

view details

Kazuaki Yabu

commit sha db5917286889be9925100dcb0d44ec2ac7f885eb

受注登録でお届け先に氏名が表示されない #4980

view details

yshirai

commit sha 95f343858900360c87bbfe1cf3c6ef7db13f44e1

受注編集時にすでに受注にセットされている顧客情報を書き換えない

view details

Chihiro Adachi

commit sha 035b0012b3865df5564366e5dec8280db437da0a

テストコードを追加

view details

matsuoshi

commit sha 705a36fa896e210f6d01d086da49b30a58e9b0af

Merge pull request #4997 from yshirai-joolen/no_update_customer_info_when_edit_order 受注編集時にすでに受注にセットされている顧客情報を書き換えない

view details

matsuoshi

commit sha 645ed79373dd47fd986278dd2f96d637a8dedae1

Merge pull request #5030 from refine-kenshi/refine-kenshi/feature/order_list_admin 受注管理画面:表示項目「出荷ID」の追加

view details

matsuoshi

commit sha 4f370145f3d6cac1aab26ff70067c1cfa235d946

Merge pull request #5066 from kazuyoshikakihara/fix/orderHelper 重複したステートメントを削除

view details

Kazuaki Yabu

commit sha 83f34621e1c8d023680ce68a1c2684d17035ce84

Merge pull request #5179 from nanasess/remove-yml パッケージから docker-composer.owaspzap.daemon.yml を削除

view details

Kazuaki Yabu

commit sha 2406d1ee292bd03006a98e87771025c9cad1fa1f

Merge pull request #5183 from nanasess/add-readme-owaspzap-ci OWASP ZAP 自動化手順の README 追加

view details

Kazuaki Yabu

commit sha e571ab8f1a8dfa1ac0e746b8a0f52aaf1ff8b86f

Merge pull request #5012 from nanasess/modify-pr-template PULL_REQUEST_TEMPLATE の修正

view details

Chihiro Adachi

commit sha 1601d1d1db451112b21aadd7d76d26ea88a3e9f0

Merge pull request #5101 from EC-CUBE/dependabot/npm_and_yarn/path-parse-1.0.7 Bump path-parse from 1.0.6 to 1.0.7

view details

push time in 15 days

created tagEC-CUBE/ec-cube

tagco/4.1-20211111

EC-CUBE is the most popular e-commerce solution in Japan

created time in 15 days

push eventEC-CUBE/ec-cube

kiy0taka

commit sha dbdee3f2525ac066efdb8dfeba7a2eaebc896309

UnitTestとE2Eのカバレッジを分離

view details

Kentaro Ohkouchi

commit sha e72432b490f9ca978a0908a58fe18f14c11f7043

, が抜けていたのを修正

view details

Kentaro Ohkouchi

commit sha fb4b8a6c7f2c8e3b2a8814f4bded3ee593038ca7

HttpMessage と Alert の型定義追加

view details

Kentaro Ohkouchi

commit sha d176c2be4c85e2a2c702fbb572abfd5e503c205d

Remove any

view details

Chihiro Adachi

commit sha 52651e8a69dd86702fef9d1ca0d1e90130919a55

Merge pull request #5113 from kiy0taka/dev/split-coverage UnitTestとE2Eのカバレッジを分離

view details

Kiyotaka Oku

commit sha 5d05d0ba6df04ce050a2cb16fcc6f0b8b26818ea

Merge pull request #5139 from nanasess/fix-arguments ファイル管理のアイコンをクリックすると JavaScript エラーになるのを修正

view details

Kiyotaka Oku

commit sha b0c61a5fe5ad44589ed28a8c2eec40baa7aa1693

Merge pull request #5178 from nanasess/add-typedefinition OWASP ZAP のテストに型定義を追加

view details

kiy0taka

commit sha 26a50d17af247daeabbb9a0a954890c14ec960a3

upgrade ca-certificates

view details

Chihiro Adachi

commit sha e9550100508a09df1b60dcca4a744fe8d56369b0

Merge pull request #5201 from kiy0taka/fix/ca-certificates 開発用コンテナでプラグインインストール時に発生する証明書エラーの対応

view details

push time in 23 days

PR opened EC-CUBE/ec-cube

開発用コンテナでプラグインインストール時に発生する証明書エラーの対応

<!-- 以下を参考にコメントを作成してください。 -->

概要(Overview・Refs Issue)

<!-- PullRequestの目的、関連するIssue番号など --> Let's Encryptの新しいルート証明書に対応してなかったので、ca-certificatesをアップデート。

方針(Policy)

<!-- このPullRequestを作るにあたって考慮したものや除外した内容  - 例)Symfony のXXにならって作成、3系で同様の仕様があったため など -->

実装に関する補足(Appendix)

<!-- コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと -->

テスト(Test)

<!-- テストを行っている範囲など、レビューアが安心できるような情報 -->

相談(Discussion)

<!-- 相談したいことや意見を求めたいこと -->

マイナーバージョン互換性保持のための制限事項チェックリスト

<!-- マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。 -->

  • [ ] 既存機能の仕様変更
  • [ ] フックポイントの呼び出しタイミングの変更
  • [ ] フックポイントのパラメータの削除・データ型の変更
  • [ ] twigファイルに渡しているパラメータの削除・データ型の変更
  • [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更
  • [ ] 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
+1 -0

0 comment

1 changed file

pr created time in a month

create barnchkiy0taka/ec-cube

branch : fix/ca-certificates

created branch time in a month

push eventEC-CUBE/ec-cube

Kentaro Ohkouchi

commit sha fb4b8a6c7f2c8e3b2a8814f4bded3ee593038ca7

HttpMessage と Alert の型定義追加

view details

Kentaro Ohkouchi

commit sha d176c2be4c85e2a2c702fbb572abfd5e503c205d

Remove any

view details

Kiyotaka Oku

commit sha b0c61a5fe5ad44589ed28a8c2eec40baa7aa1693

Merge pull request #5178 from nanasess/add-typedefinition OWASP ZAP のテストに型定義を追加

view details

push time in 2 months

PR merged EC-CUBE/ec-cube

OWASP ZAP のテストに型定義を追加 improvement

<!-- 以下を参考にコメントを作成してください。 -->

概要(Overview・Refs Issue)

<!-- PullRequestの目的、関連するIssue番号など -->

  • 開発時の利便性向上のため
  • #5169

方針(Policy)

<!-- このPullRequestを作るにあたって考慮したものや除外した内容  - 例)Symfony のXXにならって作成、3系で同様の仕様があったため など -->

  • HttpMessage と Alert の型定義を追加

実装に関する補足(Appendix)

<!-- コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと --> 型定義は以下を参考

  • https://github.com/zaproxy/zaproxy/blob/main/zap/src/main/java/org/zaproxy/zap/extension/api/ApiResponseConversionUtils.java#L80-L122
  • https://github.com/zaproxy/zaproxy/blob/main/zap/src/main/java/org/parosproxy/paros/core/scanner/Alert.java#L198

テスト(Test)

<!-- テストを行っている範囲など、レビューアが安心できるような情報 --> GitHub Actions で正常にエラーハンドリングできるのを確認(https://github.com/nanasess/ec-cube/runs/3698050584?check_suite_focus=true)

相談(Discussion)

<!-- 相談したいことや意見を求めたいこと -->

マイナーバージョン互換性保持のための制限事項チェックリスト

<!-- マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。 -->

  • [ ] 既存機能の仕様変更
  • [ ] フックポイントの呼び出しタイミングの変更
  • [ ] フックポイントのパラメータの削除・データ型の変更
  • [ ] twigファイルに渡しているパラメータの削除・データ型の変更
  • [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更
  • [ ] 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
+65 -12

2 comments

4 changed files

nanasess

pr closed time in 2 months

pull request commentEC-CUBE/ec-cube

OWASP ZAP のテストに型定義を追加

ありがとうございます。

nanasess

comment created time in 2 months

PullRequestReviewEvent

pull request commentEC-CUBE/ec-cube

ファイル管理のアイコンをクリックすると JavaScript エラーになるのを修正

ありがとうございます。

nanasess

comment created time in 2 months

push eventEC-CUBE/ec-cube

Kentaro Ohkouchi

commit sha e72432b490f9ca978a0908a58fe18f14c11f7043

, が抜けていたのを修正

view details

Kiyotaka Oku

commit sha 5d05d0ba6df04ce050a2cb16fcc6f0b8b26818ea

Merge pull request #5139 from nanasess/fix-arguments ファイル管理のアイコンをクリックすると JavaScript エラーになるのを修正

view details

push time in 2 months

PR merged EC-CUBE/ec-cube

Reviewers
ファイル管理のアイコンをクリックすると JavaScript エラーになるのを修正 bug:Low

<!-- 以下を参考にコメントを作成してください。 -->

概要(Overview・Refs Issue)

<!-- PullRequestの目的、関連するIssue番号など -->

  • ファイル管理のアイコンをクリックすると JavaScript エラーになるのを修正
  • , が抜けていた

スクリーンショット 2021-09-02 10 04 50

テスト(Test)

<!-- テストを行っている範囲など、レビューアが安心できるような情報 -->

アイコンクリックでフォルダ内に遷移するのを確認

相談(Discussion)

<!-- 相談したいことや意見を求めたいこと -->

マイナーバージョン互換性保持のための制限事項チェックリスト

<!-- マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。 -->

  • [x] 既存機能の仕様変更
  • [x] フックポイントの呼び出しタイミングの変更
  • [x] フックポイントのパラメータの削除・データ型の変更
  • [x] twigファイルに渡しているパラメータの削除・データ型の変更
  • [x] Serviceクラスの公開関数の、引数の削除・データ型の変更
  • [x] 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

  • [ ] 動作確認
  • [ ] コードレビュー
  • [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
+1 -1

1 comment

1 changed file

nanasess

pr closed time in 2 months

push eventEC-CUBE/ec-cube2

Kentaro Ohkouchi

commit sha 5de719edae46d0c6427a302a9a6d2dda546b4dd4

Windows 環境でエラーになるため, setup-php の順番を変更

view details

Kiyotaka Oku

commit sha ded2bda5800a0b8a049eed84bde657b593a1fc36

Merge pull request #478 from nanasess/fix-windows-environment Windows 環境でエラーになるため, setup-php の順番を変更

view details

push time in 2 months

PR merged EC-CUBE/ec-cube2

Windows 環境でエラーになるため, setup-php の順番を変更

Windows 環境で、 PHP5.5, 5.6 の PHPUnitがエラーになるため、以下のように修正。

  1. GitHub Actions の環境にインストールされた PHP で composer install を実行
  2. matrix で指定されたバージョンの PHP で EC-CUBE をセットアップ、 PHPUnit を実行
+8 -5

1 comment

1 changed file

nanasess

pr closed time in 2 months

push eventEC-CUBE/ec-cube2

Kentaro Ohkouchi

commit sha 9162cfaa6cd19d5472f8cb6df6aa3ef3b1632b0c

PHPStan のエラー修正 - `should return xxx but return statement is missing.`

view details

Kiyotaka Oku

commit sha da6e8fc8327e0bf622b46ef45416fa38326b12d2

Merge pull request #477 from nanasess/fix-phpstan-errors PHPStan のエラー修正

view details

push time in 2 months

PR merged EC-CUBE/ec-cube2

PHPStan のエラー修正

以下のエラーが出力されていたのを修正

  • should return xxx but return statement is missing.
+51 -13

1 comment

13 changed files

nanasess

pr closed time in 2 months

create barnchEC-CUBE/ec-cube

branch : co/4.1

created branch time in 2 months

more