small fixes rubriken und themen

app/controllers/neuigkeiten_controller.rb

@@ -9,6 +9,11 @@ class NeuigkeitenController < ApplicationController
   def show
     @neuigkeit = Neuigkeit.find(params[:id])
     @rubrik=@neuigkeit.rubrik    
+    if can?(:shownonpublic, Rubrik)
+      @rubriken = Rubrik.all
+    else
+      @rubriken = Rubrik.where(:public=>true)
+    end   
 
     if  !params[:version].nil?
       @neuigkeit.assign_attributes(@neuigkeit.translation.versions.reverse[params[:version].to_i].reify.attributes.select{|k,v| @neuigkeit.translated_attribute_names.include? k.to_sym })

app/controllers/rubriken_controller.rb

@@ -2,14 +2,25 @@ class RubrikenController < ApplicationController
   before_filter {@toolbar_elements=[]}
    load_and_authorize_resource
   def index
+    if can?(:shownonpublic, Rubrik)
       @rubriken = Rubrik.all
-    @neuigkeiten = Neuigkeit.recent
-    @calentries= Calentry.public
+    else
+      @rubriken = Rubrik.where(:public=>true)
+    end   
+    @neuigkeiten = @rubriken.collect(&:neuigkeiten).map(&:recent).flatten
+    @calentries= @rubriken.collect(&:calendar).collect(&:calentries).flatten
   end
  
   def show
+    if can?(:shownonpublic, Rubrik)
+      @rubriken = Rubrik.all
+    else
+      @rubriken = Rubrik.where(:public=>true)
+    end   
+
     @rubrik = Rubrik.find(params[:id])
     @moderatoren=User.with_role(:newsmoderator,@rubrik)
+    @calentries= @rubrik.calentries
     if can?(:showunpublished, Neuigkeit)
       @neuigkeiten = @rubrik.neuigkeiten
     else

app/models/ability.rb

@@ -76,6 +76,7 @@ class Ability
    if user.has_role?("newsadmin") || user.has_role?( "fetadmin") || user.has_role?( "fetuser") 
 	  can :manage, Rubrik
 	  can :manage, Neuigkeit
+          can :shownonpublic, Rubrik
           can :showunpublished, Neuigkeit
    end
   

app/models/lva.rb

@@ -36,14 +36,18 @@
 
 class Lva < ActiveRecord::Base
   ERLAUBTE_TYPEN = ['VO', 'UE', 'VU', 'LU', 'SE', 'andere'];
+  
   has_paper_trail :ignore=>[:desc, :pruefungsinformation]# Versionsverfolgung
+  translates :desc,:pruefungsinformation,  :fallbacks_for_empty_translations => true, :versioning=>true
+
   attr_accessible :desc, :ects, :lvanr, :name, :stunden, :modul_ids, :semester_ids, :pruefungsinformation, :lernaufwand, :typ, :lecturer_ids
+
   has_and_belongs_to_many :modul,:uniq=>true # Gehört zu einem Modul
   has_and_belongs_to_many :semester
-  #Gehört zu einem Semester( derzeit nicht implementiert)
   has_many :beispiele , :class_name => "Beispiel"
   has_and_belongs_to_many :lecturers
-  translates :desc,:pruefungsinformation,  :fallbacks_for_empty_translations => true, :versioning=>true
+  has_many :nlinks, as: :link
+  
   scope :search, ->(query) {where("name like ? or lvas.desc like ?", "%#{query}%", "%#{query}%")} 
   
   validates :lvanr,:format=>{ :with => /^[0-9][0-9][0-9]\.[0-9A][0-9][0-9]$/}, :presence=>true, :uniqueness=>true # , :uniqueness=>true # LVA-Nummer muss das Format 000.000 besitzen (uniqueness?) oder 000 für nicht 
@@ -53,7 +57,7 @@ class Lva < ActiveRecord::Base
   validates_presence_of :stunden # Stunden Eingetragen?
   validates_presence_of :modul # Zugehöriges Modul eingetragen?
   # (zumindest eines)
- has_many :nlinks, as: :link
+ 
  
   def title
     self.name

app/views/rubriken/_tabs.html.erb

@@ -1,7 +1,7 @@
 <ul class="nav nav-tabs">
  <li <%=  'class="active"'.html_safe if (request.fullpath == rubriken_path)%> >
  <%= link_to I18n.t('rubriken.alle') , rubriken_path %> </li>
-<% for r in Rubrik.all %>
+<% for r in @rubriken  %>
 <% if can?(:show, r) %>
   <li <%= (!@rubrik.nil? && r == @rubrik) ? 'class="active"'.html_safe : '' %> title="<%= r.name %>">
     <%= link_to r.name, r %>

app/views/rubriken/show.html.erb

@@ -31,7 +31,7 @@
 
 </div>
 <div class="span3">
-<%= render 'calendars/calentries', :object=>@rubrik.calendar.calentries %>
+<%= render 'calendars/calentries', :object=>@calentries %>
 <%= link_to "ics-format", calendar_path(@rubrik.calendar,:format=>:ics) %>
 </div>
 

app/views/themen/verwalten.html.erb

@@ -20,5 +20,7 @@
 <h2><%= I18n.t('attachment.title')%>:</h2>
 <div id="attachmentform">
 </div>
+
 <%= link_to new_thema_attachment_path(@thema) ,:remote=>true do %> new Attachment <% end %>
 <%= render :partial => "themen/select", :object => @thema, :locals=>{:editor => :false} %>
+<%= link_to I18n.t('common.back'), themengruppen_path, :class=>:btn %>