Merge branch 'ldap' of github.com:Rulingcom/orbit into ldap

app/controllers/admin/assets_controller.rb

@@ -30,7 +30,7 @@ class Admin::AssetsController < OrbitBackendController
   
   def create  
     @asset = Asset.new(params[:asset])
-    @asset.filename = @asset.i18n_variable[I18n.locale] rescue nil
+    @asset.filename = @asset.title[I18n.locale] rescue nil
     if @asset.filename && @asset.save
       respond_to do |format|
         format.js {

app/models/asset.rb

@@ -8,16 +8,38 @@ class Asset
   field :filename
   field :description
 
-  has_one :i18n_variable, :as => :language_value, :autosave => true, :dependent => :destroy
+  has_one :title, :class_name => "I18nVariable", :as => :language_value, :autosave => true, :dependent => :destroy
+  has_one :description, :class_name => "I18nVariable", :as => :language_value, :autosave => true, :dependent => :destroy
   
-  validates_presence_of :filename, :data, :asset_category_id
+  validates_presence_of :title, :data, :description
 
   belongs_to :asset_category
   belongs_to :assetable, polymorphic: true
   has_and_belongs_to_many :tags, :class_name => "AssetTag"
 
+  before_save :set_key
+
+  def title
+    @title ||= I18nVariable.first(:conditions => {:key => 'title', :language_value_id => self.id, :language_value_type => self.class}) rescue nil
+  end
+
+  def description
+    @description ||= I18nVariable.first(:conditions => {:key => 'description', :language_value_id => self.id, :language_value_type => self.class}) rescue nil
+  end
+
   def sorted_tags
     tags.order_by(I18n.locale, :asc)
   end
 
+  protected
+  
+  def set_key
+    if title.new_record?
+      title.key = 'title'
+    end
+    if description.new_record?
+      description.key = 'description'
+    end
+  end
+  
 end

app/views/admin/assets/_asset.html.erb

@@ -1,6 +1,6 @@
 <tr id="asset_<%= asset.id %>" class="with_action">
 	<td><%= check_box_tag 'to_delete[]', asset.id, false, :class => "checkbox_in_list" %></td>
-	<td><%= asset.i18n_variable[I18n.locale] rescue nil %></td>
+	<td><%= asset.title.i18n_variable[I18n.locale] rescue nil %></td>
 	<td>
 		<i class="icons-picture img-peview" rel="popover" data-content="<img src='<%= asset.data.url %>' />" data-original-title="<%= asset.data.filename %>"></i>
 		<div class="quick-edit">
@@ -12,7 +12,7 @@
 	</td>
 	<td><%= asset.data.file.content_type %></td>
 	<td><%= number_to_human_size(asset.data.file.file_length) %></td>
-	<td><%= asset.category.i18n_variable[I18n.locale] rescue nil %></td>
+	<td><%= asset.description.i18n_variable[I18n.locale] rescue nil %></td>
 	<td>
 		<div class="label-group">
 			<div class="label-td">

app/views/admin/assets/_sort_headers.html.erb

@@ -2,5 +2,5 @@
 													['description', 'description', 'span1-2', 'admin.data'],
 													['intro', 'intro', 'span1-2', 'admin.file_type'],
 													['intro', 'intro', 'span1-2', 'admin.file_length'],
-													['intro', 'intro', 'span1-2', 'admin.category'],
+													['intro', 'intro', 'span1-2', 'admin.description'],
 													['intro', 'intro', 'span1-2', 'admin.tags']).html_safe %>

app/views/admin/assets/file_upload.html.erb

@@ -11,7 +11,7 @@
         <div class="control-group">
             <label for="title" class="control-label error"><%= t 'admin.title' %></label>
             <div class="controls">
-                <%= f.fields_for :i18n_variable, (@asset.new_record? ? @asset.build_i18n_variable : @asset.i18n_variable) do |f| %>
+                <%= f.fields_for :title, (@asset.new_record? ? @asset.build_title : @asset.title) do |f| %>
                   <% @site_valid_locales.each do |locale| %>
                     <%= content_tag :label do -%>
                         <div>
@@ -23,19 +23,17 @@
                 <% end %>
             </div>
         </div>
-        <div class="control-group error">
-            <%= f.label :category, t('admin.category'), :class => "control-label" %>
-            <div class="controls">
-                <%= f.select :asset_category_id, @asset_categories.collect{|t| [ t.i18n_variable[I18n.locale], t.id ]}, {}, :class => "input-large" %>
-            </div>
-        </div>
         <div class="control-group">
-            <label class="control-label"><%= t 'admin.tags' %></label>
+            <label for="description" class="control-label error"><%= t 'admin.description' %></label>
             <div class="controls">
-                <% @tags.each do |tag| %>
+                <%= f.fields_for :description, (@asset.new_record? ? @asset.build_description : @asset.description) do |f| %>
-                    <%= content_tag :label, :class => "checkbox inline" do -%>
+                  <% @site_valid_locales.each do |locale| %>
-                        <%= check_box_tag 'asset[tag_ids][]', tag.id, @asset.tag_ids.include?(tag.id) %>
+                    <%= content_tag :label do -%>
-                        <%= tag[I18n.locale] %>
+                        <div>
+                            <%= I18nVariable.from_locale(locale) %>
+                            <%= f.text_field locale, :class => "input-large" %>
+                        </div>
+                    <% end %>
                   <% end %>
                 <% end %>
             </div>

app/views/layouts/_side_bar.html.erb

@@ -71,11 +71,11 @@
 	<% end -%>
 <% end -%>
 
-<%= content_tag :li, :class => active_for_controllers('assets', '/admin/asset_tags', 'asset_categories') do -%>
+<%#= content_tag :li, :class => active_for_controllers('assets', '/admin/asset_tags', 'asset_categories') do -%>
-	<%= link_to content_tag(:i, nil, :class => 'icons-link') + t('admin.asset'), admin_assets_path %>
+	<%#= link_to content_tag(:i, nil, :class => 'icons-link') + t('admin.asset'), admin_assets_path %>
-	<%= content_tag :ul, :class => ("nav nav-list " + visible_for_controllers('assets', '/admin/asset_tags', 'asset_categories')) do -%>
+	<%#= content_tag :ul, :class => ("nav nav-list " + visible_for_controllers('assets', '/admin/asset_tags', 'asset_categories')) do -%>
-		<%= content_tag :li, link_to(t('admin.all_assets'), admin_assets_path), :class => active_for_action('assets', 'index') %>
+		<%#= content_tag :li, link_to(t('admin.all_assets'), admin_assets_path), :class => active_for_action('assets', 'index') %>
-		<%= content_tag :li, link_to(t('admin.categories'), admin_asset_categories_path), :class => active_for_action('asset_categories', 'index') %>
+		<%#= content_tag :li, link_to(t('admin.categories'), admin_asset_categories_path), :class => active_for_action('asset_categories', 'index') %>
-		<%= content_tag :li, link_to(t('admin.tags'), admin_asset_tags_path), :class => active_for_action('/admin/asset_tags', 'index') %>
+		<%#= content_tag :li, link_to(t('admin.tags'), admin_asset_tags_path), :class => active_for_action('/admin/asset_tags', 'index') %>
-	<% end -%>
+	<%# end -%>
-<% end -%>
+<%# end -%>