From 585af6f162b5b93a01c056dc2fe1e26ace84b0d6 Mon Sep 17 00:00:00 2001 From: Thomas Blazek Date: Fri, 9 Aug 2013 14:15:02 +0200 Subject: [PATCH 1/5] Lva form verbessert --- .../stylesheets/formtastic-changes.css.scss | 21 +++++++ app/views/lvas/_form.html.erb | 55 ++++++++++++++----- config/locales/de.yml | 10 +++- 3 files changed, 71 insertions(+), 15 deletions(-) create mode 100644 app/assets/stylesheets/formtastic-changes.css.scss diff --git a/app/assets/stylesheets/formtastic-changes.css.scss b/app/assets/stylesheets/formtastic-changes.css.scss new file mode 100644 index 0000000..5007540 --- /dev/null +++ b/app/assets/stylesheets/formtastic-changes.css.scss @@ -0,0 +1,21 @@ +form.formtastic .stringish input { +width:90%; +} +form.formtastic textarea { +width:90%; +height:20em; +} +form.formtastic .stringish label { +font-weight: bold; +} +form.formtastic .choice-group label { +font-weight: bold; +} +form.formtastic fieldset div.control-group{} + +div.form-group { +border-width: 1px; +border-style: dashed; +padding: 10px; +border-radius: 10px; +} diff --git a/app/views/lvas/_form.html.erb b/app/views/lvas/_form.html.erb index 49bbb78..8692e8e 100755 --- a/app/views/lvas/_form.html.erb +++ b/app/views/lvas/_form.html.erb @@ -1,15 +1,44 @@ -<%= semantic_form_for @lva do |f| %> - <%= f.inputs do %> - <%= f.input :semester, :collection=>Semester.all.sort_by{|n| [:name]}%> - <%= f.input :modul %> - <%= f.input :name %> - <%= f.input :ects %> - <%= f.input :stunden %> - <%= f.input :desc %> - <%= f.input :lvanr %> - <% end %> - <%= f.actions do %> - <%= f.action :submit, :as => :input %> - <% end %> +<%= semantic_form_for @lva do |f| %> +
+ <%= f.inputs do %> + +
+
+
+
+
+ + <%= f.input :name, :wrapper_html=>{:class=>'span5'}%> +
+
+ <%= f.input :lvanr, :wrapper_html=>{:class=>'span4'}%> + + <%= f.input :ects, :wrapper_html=>{:class=>'span4'} %> + <%= f.input :stunden, :wrapper_html=>{:class=>'span4'} %> +
+
+
+
+
+
+ <%= f.input :desc %> +
+
+ +
+

BILD

