From d4568ddd40bd68c206a1b62067ed582eb3a90b45 Mon Sep 17 00:00:00 2001
From: Antonio Terceiro <terceiro@colivre.coop.br>
Date: Tue, 9 Aug 2011 15:49:14 -0700
Subject: [PATCH] Remove old crappy media panel

---
 app/controllers/my_profile/cms_controller.rb |  61 ++++++++-----------------------------------------------------
 app/views/cms/_document_link.rhtml           |  10 ----------
 app/views/cms/_image_thumb.rhtml             |  10 ----------
 app/views/cms/_media_listing.rhtml           |   3 ---
 app/views/cms/_select_folder.rhtml           |   1 -
 app/views/cms/_text_editor_sidebar.rhtml     |   3 +--
 app/views/cms/_upload_file_form.rhtml        |   4 +---
 app/views/cms/media_listing.rhtml            |  75 ---------------------------------------------------------------------------
 public/stylesheets/application.css           |  18 +-----------------
 public/stylesheets/media_listing.css         | 167 -----------------------------------------------------------------------------------------------------------------------------------------------------------------------
 test/functional/cms_controller_test.rb       | 146 ++++----------------------------------------------------------------------------------------------------------------------------------------------
 11 files changed, 15 insertions(+), 483 deletions(-)
 delete mode 100644 app/views/cms/_document_link.rhtml
 delete mode 100644 app/views/cms/_image_thumb.rhtml
 delete mode 100644 app/views/cms/_media_listing.rhtml
 delete mode 100644 app/views/cms/_select_folder.rhtml
 delete mode 100644 app/views/cms/media_listing.rhtml
 delete mode 100644 public/stylesheets/media_listing.css

diff --git a/app/controllers/my_profile/cms_controller.rb b/app/controllers/my_profile/cms_controller.rb
index c7a635d..d088eab 100644
--- a/app/controllers/my_profile/cms_controller.rb
+++ b/app/controllers/my_profile/cms_controller.rb
@@ -177,8 +177,7 @@ class CmsController < MyProfileController
     @article = @parent = check_parent(params[:parent_id])
     @target = @parent ? ('/%s/%s' % [profile.identifier, @parent.full_name]) : '/%s' % profile.identifier
     @folders = Folder.find(:all, :conditions => { :profile_id => profile })
-    @media_listing = params[:media_listing]
-    if @article && !@media_listing
+    if @article
       record_coming
     end
     if request.post? && params[:uploaded_files]
@@ -187,28 +186,17 @@ class CmsController < MyProfileController
       end
       @errors = @uploaded_files.select { |f| f.errors.any? }
       if @errors.any?
-        if @media_listing
-          flash[:notice] = _('Could not upload all files')
-          redirect_to :action => 'media_listing'
-        else
-          render :action => 'upload_files', :parent_id => @parent_id
-        end
+        render :action => 'upload_files', :parent_id => @parent_id
       else
-        if @media_listing
-          flash[:notice] = _('All files were uploaded successfully')
-          redirect_to :action => 'media_listing'
+        if @back_to
+          redirect_to @back_to
+        elsif @parent
+          redirect_to :action => 'view', :id => @parent.id
         else
-          if @back_to
-            redirect_to @back_to
-          else
-            redirect_to( if @parent
-              {:action => 'view', :id => @parent.id}
-            else
-              {:action => 'index'}
-            end)
-          end
+          redirect_to :action => 'index'
         end
       end
+      end
     end
   end
 
@@ -302,39 +290,6 @@ class CmsController < MyProfileController
     end
   end
 
-  def media_listing
-    if params[:image_folder_id]
-      folder = profile.articles.find(params[:image_folder_id]) if !params[:image_folder_id].blank?
-      @images = (folder ? folder.children : profile.top_level_articles).images
-    elsif params[:document_folder_id]
-      folder = profile.articles.find(params[:document_folder_id]) if !params[:document_folder_id].blank?
-      @documents = (folder ? folder.children : profile.top_level_articles)
-    else
-      @documents = profile.articles
-      @images = @documents.images
-      @documents -= @images
-    end
-
-    @images = @images.paginate(:per_page => per_page, :page => params[:ipage], :order => "updated_at desc") if @images
-    @documents = @documents.paginate(:per_page => per_page, :page => params[:dpage], :order => "updated_at desc", :conditions => {:is_image => false}) if @documents
-
-    @folders = profile.folders
-    @image_folders = @folders.select {|f| f.children.any? {|c| c.image?} }
-    @document_folders = @folders.select {|f| f.children.any? {|c| !c.image? && c.kind_of?(UploadedFile) } }
-
-    @media_listing = true
-
-    respond_to do |format|
-      format.html { render :layout => false}
-      format.js {
-        render :update do |page|
-          page.replace_html 'media-listing-folder-images', :partial => 'image_thumb', :locals => {:images => @images } if !@images.blank?
-          page.replace_html 'media-listing-folder-documents', :partial => 'document_link', :locals => {:documents => @documents } if !@documents.blank?
-        end
-      }
-    end
-  end
-
   def search
     query = params[:q]
     results = query.blank? ? [] : profile.articles.published.find_by_contents(query)
