From 29c54e64804b08b95c13b801cc275291a05143a5 Mon Sep 17 00:00:00 2001
From: Christophe Vilayphiou <chris.vilayphiou@gmail.com>
Date: Thu, 17 May 2012 16:40:09 +0800
Subject: [PATCH] Add mandatory fields to assets

---
 app/assets/stylesheets/message.css          | 2 +-
 app/models/asset.rb                         | 2 +-
 app/views/admin/assets/file_upload.html.erb | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/assets/stylesheets/message.css b/app/assets/stylesheets/message.css
index 9ee1d45da..7206bd727 100644
--- a/app/assets/stylesheets/message.css
+++ b/app/assets/stylesheets/message.css
@@ -1,7 +1,7 @@
 /*message*/
 
 .error{
-  color:red;
+  color: #B94A48;
   }
 	
 .notice, .message{
diff --git a/app/models/asset.rb b/app/models/asset.rb
index a4065f179..34ba57e50 100644
--- a/app/models/asset.rb
+++ b/app/models/asset.rb
@@ -10,7 +10,7 @@ class Asset
   
   has_one :i18n_variable, :as => :language_value, :autosave => true, :dependent => :destroy
   
-  validates_presence_of :filename, :data
+  validates_presence_of :filename, :data, :asset_category_id
 
   belongs_to :asset_category
   belongs_to :assetable, polymorphic: true
diff --git a/app/views/admin/assets/file_upload.html.erb b/app/views/admin/assets/file_upload.html.erb
index eeed414be..9d0c7fb03 100644
--- a/app/views/admin/assets/file_upload.html.erb
+++ b/app/views/admin/assets/file_upload.html.erb
@@ -9,7 +9,7 @@
     </div>
     <div class="modal-body form-horizontal">
         <div class="control-group">
-            <label for="title" class="control-label"><%= t 'admin.title' %></label>
+            <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| %>
                   <% @site_valid_locales.each do |locale| %>
@@ -23,8 +23,8 @@
                 <% end %>
             </div>
         </div>
-        <div class="control-group">
-            <label class="control-label"><%= f.label :category, t('admin.category') %></label>
+        <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>
@@ -41,7 +41,7 @@
             </div>
         </div>
         <div class="control-group">
-            <label class="control-label"><%= f.label :data, t('admin.data') %></label>
+            <%= f.label :data, t('admin.data'), :class => "control-label" %>
             <div class="controls">
                 <%= f.file_field :data, :class => 'upload' %>
             </div>