+ <%= f.input :modul,:label=>'Modul'.html_safe, :as=>:check_boxes%> +
+
+
+
+ + <%= f.input :semester, :label=>'Semester'.html_safe, :as=>:check_boxes%> + +
+ <% end %> + <%= f.actions do %> + <%= f.action :submit, :as => :input %> + <% end %> <% end %> diff --git a/config/locales/de.yml b/config/locales/de.yml index 94ea21c..2e2ca1e 100755 --- a/config/locales/de.yml +++ b/config/locales/de.yml @@ -91,7 +91,10 @@ de: stunden: blank: "Geben Sie die Stunden der LVA an" lvanr: - invalid: "Geben Sie eine LVA-Nummer in richtigem Format an. Wenn die LVA noch keine Nummer hat, so geben sie 000 an." + invalid: "Geben Sie eine Nummer im Format 000.000 an." + blank: "Geben sie eine Lva-Nr an" + taken: "Die Lva-Nummer ist bereits vergeben" + studium: attributes: zahl: @@ -118,4 +121,7 @@ de: verwalten: fehler: keine: "Keine Fehlermeldungen oder Warnungen" - + support: + array: + two_words_connector: ', ' + From 61fc09bcf07696e2d6d1cf23bc6aa5ebe17f14e8 Mon Sep 17 00:00:00 2001 From: Thomas Blazek Date: Fri, 9 Aug 2013 16:17:51 +0200 Subject: [PATCH 2/5] Formatierungen --- .../stylesheets/formtastic-changes.css.scss | 13 +++--- app/controllers/modulgruppen_controller.rb | 7 +-- app/views/modulgruppen/_form.html.erb | 24 +++++++--- app/views/studien/verwalten.html.erb | 44 ++++++++----------- 4 files changed, 44 insertions(+), 44 deletions(-) diff --git a/app/assets/stylesheets/formtastic-changes.css.scss b/app/assets/stylesheets/formtastic-changes.css.scss index 5007540..498eb2a 100644 --- a/app/assets/stylesheets/formtastic-changes.css.scss +++ b/app/assets/stylesheets/formtastic-changes.css.scss @@ -5,13 +5,7 @@ form.formtastic textarea { width:90%; height:20em; } -form.formtastic .stringish label { -font-weight: bold; -} -form.formtastic .choice-group label { -font-weight: bold; -} -form.formtastic fieldset div.control-group{} + div.form-group { border-width: 1px; @@ -19,3 +13,8 @@ border-style: dashed; padding: 10px; border-radius: 10px; } +div.verwalten-block { +border-width: 1px; +border-style: solid; +padding: 5px; +} \ No newline at end of file diff --git a/app/controllers/modulgruppen_controller.rb b/app/controllers/modulgruppen_controller.rb index c69f8fe..d84de6f 100755 --- a/app/controllers/modulgruppen_controller.rb +++ b/app/controllers/modulgruppen_controller.rb @@ -51,15 +51,10 @@ class ModulgruppenController < ApplicationController def create @modulgruppe = Modulgruppe.new(params[:modulgruppe]) - if !params[:studium_id].nil? - @studium=Studium.find_by_id(params[:studium_id]) - else - @studium=Studium.find_by_id(params[:modulgruppe][:studium_id]) - end respond_to do |format| if @modulgruppe.save - format.html { redirect_to @studium, notice: 'Modulgruppe was successfully created.' } + format.html { redirect_to @modulgruppe.studium, notice: 'Modulgruppe was successfully created.' } else format.html { render action: "new" } diff --git a/app/views/modulgruppen/_form.html.erb b/app/views/modulgruppen/_form.html.erb index 8191af5..22cc925 100755 --- a/app/views/modulgruppen/_form.html.erb +++ b/app/views/modulgruppen/_form.html.erb @@ -1,13 +1,25 @@ <%= semantic_form_for @modulgruppe do |f| %> <%= f.inputs do %> - <%= f.input :studium_id, :as =>:select, :collection => Studium.all %> - <%= f.input :typ ,:as => :radio, :collection=>["Pflicht","Vertiefungspflicht","Wahl"]%> - <%= f.input :phase,:as => :radio,:collection=>1..3 %> - <%= f.input :name %> - <%= f.input :desc,:input_html=>{:rows=> 3 }%> - <% end %> +
+
+ <%= f.input :name, :wrapper_html=>{:class=>"span12"}%> +
+ <%= f.input :studium_id, :as =>:select, :collection => Studium.all, :wrapper_html=>{:class=>"span4"} %> + <%= f.input :typ, :collection=>["Pflicht","Vertiefungspflicht","Wahl"], :wrapper_html=>{:class=>"span4"}%> + <%= f.input :phase, :as=>:select, :collection=>1..3, :wrapper_html=>{:class=>"span4"} %> +
+
+ + +
+ <%= f.input :desc %> + +
+ <% end %> +
<%= f.actions do %> <%= f.action :submit, :as => :input %> + <% end %> <% end %> diff --git a/app/views/studien/verwalten.html.erb b/app/views/studien/verwalten.html.erb index a834154..279026f 100644 --- a/app/views/studien/verwalten.html.erb +++ b/app/views/studien/verwalten.html.erb @@ -1,8 +1,7 @@
<%= render 'studien/tabs'%>
-
-
+

<%= I18n.t("studien.verwaltung") %>

@@ -11,7 +10,7 @@
-
+

Studien <%= link_to "new", new_studium_path %>