diff --git a/app/views/cms/_document_link.rhtml b/app/views/cms/_document_link.rhtml
deleted file mode 100644
index ee7a31c..0000000
--- a/app/views/cms/_document_link.rhtml
+++ /dev/null
@@ -1,10 +0,0 @@
-<div id='media-listing-folder-documents' >
-  <ul>
-    <% documents.each do |document| %>
-      <li><%= link_to(document.name, document.view_url, :class => icon_for_article(document)) %></li>
-    <% end %>
-  </ul>
-  <div id='pagination-documents'>
-    <%= pagination_links documents, :param_name => 'dpage', :params => {:document_folder_id => params[:document_folder_id]} %>
-  </div>
-</div>
diff --git a/app/views/cms/_image_thumb.rhtml b/app/views/cms/_image_thumb.rhtml
deleted file mode 100644
index f9c3039..0000000
--- a/app/views/cms/_image_thumb.rhtml
+++ /dev/null
@@ -1,10 +0,0 @@
-<div id='media-listing-folder-images' >
-  <ul>
-    <% images.each do |image| %>
-      <li><%= image_tag image.public_filename %></li>
-    <% end %>
-  </ul>
-  <div id='pagination-images'>
-    <%= pagination_links images, :param_name => 'ipage', :params => {:image_folder_id => params[:image_folder_id]} %>
-  </div>
-</div>
diff --git a/app/views/cms/_media_listing.rhtml b/app/views/cms/_media_listing.rhtml
deleted file mode 100644
index 394ac8a..0000000
--- a/app/views/cms/_media_listing.rhtml
+++ /dev/null
@@ -1,3 +0,0 @@
-<iframe id='media-listing-iframe' src="<%= url_for(:controller => 'cms', :action => 'media_listing', :profile => profile.identifier, :type => @type) %>">
-  <p><%= _('Your browser does not support iframes.') %></p>
-</iframe>
diff --git a/app/views/cms/_select_folder.rhtml b/app/views/cms/_select_folder.rhtml
deleted file mode 100644
index 2c64ee9..0000000
--- a/app/views/cms/_select_folder.rhtml
+++ /dev/null
@@ -1 +0,0 @@
-<%= select('folder', 'folder_id', @image_folders.collect {|f| [ f.name, f.id ] }, {:include_blank => "#{profile.identifier}"}, :onchange => remote_function(:update => 'media-listing-folder-images', :with => "'folder_id=' + value", :url => { :action => :get_images }) ) %>
diff --git a/app/views/cms/_text_editor_sidebar.rhtml b/app/views/cms/_text_editor_sidebar.rhtml
index 4669555..9c80a44 100644
--- a/app/views/cms/_text_editor_sidebar.rhtml
+++ b/app/views/cms/_text_editor_sidebar.rhtml
@@ -3,7 +3,7 @@
   <div class='text-editor-sidebar-box' id='media-upload-box'>
     <p><strong>Media upload</strong></p>
     <div id='media-upload-form'>
-      <% form_tag({ :action => 'media_upload' }, :multipart => true) do |f| %>
+      <% form_tag({ :action => 'media_upload' }, :multipart => true) do %>
         <div class='formfield'>
           <%# TODO duplicated from partial upload_file_form %>
           <%= labelled_form_field(_('Choose folder to upload files:'), select_tag('parent_id', options_for_select([[profile.identifier, '']] + profile.folders.collect {|f| [ profile.identifier + '/' +  f.full_name, f.id ] }))) %>
@@ -35,7 +35,6 @@
       <%= submit_button :search, _('Search'), :id => 'media-search-button' %>
     <% end %>
     </p>
-    </form>
     <div id='media-search-results' class='media-list-results' style='display: none'>
       <ul>
       </ul>
