dspace_plugin : refactoring

Francisco Júnior
1 parent f31d0ef4
Showing 45 changed files with 299 additions and 712 deletions Show diff stats
plugins/dspace/controllers/dspace_plugin_controller.rb
plugins/dspace/controllers/dspace_plugin_myprofile_controller.rb
plugins/dspace/lib/dspace/client.rb
plugins/dspace/lib/dspace/collection.rb
plugins/dspace/lib/dspace/item.rb
plugins/dspace/lib/dspace_plugin/collection.rb
plugins/dspace/lib/dspace_plugin/collection_helper.rb
plugins/dspace/lib/dspace_plugin/communityy.rb
plugins/dspace/lib/dspace_plugin/communityy_helper.rb
plugins/dspace/lib/dspace_plugin/dspace_block.rb
plugins/dspace/lib/dspace_plugin/item.rb
plugins/dspace/lib/dspace_plugin/item_helper.rb
plugins/dspace/lib/dspace_plugin/library.rb
plugins/dspace/lib/dspace_plugin/library_helper.rb
plugins/dspace/lib/ext/article.rb
plugins/dspace/public/icons/audio.png
plugins/dspace/public/icons/doc.png
plugins/dspace/public/icons/image.png
plugins/dspace/public/icons/pdf.png
plugins/dspace/public/icons/spreadsheet.png
@@ -7,7 +7,15 @@ class DspacePluginController &lt; PublicController
     item_id = params[:id]
     begin
-      @item = Dspace::Item.get_item_by_id item_id
+      @collection = DspacePlugin::Collection.find(collection_id)
+    rescue ActiveRecord::RecordNotFound
+      render_not_found
+      return
+    end
+
+    begin
+      dspace_server = @collection.parent.parent.dspace_server_url
+      @item = Dspace::Item.get_item_by_id dspace_server, item_id
       rescue ActiveResource::UnauthorizedAccess
       render_not_found
       return
@@ -31,20 +31,12 @@ class DspacePluginMyprofileController &lt; CmsController
     article_data.merge!(params[:article]) if params[:article]
     if @type == 'DspacePlugin::Collection'
-      ids_list = 'dspace_collections_ids'
-      names_list = 'dspace_collections_names'
+      dspace_objects = article_data['dspace_collections']
     elsif @type == 'DspacePlugin::Communityy'
-      ids_list = 'dspace_communities_ids'
-      names_list = 'dspace_communities_names'
+      dspace_objects = article_data['dspace_communities']
     end
-    index = -1
-
-    article_data[ids_list].each do |id|
-
-      index += 1
-
-      name = article_data[names_list][index]
+    dspace_objects.each do |object|
       entity = klass.new
@@ -55,18 +47,19 @@ class DspacePluginMyprofileController &lt; CmsController
         parent_id = parent.id
       end
-      entity.dspace_community_id = id if @type == 'DspacePlugin::Communityy'
-
-      if @type == 'DspacePlugin::Collection'
-        entity.dspace_collection_id = id
+      if @type == 'DspacePlugin::Communityy'
+        entity.dspace_community_id = object['id']
+      elsif @type == 'DspacePlugin::Collection'
         entity.dspace_community_id = article_data['dspace_community_id']
+        entity.dspace_collection_id = object['id']
+        entity.accept_comments = false
       end
+      entity.name = object['name']
       entity.profile = profile
       entity.author = user
       entity.last_changed_by = user
       entity.created_by = user
-      entity.name = name
       entity.save!
@@ -1,16 +0,0 @@
-class Dspace::Client
-
-  def initialize(server_url)
-    @server_url = server_url
-  end
-
-  def get_collections
-    Dspace::Collection.get_all_collections_from @server_url
-  end
-
-  def get_communities
-    Dspace::Community.get_all_communities_from @server_url
-  end
-
-end
-
@@ -7,34 +7,9 @@ class Dspace::Collection &lt; Dspace::Resource
     item_list = []
     if result.items.count > 0
-
       result.items.each { |element|
-
-        item_metadata = Dspace::Item.get_all_item_metadata_from dspace_server, element.id
-
-        # author
-        metadata = item_metadata[0].attributes
-        if metadata != {}
-          metadata = Hash[[metadata.map{|k,v| v}]]
-          author =  metadata.has_key?('dc.contributor.author') ? metadata['dc.contributor.author'] : nil
-        end
-
-        # date issued
-        metadata = item_metadata[3].attributes
-        if metadata != {}
-          metadata = Hash[[metadata.map{|k,v| v}]]
-          issue_date = metadata.has_key?('dc.date.issued') ? metadata['dc.date.issued'] : nil
-        end
-
-        item = DspacePlugin::Item.new
-
-        item.id = element.id
-        item.name = element.name
-        item.author = author
-        item.issue_date = issue_date
-
+        item = Dspace::Item.get_item_by_id dspace_server, element.id
         item_list << item
-
       }
     end
@@ -6,8 +6,8 @@ class Dspace::Item &lt; Dspace::Resource
     result.metadata
   end
-  def self.get_item_by_id(item_id)
-    self.site = 'http://dspace.maljr.net/rest/'
+  def self.get_item_by_id(dspace_server, item_id)
+    self.site = dspace_server
     result = self.find item_id, :params => { :expand => 'metadata' }
     item_metadata = Dspace::Item.get_all_item_metadata_from self.site, result.id
@@ -18,9 +18,9 @@ class DspacePlugin::Collection &lt; Article
   end
   def to_html(options = {})
-    dspace_collection = self
+    dspace_content = self
     proc do
-      render :file => 'content_viewer/collection', :locals => {:dspace_collection => dspace_collection}
+      render :file => 'content_viewer/dspace_content', :locals => { :dspace_content => dspace_content }
     end
   end
@@ -0,0 +1,14 @@
+module DspacePlugin::CollectionHelper
+
+  include ArticleHelper
+
+  def custom_options_for_article(article,tokenized_children)
+    @article = article
+
+    visibility_options(article,tokenized_children) +
+    content_tag('div',
+      hidden_field_tag('article[accept_comments]', 0)
+    )
+  end
+
+end
-class DspacePlugin::Communityy < Article
+class DspacePlugin::Communityy < Folder
   settings_items :dspace_community_id, :type => :string
@@ -17,9 +17,9 @@ class DspacePlugin::Communityy &lt; Article
   end
   def to_html(options = {})
-    dspace_community = self
+    dspace_content = self
     proc do
-      render :file => 'content_viewer/community', :locals => {:dspace_community => dspace_community}
+      render :file => 'content_viewer/dspace_content', :locals => { :dspace_content => dspace_content }
     end
   end
