hydra-collections 2.0.5 → 3.0.0.beta1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d18580cc6f615f6c2db8d172eafc3295c15112d2
-  data.tar.gz: 62d0fcd015fe10ddbea811c97d510ea851834af2
+  metadata.gz: 3b6be32ab4643ad1b7e135f56f5431cd691f06c9
+  data.tar.gz: b332c0d2f798db1e5383cc83cd880724448f0cf1
 SHA512:
-  metadata.gz: c2a9e3f91da266b5d23af91de3ff2ae4aae0252d69fb3c7feed87e9323f953528b053f9d8399418377832084e8ea212466d14370098251af22d0c3564ff77a5a
-  data.tar.gz: 0d571aa1c5108c78d46e4d6d6dc0e75c76d1b07fe77105774a068a12da2642f7e5ed3d6b2cf8d3b50fbf5af49312ab9d27a8bacfcb7e80f0ff0ca8579971864e
+  metadata.gz: 5ce2d0de4ac13bca1cf53a7ff8c705f57e401e9836882b80ca63778fd9aba7456a8b8d1f7c08204160c93040074035ca4018ae6badba6259cdc2aadc57d70894
+  data.tar.gz: 2ed72913e6c7aa79413d95a0f13f34fa0c405d883133cb4ec7390bc6ed72a05d8f3af035f047ef0c303d7fb85ab28eeb6e5282048d4f0f4e43fec3fa5a0311a8

data/.gitignore

@@ -18,3 +18,4 @@ test/tmp
 test/version_tmp
 tmp
 spec/internal
+jetty

data/.gitmodules

@@ -1,4 +0,0 @@
-[submodule "jetty"]
-	path = jetty
-	url = git://github.com/projecthydra/hydra-jetty.git
-ignore = dirty

data/.rspec

@@ -0,0 +1 @@
+--colour

data/Gemfile

@@ -3,6 +3,9 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in hydra-collections.gemspec
 gemspec
 
+gem 'active-fedora', '9.0.0.beta3'
+gem 'hydra-head', github: 'projecthydra/hydra-head', branch: 'fedora-4'
+
 group :development, :test do
   gem 'sqlite3'
   gem "factory_girl_rails"
@@ -11,3 +14,9 @@ group :development, :test do
   gem 'jettywrapper'
   gem 'byebug', require: false
 end
+
+file = File.expand_path("Gemfile", ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path("../spec/internal", __FILE__))
+if File.exists?(file)
+  puts "Loading #{file} ..." if $DEBUG # `ruby -d` or `bundle -v`
+  instance_eval File.read(file)
+end

data/README.md

@@ -35,7 +35,7 @@ Or install it yourself as:
 
 ### Make your Models Collectible
 
-Add `include Hydra::Collections::Collectible` to the models for anything that you want to be able to add to collections (ie. GenericFile, Book, Article, etc.), then add `index_collection_pids` to the solrization logic (ie. put it in to_solr)
+Add `include Hydra::Collections::Collectible` to the models for anything that you want to be able to add to collections (ie. GenericFile, Book, Article, etc.), then add `index_collection_ids` to the solrization logic (ie. put it in to_solr)
 
 Example:
 ```ruby
@@ -44,13 +44,13 @@ class GenericFile < ActiveFedora::Base
   ...
   def to_solr(solr_doc={}, opts={})
     super(solr_doc, opts)
-    index_collection_pids(solr_doc)
+    index_collection_ids(solr_doc)
     return solr_doc
   end
 end
 ```
 
-Any items that include the `Hydra::Collections::Collectible` module can look up which collections they belong to via `.collections`.  The `index_collection_pids` puts the pids of all associated collections into the `collection` facet.
+Any items that include the `Hydra::Collections::Collectible` module can look up which collections they belong to via `.collections`.  The `index_collection_ids` puts the ids of all associated collections into the `collection` facet.
 
 ### Make the Collection show as a facet in your CatalogController
 

data/Rakefile

@@ -1,12 +1,12 @@
 #!/usr/bin/env rake
-require "bundler/gem_tasks"
+begin
+  require 'bundler/setup'
+rescue LoadError
+  puts 'You must `gem install bundler` and `bundle install` to run rake tasks'
+end
 