diff --git a/app/views/cms/_upload_file_form.rhtml b/app/views/cms/_upload_file_form.rhtml
index 8c4c3ff..c5033c1 100644
--- a/app/views/cms/_upload_file_form.rhtml
+++ b/app/views/cms/_upload_file_form.rhtml
@@ -14,9 +14,7 @@
 
 <% button_bar do %>
   <%= add_upload_file_field(_('More files'), {:size => size}) %>
-  <% if @media_listing %>
-    <%= submit_button :save, _('Upload') %>
-  <% elsif @back_to %>
+  <% if @back_to %>
     <%= submit_button :save, _('Upload'), :cancel => @back_to %>
   <% else %>
     <%= submit_button :save, _('Upload'), :cancel => {:action => (@parent ? 'view' : 'index'), :id => @parent } %>
diff --git a/app/views/cms/media_listing.rhtml b/app/views/cms/media_listing.rhtml
deleted file mode 100644
index df547f0..0000000
--- a/app/views/cms/media_listing.rhtml
+++ /dev/null
@@ -1,75 +0,0 @@
-<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<%= html_language %>" lang="<%= html_language %>">
-<head>
-  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
-  <%= stylesheet_link_tag 'application', 'media_listing', :cache => 'cache-media-listing' %>
-  <%= stylesheet_import 'button', :themed_source => true %>
-  <%= javascript_include_tag :defaults %>
-  <%= javascript_include_tag 'lowpro' %>
-  <%= stylesheet_link_tag icon_theme_stylesheet_path %>
-</head>
-<body class='noosfero'>
-  <script type="text/javascript">
-    /* Adds a class to "msie" to the body element if a Microsoft browser is
-     * detected. This is needed to workaround several of their limitations.
-     */
-    if ( navigator.appVersion.indexOf("MSIE") > -1 ) {
-      document.body.className += " msie msie" +
-        navigator.appVersion.replace(/^.*MSIE\s+([0-9]+).*$/, "$1");
-    }
-    function registerDocumentSize() {
-      document.body.className = document.body.className.replace(/(^| )docSize.+( |$)/g, " ");
-      for ( var x=100; x<=1500; x+=100 ) {
-        if ( document.body.clientWidth > x ) {
-          document.body.className += " docSize-GT-" + x;
-        } else {
-          document.body.className += " docSize-LT-" + x;
-        }
-      }
-    }
-    registerDocumentSize();
-  </script>
-  <div id='media-listing-upload'>
-    <p><%= _("Include files in some folder or select from the list below to add images and documents to the text editor beside (max size %s)") % UploadedFile.max_size.to_humanreadable %></p>
-
-    <div id="notice" onclick="Element.hide('notice');" style="display:none">
-      <% unless flash[:notice].nil? %>
-        <%= flash[:notice] %>
-        <%= javascript_tag(visual_effect( :appear, 'notice')) %>
-      <% end %>
-    </div>
-
-    <% form_for('uploaded_file', :url => {:action => 'upload_files'}, :html => {:multipart => true}) do |f| %>
-      <%= hidden_field_tag('media_listing', @media_listing) %>
-      <%= render :partial => 'upload_file_form', :locals => { :size => '30' } %>
-    <% end %>
-  </div><!-- id='media-listing-upload' -->
-  <hr/>
-
-  <script type='text/javascript'>
-    document.observe("dom:loaded", function() {
-      Event.addBehavior.reassignAfterAjax = true;
-      Event.addBehavior({
-        'div#pagination-images .pagination a' : Remote.Link,
-        'div#pagination-documents .pagination a' : Remote.Link
-      })
-    });
-  </script>
-
-  <div id='media-listing'>
-    <h3><%= _('Folders') %></h3>
-    <p><%= _('Drag images and documents to add them to the text. If needed, resize images by clicking the tree icon on editor.') %></p>
-
-    <div id='media-listing-images'>
-      <h4><%= _('Images') %></h4>
-      <%= select_folder('', 'folder', 'image_folder_id', @image_folders, {}, :onchange => remote_function(:with => "'image_folder_id=' + value + '&ipage=1'", :url => { :action => :media_listing, :format => 'js' }) ) %>
-      <%= render :partial => 'image_thumb', :locals => { :images => @images } %>
-    </div><!-- id='media-listing-images' -->
-    <div id='media-listing-documents'>
-      <h4><%= _('Documents') %></h4>
-      <%= select_folder('', 'folder', 'document_folder_id', @document_folders, {}, :onchange => remote_function(:with => "'document_folder_id=' + value + '&dpage=1'", :url => { :action => :media_listing }) ) %>
-      <%= render :partial => 'document_link', :locals => { :documents => @documents } %>
-    </div><!-- id='media-listing-documents' -->
-    <br style="clear:both" />
-  </div><!-- id='media-listing' -->
-</body>
-</html>
diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css
index b8abe9c..a246e95 100644
--- a/public/stylesheets/application.css
+++ b/public/stylesheets/application.css
@@ -3344,21 +3344,7 @@ table.cms-articles .icon:hover {
   display: block;
 }
 
