SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Refactoring/comment view #252

Closed westpaddy closed 9 years ago

westpaddy commented 9 years ago

コメントの状態編集をステータスセルに即座に反映させる変更を入れるためにコードを整理しました。動作自体はほとんどもとのままのはずです。 状態反映の変更はまだ入ってません。

westpaddy commented 9 years ago

変更の補足をしておくと、一つ目のコミットはコメントフォームとコメント編集時のフォームがほとんど同じだったので一つにまとめて別のファイルに分けました。

二つ目のコミットはコメントビューが持っているコメントのデータと各コメントが持っているデータが編集後に整合性取れなくなっていたので、データはコメントビュー側で管理するように変更しました。

三つ目のコミットは、コメントビューをレンダリングするときに全部のコメントを既読にしていたのを、コメントビューがマウントされたときに既読にするように変更しました。 これはレンダリングが開いたときだけじゃなくて、上からプロップが降ってきたり、ステートが変わったりしたときにも呼ばれるので、未読にした後再読み込みしなくても勝手に既読になってしまうのを防ぐためです。

westpaddy commented 9 years ago

一部嘘書きました。一つ目のコミットはコメントフォームを別ファイルに移しただけで、編集用のフォームをコメントフォームで置き換えたのは二つ目のコミットに入ってます。

yf-fyf commented 9 years ago

一般ユーザでコメントをポストしようとすると,"Cannot read property 'getDOMNode' of undefined"とエラーがでます. 再現できますか?こちらの環境がオカシイのかも.

一般ユーザでもコメントの削除はできました.

westpaddy commented 9 years ago

直しました。