-Dir.glob('tasks/*.rake').each { |r| import r }
-
-ENV["RAILS_ROOT"] ||= 'spec/internal'
+Bundler::GemHelper.install_tasks
 
-desc 'Run CI tests in e.g. Travis environment'
-task :travis => ['clean', 'ci']
+Dir.glob('tasks/*.rake').each { |r| import r }
 
-desc 'Default: run CI'
-task :default => [:travis]
+task :default => [:ci]

data/{lib → app/controllers/concerns}/hydra/collections/selects_collections.rb

@@ -1,12 +1,8 @@
-require 'blacklight/catalog'
-
 module Hydra::Collections::SelectsCollections
-
-  extend ActiveSupport::Autoload
   extend ActiveSupport::Concern
  
   def access_levels
-    {read:[:read,:edit],edit:[:edit]}
+    { read: [:read, :edit], edit: [:edit] }
   end 
 
   # add one of the following methods as a before filter on any page that shows the form_for_select_collection
@@ -32,11 +28,10 @@ module Hydra::Collections::SelectsCollections
     
     # temporarily set solr_search_params_logic to collection_search_params_logic
     orig_solr_search_params_logic = self.class.solr_search_params_logic
-    # self.class.solr_search_params_logic += [:add_collection_filter]
     self.class.solr_search_params_logic = collection_search_params_logic
     logger.debug "Collection Search logic: "+ self.class.solr_search_params_logic.inspect
     # run the solr query to find the collections
-    (resp, doc_list) = get_search_results({:q => ''}, {:rows=>100})
+    (resp, doc_list) = get_search_results({q: ''}, {rows: 100})
     
     #reset to original solr logic
     self.class.send(:define_method, "discovery_permissions")  { original_permissions } unless access_level.blank?
@@ -48,7 +43,7 @@ module Hydra::Collections::SelectsCollections
       
   def add_collection_filter(solr_parameters, user_parameters)
     solr_parameters[:fq] ||= []
-    solr_parameters[:fq] << "#{Solrizer.solr_name("has_model", :symbol)}:\"info:fedora/afmodel:Collection\""
+    solr_parameters[:fq] << "#{Solrizer.solr_name("has_model", :symbol)}:Collection"
   end
   
   # Defines which solr_search_params_logic should be used when searching for Collections

data/app/controllers/concerns/hydra/collections_controller_behavior.rb

@@ -14,7 +14,7 @@ module Hydra
 
       # Catch permission errors
       rescue_from Hydra::AccessDenied, CanCan::AccessDenied do |exception|
-        if (exception.action == :edit)
+        if exception.action == :edit
           redirect_to(collections.url_for({:action=>'show'}), :alert => "You do not have sufficient privileges to edit this document")
         elsif current_user and current_user.persisted?
           redirect_to root_url, :alert => exception.message
@@ -46,18 +46,19 @@ module Hydra
 
     def after_create
       respond_to do |format|
+        ActiveFedora::SolrService.instance.conn.commit
         format.html { redirect_to collections.collection_path(@collection), notice: 'Collection was successfully created.' }
         format.json { render json: @collection, status: :created, location: @collection }
       end
     end
-    
+
     def after_create_error
       respond_to do |format|
         format.html { render action: "new" }
         format.json { render json: @collection.errors, status: :unprocessable_entity }
       end
     end
-        
+
     def create
       @collection.apply_depositor_metadata(current_user.user_key)
       add_members_to_collection unless batch.empty?
@@ -67,8 +68,8 @@ module Hydra
         after_create_error
       end
     end
-    
-    def after_update 
+
+    def after_update
       if flash[:notice].nil?
         flash[:notice] = 'Collection was successfully updated.'
       end
@@ -77,14 +78,14 @@ module Hydra
         format.json { render json: @collection, status: :updated, location: @collection }
       end
     end
-    
+
     def after_update_error
       respond_to do |format|
         format.html { render action: collections.edit_collection_path(@collection) }
         format.json { render json: @collection.errors, status: :unprocessable_entity }
       end
     end
-    
+
     def update
       process_member_changes
       @collection.update_attributes(params[:collection].except(:members))
@@ -94,21 +95,21 @@ module Hydra
         after_update_error
       end
     end
-    
+
     def after_destroy (id)
       respond_to do |format|
         format.html { redirect_to catalog_index_path, notice: 'Collection was successfully deleted.' }
         format.json { render json: {id:id}, status: :destroyed, location: @collection }