<% @studien.each do |s| %> <%= link_to s.name, s %> <%= (s.valid?) ? "Gültig" : "Problemfall" %>
@@ -23,7 +22,7 @@
-
+

Modulgruppen <%= link_to 'new', new_studium_modulgruppe_path(Studium.first) unless Studium.first.nil? %>

<% @modulgruppen.each do |mg| %> <%= link_to mg.name, mg %> | <%= (mg.valid?) ? "Gültig" : "Problemfall" %>
@@ -31,7 +30,7 @@
-
+

Module <%= link_to "new", new_modul_path %>

<% @module.each do |m| %> <%= link_to m.name, m %> @@ -40,7 +39,7 @@
-
+

LVas <%= link_to "new", new_lva_path %>

<% @lvas.each do |lva| %> <%= link_to lva.name, lva %> | @@ -50,7 +49,7 @@
-
+

Beispiele <%= link_to "new", new_beispiel_path %>

<% @beispiele.each do |b| %> <%= link_to b.name, b %> @@ -62,28 +61,23 @@
-
-
-
-

Error Log

+
+

Error Log

+
    +
  • Red: Error, Solve immediately
    • +
  • Black: Warning, should be solved
    • +
  • Green: Everything is fine
    • +
+
    -
  • Red: Error, Solve immediately
    • -
  • Black: Warning, should be solved
    • -
  • Green: Everything is fine
    • -
-
-
    - <% @messages.flatten.each do |m| %>
    -
  • <%= m.html_safe %>
    • - <% end %> -
