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.

Sign up for GitHub

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

Jump to bottom

[WIP] Symfony5 対応 #4528

Closed
wants to merge 62 commits into from
Closed

[WIP] Symfony5 対応 #4528

wants to merge 62 commits into from

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented May 1, 2020 •
edited

概要(Overview・Refs Issue)

方針(Policy)

  • Symfony5 でユニットテスト、 E2Eテストが通る状態とする

実装に関する補足(Appendix)

とりあえず TOP ページ表示に成功

利用できなくなってしまったパッケージ、クラス

代替方法を要検討

  • symfony/web-server-bundle → symfony/cli を使用する
  • nesbot/carbon
  • suncat/mobile-detect-bundle
  • dama/doctrine-test-bundle
  • Eccube\Twig\Extension\IgnoreRoutingNotFoundExtension

テスト（Test)

TODO

相談（Discussion）

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

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

レビュワー確認項目

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

@nanasess
非推奨の警告を表示
7b171d7
@nanasess
softDeleteFilter は使用しなくなったため削除
ee5f603 
- The Eccube\Util\EntityUtil::isEmpty method is deprecated.
@nanasess
Fix deprecated Symfony Container
70e5420 
- The "Eccube\Repository\*Repository" service is private, getting it from the container is deprecated since Symfony 3.2 and will fail in 4.0. You should either make the service public, or stop using the container directly and use dependency injection instead.
@nanasess
Symfony4.4 support
30fcc75
@nanasess
container 変数を static に変更
b783282 
see https://symfony.com/blog/new-in-symfony-4-1-simpler-service-testing
@nanasess
プレースホルダを使用しているメッセージ '' で囲む
fc88f37
@nanasess
Fix variables
059e1b7
@nanasess
Set scope to public
e8af733
@nanasess
Use Interface
97d58e0
@nanasess
Fix variables
2535553
@nanasess
Add getExtendedTypes()
dd79e8f
@nanasess
Add FIXME comment
53f4079
@nanasess
Use static
2f0e03c
@nanasess
Fixes EC-CUBE#2874 Serializable を実装
b569ac1 
- Symfony/Cache が Closure を使用している関係で Serializable を実装しないとログインに失敗するため
@nanasess
form.type_extension は autoconfiguration の対象になったため削除
a7d2fa6
@nanasess
Revert to FormExtension
b35feba
@nanasess
Fix obsolete class name
0d8e49b
@nanasess
Remove obsolete to Silex Application
ea90347
@nanasess
ComposerApiService to public
39b23c9
@nanasess
Fix functions to logger and translation
52220ac
@nanasess
AnnotationReader を Facade で設定するように修正
d22c409 
- LoadEventSubscriber は廃止
@nanasess
Customer の Serializable 実装に伴う修正
81459a8 
- 配列でセッションに保存し, オブジェクトに復元するよう修正
- OrderHelper::getNonMember() でセッションキーを指定できるよう修正
@nanasess
Add Form::isSubmitted()
f7ca09f
@nanasess
TODO エラーメッセージが翻訳されない
642a17b
@nanasess
Apply composer update
b593290 
- Updating doctrine/cache (1.9.1 => 1.10.0)
- Updating doctrine/doctrine-cache-bundle (1.3.5 => 1.4.0)
- Updating guzzlehttp/guzzle (6.4.1 => 6.5.0)
- Updating symfony/asset (v4.4.0 => v4.4.1)
- Updating symfony/browser-kit (v4.4.0 => v4.4.1)
- Updating symfony/cache (v4.4.0 => v4.4.1)
- Updating symfony/config (v4.4.0 => v4.4.1)
- Updating symfony/console (v4.4.0 => v4.4.1)
- Updating symfony/css-selector (v4.4.0 => v4.4.1)
- Updating symfony/debug (v4.4.0 => v4.4.1)
- Updating symfony/debug-bundle (v4.4.0 => v4.4.1)
- Updating symfony/dependency-injection (v4.4.0 => v4.4.1)
- Updating symfony/doctrine-bridge (v4.4.0 => v4.4.1)
- Updating symfony/dom-crawler (v4.4.0 => v4.4.1)
- Updating symfony/dotenv (v4.4.0 => v4.4.1)
- Updating symfony/error-handler (v4.4.0 => v4.4.1)
- Updating symfony/event-dispatcher (v4.4.0 => v4.4.1)
- Updating symfony/expression-language (v4.4.0 => v4.4.1)
- Updating symfony/filesystem (v4.4.0 => v4.4.1)
- Updating symfony/finder (v4.4.0 => v4.4.1)
- Updating symfony/flex (v1.4.8 => v1.5.3)
- Updating symfony/form (v4.4.0 => v4.4.1)
- Updating symfony/framework-bundle (v4.4.0 => v4.4.1)
- Updating symfony/http-foundation (v4.4.0 => v4.4.1)
- Updating symfony/http-kernel (v4.4.0 => v4.4.1)
- Updating symfony/inflector (v4.4.0 => v4.4.1)
- Updating symfony/intl (v4.4.0 => v4.4.1)
- Updating symfony/mime (v4.4.0 => v4.4.1)
- Updating symfony/monolog-bridge (v4.4.0 => v4.4.1)
- Updating symfony/options-resolver (v4.4.0 => v4.4.1)
- Updating symfony/phpunit-bridge (v4.4.0 => v4.4.1)
- Updating symfony/polyfill-ctype (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-iconv (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-intl-icu (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-intl-idn (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-mbstring (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-php70 (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-php72 (v1.13.0 => v1.13.1)
- Updating symfony/polyfill-php73 (v1.13.0 => v1.13.1)
- Updating symfony/process (v4.4.0 => v4.4.1)
- Updating symfony/property-access (v4.4.0 => v4.4.1)
- Updating symfony/proxy-manager-bridge (v4.4.0 => v4.4.1)
- Updating symfony/routing (v4.4.0 => v4.4.1)
- Updating symfony/security (v4.4.0 => v4.4.1)
- Updating symfony/security-bundle (v4.4.0 => v4.4.1)
- Updating symfony/serializer (v4.4.0 => v4.4.1)
- Updating symfony/stopwatch (v4.4.0 => v4.4.1)
- Updating symfony/templating (v4.4.0 => v4.4.1)
- Updating symfony/translation (v4.4.0 => v4.4.1)
- Updating symfony/twig-bridge (v4.4.0 => v4.4.1)
- Updating symfony/twig-bundle (v4.4.0 => v4.4.1)
- Updating symfony/validator (v4.4.0 => v4.4.1)
- Updating symfony/var-dumper (v4.4.0 => v4.4.1)
- Updating symfony/var-exporter (v4.4.0 => v4.4.1)
- Updating symfony/web-profiler-bundle (v4.4.0 => v4.4.1)
- Updating symfony/web-server-bundle (v4.4.0 => v4.4.1)
- Updating symfony/workflow (v4.4.0 => v4.4.1)
- Updating symfony/yaml (v4.4.0 => v4.4.1)
- Updating zendframework/zend-code (3.4.0 => 3.4.1)
@nanasess
Merge remote-tracking branch 'upstream/4.0' into improve/symfony4.4
5e8a1e5
@nanasess
キャッシュの再生成できるところまで修正
36f3930
@nanasess
TOPページが表示できるよう修正
b39d4e4
@nanasess
依存関係解消のために削除していたパッケージを追加
37dc985 
- symfony/cache
- codeception
- doctrine-test-bundle
@nanasess
symfony/phpunit-bridge の依存関係を使用する
6c34c1f
@okazy okazy marked this pull request as draft November 2, 2020 04:57
@okazy okazy modified the milestones: 4.1, 4.x Nov 2, 2020
@okazy
Copy link
Contributor

okazy commented Nov 2, 2020 •
edited

4.0から4.1へはプラグインの互換性を最大限保った形でのアップデートを目指しています。
Symfony5へのアップデートは変更箇所が多く、プラグインの互換性を大きく損なう恐れがあるため、マイルストンを 4.x とさせていただきます。
また、GitHubに Draft 機能がありましたので、Draftに変更させていただきました。

@nanasess nanasess mentioned this pull request Dec 25, 2020
Closed
17 tasks
@nanasess nanasess changed the base branch from 4.0 to 4.1 December 25, 2020 07:15
@nanasess nanasess mentioned this pull request Feb 17, 2021
Merged
11 tasks
@nanasess nanasess changed the base branch from 4.1 to 4.1-core March 11, 2021 01:04
@chihiro-adachi chihiro-adachi modified the milestones: 4.x, Not release Aug 26, 2021
@chihiro-adachi
Copy link
Contributor

4.1はSymfony4.4を採用のため、こちらのPRはクローズします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
experimental
Projects
None yet
Milestone
Not release
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@nanasess @ehudthelefthand @okazy @chihiro-adachi