-
Notifications
You must be signed in to change notification settings - Fork 661
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
ref #5725 商品コードバリデートの不整合 #5740
ref #5725 商品コードバリデートの不整合 #5740
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.2 #5740 +/- ##
============================================
+ Coverage 78.83% 78.84% +0.01%
- Complexity 6267 6273 +6
============================================
Files 468 469 +1
Lines 21046 21059 +13
============================================
+ Hits 16592 16605 +13
Misses 4454 4454
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@@ -84,6 +84,11 @@ public function buildForm(FormBuilderInterface $builder, array $options) | |||
]) | |||
->add('code', TextType::class, [ | |||
'required' => false, | |||
'constraints' => [ | |||
new Assert\Length([ | |||
'max' => 255, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nobuhiko
eccube_stext_len
を使っていただけるとありがたいです。
@chihiro-adachi 修正しました |
ご対応ありがとうございます! |
概要(Overview・Refs Issue)
#5725
方針(Policy)
ProductClassType と ProductClassEditType.php で内容が違うのであわせる
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目