From 9e10a3c94476ceaa17d9de84b15f200a8b9589be Mon Sep 17 00:00:00 2001
From: Harry Bomrah <harry@rulingcom.com>
Date: Tue, 10 Sep 2013 12:45:10 +0800
Subject: [PATCH] editing for the assets are done with replacable files and
 description

---
 app/controllers/admin/assets_controller.rb | 11 +---
 app/views/admin/assets/_asset.html.erb     |  2 +-
 app/views/admin/assets/_edit.html.erb      | 29 +++++------
 app/views/admin/assets/_form.html.erb      | 60 ++++++++++++----------
 app/views/admin/assets/create.html.erb     | 25 ---------
 app/views/admin/assets/edit.html.erb       |  8 ---
 app/views/admin/assets/index.html.erb      | 26 +++++-----
 app/views/admin/assets/update.html.erb     |  1 +
 8 files changed, 65 insertions(+), 97 deletions(-)
 delete mode 100644 app/views/admin/assets/create.html.erb
 create mode 100644 app/views/admin/assets/update.html.erb

diff --git a/app/controllers/admin/assets_controller.rb b/app/controllers/admin/assets_controller.rb
index 96f0e7b93..ef054a1d8 100644
--- a/app/controllers/admin/assets_controller.rb
+++ b/app/controllers/admin/assets_controller.rb
@@ -32,12 +32,8 @@ class Admin::AssetsController < OrbitBackendController
   
   def edit
     @asset = Asset.find(params[:id])
-    @asset_categories = AssetCategory.all
     @tags = get_tags
-    respond_to do |format|
-      format.html {}
-      format.js { render 'js/show_pop_up', :locals => {:partial => 'admin/assets/edit'} }
-    end
+    render layout: false
   end
   
   def create  
@@ -74,10 +70,7 @@ class Admin::AssetsController < OrbitBackendController
   def update
     @asset = Asset.find(params[:id])
     if @asset.update_attributes(params[:asset])
-      respond_to do |format|
-        format.html { redirect_to admin_assets_url }
-        format.js { render 'js/remove_pop_up_and_reload_content', :locals => {:function => 'replaceWith', :id => "asset_#{@asset.id}", :value => @asset, :values => nil, :partial => 'admin/assets/asset', :locals => nil} }
-      end
+     render layout: false
     else
       flash[:error] = t('update.fail')
       @asset_categories = AssetCategory.all
diff --git a/app/views/admin/assets/_asset.html.erb b/app/views/admin/assets/_asset.html.erb
index 860d39933..b7c30fe93 100644
--- a/app/views/admin/assets/_asset.html.erb
+++ b/app/views/admin/assets/_asset.html.erb
@@ -30,6 +30,6 @@
 		</div>
 	</td>
 	<td>
-		<a href="#" class="editform">Edit</a>
+		<a href="<%= edit_admin_asset_path(asset) %>" class="editform">Edit</a>
 	</td>
 </tr>
\ No newline at end of file
diff --git a/app/views/admin/assets/_edit.html.erb b/app/views/admin/assets/_edit.html.erb
index 5e26e0f64..5924ea1fa 100644
--- a/app/views/admin/assets/_edit.html.erb
+++ b/app/views/admin/assets/_edit.html.erb
@@ -1,18 +1,15 @@
-<div id='pop_up_content' class="main2">
-	<h1><%= t('editing.asset') %></h1>
-
-	<%= flash_messages %>
-
-	<%= form_for @asset, :url => admin_asset_path(@asset), :html => {:id => (is_html ? nil : 'ajaxForm'), :multipart => true } do |f| %>
+<%= flash_messages %>
+<div class="modal-header">
+    <button type="button" class="close" data-dismiss="modal" aria-hidden="true">&times;</button>
+    <h3><%= t('editing.asset') %></h3>
+</div>
+<%= form_for @asset, :url => admin_asset_path(@asset), :remote => true do |f| %>
+	<div class="modal-body" id="modal_body">
 	  <%= f.error_messages %>
 	  <%= render :partial => "form", :locals => { :f => f } %>
-	  <div class="button_bar">
-			<% if is_html %>
-    		<%= link_back %>
-				<%= f.submit t(:edit) %>
-			<% else %>
-				<a id='ajax_form_submit' class="btn btn-primary"><%= t(:update_) %></a>
-			<% end %>
-	  </div>
-	<% end %>
-</div>
\ No newline at end of file
+	</div>
+	<div class="modal-footer">
+		<a href="#" class="btn" data-dismiss="modal">Close</a>
+	    <%= f.submit t('submit'), class: 'btn btn-primary' %>
+	</div>
+<% end %>
\ No newline at end of file
diff --git a/app/views/admin/assets/_form.html.erb b/app/views/admin/assets/_form.html.erb
index 32817f82f..1e080678b 100644
--- a/app/views/admin/assets/_form.html.erb
+++ b/app/views/admin/assets/_form.html.erb
@@ -1,38 +1,46 @@
 <div class="control-group">
-    <label for="title" class="control-label"><%= t :title %></label>
-    <div class="controls">
-        <%= f.fields_for :title_translations do |f| %>
-          <% @site_valid_locales.each do |locale| %>
-            <%= content_tag :label do -%>
-                <div>
-                    <%= I18nVariable.from_locale(locale) %>
-                    <%= f.text_field locale, :class => "input-large", :value => (@asset.title_translations[locale] rescue nil) %>
+    <ul class="nav nav-pills language-nav">
+        <% @site_valid_locales.each_with_index do |locale, i| %>
+            <li <%= ( i == 0 ) ? "class=active" : '' %>><a data-toggle="tab" href=".<%= locale %>"><%= I18nVariable.from_locale(locale) %></a></li>
+        <% end %>
+    </ul>
+    <% @site_valid_locales.each_with_index do |locale, i| %>
+        <div class="<%= locale %> fade tab-pane <%= ( i == 0 ) ? "in active" : '' %>">
+            <div class="control-group input-title">
+                <%= f.label :title , t('asset.title'), :class=>"control-label muted" %>
+                <div class="controls">
+                    <%= f.fields_for :title_translations do |f| %>
+                        <%= f.text_field locale, :class=>'post-title', :value => (@asset.title_translations[locale] rescue nil) %>
+                    <% end %>
                 </div>