-/* Media listing */
-
-.controller-cms #media-listing-iframe {
-  float: right;
-  width: 380px;
-  height: 630px;
-  border: none;
-  margin-top: 104px;
-  padding: 0px;
-  overflow: hidden;
-}
-
-.controller-cms .msie #media-listing-iframe {
-  height: 610px;
-}
+/* Text editors sidebar */
 
 .controller-cms div.with_media_panel {
   float: left;
@@ -3369,8 +3355,6 @@ div.with_media_panel .formfield input {
   width: 100%;
 }
 
-/* Textile sidebar */
-
 .text-editor-sidebar {
   position: absolute;
   width: 380px;
diff --git a/public/stylesheets/media_listing.css b/public/stylesheets/media_listing.css
deleted file mode 100644
index 26304fe..0000000
--- a/public/stylesheets/media_listing.css
+++ /dev/null
@@ -1,167 +0,0 @@
-body {
-  padding: 0px;
-  margin: 0px;
-  font-family: Verdana, sans-serif;
-  font-size: 14px;
-  color: #444;
-  background-color: #F0F0EE;
-  overflow: hidden;
-}
-
-h3, h4, h5 {
-  margin: 5px 0px;
-}
-
-h3 {
-  font-size: 18px;
-}
-
-h4 {
-  font-size: 16px;
-}
-
-#media-listing {
-  width: 100%;
-  height: 48%;
-  margin: 0px;
-  padding: 0px;
-  padding-bottom: 5px;
-}
-
-#media-listing-upload p,
-#media-listing p {
-  font-size: 13px;
-  font-weight: bold;
-  margin: 5px 5px;
-}
-
-#media-listing li {
-  list-style: none;
-  margin: 0px;
-}
-
-#media-listing li:hover {
-  background-color: #CCC;
-}
-
-#media-listing a {
-  text-decoration: none;
-}
-
-#media-listing select {
-  width: 80%;
-}
-
-#media-listing-images {
-  width: 46%;
-  float: left;
-  text-align: center;
-}
-
-#media-listing-images img {
-  max-width: 80px;
-  max-height: 60px;
-}
-
-.msie6 #media-listing-images img,
-.msie7 #media-listing-images img {
-  width: 80px;
-  height: 60px;
-}
-
-#media-listing-folder-images,
-#media-listing-folder-documents {
-  height: 80%;
-}
-
-#media-listing ul {
-  padding: 0px;
-  margin: 5px;
-  height: 40%;
-  overflow: auto;
-  width: 98%;
-}
-
-#media-listing ul {
-  height: 85%;
-}
-
-#media-listing-images,
-#media-listing-documents {
-  height: 55%;
-}
-
-#media-listing-documents ul {
-  text-align: left;
-}
-
-#media-listing-documents {
-  width: 52%;
-  float: left;
-  text-align: center;
-}
-
-#media-listing-documents li {
-  padding-bottom: 5px;
-}
-
-#media-listing-folder-documents a.icon {
-  background-repeat: no-repeat;
-  padding-left: 20px;
-  border: none;
-}
-
-#media-listing-folder-documents .icon:hover {
-  background-color: transparent;
-}
-
-#media-listing .icon-rss-feed {
-  background-image: url(../images/icons-mime/rss-feed-16.png);
-}
-
-#media-listing-upload {
-  width: 98%;
-  padding: 3px;
-}
-
-#media-listing-upload p {
-  margin: 5px 0px;
-}
-
-#media-listing-upload select {
-  width: 90%;
-}
-
-#uploaded_files {
-  overflow-x: hidden;
-  overflow-y: scroll;
-  height: 100px;
-  margin-top: 5px;
-}
-
-.msie #uploaded_files {
-  height: 100px;
-  padding: 0px;
-}
-
-.formlabel {
-  font-size: 11px;
-  display: block;
-}
-
-/* Notice */
-
-div#notice {
-  background: #fee;
-  border: 1px solid #933;
-  top: 150px;
-  color: black;
-  cursor: pointer;
-  font-weight: bold;
-  left: 50%;
-  margin-left: -150px;
-  padding: 5px;
-  position: absolute;
-  text-align: center;
-  width: 300px;
-}
diff --git a/test/functional/cms_controller_test.rb b/test/functional/cms_controller_test.rb
index dae4d5d..4db6977 100644
--- a/test/functional/cms_controller_test.rb
+++ b/test/functional/cms_controller_test.rb
@@ -1064,7 +1064,7 @@ class CmsControllerTest < Test::Unit::TestCase
     assert_tag :tag => 'input', :attributes => { :name => 'article[external_feed_builder][only_once]', :checked => 'checked', :value => 'true' }
   end
 
