From a1c43ef07934b0f3c15a5426c03be3085ae7df74 Mon Sep 17 00:00:00 2001 From: Thomas Blazek Date: Tue, 23 Jul 2013 17:10:28 +0200 Subject: [PATCH] FIX: Modulgruppen melden Fehler bei Studium blank MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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. --- app/controllers/modulgruppen_controller.rb | 4 ++-- app/models/modulgruppe.rb | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/app/controllers/modulgruppen_controller.rb b/app/controllers/modulgruppen_controller.rb index f0d8367..c8167d6 100755 --- a/app/controllers/modulgruppen_controller.rb +++ b/app/controllers/modulgruppen_controller.rb @@ -44,9 +44,9 @@ class ModulgruppenController < ApplicationController def create @modulgruppe = Modulgruppe.new(params[:modulgruppe]) if !params[:studium_id].nil? - @studium=Studium.find(params[:studium_id]) + @studium=Studium.find_by_id(params[:studium_id]) else - @studium=Studium.find(params[:modulgruppe][:studium_id]) + @studium=Studium.find_by_id(params[:modulgruppe][:studium_id]) end respond_to do |format| if @modulgruppe.save diff --git a/app/models/modulgruppe.rb b/app/models/modulgruppe.rb index 7f21489..30451e9 100755 --- a/app/models/modulgruppe.rb +++ b/app/models/modulgruppe.rb @@ -3,7 +3,8 @@ class Modulgruppe < ActiveRecord::Base belongs_to :studium, :foreign_key => "studium_id" has_and_belongs_to_many :moduls 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 :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"] }