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

postgresql利用時のtranslateのFROMの修正 #5472

Conversation

kazuyoshikakihara
Copy link
Contributor

概要(Overview・Refs Issue)

PostgreSQL使用時に使われるtranslate用のFROM文字列が一部間違っており、意図どおりにtranslateされず、「ウヰスキー」を「うゐすきー」で検索してもヒットしなかったりするので修正

方針(Policy)

最短で修正

実装に関する補足(Appendix)

特になし

テスト(Test)

PostgreSQL固有の問題のため自動テストなし。
ローカルの環境で商品名に「ウヰスキー」を登録し、「うゐすきー」で検索した場合、修正前はヒットしないが、修正後はヒットすることを確認。

相談(Discussion)

特になし

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

@codecov-commenter
Copy link

codecov-commenter commented Jul 25, 2022

Codecov Report

Merging #5472 (6a437d9) into 4.2 (9bafbe4) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                4.2    #5472   +/-   ##
=========================================
  Coverage     78.74%   78.75%           
  Complexity     6251     6251           
=========================================
  Files           468      468           
  Lines         21007    21010    +3     
=========================================
+ Hits          16543    16547    +4     
+ Misses         4464     4463    -1     
Flag Coverage Δ
E2E 64.35% <ø> (+0.01%) ⬆️
Unit 76.99% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/Eccube/Doctrine/ORM/Query/Normalize.php 64.70% <ø> (ø)
src/Eccube/Form/Type/Install/Step3Type.php 100.00% <0.00%> (ø)
src/Eccube/Form/Type/Admin/ChangePasswordType.php 94.11% <0.00%> (+0.36%) ⬆️
...ontroller/Admin/Product/ProductClassController.php 90.85% <0.00%> (+0.57%) ⬆️
src/Eccube/Form/Type/RepeatedPasswordType.php 85.00% <0.00%> (+0.78%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bafbe4...6a437d9. Read the comment docs.

@kiy0taka
Copy link
Contributor

@kazuyoshikakihara ありがとうございます。
GitHub Actionsでの自動テストではPostgreSQLとMySQLの両方で実行しているのでテストケースがあると助かるのですが、追加してもらえますか?

@kazuyoshikakihara
Copy link
Contributor Author

@kiy0taka なるほど了解しました。
該当箇所を探して2~3日中にテストケース追加します。

@kazuyoshikakihara
Copy link
Contributor Author

PostgreSQL固有の問題なので、PostgreSQLの場合のみ動作する形でテストケース追加しました。

@chihiro-adachi chihiro-adachi added this to the 4.2.0 milestone Aug 1, 2022
@chihiro-adachi chihiro-adachi merged commit ef16a1a into EC-CUBE:4.2 Aug 1, 2022
@chihiro-adachi
Copy link
Contributor

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

@carkn carkn added the bugbounty2022:pr:entry バグバウンティ2022 バグ登録 PR登録 label Aug 2, 2022
@chihiro-adachi chihiro-adachi added bugbounty2022:pr:testcode バグバウンティ2022 バグ登録 テストコードの記述 bugbounty2022:pr:recognition バグバウンティ2022 バグ登録 PR取込 labels Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:Low bugbounty2022:pr:entry バグバウンティ2022 バグ登録 PR登録 bugbounty2022:pr:recognition バグバウンティ2022 バグ登録 PR取込 bugbounty2022:pr:testcode バグバウンティ2022 バグ登録 テストコードの記述
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants