andis/fetsite
1
0
Fork 0
forked from bofh/fetsite
Code Issues Pull Requests Releases Wiki Activity

FIX: Modulgruppen melden Fehler bei Studium blank

Browse Source 
Der Fix inkludiert zwei Änderungen: erstens ist die Abfrage im
Controller create jetzt mit find_by_id anstatt find, das behebt den
hauptsächlichen Fehler.
Zweitens hab ich ins modell studium_id als validation aufgenommen, ohne
dem wird es nicht richtig als Fehler dargestellt.
This commit is contained in:
Thomas Blazek
2013-07-23 17:10:28 +02:00
parent 5e9ffdf0d6
commit a1c43ef079
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/controllers/modulgruppen_controller.rb
View File

@@ -44,9 +44,9 @@ class ModulgruppenController < ApplicationController
def create def create
@modulgruppe = Modulgruppe.new(params[:modulgruppe]) @modulgruppe = Modulgruppe.new(params[:modulgruppe])
if !params[:studium_id].nil? if !params[:studium_id].nil?
@studium=Studium.find(params[:studium_id]) @studium=Studium.find_by_id(params[:studium_id])
else else
@studium=Studium.find(params[:modulgruppe][:studium_id]) @studium=Studium.find_by_id(params[:modulgruppe][:studium_id])
end end
respond_to do |format| respond_to do |format|
if @modulgruppe.save if @modulgruppe.save

3
app/models/modulgruppe.rb
View File

@@ -3,7 +3,8 @@ class Modulgruppe < ActiveRecord::Base
belongs_to :studium, :foreign_key => "studium_id" belongs_to :studium, :foreign_key => "studium_id"
has_and_belongs_to_many :moduls has_and_belongs_to_many :moduls
resourcify resourcify
validates :studium, :presence=>true validates :studium_id, :presence => true
validates :studium, :presence => true
validates :name, :presence=>true,:uniqueness =>{:scope => :studium, :message => "Nur einmal je Studium erlaubt"} validates :name, :presence=>true,:uniqueness =>{:scope => :studium, :message => "Nur einmal je Studium erlaubt"}
validates :phase, :numericality => { :only_integer => true },:inclusion => {:in => [1, 2, 3, 4], :message => "%{value} is not valid, choose phase 1 to 4"}, :presence=>true validates :phase, :numericality => { :only_integer => true },:inclusion => {:in => [1, 2, 3, 4], :message => "%{value} is not valid, choose phase 1 to 4"}, :presence=>true
validates :typ, :inclusion => {:in => ["Pflicht","Vertiefungspflicht","Wahl"] } validates :typ, :inclusion => {:in => ["Pflicht","Vertiefungspflicht","Wahl"] }