-  should 'display iframe for media listing when it is TinyMceArticle and enabled on environment' do
+  should 'display media listing when it is TinyMceArticle and enabled on environment' do
     e = Environment.default
     e.enable('media_panel')
     e.save!
@@ -1076,10 +1076,10 @@ class CmsControllerTest < Test::Unit::TestCase
     file = UploadedFile.create!(:profile => profile, :parent => non_image_folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
 
     get :new, :profile => profile.identifier, :type => 'TinyMceArticle'
-    assert_tag :tag => 'iframe', :attributes => { :src => "/myprofile/#{profile.identifier}/cms/media_listing?type=TinyMceArticle" }
+    assert_tag :div, :attributes => { :class => "text-editor-sidebar" }
   end
 
-  should 'not display iframe for media listing when it is Folder' do
+  should 'not display media listing when it is Folder' do
     image_folder = Folder.create(:profile => profile, :name => 'Image folder')
     non_image_folder = Folder.create(:profile => profile, :name => 'Non image folder')
 
@@ -1087,145 +1087,7 @@ class CmsControllerTest < Test::Unit::TestCase
     file = UploadedFile.create!(:profile => profile, :parent => non_image_folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
 
     get :new, :profile => profile.identifier, :type => 'Folder'
-    assert_no_tag :tag => 'iframe', :attributes => { :src => "/myprofile/#{profile.identifier}/cms/media_listing" }
-  end
-
-  should 'display list of images' do
-    file = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    process_delayed_job_queue
-    get :media_listing, :profile => profile.identifier
-
-    assert_tag :tag => 'div', :attributes => { :id => 'media-listing-images' }, :descendant => { :tag => 'img', :attributes => {:src => /rails.png/}}
-  end
-
-  should 'display loading image if not processed yet' do
-    file = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    get :media_listing, :profile => profile.identifier
-
-    assert_tag :tag => 'div', :attributes => { :id => 'media-listing-images' }, :descendant => { :tag => 'img', :attributes => {:src => /image-loading-thumb.png/}}
-  end
-
-
-  should 'display list of documents' do
-    file = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-    get :media_listing, :profile => profile.identifier
-    assert_tag :tag => 'div', :attributes => { :id => 'media-listing-documents' }, :descendant => { :tag => 'a', :attributes => {:href => /#{file.name}/}}
-  end
-
-  should 'list image folders to select' do
-    image_folder = Folder.create(:profile => profile, :name => 'Image folder')
-    non_image_folder = Folder.create(:profile => profile, :name => 'Non image folder')
-
-    image = UploadedFile.create!(:profile => profile, :parent => image_folder, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    file = UploadedFile.create!(:profile => profile, :parent => non_image_folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-
-    get :media_listing, :profile => profile.identifier
-    assert_tag :tag => 'div', :attributes => { :id => 'media-listing-images' }, :descendant => { :tag => 'option', :content => /#{image_folder.name}/, :attributes => { :value => image_folder.id}}
-    assert_no_tag :tag => 'div', :attributes => { :id => 'media-listing-images' }, :descendant => { :tag => 'option', :content => /#{non_image_folder.name}/, :attributes => { :value => non_image_folder.id}}
-  end
-
-  should 'list documents folders to select' do
-    image_folder = Folder.create(:profile => profile, :name => 'Image folder')
-    non_image_folder = Folder.create(:profile => profile, :name => 'Non image folder')
-
-    image = UploadedFile.create!(:profile => profile, :parent => image_folder, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    file = UploadedFile.create!(:profile => profile, :parent => non_image_folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-
-    get :media_listing, :profile => profile.identifier
-    assert_no_tag :tag => 'div', :attributes => { :id => 'media-listing-documents' }, :descendant => { :tag => 'option', :content => /#{image_folder.name}/, :attributes => { :value => image_folder.id}}
-    assert_tag :tag => 'div', :attributes => { :id => 'media-listing-documents' }, :descendant => { :tag => 'option', :content => /#{non_image_folder.name}/, :attributes => { :value => non_image_folder.id}}
-  end
-
-  should 'get a list of images from a image folder' do
-    folder = Folder.create(:profile => profile, :name => 'Image folder')
-    other_folder = Folder.create(:profile => profile, :name => 'Non image folder')
-    image = UploadedFile.create!(:profile => profile, :parent => folder, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    file_in_folder = UploadedFile.create!(:profile => profile, :parent => folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-    image_in_other_folder = UploadedFile.create!(:profile => profile, :parent => other_folder, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-
-    get :media_listing, :profile => profile.identifier, :image_folder_id => folder.id, :format => 'js'
-
-    assert_includes assigns(:images), image
-    assert_not_includes assigns(:images), file_in_folder
-    assert_not_includes assigns(:images), image_in_other_folder
-  end
-
-  should 'get a list of images from profile' do
-    image = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    folder = Folder.create(:profile => profile, :name => 'Image folder')
-    image_in_folder = UploadedFile.create!(:profile => profile, :parent => folder, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    get :media_listing, :profile => profile.identifier, :image_folder_id => '', :format => 'js'
-
-    assert_includes assigns(:images), image
-    assert_not_includes assigns(:images), image_in_folder
-  end
-
-  should 'get a list of documents from a document folder' do
-    folder = Folder.create(:profile => profile, :name => 'Non images folder')
-    other_folder = Folder.create(:profile => profile, :name => 'Image folder')
-    file = UploadedFile.create!(:profile => profile, :parent => folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-    image = UploadedFile.create!(:profile => profile, :parent => folder, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'))
-    file_in_other_folder = UploadedFile.create!(:profile => profile, :parent => other_folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-
-    get :media_listing, :profile => profile.identifier, :document_folder_id => folder.id, :format => 'js'
-
-    assert_includes assigns(:documents), file
-    assert_not_includes assigns(:documents), image
-    assert_not_includes assigns(:documents), file_in_other_folder
-  end
-
-  should 'get a list of documents from profile' do
-    file = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-    folder = Folder.create(:profile => profile, :name => 'Image folder')
-    file_in_folder = UploadedFile.create!(:profile => profile, :parent => folder, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-
-    get :media_listing, :profile => profile.identifier, :document_folder_id => '', :format => 'js'
-
-    assert_includes assigns(:documents), file
-    assert_not_includes assigns(:documents), file_in_folder
-  end
-
-  should 'display pagination links of images' do
-    @controller.stubs(:per_page).returns(1)
-
-    image = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/other-pic.jpg', 'image/jpg'))
-    image2 = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png'), :created_at => 1.day.ago)
-    image2.updated_at = 1.day.ago
-    image2.send :update_without_callbacks
-
-    get :media_listing, :profile => profile.identifier
-
-    assert_includes assigns(:images), image
-    assert_not_includes assigns(:images), image2
-  end
-
-  should 'display pagination links of documents' do
-    @controller.stubs(:per_page).returns(1)
-    profile.articles.destroy_all
-    file = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/feed.xml', 'text/xml'))
-    file2 = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('/files/test.txt', 'text/plain'))
-    file2.created_at = 1.day.ago
-    file2.save!
-
-    get :media_listing, :profile => profile.identifier
-
-    assert_includes assigns(:documents), file
-    assert_not_includes assigns(:documents), file2
-  end
-
-
-  should 'redirect to media listing when upload files from there' do
-    post :upload_files, :profile => profile.identifier, :media_listing => true, :uploaded_files => [fixture_file_upload('files/rails.png', 'image/png')]
-    assert_template nil
-    assert_redirected_to :action => 'media_listing'
-  end
-
-  should 'redirect to media listing when occur errors when upload files from there' do
-    file = UploadedFile.create!(:profile => profile, :uploaded_data => fixture_file_upload('files/rails.png', 'image/png'))
-
-    post :upload_files, :profile => profile.identifier, :media_listing => true, :uploaded_files => [fixture_file_upload('files/rails.png', 'image/png')]
-    assert_template nil
-    assert_redirected_to :action => 'media_listing'
+    assert_no_tag :div, :attributes => { :id => "text-editor-sidebar" }
   end
 
   should "display 'Publish' when profile is a person" do
--
libgit2 0.21.2