search

hoaivan / Writing-Planet

1 stars 0 forks source link

màu highlight cho teacher chưa đúng như template #48

Closed hoaivan closed 12 years ago

hoaivan commented 12 years ago

theo màu trong file pdf khách gửi

cuongmc commented 12 years ago

Đổi sang #DF5F32

Done @ rev.201

hoaivan commented 12 years ago

test lại chưa thấy đổi màu mà còn bị chuyển thành màu xanh như student. Anh kiểm tra 1 user có là teacher ko theo thuật toán thế nào? => Hàm check xem user là teacher của 1 course nào đó trong cả hệ thống thế này: WpRoleUtils.isTeacher()

cuongmc commented 12 years ago

Anh check có phải là Teacher theo cái này:

Check có phải là để hệ thống gửi hay không là theo cái này (anh coi là Admin gửi thì vào assignment và màu xanh)

Anh xem source code thấy WpRoleUtils.isTeacher() gặp Admin cũng tính là Teacher luôn? Vậy anh update lại sử dụng 1 method isTeacher cho giống với WpRoleUtils.isTeacher() nhé?

hoaivan commented 12 years ago

Anh check có phải là Teacher theo cái này:

  • PrivateMessagingUtil.isTeacher() <== chỉ những thằng nào có chứa role = Teachers

Check có phải là để hệ thống gửi hay không là theo cái này:

  • PrivateMessagingUtil.isAdministrator() <== những thằng có chưa role = Administrator, Site Administrator, Site Owner

Sao lại cần check chỗ này nhỉ? Check có phải hệ thống gửi ko thì dựa vào TYPE chứ: public static final int PRIVATE_MESSAGING_TYPE_SYSTEM = 2;

Anh xem source code thấy WpRoleUtils.isTeacher() gặp Admin cũng tính là

Teacher luôn? Vậy anh update lại sử dụng 1 method isTeacher cho giống với WpRoleUtils.isTeacher() nhé?

Yes. Vì mặc dù ko có được assign role là teacher nhưng admin, owner cũng có đủ tất cả các quyền mà teacher có.

Reply to this email directly or view it on GitHub: https://github.com/hoaivan/Writing-Planet/issues/48#issuecomment-5565809

cuongmc commented 12 years ago

2012/5/8 hoaivan < reply@reply.github.com

Anh check có phải là Teacher theo cái này:

  • PrivateMessagingUtil.isTeacher() <== chỉ những thằng nào có chứa role = Teachers

Check có phải là để hệ thống gửi hay không là theo cái này:

  • PrivateMessagingUtil.isAdministrator() <== những thằng có chưa role = Administrator, Site Administrator, Site Owner

Sao lại cần check chỗ này nhỉ? Check có phải hệ thống gửi ko thì dựa vào TYPE chứ: public static final int PRIVATE_MESSAGING_TYPE_SYSTEM = 2;

Cái này anh viết hơi ngược. Anh đang thực hiện check tại thời điểm gửi mesage đi.

  1. Nếu thằng gửi là Teacher => set Type = 1
  2. Nếu thằng gửi là Admin => set Type = 2

Đấy cũng là lý do tại sao em có câu hỏi này tại #55 Thêm nữa hiện tại teacher send msg (new message) thì chui vào assignment => vào inbox thôi chứ? Msg trong assignment đều là các msg do hệ thống tự tạo từ đầu thôi. => Anh đang hiểu sai rồi. Giờ anh cập nhật lại lúc send cứ cho vào Inbox như thường, bất kể là thằng nào gửi. Chỉ có phần của em làm tự động gửi thì mới quan tâm set Type = 1, Type = 2.

Anh xem source code thấy WpRoleUtils.isTeacher() gặp Admin cũng tính là

Teacher luôn? Vậy anh update lại sử dụng 1 method isTeacher cho giống với WpRoleUtils.isTeacher() nhé?

Yes. Vì mặc dù ko có được assign role là teacher nhưng admin, owner cũng có đủ tất cả các quyền mà teacher có.

=> OK

hoaivan commented 12 years ago

Cái này anh viết hơi ngược. Anh đang thực hiện check tại thời điểm gửi mesage đi.

  1. Nếu thằng gửi là Teacher => set Type = 1
  2. Nếu thằng gửi là Admin => set Type = 2

Đấy cũng là lý do tại sao em có câu hỏi này tại #55 Thêm nữa hiện tại teacher send msg (new message) thì chui vào assignment => vào inbox thôi chứ? Msg trong assignment đều là các msg do hệ thống tự tạo từ đầu thôi. => Anh đang hiểu sai rồi. Giờ anh cập nhật lại lúc send cứ cho vào Inbox như thường, bất kể là thằng nào gửi. Chỉ có phần của em làm tự động gửi thì mới quan tâm set Type = 1, Type = 2.

Đúng rồi. Nhưng em sẽ ko dùng type này đâu: PRIVATE_MESSAGING_TYPE_TEACHER = 1. Sẽ chỉ set Type = SYSTEM thôi. Khi display mesage anh sẽ cần check role của user để highlight name. Tóm lại là Type Teacher bỏ đi cũng được vì ko cần dùng chỗ nào hết.

cuongmc commented 12 years ago

2012/5/8 hoaivan < reply@reply.github.com

Đúng rồi. Nhưng em sẽ ko dùng type này đâu: PRIVATE_MESSAGING_TYPE_TEACHER = 1. Sẽ chỉ set Type = SYSTEM thôi. Khi display mesage anh sẽ cần check role của user để highlight name. Tóm lại là Type Teacher bỏ đi cũng được vì ko cần dùng chỗ nào hết.

Ok

cuongmc commented 12 years ago

Done @ rev.212