Conform to standard option and add YARD doc

samvera · Jun 13, 2017 · 5a4fd21 · 5a4fd21
1 parent 275e92d
commit 5a4fd21
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/app/services/hyrax/file_set_derivatives_service.rb b/app/services/hyrax/file_set_derivatives_service.rb
@@ -82,6 +82,10 @@ def derivative_path_factory
         Hyrax::DerivativePath
       end
 
+      # Calls the Hydra::Derivates::FulltextExtraction unless the extract_full_text
+      # configuration option is set to false
+      # @param [String] filename of the object to be used for full text extraction
+      # @param [String] uri to the files set (deligated to file_set)
       def extract_full_text(filename, uri)
         return unless Hyrax.config.extract_full_text?
         Hydra::Derivatives::FullTextExtract.create(filename,

diff --git a/lib/generators/hyrax/templates/config/hyrax.rb b/lib/generators/hyrax/templates/config/hyrax.rb
@@ -75,8 +75,8 @@
   # config.libreoffice_path = "soffice"
 
   # Option to enable/disable full text extraction from PDFs
-  # Default is false, set to true to skip extraction
-  config.extract_full_text = true
+  # Default is true, set to false to disable full text extraction
+  # config.extract_full_text = true
 
   # How many seconds back from the current time that we should show by default of the user's activity on the user's dashboard
   # config.activity_to_show_default_seconds_since_now = 24*60*60

diff --git a/lib/hyrax/configuration.rb b/lib/hyrax/configuration.rb
@@ -375,7 +375,8 @@ def subject_prefix
 
     attr_writer :extract_full_text
     def extract_full_text?
-      @extract_full_text ||= false
+      return @extract_full_text unless @extract_full_text.nil?
+      @extract_full_text = true
     end
 
     private