mindpin / kc_courses

MIT License
0 stars 6 forks source link

增加参加课程的模型方法 #4

Closed fushang318 closed 8 years ago

fushang318 commented 8 years ago

参加课程的功能,建议增加如下方法

# 查询 user 所有参加的课程
user.join_courses

# 如果引入了 Kaminari,可以支持分页查询
user.join_courses do |q|
  q.page(7).per(50)
end

# 参加某一个课程
user.join_course(course)

# 取消参加某一个课程
user.cancel_join_course(course)

另外
User.has_many :joins 建议改成 User.has_many :course_joins ,因为 joins 这个名字比较宽泛,当很多功能模块后,容易冲突和不易维护

destinyd commented 8 years ago

https://github.com/mindpin/kc_courses/pull/7

destinyd commented 8 years ago
# 返回加入课程
user.join_courses

此方法处理时理解有误,需要稍作处理

destinyd commented 8 years ago

user.join_courses 已照要求实现,并带有 kaminari 相关测试