circularnotices's Introduction
circularnotices's People
circularnotices's Issues
viewVars['userId']の削除
詳しくは、見てないのですが、下記の$userIdですが、viewVarsの中の$userIdを参照しているのでしたら、Current::read('User.id')に変えてください。
https://github.com/NetCommons3/CircularNotices/blob/master/View/CircularNotices/index.ctp#L111
https://github.com/NetCommons3/CircularNotices/blob/master/View/CircularNotices/view.ctp#L45
ちなみに、セットしているところは、下記です。
https://github.com/NetCommons3/NetCommons/blob/master/Controller/NetCommonsAppController.php#L158
ソースファイルの改行コードがCRLFになっている。
ソースファイルの改行コードがCRLFになっています。UTF8のLFにしてください。
一時保存や回覧期間終了の回覧を開いたときでも既読になる
未読>既読にする条件は「現時点で未読」且つ「回覧中か回答受付終了」
回覧機関のエラーメッセージがおかしい
【一覧画面】回覧の絞り込みと表示順ソート処理を修正
回覧を削除する際、ループする必要ありますか?
回覧を削除する際、わざわざループしていますが、ループする必要ありますか?
https://github.com/NetCommons3/CircularNotices/blob/master/Model/CircularNoticeContent.php#L480-L490
公開期間、回答期間のタイムゾーンが正しくセットされていない
回答内容登録処理について
CircularNoticesController::Viewアクションに登録処理がありますが、
コントローラー分けたほうが良い気がしています。
CircularNoticesAnswerController::editアクションがCakePHPっぽいと思います。
新着情報の削除処理の追加依頼
#82
このIssueのため、組み込めません。
こちらを参考に、
$this->contentKeyのセットとdeleteAllの第三引数をtrue(コールバックの有効)にしてください。
https://github.com/s-nakajima/Bbses/blob/master/Model/BbsArticle.php#L269-L270
回覧作成権限チェック処理を追加
FrameIdがなくても、エラーにせず、デフォルト値を戻してください。
Chromeで回覧を作成する際にDateTimePickerがTimeのみになる
Chromeのみで発生する現象。Safari、IE、Firefox は問題ない。
Chromeでは、DateTimePickerが表示されるが、テキストボックスには時刻の表示になっている。
しかも、時刻を選択しても何も反映されない。
メール関連処理を実装
datepickerが組み込まれていない
回覧期間、回答期限の日付入力部にdatepickerの組み込みが必要。
recursiveについて
'recursive' => 0
で良いと思われます。
recursiveの値については、勘違いしやすい部分なので説明のサイトあげときます。
ちょと古いですが、考え方は変わっていないと思います。
http://www.cpa-lab.com/tech/081
ログイン者以外が作成した回覧の場合の条件の追加
ログイン者が作成した回覧以外の場合は取得条件の追加が必要。
・ログイン者が回覧先に含まれていること
・回覧が公開されていること(一時保存でないこと)
・回覧期間内であること
サブクエリーついて
確認です。
回覧板はサブクエリー使用してないんでしょうか?
ファイルのパーミッション
回答内容データの保持
回答内容を保持しているようですが、何に使用するのでしょうか?
ログインしていないとき、Noticeがでる。
【詳細画面】回覧先ユーザのソート処理を修正
'Pages.PageLayout'コンポーネントの定義を追加してください。
左右おかずつける方法は、'Pages.PageLayout'コンポーネントの定義を一行追加してください。
https://github.com/NetCommons3/Announcements/blob/master/Controller/AnnouncementsAppController.php#L29
また、テストについては、この辺を参考に設定してください。
https://github.com/NetCommons3/Announcements/blob/master/Test/Case/Controller/AnnouncementsControllerTestBase.php
【登録画面】ルームに参加する全ユーザを取得して回覧先に設定する処理を追加
CircularNoticeBlockRolePermissionsControllerでFatal Error
3a7111d#diff-71854a4e65ad3ba541f79469c6b80e27
でFrameモデルを使用するように修正しています。
usesにないので、
Call to a member function findById() on a non-object
のエラーが出ます。
そもそも、この処理いらない気がします。
削除しても動いています。
回覧のステータスや回答状況による編集可否判断
例:回覧期間中の場合は編集ができないなど
【登録画面】新規登録時にデフォルトで自分自身が回覧先として選択された状態にする
セッティングモードの呼び出しアクション
CircularNoticeBlocksController::indexでリダイレクトしていますが、
最初からCircularNoticeBlockRolePermissionsController::edit呼び出せば良いように思います。
の値を変更するmigrationファイル作成で行けると思います。
回覧板の追加でNoticeがでる。
無駄なHelperの定義ある
無駄なHelperの定義があります。
NetCommons3/NetCommons3#221
Issueのほか、このHelperも使ってますか?
https://github.com/NetCommons3/CircularNotices/blob/master/Controller/CircularNoticesController.php#L55
一度、見直してください。
また、ログインしているユーザIDを使う場合は、AuthではなくCurrentを使ってください。
https://github.com/NetCommons3/CircularNotices/blob/master/Controller/CircularNoticesController.php#L299
CircularNoticeTargetUserのuser_status
virtualFieldsとして保持してますが、実態を持った方が分かりやすい気がします。
登録時に更新するようなイメージです。
メンバー変数は、メソッドより上が良いのでは。
ブロック管理(フレームの歯車)に遷移しようとすると、エラーになる
バリデーションの実装
バリデーションが未完
【編集画面】決定ボタン押下時の処理を修正
回答状況一覧のCSVダウンロード処理を実装
多言語対応について
CircularNoticeContentにlanguage_idがないので、多言語化できない感じですが、
回覧板は多言語化ないのでしょうか?
後々追加予定?
あと、回答データ(CircularNoticeTargetUser)は、多言語化するのでしょうかね。。。
これは、アンケートも同じですよね。
もくもく会で確認させてください。
初期データ登録処理
CircularNoticesAppController::beforeFilterで初期データ登録処理を呼んでいます。
CircularNoticeSetting::prepareCircularNoticeSettingと
CircularNoticeFrameSetting::prepareCircularNoticeFrameSettingです。
CircularNoticeBlockRolePermissionsController::editアクションの時だけ呼び出すのが良いように思います。
【登録画面】回覧先のラジオボタン切り替えによるユーザ選択画面の表示・非表示切り替え
回覧先はデフォルトで開いている状態に変更
詳細の回覧先一覧がデフォルトで閉じているため、ページネーションによる画面遷移のたびに回覧先一覧が閉じてしまう。
回覧のステータスによる回答可否判断
例:ログイン者が作成した回覧で回覧先に自身が含まれていない場合は出さない
例:回覧受付終了となっている場合は出さない
など
閲覧件数、回答件数、未読件数処理について
コメントに書いてありますが、issueにもあげときます。
ContentCommentsプラグインにもあげたのですが、件数の取得処理が厄介な感じです。
NetCommons3/ContentComments#2
CakePHPだと、counterCacheという仕組みが提供されています。
http://book.cakephp.org/2.0/ja/models/associations-linking-models-together.html#countercache-count
これだと、テーブルにフィールド追加になります。
状態ごとに件数を取得することを考えると、counterCacheはきびしいと思ったのですが、
状態ごとの件数をフィールドにもつと行けそうです。
案としてあげときます。
回答の入力タイプ共通化
【詳細画面】「回答する」ボタンを除去
バリデーションチェックでNGとなった際に回覧先ユーザの選択状態を保持できるように修正
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.