tera

Results 3 comments of tera

React DnDのissueに上がっていた下記と同様の可能性が高い https://github.com/react-dnd/react-dnd/issues/3425#issuecomment-1214554950

@kumabecu どちらかというと表示画面の文字を任意とするところを必須にしてしまっている気がします。 basercms4時代のblog_comments_scripts.jsを参照してもメールアドレスは必須判定しておらず、送信できます。 https://github.com/baserproject/basercms/blob/b75f280c72432ebe981b7eff9b8d35fa9fda28df/lib/Baser/Plugin/Blog/webroot/js/blog_comments_scripts.js#L38 あとバックエンド側のbasercms5のBlogCommentsTable.phpのvalidationDefaultメソッドもメールアドレスも必須になっていないです。 この観点から元々は任意としたかったんだと思いました。 謎なのがbasercms5.1系のbundleされたblog_comment.bundle.jsとblog_comment.jsの内容に相違があるように思えます。 (ビルドまでして確認したわけではないですが、5.1系はフロントのemailのバリデーションが実装されていそう。) https://github.com/baserproject/basercms/blob/89e2554f944d405bb526fe11051bb8c8837424a9/plugins/bc-front/src/bc_blog/js/blog_comment.js#L154 https://github.com/baserproject/basercms/blob/4ec1cabcbfbf57c43c898129ee58116c86820ee6/plugins/bc-front/src/bc_blog/js/blog_comment.js#L154

いえ、お手間を取らせ申し訳ないです。 4系も表示画面自体は必須とラベル表示されているので、どちらが正しい仕様なのか微妙なんですよね... 必須にするなら、フロントのバリデーションチェックとバックエンド側のチェックどちらかを実装する際に気付きそうですし