-            <% end %>
-          <% end %>
-        <% end %>
-    </div>
+            </div>
+            <div class="control-group input-content">
+                <%= f.label :description ,t('asset.description'), :class => "control-label muted" %>
+                <%= f.fields_for :description_translations do |f| %>
+                    <div class="controls">
+                        <div class="textarea">
+                            <%= f.text_area locale, :style=>"width:100%;height:100px", :class => '', :value => (@asset.description_translations[locale] rescue nil) %>
+                        </div>
+                    </div>
+                <% end %>
+            </div>
+        </div>
+    <% end %>
+    
 </div>
 <div class="control-group">
-    <label class="control-label"><%= f.label :category, t(:category) %></label>
-    <div class="controls">
-        <%= f.select :asset_category_id, @asset_categories.collect{|t| [ t.title, t.id ]}, {}, :class => "input-large" %>
-    </div>
-</div>
-<div class="control-group">
-    <label class="control-label"><%= t :tags %></label>
-    <div class="controls">
+    <label class="control-label muted">Tag</label>
+    <div class="controls" data-toggle="buttons-checkbox">
         <% @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) %>
+          <label class="checkbox inline btn <%= 'active' if @asset.tag_ids.include?(tag.id) %>">
+                <%= check_box_tag 'asset[tag_ids][]', tag.id, @asset.tag_ids.include?(tag.id)%>
                 <%= tag.name %>
-                <%= hidden_field_tag 'asset[tag_ids][]', '' %>
-            <% end %>
+          </label>
         <% end %>
     </div>
-</div>
+  </div>
 <div class="control-group">
-    <label class="control-label"><%= f.label :data, t(:data) %></label>
+    <label class="control-label muted"><%= f.label :data, t(:data) %></label>
     <div class="controls">
         <%= f.file_field :data, :class => 'upload' %>
     </div>
diff --git a/app/views/admin/assets/create.html.erb b/app/views/admin/assets/create.html.erb
deleted file mode 100644
index 81b112dc4..000000000
--- a/app/views/admin/assets/create.html.erb
+++ /dev/null
@@ -1,25 +0,0 @@
-<% if !request.xhr? %>
-<textarea>
-<% end %>
-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;
-if(win.document.getElementById(tinyMCEPopup.getWindowArg("alt")))
-	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)
-                win.ImageDialog.getImageData();
-
-            // ... and preview if necessary
-            if (win.ImageDialog.showPreviewImage)
-                win.ImageDialog.showPreviewImage(returnurl);
-        }
-tinyMCEPopup.close();
-<% if !request.xhr? %>
-</textarea>
-<% end %>
\ No newline at end of file
diff --git a/app/views/admin/assets/edit.html.erb b/app/views/admin/assets/edit.html.erb
index b4b57fabf..c4c33e4ba 100644
--- a/app/views/admin/assets/edit.html.erb
+++ b/app/views/admin/assets/edit.html.erb
@@ -1,9 +1 @@
-<% content_for :secondary do %>
-<div class="assets_setup">
-  <ul class="list">
-	<li><%= link_to t('new.asset'), new_admin_asset_path, :remote => true, :class => 'button positive' %></li>
-  </ul>
-</div>
-<% end -%>
-
 <%= render :partial => 'edit', :locals => {:is_html => true} %>
\ No newline at end of file
diff --git a/app/views/admin/assets/index.html.erb b/app/views/admin/assets/index.html.erb
index 070c640d1..fc373d2e0 100644
--- a/app/views/admin/assets/index.html.erb
+++ b/app/views/admin/assets/index.html.erb
@@ -302,17 +302,9 @@
   </form>
 </div>
 <div id="editform" class="modal hide fade">
-  <div class="modal-header">
-    <button type="button" class="close" data-dismiss="modal" aria-hidden="true">&times;</button>
-    <h3>Modal header</h3>
-  </div>
-  <div class="modal-body">
-    <p>One fine body…</p>
-  </div>
-  <div class="modal-footer">
-    <a href="#" class="btn" data-dismiss="modal">Close</a>
-    <a href="#" class="btn btn-primary">Save changes</a>
-  </div>
+  
+   
+  
 </div>
 
 <% content_for :page_specific_javascript do %>
@@ -402,8 +394,18 @@ $(document).ready(function(){
 			deleteBtn.hide();
 		})
 	})
+	var currentEdit = null;
+	$(document).on("ajax:success","form[data-remote=true]",function(evt, data, xhr){
+		currentEdit.replaceWith($(data));
+		$("#editform").modal("hide");
+	})
+
 	$("#asset_sort_list").on(clickEvent,".editform",function(){
-		$("#editform").modal("show");
+		currentEdit = $(this).parent().parent();
+		$.get($(this).attr("href"),function(data){
+			$("#editform").html(data).modal("show");
+		})
+		return false;
 	})
 
 	function getUrlParam( paramName ) {
diff --git a/app/views/admin/assets/update.html.erb b/app/views/admin/assets/update.html.erb
new file mode 100644
index 000000000..9154b6948
--- /dev/null
+++ b/app/views/admin/assets/update.html.erb
@@ -0,0 +1 @@
+<%= render :partial => 'asset', :object => @asset %>
\ No newline at end of file