-      end      
+      end
     end
 
     def after_destroy_error (id)
       respond_to do |format|
         format.html { redirect_to catalog_index_path, notice: 'Collection could not be deleted.' }
         format.json { render json: {id:id}, status: :destroy_error, location: @collection }
-      end      
+      end
     end
-    
+
     def destroy
       if @collection.destroy
          after_destroy(params[:id])
@@ -116,16 +117,16 @@ module Hydra
         after_destroy_error(params[:id])
       end
     end
-    
+
     protected
 
-    # Queries Solr for members of the collection.  
+    # Queries Solr for members of the collection.
     # Populates @response and @member_docs similar to Blacklight Catalog#index populating @response and @documents
     def query_collection_members
       query = params[:cq]
 
       #default the rows to 100 if not specified then merge in the user parameters and the attach the collection query
-      solr_params =  {rows:100}.merge(params.symbolize_keys).merge({:q => query})
+      solr_params =  { rows: 100 }.merge(params.symbolize_keys).merge(q: query)
 
       # run the solr query to find the collections
       (@response, @member_docs) = get_search_results(solr_params)
@@ -142,17 +143,15 @@ module Hydra
 
     def add_members_to_collection collection = nil
       collection ||= @collection
-      collection.members(true)
       collection.member_ids = batch.concat(collection.member_ids)
     end
 
     def remove_members_from_collection
-      @collection.members(true)
-      @collection.member_ids = (@collection.member_ids - batch)
+      @collection.members.delete(batch.map { |pid| ActiveFedora::Base.find(pid) })
     end
 
     def assign_batch_to_collection
-      @collection.members(true)
+      @collection.members(true) #Force the members to get cached before (maybe) removing some of them
       @collection.member_ids = batch
     end
 
@@ -166,11 +165,11 @@ module Hydra
         flash[:error] = "An error occured. Files were not moved to #{destination_collection.title} Collection."
       end
     end
-    
+
     # include filters into the query to only include the collection memebers
     def include_collection_ids(solr_parameters, user_parameters)
       solr_parameters[:fq] ||= []
-      solr_parameters[:fq] << Solrizer.solr_name(:collection)+':"'+@collection.id+'"'
+      solr_parameters[:fq] << Solrizer.solr_name(:collection, :facetable)+':"'+@collection.id+'"'
     end
   end # module CollectionsControllerBehavior
 end # module Hydra

data/app/helpers/collections_search_helper.rb

@@ -4,7 +4,8 @@ module CollectionsSearchHelper
   # @param [String] collection_pid the pid of a collection
   # @return [String] the title of the collection if available, otherwise its pid
   def collection_name(collection_pid)
-    Collection.load_instance_from_solr(collection_pid).title || collection_pid
+    #TODO this can be loaded from solr
+    Collection.find(collection_pid).title || collection_pid
   end
 
 end

data/app/models/concerns/hydra/collection.rb

@@ -7,26 +7,87 @@ module Hydra
     include Hydra::Collections::Collectible
 
     included do
-      has_metadata "descMetadata", type: Hydra::CollectionRdfDatastream
-      has_metadata "properties", type: Hydra::Datastream::Properties
+      has_and_belongs_to_many :members, predicate: ActiveFedora::RDF::Fcrepo::RelsExt.hasCollectionMember, class_name: "ActiveFedora::Base" , after_remove: :update_member
 
-      has_and_belongs_to_many :members, property: :has_collection_member, class_name: "ActiveFedora::Base" , after_remove: :update_member
+      property :depositor, predicate: RDF::URI.new("http://id.loc.gov/vocabulary/relators/dpt")
 