- <%= I18n.t("verwalten.fehler.keine").html_safe if @messages.empty?%> -
+ <% @messages.flatten.each do |m| %>
+
  • <%= m.html_safe %>
    • + <% end %> + + <%= I18n.t("verwalten.fehler.keine").html_safe if @messages.empty?%>
    - -
    From b6671879fc906dbb8dca561e53c28ca1fec4e7e1 Mon Sep 17 00:00:00 2001 From: Thomas Blazek Date: Fri, 9 Aug 2013 18:49:30 +0200 Subject: [PATCH 3/5] Bug fix Die persistent variables waren schlecht benannt. hab sie eindeutig benannt --- app/controllers/studien_controller.rb | 32 +++++++++++----------- app/views/studien/_verwalten_menu.html.erb | 32 +++++++++++----------- 2 files changed, 32 insertions(+), 32 deletions(-) diff --git a/app/controllers/studien_controller.rb b/app/controllers/studien_controller.rb index 4d0a0b3..f2c9231 100755 --- a/app/controllers/studien_controller.rb +++ b/app/controllers/studien_controller.rb @@ -92,7 +92,7 @@ class StudienController < ApplicationController end def verwalten - @new_params={:studium=>params[:studium], :modulgruppe=>params[:modulgruppe], :modul=>params[:modul], :lva=>params[:lva], :beispiel=>params[:beispiel]} + @new_params={:std_verw=>params[:std_verw], :mg_verw=>params[:mg_verw], :m_verw=>params[:m_verw], :lva_verw=>params[:lva_verw], :b_verw=>params[:b_verw]} if @new_params.values.compact.empty? @studien=Studium.all @modulgruppen=Modulgruppe.all @@ -100,13 +100,13 @@ class StudienController < ApplicationController @lvas=Lva.all @beispiele=Beispiel.all else - if !@new_params[:studium].nil? - @studien = [Studium.find(@new_params[:studium])] + if !@new_params[:std_verw].nil? + @studien = [Studium.find(@new_params[:std_verw])] else @studien = Studium.all end - if !@new_params[:modulgruppe].nil? - @modulgruppen = [Modulgruppe.find(@new_params[:modulgruppe])] + if !@new_params[:mg_verw].nil? + @modulgruppen = [Modulgruppe.find(@new_params[:mg_verw])] temp = @modulgruppen.map{|x| x.studium}.flatten.uniq # Force Studien @studien=@studien.select{|k| temp.include?(k)} @@ -115,8 +115,8 @@ class StudienController < ApplicationController temp = @studien.map{|x| x.modulgruppen}.flatten.uniq # Studien forcen Modulgruppen @modulgruppen = @modulgruppen.select{|k| temp.include?(k)} end - if !@new_params[:modul].nil? - @module = [Modul.find(@new_params[:modul])] + if !@new_params[:m_verw].nil? + @module = [Modul.find(@new_params[:m_verw])] temp = @module.map{|x| x.modulgruppen}.flatten.uniq # Force Modulgruppen @modulgruppen = @modulgruppen.select{|k| temp.include?(k)} temp = @modulgruppen.map{|x| x.studium}.flatten.uniq # Force Studien @@ -127,8 +127,8 @@ class StudienController < ApplicationController temp = @modulgruppen.map{|x| x.moduls}.flatten.uniq @module=@module.select{|k| temp.include?(k)} end - if !@new_params[:lva].nil? - @lvas = [Lva.find(@new_params[:lva])] + if !@new_params[:lva_verw].nil? + @lvas = [Lva.find(@new_params[:lva_verw])] temp = @lvas.map{|x| x.modul}.flatten.uniq @module=@module.select{|k| temp.include?(k)} temp = @module.map{|x| x.modulgruppen}.flatten.uniq # Force Modulgruppen @@ -140,8 +140,8 @@ class StudienController < ApplicationController temp = @module.map{|x| x.lvas}.flatten.uniq #Force Module @lvas=@lvas.select{|k| temp.include?(k)} end - if !@new_params[:beispiel].nil? - @beispiele = [Beispiel.find(@new_params[:beispiel])] + if !@new_params[:b_verw].nil? + @beispiele = [Beispiel.find(@new_params[:b_verw])] temp = @lvas.map{|x| x.beispiele}.flatten.uniq #Force Force Lvas @lvas=@lvas.select{|k| temp.include?(k)} temp = @module.map{|x| x.lvas}.flatten.uniq #Force Module @@ -226,11 +226,11 @@ class StudienController < ApplicationController def default_url_options {:ansicht=> params[:ansicht], - :studium=> params[:studium], - :modulgruppe=> params[:modulgruppe], - :modul=>params[:modul], - :lva=>params[:lva], - :beispiel=>params[:beispiel]}.merge(super) + :std_verw=> params[:std_verw], + :mg_verw=> params[:mg_verw], + :m_verw=>params[:m_verw], + :lva_verw=>params[:lva_verw], + :b_verw=>params[:b_verw]}.merge(super) end end diff --git a/app/views/studien/_verwalten_menu.html.erb b/app/views/studien/_verwalten_menu.html.erb index 0c37f91..3f7c051 100644 --- a/app/views/studien/_verwalten_menu.html.erb +++ b/app/views/studien/_verwalten_menu.html.erb @@ -1,68 +1,68 @@
    - <%= params[:studium].nil? ? I18n.t("filter.studium") : Studium.find(params[:studium]).name %> + <%= params[:std_verw].nil? ? I18n.t("filter.studium") : Studium.find(params[:std_verw]).name %>
      -
    • <%= link_to 'Alle Studien', studien_verwalten_path(:studium=>nil) %>
      • +
    • <%= link_to 'Alle Studien', studien_verwalten_path(:std_verw=>nil) %>
      • <% for s in @studien %> -
    • <%= link_to s.name, studien_verwalten_path(:studium=>s)%>
      • +
    • <%= link_to s.name, studien_verwalten_path(:std_verw=>s)%>
      • <% end %>
    - <%= params[:modulgruppe].nil? ? I18n.t("filter.modulgruppe") : Modulgruppe.find(params[:modulgruppe]).name %> + <%= params[:mg_verw].nil? ? I18n.t("filter.modulgruppe") : Modulgruppe.find(params[:mg_verw]).name %>
      -
    • <%= link_to 'Alle Modulgruppen', studien_verwalten_path(:modulgruppe=>nil) %>
      • +
    • <%= link_to 'Alle Modulgruppen', studien_verwalten_path(:mg_verw=>nil) %>
      • <% for mg in @modulgruppen %> -
    • <%= link_to mg.name, studien_verwalten_path(:modulgruppe=>mg)%>
      • +
    • <%= link_to mg.name, studien_verwalten_path(:mg_verw=>mg)%>
      • <% end %>
    - <%= params[:modul].nil? ? I18n.t("filter.modul") : Modul.find(params[:modul]).name %> + <%= params[:m_verw].nil? ? I18n.t("filter.modul") : Modul.find(params[:m_verw]).name %>
      -
    • <%= link_to 'Alle Module', studien_verwalten_path(:modul=>nil) %>
      • +
    • <%= link_to 'Alle Module', studien_verwalten_path(:m_verw=>nil) %>
      • <% for m in @module %> -
    • <%= link_to m.name, studien_verwalten_path(:modul=>m)%>
      • +
    • <%= link_to m.name, studien_verwalten_path(:m_verw=>m)%>
      • <% end %>
    - <%= params[:lva].nil? ? I18n.t("filter.lva") : Lva.find(params[:lva]).name %> + <%= params[:lva_verw].nil? ? I18n.t("filter.lva") : Lva.find(params[:lva_verw]).name %>
      -
    • <%= link_to 'Alle Lvas', studien_verwalten_path(:lva=>nil) %>
      • +
    • <%= link_to 'Alle Lvas', studien_verwalten_path(:lva_verw=>nil) %>
      • <% for lva in @lvas %> -
    • <%= link_to lva.name, studien_verwalten_path(:lva=>lva)%>
      • +
    • <%= link_to lva.name, studien_verwalten_path(:lva_verw=>lva)%>
      • <% end %>
    - <%= params[:beispiel].nil? ? I18n.t("filter.beispiel") : Beispiel.find(params[:beispiel]).name %> + <%= params[:b_verw].nil? ? I18n.t("filter.beispiel") : Beispiel.find(params[:b_verw]).name %>
      -
    • <%= link_to 'Alle Beispiele', studien_verwalten_path(:beispiel=>nil) %>
      • +
    • <%= link_to 'Alle Beispiele', studien_verwalten_path(:b_verw=>nil) %>
      • <% for b in @beispiele %> -
    • <%= link_to b.name, studien_verwalten_path(:beispiel=>b)%>
      • +
    • <%= link_to b.name, studien_verwalten_path(:b_verw=>b)%>
      • <% end %>
    - <%= link_to I18n.t("filter.reset"), studien_verwalten_path(:studium=>nil, :modulgruppe=>nil, :modul=>nil, :lva=>nil, :beispiel=>nil), :class=>"btn" %> + <%= link_to I18n.t("filter.reset"), studien_verwalten_path(:std_verw=>nil, :mg_verw=>nil, :m_verw=>nil, :lva_verw=>nil, :b_verw=>nil), :class=>"btn" %>
    From 9bf4768e0981313ca33db9a79385be837435371a Mon Sep 17 00:00:00 2001 From: Thomas Blazek Date: Fri, 9 Aug 2013 18:49:56 +0200 Subject: [PATCH 4/5] Form designs --- .../stylesheets/formtastic-changes.css.scss | 4 +-- app/views/moduls/_form.html.erb | 31 ++++++++++++++----- app/views/studien/_form.html.erb | 29 ++++++++++++----- 3 files changed, 47 insertions(+), 17 deletions(-) diff --git a/app/assets/stylesheets/formtastic-changes.css.scss b/app/assets/stylesheets/formtastic-changes.css.scss index 498eb2a..192ee43 100644 --- a/app/assets/stylesheets/formtastic-changes.css.scss +++ b/app/assets/stylesheets/formtastic-changes.css.scss @@ -9,9 +9,9 @@ height:20em; div.form-group { border-width: 1px; -border-style: dashed; +border-style: none; padding: 10px; -border-radius: 10px; +// border-radius: 10px; } div.verwalten-block { border-width: 1px; diff --git a/app/views/moduls/_form.html.erb b/app/views/moduls/_form.html.erb index fb6b68b..db03f9d 100755 --- a/app/views/moduls/_form.html.erb +++ b/app/views/moduls/_form.html.erb @@ -1,15 +1,30 @@ +
    + <%= semantic_form_for @modul do |f| %> -<%= semantic_form_for @modul do |f| %> <%= f.inputs do %> - <% # f.input :modulgruppen,:as => :select, :collection => Hash[Modulgruppe.all.map{|m| [m.studium.name + " " + m.name,m.id]}] %> - <%= f.input :modulgruppen %> - <%= f.input :name ,:hint=>true%> - <%= f.input :desc, :as=>:tinymce_text %> - <%= f.input :depend %> +
    +
    + <%= f.input :name ,:hint=>true%> +
    +
    + <%= f.input :desc, :as=>:tinymce_text %> +
    +
    + <%= f.input :depend %> +
    +
    +
    +
    + <%= f.input :modulgruppen,:as => :check_boxes, :collection => Hash[Modulgruppe.all.map{|m| [m.studium.name + " " + m.name,m.id]}.sort] %> +
    +
    + <% end %> <%= f.actions do %> - <%= f.action :submit, :as => :input %> + <%= f.action :submit, :as => :input %> <% end %> -<% end %> + + <% end %> +
    <%= tinymce %> diff --git a/app/views/studien/_form.html.erb b/app/views/studien/_form.html.erb index 471a8c3..0973ef5 100755 --- a/app/views/studien/_form.html.erb +++ b/app/views/studien/_form.html.erb @@ -1,13 +1,28 @@ +
    <%= tinymce_assets %> -<%= semantic_form_for @studium do |f| %> + <%= semantic_form_for @studium do |f| %> +
    <%= f.inputs do %> - <%= f.input :zahl %> - <%= f.input :name,:append=>"NAME" %> + +
    +
    + <%= f.input :name,:append=>"NAME" %> +
    +
    + <%= f.input :zahl %> +
    +
    + <%= f.input :typ, :as=>:select, :include_blank=>false, :collection=>["Bachelor","Master"]%> +
    +
    +
    <%= f.input :desc, :as=>:tinymce_text %> - <%= f.input :typ, :as=>:radio, :collection=>["Bachelor","Master"]%> +
    <% end %> -<%= tinymce %> + <%= tinymce %> <%= f.actions do %> - <%= f.action :submit, :as => :input %> + <%= f.action :submit, :as => :input %> <% end %> -<% end %> +
    + <% end %> +
    From a8c6ecd08cfff0995c1ac704f06c44364aeebcbb Mon Sep 17 00:00:00 2001 From: Thomas Blazek Date: Fri, 9 Aug 2013 19:09:01 +0200 Subject: [PATCH 5/5] beispiel formular --- app/views/beispiele/_form.html.erb | 39 ++++++++++++++++++++++-------- 1 file changed, 29 insertions(+), 10 deletions(-) diff --git a/app/views/beispiele/_form.html.erb b/app/views/beispiele/_form.html.erb index bf0c027..1e3833c 100755 --- a/app/views/beispiele/_form.html.erb +++ b/app/views/beispiele/_form.html.erb @@ -1,13 +1,32 @@ -<%= semantic_form_for @beispiel, :html => { :multipart => true } do |f| %> - <%= f.inputs do %> - <%= f.input :name %> - <%= f.input :desc %> - <%= f.file_field :beispieldatei %> - <%= f.hidden_field :beispieldatei_cache %> - <%= f.input :lva, :as=>:select, :collection => Lva.all%> - <% end %> +
    +
    + <%= semantic_form_for @beispiel, :html => { :multipart => true } do |f| %> + + <%= f.inputs do %> +
    +
    + <%= f.file_field :beispieldatei %> + <%= f.hidden_field :beispieldatei_cache %> +
    +
    + <%= f.input :lva, :as=>:select, :collection => Lva.all%> + +
    +
    + <%= f.input :name %> +
    +
    + +
    + <%= f.input :desc %> +
    + <% end %> <%= f.actions do %> - <%= f.action :submit, :as => :input %> + <%= f.action :submit, :as => :input %> + <% end %> -<% end %> + + <% end %> +
    +