water / mainline

Core web service
GNU Affero General Public License v3.0
9 stars 0 forks source link

Feature/assistant relation fix #159

Closed jesjos closed 12 years ago

jesjos commented 12 years ago

En Assistant kan ha flera LabHasGroups via AssistantRegisteredToGivenCourse och AssistantRegisteredToGivenCourseHasLabHasGroup.

Join-entiteten AssistantRegisteredToGivenCourseHasLabHasGroup används istället för en has_many och belongs_to-lösning för att undvika en null-value approach.

jesjos commented 12 years ago

Kortare namn på till exempel AssistantRegisteredToGivenCourseHasLabHasGroup vore ju trevligt.

Vi kanske kan våga oss på lite förkortningar?

Typ: AsstRegToGivCrsHasLabHasGrp

Fast, haha, det blev ju nästan lika illa....

jesjos commented 12 years ago

Jag har förstått att Linus är upptagen i helgen, men kanske @spontus @anjonas @karinsofia också skulle kunna ta en titt på pull requesten?

anjonas commented 12 years ago

Har lite svårt att förstå Rails relationerna , men det verkar bra. Finns det någon validering som ser till att varje lab_has_group bara har en arfgc?

jesjos commented 12 years ago

Tack för din feeback @anjonas! Om det är någon speciell relation som är otydlig så ställer jag gärna upp och förklarar. Det som ser till att varje LHG bara har en arfgc är denna rad:

class AssistantRegisteredToGivenCourseHasLabHasGroup < ActiveRecord::Base
  # ...
  validates_uniqueness_of :lab_has_group_id
  # ...
end
jesjos commented 12 years ago

Om inga protester trillat in innan 21:00 ikväll så kommer jag att mergea in detta.

Ping @oleander @spontus @karinsofia @anjonas @Tarrasch

Tarrasch commented 12 years ago

Jag är för merge!