-      has_attributes :depositor, datastream: :properties, multiple: false
-      
-      has_attributes :title, :date_uploaded, :date_modified, :description, 
-                     datastream: :descMetadata, multiple: false
-      has_attributes :creator, :contributor, :based_near, :part_of, :publisher,
-                     :date_created, :subject,:resource_type, :rights,
-                     :identifier, :language, :tag, :related_url,
-                     datastream: :descMetadata, multiple: true
+      property :part_of, predicate: RDF::DC.isPartOf
+      property :contributor, predicate: RDF::DC.contributor do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :creator, predicate: RDF::DC.creator do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :title, predicate: RDF::DC.title do |index|
+        index.as :stored_searchable
+      end
+      property :description, predicate: RDF::DC.description do |index|
+        index.type :text
+        index.as :stored_searchable
+      end
+      property :publisher, predicate: RDF::DC.publisher do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :date_created, predicate: RDF::DC.created do |index|
+        index.as :stored_searchable
+      end
+      property :date_uploaded, predicate: RDF::DC.dateSubmitted do |index|
+        index.type :date
+        index.as :stored_sortable
+      end
+      property :date_modified, predicate: RDF::DC.modified do |index|
+        index.type :date
+        index.as :stored_sortable
+      end
+      property :subject, predicate: RDF::DC.subject do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :language, predicate: RDF::DC.language do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :rights, predicate: RDF::DC.rights do |index|
+        index.as :stored_searchable
+      end
+      property :resource_type, predicate: RDF::DC.type do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :identifier, predicate: RDF::DC.identifier do |index|
+        index.as :stored_searchable
+      end
+      property :based_near, predicate: RDF::FOAF.based_near do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :tag, predicate: RDF::DC.relation do |index|
+        index.as :stored_searchable, :facetable
+      end
+      property :related_url, predicate: RDF::RDFS.seeAlso
+
+      # Single-valued properties
+      def depositor
+        super.first
+      end
+
+      def title
+        super.first
+      end
+
+      def date_uploaded
+        super.first
+      end
+
+      def date_modified
+        super.first
+      end
+
+      def description
+        super.first
+      end
 
       before_create :set_date_uploaded
       before_save :set_date_modified
+      before_destroy :update_all_members
 
       after_save :update_all_members
-
-      before_destroy :update_all_members
     end
 
     def terms_for_editing
@@ -34,7 +95,11 @@ module Hydra
     end
 
     def terms_for_display
-      self.descMetadata.class.config.keys.map{|v| v.to_sym}
+      [
+        :part_of, :contributor, :creator, :title, :description, :publisher, 
+        :date_created, :date_uploaded, :date_modified, :subject, :language, :rights, 
+        :resource_type, :identifier, :based_near, :tag, :related_url
+      ]
     end
 
     def update_all_members
@@ -58,6 +123,5 @@ module Hydra
       self.date_modified = Date.today
     end
 
-  end
-
+ end
 end

data/app/views/collections/_search_form.html.erb

@@ -1,19 +1,3 @@
-<%#
-Copyright © 2012 The Pennsylvania State University
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-%>
-
 <div style="float: right;" >
   <%= form_for([collections, @collection] , :method => :get, :class => "well form-search") do |f| %>
 

data/hydra-collections.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |spec|
   spec.email         = ["cam156@psu.edu"]
   spec.description   = "A rails engine for managing Hydra Collections"
   spec.summary       = "A rails engine for managing Hydra Collections"
-  spec.homepage      = "https://github.com/psu-stewardship/hydra-collections"
+  spec.homepage      = "https://github.com/projecthydra-labs/hydra-collections"
   spec.license       = "APACHE2"
 
   spec.files         = `git ls-files`.split($/)
@@ -19,9 +19,9 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "blacklight"
-  spec.add_dependency "hydra-head", "~> 7.0"
+  spec.add_dependency "hydra-head", "~> 8.0.0.alpha"
+  spec.add_dependency 'deprecation'
 
-  spec.add_development_dependency "bundler", "~> 1.3"
-  spec.add_development_dependency "rake"
+  spec.add_development_dependency "engine_cart"
   spec.add_development_dependency "rspec-rails"
 end

data/lib/hydra/collections/accepts_batches.rb

@@ -18,12 +18,12 @@ module Hydra
       protected
       
       def batch_ids_from_params
-        if params["batch_document_ids"].nil? || params["batch_document_ids"].empty?
-          return []
+        if params["batch_document_ids"].blank?
+          []
         elsif params["batch_document_ids"] == "all"
-          return Hydra::Collections::SearchService.new(session, current_user.user_key).last_search_documents.map(&:id)
+          Hydra::Collections::SearchService.new(session, current_user.user_key).last_search_documents.map(&:id)
         else
-          return params["batch_document_ids"]
+          params["batch_document_ids"]
         end
       end
       
@@ -52,4 +52,4 @@ module Hydra
       
     end
   end
-end
+end