@@ -0,0 +1,14 @@
+module DspacePlugin::CommunityyHelper
+
+  include ArticleHelper
+
+  def custom_options_for_article(article,tokenized_children)
+    @article = article
+
+    visibility_options(article,tokenized_children) +
+    content_tag('div',
+      hidden_field_tag('article[accept_comments]', 0)
+    )
+  end
+
+end
@@ -1,35 +0,0 @@
-class DspacePlugin::DspaceBlock < Block
-
-  settings_items :dspace_server_url, :type => :string, :default => ""
-  settings_items :collections, :type => :string, :default => ""
-
-  attr_accessible :dspace_server_url, :collections
-
-  def self.description
-    _('DSpace library')
-  end
-
-  def help
-    _('This block displays a DSpace content.')
-  end
-
-  def content(args={})
-    block = self
-    proc do
-      dspace_client = Dspace::Client.new(block.dspace_server_url)
-      collection_items = dspace_client.get_collection_items(block.collections)
-      if !collection_items.blank?
-        content_tag('div',
-          render(:file => 'blocks/dspace', :locals => {:collection_items => collection_items})
-        )
-      else
-        ''
-      end
-    end
-  end
-
-  def cacheable?
-    false
-  end
-
-end
 class DspacePlugin::Item
-  attr_accessor :id, :name, :author, :issue_date, :abstract, :description, :uri, :files
+  include DspacePlugin::ItemHelper
+
+  attr_accessor :id, :name, :author, :issue_date, :abstract, :description, :uri, :files, :mimetype
   def initialize
     self.files = []
@@ -0,0 +1,8 @@
+module DspacePlugin::ItemHelper
+
+  def remove_slash_at_end_url(url)
+    url.gsub!(/\/$/,'') if url =~ /\/$/
+    url
+  end
+
+end
-class DspacePlugin::Library < Blog
+class DspacePlugin::Library < Folder
   settings_items :dspace_server_url, :type => :string
   attr_accessible :dspace_server_url
   def dspace_server_url_valid
+
+    if self.dspace_server_url.blank?
+      errors.add(:dspace_server_url, _("can't be blank") )
+      return
+    end
+
     errors.add(self.dspace_server_url, _("is not a valid URL. Please correct it and resubmit.")) unless url_valid?(self.dspace_server_url)
   end
@@ -23,9 +29,9 @@ class DspacePlugin::Library &lt; Blog
   end
   def to_html(options = {})
-    dspace_library = self
+    dspace_content = self
     proc do
-      render :file => 'content_viewer/library', :locals => {:dspace_library => dspace_library}
+      render :file => 'content_viewer/dspace_content', :locals => { :dspace_content => dspace_content }
     end
   end
@@ -0,0 +1,14 @@
+module DspacePlugin::LibraryHelper
+
+  include ArticleHelper
+
+  def custom_options_for_article(article,tokenized_children)
+    @article = article
+
+    visibility_options(article,tokenized_children) +
+    content_tag('div',
+      hidden_field_tag('article[accept_comments]', 0)
+    )
+  end
+
+end
@@ -1,95 +0,0 @@
-require_dependency 'article'
-
-class Article
-
-  scope :dspace, :conditions => ["articles.published = true and (articles.type != 'UploadedFile' and articles.type != 'Blog' and articles.type != 'RssFeed') OR articles.type is NULL"]
-
-  def self.articles_columns
-    Article.column_names.map {|c| "articles.#{c}"} .join(",")
-  end
-
-  def self.most_accessed(owner, limit = nil)
-    conditions = owner.kind_of?(Environment) ?  ["hits > 0"] : ["profile_id = ? and hits > 0", owner.id]
-    result = Article.dspace.find(
-      :all,
-      :order => 'hits desc',
-      :limit => limit,
-      :conditions => conditions)
-    result.paginate({:page => 1, :per_page => limit})
-  end
-
-  def self.most_commented_dspace(owner, limit)
-      conditions = owner.kind_of?(Environment) ? ["comments_count > 0"] : ["profile_id = ? and comments_count > 0", owner.id]
-      result = Article.dspace.find(
-        :all,
-        :order => 'comments_count desc',
-        :limit => limit,
-        :conditions => conditions)
-      result.paginate({:page => 1, :per_page => limit})
-  end
-
-  def self.more_positive_votes(owner, limit = nil)
-      conditions = owner.kind_of?(Environment) ? {'votes.voteable_type' => 'Article'} : ["profile_id = ? and votes.voteable_type = ? ", owner.id, 'Article']
-      result = Article.dspace.find(
-        :all,
-        :order => 'sum(vote) desc',
-        :group => 'voteable_id, ' + articles_columns,
-        :limit => limit,
-        :having => ['sum(vote) > 0'],
-        :conditions => conditions,
-        :joins => 'INNER JOIN votes ON articles.id = votes.voteable_id')
-      result.paginate({:page => 1, :per_page => limit})
-  end
-
-  def self.more_negative_votes(owner, limit = nil)
-      conditions = owner.kind_of?(Environment) ? {'votes.voteable_type' => 'Article'} : ["profile_id = ? and votes.voteable_type = 'Article' ", owner.id]
-      result = Article.dspace.find(
-        :all,
-        :order => 'sum(vote) asc',
-        :group => 'voteable_id, ' + articles_columns,
-        :limit => limit,
-        :having => ['sum(vote) < 0'],
-        :conditions => conditions,
-        :joins => 'INNER JOIN votes ON articles.id = votes.voteable_id'
-      )
-      result.paginate({:page => 1, :per_page => limit})
-  end
-
-  def self.most_liked(owner, limit = nil)
-      conditions = owner.kind_of?(Environment) ? ["votes.voteable_type = 'Article' and vote > 0"] : ["votes.voteable_type = 'Article' and vote > 0 and profile_id = ? ", owner.id]
-      result = Article.dspace.find(
-        :all,
-        :select => articles_columns,
-        :order => 'count(voteable_id) desc',
-        :group => 'voteable_id, ' + articles_columns,
-        :limit => limit,
-        :conditions => conditions,
-        :joins => 'INNER JOIN votes ON articles.id = votes.voteable_id')
-      result.paginate({:page => 1, :per_page => limit})
-  end
-
-  def self.most_disliked(owner, limit = nil)
-      conditions = owner.kind_of?(Environment) ? ["votes.voteable_type = 'Article' and vote < 0"] : ["votes.voteable_type = 'Article' and vote < 0 and profile_id = ? ", owner.id]
-      result = Article.dspace.find(
-        :all,
-        :order => 'count(voteable_id) desc',
-        :group => 'voteable_id, ' + articles_columns,
-        :limit => limit,
-        :conditions => conditions,
-        :joins => 'INNER JOIN votes ON articles.id = votes.voteable_id')
-       result.paginate({:page => 1, :per_page => limit})
-  end
-
-  def self.most_voted(owner, limit = nil)
-    conditions = owner.kind_of?(Environment) ? ["votes.voteable_type = 'Article'"] : ["votes.voteable_type = 'Article' and profile_id = ? ", owner.id]
-      result = Article.dspace.find(
-        :all,
-        :select => articles_columns,
-        :order => 'count(voteable_id) desc',
-        :group => 'voteable_id, ' + articles_columns,
-        :limit => limit,
-        :conditions => conditions,
-        :joins => 'INNER JOIN votes ON articles.id = votes.voteable_id')
-      result.paginate({:page => 1, :per_page => limit})
-  end
-end
+/**
+
 function selectCommunity(element, community_slug) {
   var hidden_field = jQuery('<input>').attr({
       id: 'article_dspace_community_name_',
@@ -9,6 +11,7 @@ function selectCommunity(element, community_slug) {
   jQuery(hidden_field).insertAfter(element);
 }
+
 function selectCollection(element, collection_slug) {
   var hidden_field = jQuery('<input>').attr({
       id: 'article_dspace_collection_name_',
@@ -20,6 +23,10 @@ function selectCollection(element, collection_slug) {
   jQuery(hidden_field).insertAfter(element);
 }
+function select_action(field_active) {
+}
+**/
+
 jQuery(document).ready(function() {
   url_base = window.location.protocol + '//' + window.location.host;
   forms = jQuery('form');
@@ -29,4 +36,50 @@ jQuery(document).ready(function() {
       forms[f].action = url_action.replace("/cms/new", "/plugin/dspace/new").replace(url_base,'');
     }
   });
+
+  function check_fields(check, table_id) {
+    var checkboxes = jQuery("#" + table_id + " tbody tr td input[type='checkbox']")
+    for (var i = 0; i < checkboxes.length; i++) {
+      if (checkboxes[i].disabled == false) {
+        checkboxes[i].checked = check
+      }
+    }
+  }
+
+  function verify_checked(field_id){
+    var checkboxes = jQuery("#" + field_id + "_fields_conf tbody tr td input[type='checkbox']")
+    var allchecked = true;
+    for (var j = 1; j < checkboxes.length; j++) {
+      if(!checkboxes[j].checked) {
+        allchecked = false;
+        break;
+      }
+    }
+
+    var checkbox = jQuery("#" + field_id + "_active");
+    checkbox.attr("checked", allchecked);
+  }
+
+
+  function check_all(field_id) {
+    jQuery("#" + field_id + "_active").click(function (){
+      check_fields(this.checked, field_id + "_fields_conf")
+    });
+    verify_checked(field_id);
+  }
+
+  check_all("community");
+  check_all("collection");
+
+  jQuery("input[type='checkbox']").click(function () {
+    var checkbox = jQuery(this).attr("id").split("_");
+    verify_checked(checkbox.first());
+
+    if(this.checked == false) {
+      jQuery("#" + checkbox.first() + "_" + checkbox.last()).attr("checked", false)
+    }
+
+    jQuery(this).next().attr("disabled", !this.checked);
+  })
+
 });
 #dspace_library ul {
   margin: 0;
