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

app/assets/javascripts/tinymce_orbit.js.erb

@@ -62,14 +62,16 @@ function myFileBrowser(field_name, url, type, win) {
     tinyMCE.activeEditor.windowManager.open({
         file : cmsURL,
         title : 'File Browser',
-        width : 850,  // Your dimensions may differ - toy around with them!
-        height : 455,
+        width : 530,  // Your dimensions may differ - toy around with them!
+        height : 350,
         resizable : "no",
         inline : "no",  // This parameter only has an effect if you use the inlinepopups plugin!
         close_previous : "no"
     }, {
         window : win,
         input : field_name,
+        alt : "alt",
+        title : "title"
     });
     return false;
   }

app/assets/stylesheets/message.css

@@ -1,7 +1,7 @@
 /*message*/
 
 .error{
-  color:red;
+  color: #B94A48;
   }
 	
 .notice, .message{

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/helpers/admin/asset_helper.rb

@@ -0,0 +1,11 @@
+module Admin::AssetHelper
+
+  def show_all_fields(asset, field)
+    a = []
+    @site_valid_locales.each do |locale|
+      a << asset.send(field)[locale] rescue nil
+    end 
+    a.join(' - ')
+  end
+
+end

app/models/asset.rb

@@ -3,21 +3,43 @@ class Asset
   include Mongoid::Document
   include Mongoid::Timestamps
 
-  mount_uploader :data, AssetUploader
+  mount_uploader :data, FileAssetUploader
 
   field :filename
   field :description
+
+  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
   
-  has_one :i18n_variable, :as => :language_value, :autosave => true, :dependent => :destroy
-  
-  validates_presence_of :filename, :data
+  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/uploaders/asset_uploader.rb

@@ -48,8 +48,8 @@ class AssetUploader < CarrierWave::Uploader::Base
   # end
 
   # Override the filename of the uploaded files:
-  def filename
-    model.filename
-  end
+  # def filename
+  #   model.filename
+  # end
 
 end

app/uploaders/file_asset_uploader.rb

@@ -0,0 +1,55 @@
+# encoding: utf-8
+
+class FileAssetUploader < CarrierWave::Uploader::Base
+  require 'mime/types'
+  
+  process :set_content_type
+
+  def set_content_type(*args)
+    content_type = file.content_type == ('binary/octet-stream' || 'application/octet-stream') || file.content_type.blank? ? MIME::Types.type_for(original_filename).first.to_s : file.content_type
+    self.file.instance_variable_set(:@content_type, content_type)
+  end
+
+  # Include RMagick or ImageScience support:
+  # include CarrierWave::RMagick
+  # include CarrierWave::ImageScience
+
+  # Choose what kind of storage to use for this uploader:
+  # storage :file
+  # storage :s3
+
+  # Override the directory where uploaded files will be stored.
+  # This is a sensible default for uploaders that are meant to be mounted:
+  def store_dir
+    "assets/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
+  end
+
+  # Provide a default URL as a default if there hasn't been a file uploaded:
+  # def default_url
+  #   "/images/fallback/" + [version_name, "default.png"].compact.join('_')
+  # end
+
+  # Process files as they are uploaded:
+  # process :scale => [200, 300]
+  #
+  # def scale(width, height)
+  #   # do something
+  # end
+
+  # Create different versions of your uploaded files:
+  # version :thumb do
+  #   process :scale => [50, 50]
+  # end
+
+  # Add a white list of extensions which are allowed to be uploaded.
+  # For images you might use something like this:
+  # def extension_white_list
+  #   %w(jpg jpeg gif png)
+  # end
+
+  # Override the filename of the uploaded files:
+  # def filename
+  #   model.filename.force_encoding("UTF-8")
+  # 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/create.js.erb

@@ -3,10 +3,13 @@
 // document.domain = 'moxiecode.com';
 
 var tinymce=null,tinyMCEPopup,tinyMCE;tinyMCEPopup={init:function(){var b=this,a,c;a=b.getWin();tinymce=a.tinymce;tinyMCE=a.tinyMCE;b.editor=tinymce.EditorManager.activeEditor;b.params=b.editor.windowManager.params;b.features=b.editor.windowManager.features;b.dom=b.editor.windowManager.createInstance("tinymce.dom.DOMUtils",document);if(b.features.popup_css!==false){b.dom.loadCSS(b.features.popup_css||b.editor.settings.popup_css)}b.listeners=[];b.onInit={add:function(e,d){b.listeners.push({func:e,scope:d})}};b.isWindow=!b.getWindowArg("mce_inline");b.id=b.getWindowArg("mce_window_id");b.editor.windowManager.onOpen.dispatch(b.editor.windowManager,window)},getWin:function(){return(!window.frameElement&&window.dialogArguments)||opener||parent||top},getWindowArg:function(c,b){var a=this.params[c];return tinymce.is(a)?a:b},getParam:function(b,a){return this.editor.getParam(b,a)},getLang:function(b,a){return this.editor.getLang(b,a)},execCommand:function(d,c,e,b){b=b||{};b.skip_focus=1;this.restoreSelection();return this.editor.execCommand(d,c,e,b)},resizeToInnerSize:function(){var a=this;setTimeout(function(){var b=a.dom.getViewPort(window);a.editor.windowManager.resizeBy(a.getWindowArg("mce_width")-b.w,a.getWindowArg("mce_height")-b.h,a.id||window)},10)},executeOnLoad:function(s){this.onInit.add(function(){eval(s)})},storeSelection:function(){this.editor.windowManager.bookmark=tinyMCEPopup.editor.selection.getBookmark(1)},restoreSelection:function(){var a=tinyMCEPopup;if(!a.isWindow&&tinymce.isIE){a.editor.selection.moveToBookmark(a.editor.windowManager.bookmark)}},requireLangPack:function(){var b=this,a=b.getWindowArg("plugin_url")||b.getWindowArg("theme_url");if(a&&b.editor.settings.language&&b.features.translate_i18n!==false&&b.editor.settings.language_load!==false){a+="/langs/"+b.editor.settings.language+"_dlg.js";if(!tinymce.ScriptLoader.isDone(a)){document.write('<script type="text/javascript" src="'+tinymce._addVer(a)+'"><\/script>');tinymce.ScriptLoader.markDone(a)}}},pickColor:function(b,a){this.execCommand("mceColorPicker",true,{color:document.getElementById(a).value,func:function(e){document.getElementById(a).value=e;try{document.getElementById(a).onchange()}catch(d){}}})},openBrowser:function(a,c,b){tinyMCEPopup.restoreSelection();this.editor.execCallback("file_browser_callback",a,document.getElementById(a).value,c,window)},confirm:function(b,a,c){this.editor.windowManager.confirm(b,a,c,window)},alert:function(b,a,c){this.editor.windowManager.alert(b,a,c,window)},close:function(){var a=this;function b(){a.editor.windowManager.close(window);tinymce=tinyMCE=a.editor=a.params=a.dom=a.dom.doc=null}if(tinymce.isOpera){a.getWin().setTimeout(b,0)}else{b()}},_restoreSelection:function(){var a=window.event.srcElement;if(a.nodeName=="INPUT"&&(a.type=="submit"||a.type=="button")){tinyMCEPopup.restoreSelection()}},_onDOMLoaded:function(){var b=tinyMCEPopup,d=document.title,e,c,a;if(b.domLoaded){return}b.domLoaded=1;if(b.features.translate_i18n!==false){c=document.body.innerHTML;if(tinymce.isIE){c=c.replace(/ (value|title|alt)=([^"][^\s>]+)/gi,' $1="$2"')}document.dir=b.editor.getParam("directionality","");if((a=b.editor.translate(c))&&a!=c){document.body.innerHTML=a}if((a=b.editor.translate(d))&&a!=d){document.title=d=a}}if(!b.editor.getParam("browser_preferred_colors",false)||!b.isWindow){b.dom.addClass(document.body,"forceColors")}document.body.style.display="";if(tinymce.isIE){document.attachEvent("onmouseup",tinyMCEPopup._restoreSelection);b.dom.add(b.dom.select("head")[0],"base",{target:"_self"})}b.restoreSelection();b.resizeToInnerSize();if(!b.isWindow){b.editor.windowManager.setTitle(window,d)}else{window.focus()}if(!tinymce.isIE&&!b.isWindow){tinymce.dom.Event._add(document,"focus",function(){b.editor.windowManager.focus(b.id)})}tinymce.each(b.dom.select("select"),function(f){f.onkeydown=tinyMCEPopup._accessHandler});tinymce.each(b.listeners,function(f){f.func.call(f.scope,b.editor)});if(b.getWindowArg("mce_auto_focus",true)){window.focus();tinymce.each(document.forms,function(g){tinymce.each(g.elements,function(f){if(b.dom.hasClass(f,"mceFocus")&&!f.disabled){f.focus();return false}})})}document.onkeyup=tinyMCEPopup._closeWinKeyHandler},_accessHandler:function(a){a=a||window.event;if(a.keyCode==13||a.keyCode==32){a=a.target||a.srcElement;if(a.onchange){a.onchange()}return tinymce.dom.Event.cancel(a)}},_closeWinKeyHandler:function(a){a=a||window.event;if(a.keyCode==27){tinyMCEPopup.close()}},_wait:function(){if(document.attachEvent){document.attachEvent("onreadystatechange",function(){if(document.readyState==="complete"){document.detachEvent("onreadystatechange",arguments.callee);tinyMCEPopup._onDOMLoaded()}});if(document.documentElement.doScroll&&window==window.top){(function(){if(tinyMCEPopup.domLoaded){return}try{document.documentElement.doScroll("left")}catch(a){setTimeout(arguments.callee,0);return}tinyMCEPopup._onDOMLoaded()})()}document.attachEvent("onload",tinyMCEPopup._onDOMLoaded)}else{if(docuLine truncated
-
 var returnurl = "<%= @asset.data.url %>";
+var returntitle = "<%= show_all_fields(@asset, 'title') %>";
+var returndescription = "<%= show_all_fields(@asset, 'description') %>";
 var win = tinyMCEPopup.getWindowArg("window");
 win.document.getElementById(tinyMCEPopup.getWindowArg("input")).value = returnurl;
+win.document.getElementById(tinyMCEPopup.getWindowArg("alt")).value = returndescription;
+win.document.getElementById(tinyMCEPopup.getWindowArg("title")).value = returntitle;
   if (typeof(win.ImageDialog) != "undefined") {
             // we are, so update image dimensions...
             if (win.ImageDialog.getImageData)

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

@@ -9,39 +9,41 @@
     </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| %>
-                    <%= content_tag :label do -%>
-                        <div>
-                            <%= I18nVariable.from_locale(locale) %>
-                            <%= f.text_field locale, :class => "input-large" %>
-                        </div>
-                    <% end %>
-                  <% end %>
+                <%= f.fields_for :title, (@asset.new_record? ? @asset.build_title : @asset.title) do |f| %>
+                    <div>
+                    <% @site_valid_locales.each do |locale| %>
+                        <%= content_tag :label do -%>
+                            <div>
+                                <%= I18nVariable.from_locale(locale) %>
+                                <%= f.text_field locale, :class => "input-large" %>
+                            </div>
+                        <% end %>
+                      <% end %>
+                    </div>
                 <% end %>
             </div>
         </div>
         <div class="control-group">
-            <label class="control-label"><%= f.label :category, t('admin.category') %></label>
+            <label for="description" class="control-label error"><%= t 'admin.description' %></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>
-            <div class="controls">
-                <% @tags.each do |tag| %>
-                    <%= content_tag :label, :class => "checkbox inline" do -%>
-                        <%= check_box_tag 'asset[tag_ids][]', tag.id, @asset.tag_ids.include?(tag.id) %>
-                        <%= tag[I18n.locale] %>
-                    <% end %>
+                <%= f.fields_for :description, (@asset.new_record? ? @asset.build_description : @asset.description) do |f| %>
+                    <div>
+                    <% @site_valid_locales.each do |locale| %>
+                        <%= content_tag :label do -%>
+                            <div>
+                                <%= I18nVariable.from_locale(locale) %>
+                                <%= f.text_field locale, :class => "input-large" %>
+                            </div>
+                        <% end %>
+                      <% end %>
+                    </div>
                 <% end %>
             </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>

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 -%>
-	<%= 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 :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.tags'), admin_asset_tags_path), :class => active_for_action('/admin/asset_tags', 'index') %>
-	<% end -%>
-<% end -%>
+<%#= 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 %>
+	<%#= 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.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') %>
+	<%# end -%>
+<%# end -%>

vendor/built_in_modules/announcement/app/models/bulletin.rb

@@ -13,8 +13,8 @@ class Bulletin
   has_one :text, :class_name => "I18nVariable", :as => :language_value, :autosave => true, :dependent => :destroy
   has_and_belongs_to_many :tags, :class_name => "AnnouncementTag"
 
-  field :postdate , :type => Date
-  field :deadline , :type => Date
+  field :postdate , :type => DateTime
+  field :deadline , :type => DateTime
   # field :url
   field :create_user_id
   field :update_user_id, :class_name => "User"

vendor/built_in_modules/news/app/models/news_bulletin.rb

@@ -13,8 +13,8 @@ class NewsBulletin
   has_one :text, :class_name => "I18nVariable", :as => :language_value, :autosave => true, :dependent => :destroy
   has_and_belongs_to_many :tags, :class_name => "NewsTag"
 
-  field :postdate , :type => Date
-  field :deadline , :type => Date
+  field :postdate , :type => DateTime
+  field :deadline , :type => DateTime
   # field :url
   field :create_user_id
   field :update_user_id, :class_name => "User"