-  padding: 0 0 0 20px;
+  padding: 0 0 0 5px;
 }
 #dspace_library li {
-  list-style-image: url(/designs/themes/base/imgs/li-recent.gif);
+  /**list-style-image: url(/designs/themes/base/imgs/li-recent.gif);*/
+  list-style-type: none;
 }
 #dspace_library li.item,
@@ -28,7 +29,7 @@
 }
 .dspace_item_section {
-  margin-left: 20px;
+  margin: 15px;
 }
 .dspace_item_section_title {
@@ -1 +0,0 @@
-require File.dirname(__FILE__) + '/../../../test/test_helper'
@@ -1,148 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper'
-
-require 'comment_controller'
-# Re-raise errors caught by the controller.
-class CommentController; def rescue_action(e) raise e end; end
-
-class DspaceBlockTest < ActiveSupport::TestCase
-
-  include AuthenticatedTestHelper
-  fixtures :users, :environments
-
-  def setup
-    @controller = CommentController.new
-    @request    = ActionController::TestRequest.new
-    @response   = ActionController::TestResponse.new
-    @profile = create_user('testinguser').person
-    @environment = @profile.environment
-  end
-  attr_reader :profile, :environment
-
-  def enable_vote_plugin
-    enabled = false
-    environment=Environment.default
-    if Noosfero::Plugin.all.include?('VotePlugin')
-      if not environment.enabled_plugins.include?(:vote)
-        environment.enable_plugin(Vote)
-        environment.save!
-      end
-      enabled = true
-    end
-    enabled
-  end
-
- should 'list most commented articles' do
-    Article.delete_all
-    a1 = create(TextileArticle, :name => "art 1", :profile_id => profile.id)
-    a2 = create(TextileArticle, :name => "art 2", :profile_id => profile.id)
-    a3 = create(TextileArticle, :name => "art 3", :profile_id => profile.id)
-
-    2.times { Comment.create(:title => 'test', :body => 'asdsad', :author => profile, :source => a2).save! }
-    4.times { Comment.create(:title => 'test', :body => 'asdsad', :author => profile, :source => a3).save! }
-
-    # should respect the order (more commented comes first)
-    assert_equal a3.name, profile.articles.most_commented_dspace(Environment.default, 3).first.name
-    # It is a2 instead of a1 since it does not list articles without comments
-    assert_equal a2.name, profile.articles.most_commented_dspace(Environment.default, 3).last.name
-  end
-
-
-  should 'find the most voted' do
-    if not enable_vote_plugin
-      return
-    end
-    article = fast_create(Article, {:name=>'2 votes'})
-    2.times{
-      person = fast_create(Person)
-      person.vote_for(article)
-    }
-    article = fast_create(Article, {:name=>'10 votes'})
-    10.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    article = fast_create(Article, {:name=>'5 votes'})
-    5.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    articles = Article.most_voted(Environment.default, 5)
-    assert_equal '10 votes', articles.first.name
-    assert_equal '2 votes', articles.last.name
-  end
-
-  should 'list the most postive' do
-    if not enable_vote_plugin
-      return
-    end
-    article = fast_create(Article, {:name=>'23 votes for 20 votes against'})
-    20.times{
-        person = fast_create(Person)
-        person.vote_against(article)
-    }
-    23.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    article = fast_create(Article, {:name=>'10 votes for 5 votes against'})
-    10.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    5.times{
-        person = fast_create(Person)
-        person.vote_against(article)
-    }
-    article = fast_create(Article, {:name=>'2 votes against'})
-    2.times{
-        person = fast_create(Person)
-        person.vote_against(article)
-    }
-
-    article = fast_create(Article, {:name=>'7 votes for'})
-    7.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    articles = Article.more_positive_votes(Environment.default, 5)
-    assert_equal '7 votes for', articles.first.name
-    assert_equal '23 votes for 20 votes against', articles.last.name
-  end
-
-  should 'list the most negative' do
-    if not enable_vote_plugin
-      return
-    end
-    article = fast_create(Article, {:name=>'23 votes for 29 votes against'})
-    29.times{
-        person = fast_create(Person)
-        person.vote_against(article)
-    }
-    23.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    article = fast_create(Article, {:name=>'10 votes for 15 votes against'})
-    10.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    15.times{
-        person = fast_create(Person)
-        person.vote_against(article)
-    }
-    article = fast_create(Article, {:name=>'2 votes against'})
-    2.times{
-        person = fast_create(Person)
-        person.vote_against(article)
-    }
-    article = fast_create(Article, {:name=>'7 votes for'})
-    7.times{
-        person = fast_create(Person)
-        person.vote_for(article)
-    }
-    articles = Article.more_negative_votes(Environment.default, 5)
-    assert_equal '23 votes for 29 votes against', articles.first.name
-    assert_equal '2 votes against', articles.last.name
-  end
-end
 \ No newline at end of file
@@ -1,29 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper'
-
-class DspacePluginTest < ActiveSupport::TestCase
-
-  def setup
-    @plugin = DspacePlugin.new
-  end
-
-  should 'be a noosfero plugin' do
-    assert_kind_of Noosfero::Plugin, @plugin
-  end
-
-  should 'have name' do
-    assert_equal 'Relevant Content Plugin', DspacePlugin.plugin_name
-  end
-
-  should 'have description' do
-    assert_equal  _("A plugin that lists the most accessed, most commented, most liked and most disliked contents."), DspacePlugin.plugin_description
-  end
-
-  should 'have stylesheet' do
-    assert @plugin.stylesheet?
-  end
-
-  should "return DspaceBlock in extra_blocks class method" do
-    assert  DspacePlugin.extra_blocks.keys.include?(DspacePlugin::DspaceBlock)
-  end
-
-end
@@ -10,10 +10,14 @@
 <% collections = Dspace::Community.get_all_collections_from( dspace_server_url, community_id ).map { |collection| item = [_(collection.name), collection.id] } %>
-<table border="0" style="border-bottom: 1px solid #c0c0c0;">
-  <tr style="background-color: #f0f0f0; border-bottom: 1px solid #c0c0c0;">
-    <th style="border-bottom: none;">&nbsp;</th>
-    <th style="border-bottom: none;" align="left"><%= _('Collection name') %></th>
+<table id="collection_fields_conf" border="0" style="border-bottom: 1px solid #c0c0c0;">
+  <tr style="background-color: #f0f0f0; border-top: 1px solid #c0c0c0; border-bottom: 1px solid #c0c0c0;">
+    <td style="font-style: italic">
+      <%= _("Check/Uncheck All Unlocked")%>
+    </td>
+    <td align="center">
+      <input type="checkbox" id="collection_active" />
+    </td>
   </tr>
   <% dspace_collections_ids = DspacePlugin::Collection.find(:all, :conditions => { :parent_id => @article.parent_id}).map { |collection| ids = collection.dspace_collection_id.to_i } %>
@@ -21,14 +25,22 @@
   <% collections.each do |collection| %>
   <tr>
-    <% if dspace_collections_ids.include? collection[1] %>
-    <td align="center"><%= check_box_tag "article[dspace_collections_ids][]", collection[1], true, :disabled => 'disabled', :onclick => "selectCollection(this,'#{collection[0]}')" %> </td>
-    <% else %>
-    <td align="center"><%= check_box_tag "article[dspace_collections_ids][]", collection[1], false, :onclick => "selectCollection(this,'#{collection[0]}')" %> </td>
-    <% end %>
     <td>
       <%= collection[0] %>
     </td>
+
+    <% if dspace_collections_ids.include? collection[1] %>
+    <td align="center">
+      <%= check_box_tag "article[dspace_collections][][id]", collection[1], true, :disabled => 'disabled', :id => "collection_id_#{collection[1]}" %>
+      <%= hidden_field_tag "article[dspace_collections][][name]", collection[0], :disabled => 'disabled' %>
+    </td>
+    <% else %>
+    <td align="center">
+      <%= check_box_tag "article[dspace_collections][][id]", collection[1], false, :id => "collection_id_#{collection[1]}" %>
+     <%= hidden_field_tag "article[dspace_collections][][name]", collection[0], :disabled => 'disabled', :id => "collection_name_#{collection[1]}" %>
+    </td>
+    <% end %>
+
   </tr>
   <% end %>
@@ -6,10 +6,14 @@
 <% communities = Dspace::Community.get_all_communities_from(dspace_server_url).map { |community| item = [_(community.name), community.id] } %>
-<table border="0" style="border-bottom: 1px solid #c0c0c0;">
-  <tr style="background-color: #f0f0f0; border-bottom: 1px solid #c0c0c0;">
-    <th style="border-bottom: none;">&nbsp;</th>
-    <th style="border-bottom: none;" align="left"><%= _('Community name') %></th>
+<table id="community_fields_conf" border="0" style="border-bottom: 1px solid #c0c0c0;">
+  <tr style="background-color: #f0f0f0; border-top: 1px solid #c0c0c0; border-bottom: 1px solid #c0c0c0;">
+    <td style="font-style: italic">
+      <%= _("Check/Uncheck All Unlocked")%>
+    </td>
+    <td align="center">
+      <input type="checkbox" id="community_active" />
+    </td>
   </tr>
   <% dspace_communities_ids = DspacePlugin::Communityy.find(:all, :conditions => { :parent_id => @article.parent_id }).map { |community| ids = community.dspace_community_id.to_i } %>
@@ -17,14 +21,22 @@
   <% communities.each do |community| %>
   <tr>
-    <% if dspace_communities_ids.include? community[1] %>
-      <td align="center"><%= check_box_tag "article[dspace_communities_ids][]", community[1], true, :disabled => 'disabled', :onclick => "selectCommunity(this,'#{community[0]}')" %> </td>
-    <% else %>
-      <td align="center"><%= check_box_tag "article[dspace_communities_ids][]", community[1], false, :onclick => "selectCommunity(this,'#{community[0]}')" %> </td>
-    <% end %>
     <td>
       <%= community[0] %>
     </td>
+
+    <% if dspace_communities_ids.include? community[1] %>
+    <td align="center">
+      <%= check_box_tag "article[dspace_communities][][id]", community[1], true, :disabled => 'disabled', :id => "community_id_#{community[1]}" %>
+      <%= hidden_field_tag "article[dspace_communities][][name]", community[0], :disabled => 'disabled' %>
+    </td>
+    <% else %>
+    <td align="center">
+      <%= check_box_tag "article[dspace_communities][][id]", community[1], false, :id => "community_id_#{community[1]}" %>
+      <%= hidden_field_tag "article[dspace_communities][][name]", community[0], :disabled => 'disabled', :id => "community_name_#{community[1]}" %>
+    </td>
+    <% end %>
+
   </tr>
   <% end %>
@@ -1,23 +0,0 @@
-<h1><%= _('DSpace Item') %></h1>
-
-<%= required_fields_message %>
-
-<%= required f.text_field('name', :size => '64', :maxlength => 150) %>
-
-<%= hidden_field_tag 'article[parent_id]', @article.parent_id %>
-
-<% dspace_client = Dspace::Client.new('') %>
-
-<% collections = dspace_client.get_collections %>
-
-<%
-collections_2 = collections.map { |collection|
-  item = [_(collection.name), collection.id]
-}
-%>
-
-<%= labelled_form_field(_('Collections:'),
-      select(:article,
-             :dspace_collection_id,
-             options_for_select_with_title(collections_2))) %>
-
-<%= error_messages_for 'library' %>
-
-<h1><%= _('My Library') %></h1>
+<h1><%= _('My DSpace Library') %></h1>
 <%= render :file => 'shared/tiny_mce' %>
@@ -10,61 +8,4 @@
 <%= render :partial => 'general_fields' %>
-<script type="text/javascript">
-  function submit_button(index) {
-     return $("article_slug").form.select("input.submit")[index];
-  }
-  function warn_value_change() {
-     show_warning('article-formitem', "slug-change-confirmation");
-     disable_button(submit_button(0));
-     disable_button(submit_button(1));
-  }
-  function confirm_change() {
-     enable_button(submit_button(0));
-     enable_button(submit_button(1));
-     hide_warning('slug-change-confirmation');
-  }
-  function no_change() {
-     $("article_slug").value = $("old_article_slug").value;
-     enable_button(submit_button(0));
-     enable_button(submit_button(1));
-     hide_warning('slug-change-confirmation');
-  }
-</script>
-
-<%= hidden_field_tag 'old_article_slug', @article.slug %>
-<div id="article-formitem">
-  <%= labelled_form_field( _('Address'),
-        content_tag('code',
-          url_for(@article.url).gsub(/#{@article.slug}$/, '') +
-          text_field(:article, :slug, :onchange => "warn_value_change()", :size => 25)
-        ) +
-        content_tag('div',
-          content_tag('strong', _('WARNING!')) + '&nbsp;' +
-            _("You are about to change the address, and this will break external links to this blog or to posts inside it. Do you really want to change?") +
-          content_tag('div',
-            button_to_function(:ok, _("Yes"), "confirm_change()") + ' ' +
-            button_to_function(:cancel, _('No'), 'no_change()')
-          ),
-          :id => 'slug-change-confirmation',
-          :class => 'change-confirmation',
-          :style => 'display: none;'
-        )
-      )
-  %>
-</div>
-
-<%= labelled_form_field(_('Description:'), text_area(:article, :body, :rows => 10)) %>
-
-<%= f.fields_for :image_builder, @article.image do |i| %>
-  <%= file_field_or_thumbnail(_('Cover image:'), @article.image, i)%>
-  <%= _("Max size: %s (.jpg, .gif, .png)")% Image.max_size.to_humanreadable %>
-<% end %>
-
-<% unless @article.image.nil? %>
-  <%= labelled_check_box(_('Remove cover image'),'remove_image',true,false)%>
-<% end %>
-
-<%= labelled_form_field(_('Show catalog as:'), f.select(:visualization_format, [ [ _('Listing with thumbnail'), 'full'], [ _('Listing without thumbnail'), 'short'], [ _('Type catalog'), 'catalog'] ])) %>
-
-<%= labelled_form_field(_('Items per page:'), f.select(:posts_per_page, Blog.posts_per_page_options)) %>
+<%= labelled_form_field(_('Description:'), text_area(:article, :body, :cols => 68, :rows => 10)) %>
@@ -1,3 +0,0 @@
-<li class="collection">
-  <span class="title"><%= link_to collection_item.title, :controller => 'content_viewer', :action => 'view_page', :page => collection_item.path %></span>
-</li>
@@ -0,0 +1,3 @@
+<li class="community">
+  <span class="title"><%= link_to community.title, :controller => 'content_viewer', :action => 'view_page', :page => community.path %></span>
+</li>
@@ -1,3 +0,0 @@
-<li class="community">
-  <span class="title"><%= link_to community_item.title, :controller => 'content_viewer', :action => 'view_page', :page => community_item.path %></span>
-</li>
 <li class="item">
-  <span class="name"><%= link_to item.name, :controller => 'dspace_plugin', :action => 'view_item', :id => item.id, :collection_id => @page.id %></span><br />
-  <span class="authors"><%= item.author %></span> <span class="date_issued">(<%= item.issue_date %>)</span>
+  <div>
+
+    <% if item.files.first.mimetype == 'application/pdf' %>
+      <img src="/plugins/dspace/icons/pdf.png" style="display: inline-block;"/>
+    <% else %>
+      <img src="/plugins/dspace/icons/text.png" style="display: inline-block;"/>
+    <% end %>
+
+    <div style="display: inline-block; width: 92%;">
+      <span class="name"><%= link_to item.name, :controller => 'dspace_plugin', :action => 'view_item', :id => item.id, :collection_id => @page.id %></span><br />
+      <span class="authors"><%= item.author %></span> <span class="date_issued">(<%= item.issue_date %>)</span>
+    </div>
+
+  </div>
 </li>
@@ -1,10 +0,0 @@
-<% collection_id = dspace_collection.dspace_collection_id %>
-<% dspace_server = dspace_collection.parent.parent.dspace_server_url %>
-
-<div id="dspace_library">
-  <div id="dspace_collection">
-    <ul id="items_list">
-      <%= render :partial => 'item', :collection => dspace_collection.items(dspace_server, collection_id) %>
-    </ul>
-  </div>
-</div>
@@ -1,24 +0,0 @@
-<% community_id = dspace_community.dspace_community_id %>
-<% dspace_server = dspace_community.parent.dspace_server_url %>
-
-<div id="dspace_library">
-
-  <% if dspace_community.allow_create?(user) %>
-
-  <div id="actions">
-    <%= content_tag('a', :href => url_for({:controller => 'cms', :action => 'new', :type => "DspacePlugin::Collection", :parent_id => dspace_community.id}), :class => 'button with-text icon-add') do %>
-          <%= _("Add a %s") % DspacePlugin::Collection.short_description %>
-    <% end %>
-  </div>
-
-  <% end %>
-
-  <% collections = dspace_community.collections dspace_server, community_id %>
-
-  <div id="dspace_community">
-    <ul id="collection_list">
-      <%= render :partial => 'collection', :collection => collections %>
-    </ul>
-  </div>
-
-</div>
@@ -0,0 +1,59 @@
+<div id="dspace_library">
+
+  <% if dspace_content.allow_create?(user) %>
+
+  <div id="actions">
+
+    <% if dspace_content.is_a? DspacePlugin::Library %>
+
+      <%= content_tag('a', :href => url_for({:controller => 'cms', :action => 'new', :type => "DspacePlugin::Communityy", :parent_id => dspace_content.id}), :class => 'button with-text icon-add') do %>
+        <%= _("Add %s") % DspacePlugin::Communityy.short_description %>
+      <% end %>
+
+    <% elsif dspace_content.is_a? DspacePlugin::Communityy %>
+
+      <%= content_tag('a', :href => url_for({:controller => 'cms', :action => 'new', :type => "DspacePlugin::Collection", :parent_id => dspace_content.id}), :class => 'button with-text icon-add') do %>
+        <%= _("Add %s") % DspacePlugin::Collection.short_description %>
+      <% end %>
+
+    <% end %>
+
+  </div>
+
+  <% end %>
+
+  <% if dspace_content.is_a? DspacePlugin::Library %>
+
+    <% communities = dspace_content.communities %>
+
+  <ul id="communities_list">
+    <%= render :partial => 'community', :collection => communities %>
+  </ul>
+
+  <% elsif dspace_content.is_a? DspacePlugin::Communityy %>
+
+    <% community_id = dspace_content.dspace_community_id %>
+    <% dspace_server = dspace_content.parent.dspace_server_url %>
+
+    <% collections = dspace_content.collections dspace_server, community_id %>
+
+  <ul id="collections_list">
+    <%= render :partial => 'collection', :collection => collections %>
+  </ul>
+
+  <% elsif dspace_content.is_a? DspacePlugin::Collection %>
+
+    <% collection_id = dspace_content.dspace_collection_id %>
+    <% dspace_server = dspace_content.parent.parent.dspace_server_url %>
+
+    <div id="dspace_library">
+      <div id="dspace_collection">
+        <ul id="items_list">
+          <%= render :partial => 'item', :collection => dspace_content.items(dspace_server, collection_id) %>
+        </ul>
+      </div>
+    </div>
+
+  <% end %>
+
+</div>
@@ -1,19 +0,0 @@
-<div id="dspace_library">
-
-  <% if dspace_library.allow_create?(user) %>
-
-  <div id="actions">
-    <%= content_tag('a', :href => url_for({:controller => 'cms', :action => 'new', :type => "DspacePlugin::Communityy", :parent_id => dspace_library.id}), :class => 'button with-text icon-add') do %>
-          <%= _("Add a %s") % DspacePlugin::Communityy.short_description %>
-    <% end %>
-  </div>
-
-  <% end %>
-
-  <% communities = dspace_library.communities %>
-
-  <ul id="sortable" class="communities_list">
-    <%= render :partial => 'community_item', :collection => communities %>
-  </ul>
-
-</div>
@@ -0,0 +1,12 @@
+<li>
+  <ul class="item_file_attributes_list">
+    <li>
+      <span><%= _('File:') %></span> <%= link_to item_file.name, dspace_server_url + item_file.retrieve_link, :target => '_blank' %></li>
+    <% unless item_file.description.blank? %>
+    <li>
+      <span><%= _('Description:') %> </span> <%= item_file.description %></li>
+    <% end %>
+    <li>
+      <span><%= _('Size:') %> </span> <%= number_to_human_size( item_file.size_bytes ) %></li>
+  </ul>
+</li>
@@ -0,0 +1,8 @@
+<div class="dspace_item_section">
+  <div class="dspace_item_section_title">
+   <%= item_section[:title] %>
+  </div>
+  <div class="dspace_item_section_content">
+    <%= item_section[:content] %>
+  </div>
+</div>
-<% dspace_server_url = @collection.parent.parent.dspace_server_url %>
-
-<% if dspace_server_url =~ /\/$/ %>
-  <% dspace_server_url.gsub!(/\/$/,'') %>
-<% end %>
+<% extend DspacePlugin::ItemHelper %>
 <div id="article-parent">
   <%= button(:back, _('Go back to %s') % @collection.short_title, @collection.url) %>
@@ -11,58 +7,22 @@
 <div id="dspace_library_item">
   <div id="dspace_item_title">
-    <%= content_tag 'h1', @item.name, :class => 'title' %>
-  </div>
-
-  <br />
-
-  <div class="dspace_item_section">
-    <%= content_tag 'div', _('Authors:'), :class => 'dspace_item_section_title' %>
-    <%= content_tag 'div', @item.author, :class => 'dspace_item_section_value' %>
+    <h1 class="title"><%= @item.name %></h1>
   </div>
-  <br />
+  <% item_files = render :partial => 'item_file',
+               :locals => { :dspace_server_url => remove_slash_at_end_url(@collection.parent.parent.dspace_server_url) },
+               :collection => @item.files %>
-  <div class="dspace_item_section">
-    <%= content_tag 'div', _('Issue date:'), :class => 'dspace_item_section_title' %>
-    <%= content_tag 'div', @item.issue_date, :class => 'dspace_item_section_value' %>
-  </div>
-  <br />
-
-  <div class="dspace_item_section">
-    <%= content_tag 'div', _('Abstract:'), :class => 'dspace_item_section_title' %>
-    <%= content_tag 'div', @item.abstract, :class => 'dspace_item_section_value' %>
-  </div>
+  <% item_files_content = content_tag('ul', item_files, :id => 'item_files_list') %>
-  <br />
-  <div class="dspace_item_section">
-    <%= content_tag 'div', _('Description:'), :class => 'dspace_item_section_title' %>
-    <%= content_tag 'div', @item.description, :class => 'dspace_item_section_value' %>
-  </div>
+  <%= render :partial => 'item_section',
+             :collection => [ { :title => _('Authors:'), :content => @item.author },
+                              { :title => _('Issue date:'), :content => @item.issue_date },
+                              { :title => _('Abstract:'), :content => @item.abstract },
+                              { :title => _('Description:'), :content => @item.description },
+                              { :title => _('URI:'), :content => link_to(@item.uri, @item.uri, :target => '_blank') },
+                              { :title => _('Files in this item'), :content => item_files_content } ] %>
-  <br />
-
-  <div class="dspace_item_section">
-    <%= content_tag 'div', _('URI:'), :class => 'dspace_item_section_title' %>
-    <%= content_tag 'div', link_to(@item.uri, @item.uri), :class => 'dspace_item_section_value' %>
-  </div>
-
-  <br />
-
-  <div class="dspace_item_section">
-    <%= content_tag 'div', _('Files in this item'), :class => 'dspace_item_section_title' %>
-    <ul id="item_files_list">
-    <% @item.files.each do |file| %>
-      <li>
-        <ul class="item_file_attributes_list">
-          <li><span>File:</span> <%= link_to file.name, dspace_server_url + file.retrieve_link %></li>
-          <li><span>Description:</span> <%= file.description %></li>
-          <li><span>Size:</span> <%= number_to_human_size( file.size_bytes ) %></li>
-        </ul>
-      </li>
-    <% end %>
-    </ul>
-  </div>
-
-<div>
+</div>
@@ -1,38 +0,0 @@
-<h1><%= _('DSpace Collections') %></h1>
-
-<%= hidden_field_tag 'article[parent_id]', @article.parent_id %>
-
-<%= hidden_field_tag 'article[dspace_community_id]', @article.parent.dspace_community_id %>
-
-<% dspace_server_url = @article.parent.parent.dspace_server_url %>
-
-<% community_id = @article.parent.dspace_community_id %>
-
-<% collections = Dspace::Community.get_all_collections_from( dspace_server_url, community_id ).map { |collection| item = [_(collection.name), collection.id] } %>
-
-<table border="0" style="border-bottom: 1px solid #c0c0c0;">
-  <tr style="background-color: #f0f0f0; border-bottom: 1px solid #c0c0c0;">
-    <th style="border-bottom: none;">&nbsp;</th>
-    <th style="border-bottom: none;" align="left"><%= _('Collection name') %></th>
-  </tr>
-
-  <% dspace_collections_ids = DspacePlugin::Collection.find(:all, :conditions => { :parent_id => @article.parent_id}).map { |collection| ids = collection.dspace_collection_id.to_i } %>
-
-  <% collections.each do |collection| %>
-
-  <tr>
-    <% if dspace_collections_ids.include? collection[1] %>
-    <td align="center"><%= check_box_tag "article[dspace_collections_ids][]", collection[1], true, :disabled => 'disabled', :onclick => "selectCollection(this,'#{collection[0]}')" %> </td>
-    <% else %>
-    <td align="center"><%= check_box_tag "article[dspace_collections_ids][]", collection[1], false, :onclick => "selectCollection(this,'#{collection[0]}')" %> </td>
-    <% end %>
-    <td>
-      <%= collection[0] %>
-    </td>
-  </tr>
-
-  <% end %>
-
-</table>
-
-<%= javascript_include_tag 'plugins/dspace/javascripts/dspace_plugin' %>
@@ -1,34 +0,0 @@
-<h1><%= _('DSpace Communities') %></h1>
-
-<%= hidden_field_tag 'article[parent_id]', @article.parent_id %>
-
-<% dspace_server_url = @article.parent.dspace_server_url %>
-
-<% communities = Dspace::Community.get_all_communities_from(dspace_server_url).map { |community| item = [_(community.name), community.id] } %>
-
-<table border="0" style="border-bottom: 1px solid #c0c0c0;">
-  <tr style="background-color: #f0f0f0; border-bottom: 1px solid #c0c0c0;">
-    <th style="border-bottom: none;">&nbsp;</th>
-    <th style="border-bottom: none;" align="left"><%= _('Community name') %></th>
-  </tr>
-
-  <% dspace_communities_ids = DspacePlugin::Communityy.find(:all, :conditions => { :parent_id => @article.parent_id }).map { |community| ids = community.dspace_community_id.to_i } %>
-
-  <% communities.each do |community| %>
-
-  <tr>
-    <% if dspace_communities_ids.include? community[1] %>
-      <td align="center"><%= check_box_tag "article[dspace_communities_ids][]", community[1], true, :disabled => 'disabled', :onclick => "selectCommunity(this,'#{community[0]}')" %> </td>
-    <% else %>
-      <td align="center"><%= check_box_tag "article[dspace_communities_ids][]", community[1], false, :onclick => "selectCommunity(this,'#{community[0]}')" %> </td>
-    <% end %>
-    <td>
-      <%= community[0] %>
-    </td>
-  </tr>
-
-  <% end %>
-
-</table>
-
-<%= javascript_include_tag 'plugins/dspace/javascripts/dspace_plugin' %>
@@ -1,42 +0,0 @@
-<%= error_messages_for 'article' %>
-
-<div class='<%= (environment.enabled?('media_panel') ? 'with_media_panel' : 'no_media_panel') %>'>
-
-<%= labelled_form_for 'article', :url => {action: "new_dspace_content"}, :html => { :multipart => true, :class => @type } do |f| %>
-
-  <%= hidden_field_tag("type", @type) if @type %>
-
-  <%= hidden_field_tag('parent_id', @parent_id) if @parent_id %>
-
-  <%= hidden_field_tag('back_to', @back_to) %>
-
-  <%= hidden_field_tag('success_back_to', @success_back_to) %>
-
-  <%= render :partial => partial_for_class(@article.class), :locals => { :f => f } %>
-
-  <% button_bar do %>
-    <%= submit_button :save, _('Save') %>
-
-    <% if @back_to %>
-      <%= button :cancel, _('Cancel'), @back_to %>
-    <% elsif @parent_id || @article.parent %>
-      <%= button :cancel, _('Cancel'), :action => 'view', :id => @parent_id || @article.parent %>
-    <% else %>
-      <%= button :cancel, _('Cancel'), :action => 'index' %>
-    <% end %>
-
-    <% unless @article.new_record? %>
-      <%= button :delete, _('Delete'), {:controller => :cms, :action => :destroy, :id => @article},
-        :method => :post, :confirm => delete_article_message(@article) %>
-    <% end %>
-  <% end %>
-<% end %>
-</div>
-
-<% if environment.enabled?('media_panel') && [TinyMceArticle, TextileArticle, Event, EnterpriseHomepage].any?{|klass| @article.kind_of?(klass)} %>
-  <%= render :partial => 'text_editor_sidebar' %>
-<% end %>
-
-<br style='clear: both'/>
-
-<%= javascript_include_tag "article.js" %>
	@@ -7,7 +7,15 @@ class DspacePluginController < PublicController		@@ -7,7 +7,15 @@ class DspacePluginController < PublicController
7	item_id = params[:id]	7	item_id = params[:id]
8		8
9	begin	9	begin
10	- @item = Dspace::Item.get_item_by_id item_id	10	+ @collection = DspacePlugin::Collection.find(collection_id)
		11	+ rescue ActiveRecord::RecordNotFound
		12	+ render_not_found
		13	+ return
		14	+ end
		15	+
		16	+ begin
		17	+ dspace_server = @collection.parent.parent.dspace_server_url
		18	+ @item = Dspace::Item.get_item_by_id dspace_server, item_id
11	rescue ActiveResource::UnauthorizedAccess	19	rescue ActiveResource::UnauthorizedAccess
12	render_not_found	20	render_not_found
13	return	21	return
	@@ -1,16 +0,0 @@	@@ -1,16 +0,0 @@
1	-class Dspace::Client
2	-
3	- def initialize(server_url)
4	- @server_url = server_url
5	- end
6	-
7	- def get_collections
8	- Dspace::Collection.get_all_collections_from @server_url
9	- end
10	-
11	- def get_communities
12	- Dspace::Community.get_all_communities_from @server_url
13	- end
14	-
15	-end
16	-
	@@ -7,34 +7,9 @@ class Dspace::Collection < Dspace::Resource		@@ -7,34 +7,9 @@ class Dspace::Collection < Dspace::Resource
7	item_list = []	7	item_list = []
8		8
9	if result.items.count > 0	9	if result.items.count > 0
10	-
11	result.items.each { \|element\|	10	result.items.each { \|element\|
12	-
13	- item_metadata = Dspace::Item.get_all_item_metadata_from dspace_server, element.id
14	-
15	- # author
16	- metadata = item_metadata[0].attributes
17	- if metadata != {}
18	- metadata = Hash[[metadata.map{\|k,v\| v}]]
19	- author = metadata.has_key?('dc.contributor.author') ? metadata['dc.contributor.author'] : nil
20	- end
21	-
22	- # date issued
23	- metadata = item_metadata[3].attributes
24	- if metadata != {}
25	- metadata = Hash[[metadata.map{\|k,v\| v}]]
26	- issue_date = metadata.has_key?('dc.date.issued') ? metadata['dc.date.issued'] : nil
27	- end
28	-
29	- item = DspacePlugin::Item.new
30	-
31	- item.id = element.id
32	- item.name = element.name
33	- item.author = author
34	- item.issue_date = issue_date
35	-	11	+ item = Dspace::Item.get_item_by_id dspace_server, element.id
36	item_list << item	12	item_list << item
37	-
38	}	13	}
39	end	14	end
40		15
	@@ -18,9 +18,9 @@ class DspacePlugin::Collection < Article		@@ -18,9 +18,9 @@ class DspacePlugin::Collection < Article
18	end	18	end
19		19
20	def to_html(options = {})	20	def to_html(options = {})
21	- dspace_collection = self	21	+ dspace_content = self
22	proc do	22	proc do
23	- render :file => 'content_viewer/collection', :locals => {:dspace_collection => dspace_collection}	23	+ render :file => 'content_viewer/dspace_content', :locals => { :dspace_content => dspace_content }
24	end	24	end
25	end	25	end
26		26
@@ -0,0 +1,14 @@		@@ -0,0 +1,14 @@
	1	+module DspacePlugin::CollectionHelper
	2	+
	3	+ include ArticleHelper
	4	+
	5	+ def custom_options_for_article(article,tokenized_children)
	6	+ @article = article
	7	+
	8	+ visibility_options(article,tokenized_children) +
	9	+ content_tag('div',
	10	+ hidden_field_tag('article[accept_comments]', 0)
	11	+ )
	12	+ end
	13	+
	14	+end
1	-class DspacePlugin::Communityy < Article	1	+class DspacePlugin::Communityy < Folder
2		2
3	settings_items :dspace_community_id, :type => :string	3	settings_items :dspace_community_id, :type => :string
4		4
	@@ -17,9 +17,9 @@ class DspacePlugin::Communityy < Article		@@ -17,9 +17,9 @@ class DspacePlugin::Communityy < Article
17	end	17	end
18		18
19	def to_html(options = {})	19	def to_html(options = {})
20	- dspace_community = self	20	+ dspace_content = self
21	proc do	21	proc do
22	- render :file => 'content_viewer/community', :locals => {:dspace_community => dspace_community}	22	+ render :file => 'content_viewer/dspace_content', :locals => { :dspace_content => dspace_content }
23	end	23	end
24	end	24	end
25		25
@@ -0,0 +1,14 @@		@@ -0,0 +1,14 @@
	1	+module DspacePlugin::CommunityyHelper
	2	+
	3	+ include ArticleHelper
	4	+
	5	+ def custom_options_for_article(article,tokenized_children)
	6	+ @article = article
	7	+
	8	+ visibility_options(article,tokenized_children) +
	9	+ content_tag('div',
	10	+ hidden_field_tag('article[accept_comments]', 0)
	11	+ )
	12	+ end
	13	+
	14	+end
	@@ -1,35 +0,0 @@	@@ -1,35 +0,0 @@
1	-class DspacePlugin::DspaceBlock < Block
2	-
3	- settings_items :dspace_server_url, :type => :string, :default => ""
4	- settings_items :collections, :type => :string, :default => ""
5	-
6	- attr_accessible :dspace_server_url, :collections
7	-
8	- def self.description
9	- _('DSpace library')
10	- end
11	-
12	- def help
13	- _('This block displays a DSpace content.')
14	- end
15	-
16	- def content(args={})
17	- block = self
18	- proc do
19	- dspace_client = Dspace::Client.new(block.dspace_server_url)
20	- collection_items = dspace_client.get_collection_items(block.collections)
21	- if !collection_items.blank?
22	- content_tag('div',
23	- render(:file => 'blocks/dspace', :locals => {:collection_items => collection_items})
24	- )
25	- else
26	- ''
27	- end
28	- end
29	- end
30	-
31	- def cacheable?
32	- false
33	- end
34	-
35	-end
1	class DspacePlugin::Item	1	class DspacePlugin::Item
2		2
3	- attr_accessor :id, :name, :author, :issue_date, :abstract, :description, :uri, :files	3	+ include DspacePlugin::ItemHelper
		4	+
		5	+ attr_accessor :id, :name, :author, :issue_date, :abstract, :description, :uri, :files, :mimetype
4		6
5	def initialize	7	def initialize
6	self.files = []	8	self.files = []
@@ -0,0 +1,8 @@		@@ -0,0 +1,8 @@
	1	+module DspacePlugin::ItemHelper
	2	+
	3	+ def remove_slash_at_end_url(url)
	4	+ url.gsub!(/\/$/,'') if url =~ /\/$/
	5	+ url
	6	+ end
	7	+
	8	+end