hydra-works 0.16.0 → 2.0.0

data/CODE_OF_CONDUCT.md

@@ -0,0 +1,36 @@
+The Samvera community is dedicated to providing a welcoming and
+positive experience for all its members, whether they are at a formal
+gathering, in a social setting, or taking part in activities online.
+The Samvera community welcomes participation from people all over the
+world and these members bring with them a wide variety of
+professional, personal and social backgrounds; whatever these may be,
+we treat colleagues with dignity and respect.
+
+Community members communicate primarily in English, though for many of
+them this is not their native language. We therefore strive to express
+ourselves simply and clearly remembering that unnecessary use of
+jargon and slang will be a barrier to understanding for many of our
+colleagues.  We are sensitive to the fact that the international
+nature of the community means that we span many different social norms
+around language and behaviour and we strive to conduct ourselves,
+online and in person, in ways that are unlikely to cause offence.
+
+Samvera conversations are often information-rich and intended to
+generate discussion and debate.  We discuss ideas from a standpoint of
+mutual respect and reasoned argument.
+
+Community members work together to promote a respectful and safe
+community. In the event that someone’s conduct is causing offence or
+distress, Samvera has a detailed
+[Anti-Harassment Policy and Protocol](https://wiki.duraspace.org/display/samvera/Anti-Harassment+Policy)
+which can be applied to address the problem. The first step in dealing
+with any serious misconduct is to contact a local meeting organizer,
+the
+[Samvera community helpers](https://wiki.duraspace.org/display/samvera/Samvera+Community+Helpers)
+([email](mailto:helpers@samvera.org)), a community member you
+trust, or the
+[Samvera Steering Group](https://wiki.duraspace.org/display/samvera/Samvera+Steering+Group+membership)
+immediately; at Samvera events, these people can be identified by
+distinctive name badges. The
+[Policy and Protocol](https://wiki.duraspace.org/display/samvera/Anti-Harassment+Policy)
+should be consulted for fuller details.

data/CONTRIBUTING.md

@@ -1,21 +1,24 @@
 # How to Contribute
 
-We want your help to make Project Hydra great.
-There are a few guidelines that we need contributors to follow so that we can have a chance of keeping on top of things.
+We want your help to make the Samvera community great. There are a few guidelines
+that we need contributors to follow so that we can have a chance of
+keeping on top of things.
 
 ## Code of Conduct
 
-The Hydra community is dedicated to providing a welcoming and positive experience for all its
-members, whether they are at a formal gathering, in a social setting, or taking part in activities
-online.  Please see our [Code of Conduct](https://wiki.duraspace.org/display/hydra/Code+of+Conduct)
-for more information.
+The Samvera Community is dedicated to providing a welcoming and positive
+experience for all its members, whether they are at a formal gathering, in
+a social setting, or taking part in activities online. Please see our
+[Code of Conduct](CODE_OF_CONDUCT.md) for more information.
 
-## Hydra Project Intellectual Property Licensing and Ownership
+## Samvera Community Intellectual Property Licensing and Ownership
 
-All code contributors must have an Individual Contributor License Agreement (iCLA) on file with the Hydra Project Steering Group.
-If the contributor works for an institution, the institution must have a Corporate Contributor License Agreement (cCLA) on file.
+All code contributors must have an Individual Contributor License Agreement
+(iCLA) on file with the Samvera Steering Group. If the contributor works for
+an institution, the institution must have a Corporate Contributor License
+Agreement (cCLA) on file.
 
-https://wiki.duraspace.org/display/hydra/Hydra+Project+Intellectual+Property+Licensing+and+Ownership
+https://wiki.duraspace.org/display/samvera/Samvera+Community+Intellectual+Property+Licensing+and+Ownership
 
 You should also add yourself to the `CONTRIBUTORS.md` file in the root of the project.
 
@@ -31,7 +34,7 @@ You should also add yourself to the `CONTRIBUTORS.md` file in the root of the pr
 ### Reporting Issues
 
 * Make sure you have a [GitHub account](https://github.com/signup/free)
-* Submit a [Github issue](./issues) by:
+* Submit a [Github issue](https://github.com/samvera/hydra-works/issues/) by:
   * Clearly describing the issue
     * Provide a descriptive summary
     * Explain the expected behavior
@@ -48,7 +51,7 @@ You should also add yourself to the `CONTRIBUTORS.md` file in the root of the pr
   * Please avoid working directly on the `master` branch.
   * You may find the [hub suite of commands](https://github.com/defunkt/hub) helpful
 * Make sure you have added sufficient tests and documentation for your changes.
-  * Test functionality with RSpec; est features / UI with Capybara.
+  * Test functionality with RSpec; Test features / UI with Capybara.
 * Run _all_ the tests to assure nothing else was accidentally broken.
 
 ### Documenting Code
@@ -60,15 +63,11 @@ You should also add yourself to the `CONTRIBUTORS.md` file in the root of the pr
   * If you don't know exactly what a bit of code does, it is extra likely that it needs to be documented. Take a stab at it and ask for feedback in your pull request. You can use the 'blame' button on GitHub to identify the original developer of the code and @mention them in your comment.
   * This work greatly increases the usability of the code base and supports the on-ramping of new committers.
   * We will all be understanding of one another's time constraints in this area.
-* YARD examples:
-  * [Hydra::Works::RemoveGenericFile](https://github.com/projecthydra-labs/hydra-works/blob/master/lib/hydra/works/services/generic_work/remove_generic_file.rb)
-  * [ActiveTriples::LocalName::Minter](https://github.com/ActiveTriples/active_triples-local_name/blob/master/lib/active_triples/local_name/minter.rb)
 * [Getting started with YARD](http://www.rubydoc.info/gems/yard/file/docs/GettingStarted.md)
 
 ### Committing changes
 
 * Make commits of logical units.
-  * Your commit should include a high level description of your work in HISTORY.textile
 * Check for unnecessary whitespace with `git diff --check` before committing.
 * Make sure your commit messages are [well formed](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).
 * If you created an issue, you can close it by including "Closes #issue" in your commit message. See [Github's blog post for more details](https://github.com/blog/1386-closing-issues-via-commit-messages)
@@ -103,6 +102,10 @@ You should also add yourself to the `CONTRIBUTORS.md` file in the root of the pr
       long to fit in 72 characters
 ```
 
+* Make sure you have added the necessary tests for your changes.
+* Run _all_ the tests to assure nothing else was accidentally broken.
+* When you are ready to submit a pull request
+
 ### Submitting Changes
 
 * Read the article ["Using Pull Requests"](https://help.github.com/articles/using-pull-requests) on GitHub.
@@ -125,9 +128,7 @@ We adopted [Github's Pull Request Review](https://help.github.com/articles/about
 Common checks that may occur in our repositories:
 
 1. Travis CI - where our automated tests are running
-2. Hound CI - where we check for style violations
-3. Approval Required - Github enforces at least one person approve a pull request. Also, all reviewers that have chimed in must approve.
-4. CodeClimate - is our code remaining healthy (at least according to static code analysis)
+2. Approval Required - Github enforces at least one person approve a pull request. Also, all reviewers that have chimed in must approve.
 
 If one or more of the required checks failed (or are incomplete), the code should not be merged (and the UI will not allow it). If all of the checks have passed, then anyone on the project (including the pull request submitter) may merge the code.
 
@@ -144,16 +145,17 @@ First, the person contributing the code is putting themselves out there. Be mind
 This is your chance for a mentoring moment of another developer. Take time to give an honest and thorough review of what has changed. Things to consider:
 
   * Does the commit message explain what is going on?
-  * Does the code changes have tests? _Not all changes need new tests, some changes are refactors_
+  * Does the code changes have tests? _Not all changes need new tests, some changes are refactorings_
   * Do new or changed methods, modules, and classes have documentation?
   * Does the commit contain more than it should? Are two separate concerns being addressed in one commit?
   * Does the description of the new/changed specs match your understanding of what the spec is doing?
+  * Did the Travis tests complete successfully?
 
 If you are uncertain, bring other contributors into the conversation by assigning them as a reviewer.
 
 # Additional Resources
 
 * [General GitHub documentation](http://help.github.com/)
-* [GitHub pull request documentation](http://help.github.com/send-pull-requests/)
+* [GitHub pull request documentation](https://help.github.com/articles/about-pull-requests/)
 * [Pro Git](http://git-scm.com/book) is both a free and excellent book about Git.
 * [A Git Config for Contributing](http://ndlib.github.io/practices/my-typical-per-project-git-config/)

data/Gemfile

@@ -6,8 +6,16 @@ gemspec
 gem 'slop', '~> 3.6' # For byebug
 
 group :development, :test do
-  gem 'rubocop', '~> 0.47.0', require: false
-  gem 'rubocop-rspec', require: false
-  gem 'pry' unless ENV['CI']
+  gem 'clamby'
   gem 'pry-byebug' unless ENV['CI']
+  gem 'rubocop', require: false
+  gem 'rubocop-rspec', require: false
+end
+
+if ENV['RAILS_VERSION']
+  if ENV['RAILS_VERSION'] == 'edge'
+    gem 'rails', github: 'rails/rails'
+  else
+    gem 'rails', ENV['RAILS_VERSION']
+  end
 end

data/LICENSE

@@ -1,16 +1,14 @@
-##########################################################################
-#
-# Copyright 2014 University of Notre Dame, Northwestern University, and Data Curation Experts
-# Additional copyright may be held by others, as reflected in the commit log
-#
-# 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.
+Copyright 2014 University of Notre Dame, Northwestern University, and Data Curation Experts
+Additional copyright may be held by others, as reflected in the commit history.
+
+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.

data/README.md

@@ -1,14 +1,34 @@
 # Hydra::Works
 
 [![Version](https://badge.fury.io/rb/hydra-works.png)](http://badge.fury.io/rb/hydra-works)
-[![Build Status](https://travis-ci.org/projecthydra/hydra-works.svg?branch=master)](https://travis-ci.org/projecthydra/hydra-works)
-[![Coverage Status](https://coveralls.io/repos/projecthydra/hydra-works/badge.svg?branch=master)](https://coveralls.io/r/projecthydra/hydra-works?branch=master)
-[![Code Climate](https://codeclimate.com/github/projecthydra/hydra-works/badges/gpa.svg)](https://codeclimate.com/github/projecthydra/hydra-works)
+[![CircleCI](https://circleci.com/gh/samvera/hydra-works.svg?style=svg)](https://circleci.com/gh/samvera/hydra-works)
+[![Coverage Status](https://coveralls.io/repos/samvera/hydra-works/badge.svg?branch=master)](https://coveralls.io/r/samvera/hydra-works?branch=master)
+[![Code Climate](https://codeclimate.com/github/samvera/hydra-works/badges/gpa.svg)](https://codeclimate.com/github/samvera/hydra-works)
 [![Apache 2.0 License](http://img.shields.io/badge/APACHE2-license-blue.svg)](./LICENSE)
 [![Contribution Guidelines](http://img.shields.io/badge/CONTRIBUTING-Guidelines-blue.svg)](./CONTRIBUTING.md)
 [![API Docs](http://img.shields.io/badge/API-docs-blue.svg)](http://rubydoc.info/gems/hydra-works)
 
-The Hydra::Works gem implements the [PCDM](https://github.com/duraspace/pcdm/wiki) [Works](https://github.com/duraspace/pcdm/blob/master/pcdm-ext/works.rdf) data model using ActiveFedora-based models. In addition to the models, Hydra::Works includes associated behaviors around the broad concept of describable "works" or intellectual entities, the need for which was expressed by a variety of [Hydra community use cases](https://github.com/projecthydra/hydra-works/tree/master/use-cases). The PCDM Works domain model includes the following high-level entities:
+Jump in: [![Slack Status](http://slack.samvera.org/badge.svg)](http://slack.samvera.org/)
+
+# What is hydra-works?
+The Hydra::Works gem implements the [PCDM](https://github.com/duraspace/pcdm/wiki) [Works](https://github.com/duraspace/pcdm/blob/master/pcdm-ext/works.rdf) data model using ActiveFedora-based models. In addition to the models, Hydra::Works includes associated behaviors around the broad concept of describable "works" or intellectual entities, the need for which was expressed by a variety of [Samvera community use cases](https://github.com/samvera/hydra-works/tree/master/use-cases).
+
+## Product Owner & Maintenance
+
+hydra-works is a Core Component of the Samvera community. The documentation for
+what this means can be found
+[here](http://samvera.github.io/core_components.html#requirements-for-a-core-component).
+
+### Product Owner
+
+[mark-dce](https://github.com/mark-dce)
+
+# Help
+
+The Samvera community is here to help. Please see our [support guide](./SUPPORT.md).
+
+# Getting Started
+The PCDM Works domain model includes the following high-level entities:
 
  * **Collection**: a *pcdm:Collection* that indirectly contains zero or more **Works** and zero or more **Collection**s
  * **Work**: a *pcdm:Object* that holds zero or more **FileSets** and zero or more **Works**
@@ -23,7 +43,19 @@ Behaviors included in the model include:
  * Virus checking of original files
  * Full-text extraction from original files
 
-Check out the [Hydra::Derivatives README](https://github.com/projecthydra/hydra-derivatives#dependencies) for additional dependencies.
+## Dependencies
+
+Check out the [Hydra::Derivatives README](https://github.com/samvera/hydra-derivatives#dependencies) for dependencies.
+
+## Additional dependencies required for specs
+
+#### ClamAV
+* Mac installation
+  ```
+  $ brew install clamav
+  $ cp /usr/local/etc/clamav/freshclam.conf.sample /usr/local/etc/clamav/freshclam.conf
+  $ freshclam
+  ```
 
 ## Installation
 
@@ -73,9 +105,11 @@ page.save
 
 ## Virus Detection
 
-To turn on virus detection, install clamav on your system and add the `clamav` gem to your Gemfile
+To turn on virus detection, install [ClamAV](https://www.clamav.net/documents/installing-clamav) on your system and add the `clamby` gem to your Gemfile
 
-    gem 'clamav'
+```ruby
+gem 'clamby'
+```
 
 Then include the `VirusCheck` module in your `FileSet` class:
 
@@ -88,7 +122,7 @@ end
 
 ## Access controls
 
-We are using [Web ACL](http://www.w3.org/wiki/WebAccessControl) as implemented by [hydra-access-controls](https://github.com/projecthydra/hydra-head/tree/master/hydra-access-controls).
+We are using [Web ACL](http://www.w3.org/wiki/WebAccessControl) as implemented by [hydra-access-controls](https://github.com/samvera/hydra-head/tree/master/hydra-access-controls).
 
 ## How to contribute
 
@@ -108,21 +142,29 @@ rake
 
 ### Testing manually
 
-If you want to run the tests manually, follow these instructions:
+If you want to run the tests manually, first run solr and FCRepo. To start solr:
 
 ```bash
-solr_wrapper -d solr/config/
+solr_wrapper -v -d solr/config/ -n hydra-test -p 8985
 ```
 
 To start FCRepo, open another shell and run:
 
 ```bash
-fcrepo_wrapper -p 8984
+fcrepo_wrapper -v -p 8986 --no-jms
 ```
+Note you won't find these ports mentioned in this codebase, as testing behavior is inherited from ActiveFedora.
 
 Now you’re ready to run the tests. In the directory where hydra-works
 is installed, run:
 
 ```bash
-rake spec
+rake works:spec
 ```
+
+# Acknowledgments
+
+This software has been developed by and is brought to you by the Samvera community.  Learn more at the
+[Samvera website](http://samvera.org/).
+
+![Samvera Logo](https://wiki.duraspace.org/download/thumbnails/87459292/samvera-fall-font2-200w.png?version=1&modificationDate=1498550535816&api=v2)

data/SUPPORT.md

@@ -0,0 +1,5 @@
+If you would like to report an issue, first search [the list of issues](https://github.com/samvera/hydra-works/issues/) to see if someone else has already reported it, and then feel free to [create a new issue](https://github.com/samvera/hydra-works/issues/new).
+i
+If you have questions or need help, please email [the Samvera community tech list](https://groups.google.com/forum/#!forum/samvera-tech) or stop by the #dev channel in [the Samvera community Slack team](https://wiki.duraspace.org/pages/viewpage.action?pageId=87460391#Getintouch!-Slack).
+
+You can learn more about the various Samvera communication channels on the [Get in touch!](https://wiki.duraspace.org/pages/viewpage.action?pageId=87460391) wiki page.

data/hydra-works.gemspec

@@ -2,33 +2,35 @@
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'hydra/works/version'
+DO_NOT_SHIP          = ['spec/fixtures/eicar.txt']
 
 Gem::Specification.new do |spec|
   spec.name          = 'hydra-works'
   spec.version       = Hydra::Works::VERSION
   spec.authors       = ['Justin Coyne']
   spec.email         = ['justin@curationexperts.com']
-  spec.summary       = %q{Fundamental repository data model for hydra}
-  spec.description   = %q{Using this data model should enable easy collaboration amongst hydra projects.}
-  spec.homepage      = 'https://github.com/projecthydra-labs/hydra-works'
+  spec.summary       = %q{Fundamental repository data model for Samvera applications}
+  spec.description   = %q{Using this data model should enable easy collaboration amongst Samvera projects.}
+  spec.homepage      = 'https://github.com/samvera/hydra-works'
   spec.license       = 'APACHE2'
 
-  spec.files         = `git ls-files -z`.split("\x0")
+  spec.files         = `git ls-files -z`.split("\x0") - DO_NOT_SHIP
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
+  spec.add_dependency 'activesupport', '>= 5.2', '< 7.0'
+  spec.add_dependency 'hydra-derivatives', '~> 3.6'
+  spec.add_dependency 'hydra-file_characterization', '~> 1.0'
   spec.add_dependency 'hydra-pcdm', '>= 0.9'
-  spec.add_dependency 'hydra-derivatives', '~> 3.0'
-  spec.add_dependency 'hydra-file_characterization', '~> 0.3', '>= 0.3.3'
-  spec.add_dependency 'om', '~> 3.1'
 
-  spec.add_development_dependency 'bundler', '~> 1.7'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'rspec-rails', '~> 3.1'
-  spec.add_development_dependency 'sqlite3'
-  spec.add_development_dependency 'solr_wrapper', '~> 0.4'
-  spec.add_development_dependency 'fcrepo_wrapper', '~> 0.1'
+  spec.add_development_dependency 'bundler', '>= 1.7'
   spec.add_development_dependency 'coveralls'
+  spec.add_development_dependency 'fcrepo_wrapper', '~> 0.1'
+  spec.add_development_dependency 'rake', '~> 12.3'
   spec.add_development_dependency 'rspec'
+  spec.add_development_dependency 'rspec-rails'
+  spec.add_development_dependency 'solr_wrapper', '~> 2.0'
+  spec.add_development_dependency 'sqlite3'
+  spec.add_development_dependency 'rspec_junit_formatter'
 end

data/lib/hydra/works/characterization.rb

@@ -9,10 +9,11 @@ module Hydra::Works
       end
 
       def mapper_defaults
-        { audio_duration: :duration, audio_sample_rate: :sample_rate, exif_tool_version: :exif_version,
-          file_author: :creator, file_language: :language, file_mime_type: :mime_type,
-          video_audio_sample_rate: :sample_rate, video_duration: :duration, video_height: :height,
-          video_sample_rate: :sample_rate, video_width: :width }
+        { file_author: :creator, file_language: :language, file_mime_type: :mime_type,
+          audio_duration: :duration, audio_sample_rate: :sample_rate, audio_bit_rate: :bit_rate,
+          video_audio_sample_rate: :sample_rate, track_frame_rate: :frame_rate,
+          video_duration: :duration, video_sample_rate: :sample_rate, video_bit_rate: :bit_rate,
+          video_width: :width, video_track_width: :width, video_height: :height, video_track_height: :height }
       end
     end
 

data/lib/hydra/works/characterization/fits_document.rb

@@ -1,149 +1,353 @@
-require 'om'
-
 module Hydra::Works::Characterization
   class FitsDocument
-    include OM::XML::Document
-
-    set_terminology do |t|
-      t.root(path: 'fits',
-             xmlns: 'http://hul.harvard.edu/ois/xml/ns/fits/fits_output',
-             schema: 'http://hul.harvard.edu/ois/xml/xsd/fits/fits_output.xsd')
-      t.fits_v(path: { attribute: 'version' })
-      t.identification do
-        t.identity do
-          t.format_label(path: { attribute: 'format' })
-          t.mime_type(path: { attribute: 'mimetype' })
-          t.tool(attributes: { toolname: "Exiftool" }) do
-            t.exif_tool_version(path: { attribute: 'toolversion' })
-          end
-        end
-      end
-      t.fileinfo do
-        t.file_size(path: 'size')
-        t.last_modified(path: 'lastmodified', attributes: { toolname: "Exiftool" })
-        t.filename(path: 'filename')
-        t.original_checksum(path: 'md5checksum')
-        t.date_created(path: 'created')
-        t.rights_basis(path: 'rightsBasis')
-        t.copyright_basis(path: 'copyrightBasis')
-        t.copyright_note(path: 'copyrightNote')
-      end
-      t.filestatus do
-        t.well_formed(path: 'well-formed')
-        t.valid(path: 'valid')
-        t.status_message(path: 'message')
-      end
-      t.metadata do
-        t.document do
-          t.file_title(path: 'title')
-          t.file_author(path: 'author')
-          t.file_language(path: 'language')
-          t.page_count(path: 'pageCount')
-          t.word_count(path: 'wordCount')
-          t.character_count(path: 'characterCount')
-          t.paragraph_count(path: 'paragraphCount')
-          t.line_count(path: 'lineCount')
-          t.table_count(path: 'tableCount')
-          t.graphics_count(path: 'graphicsCount')
-        end
-        t.image do
-          t.byte_order(path: 'byteOrder')
-          t.compression(path: 'compressionScheme')
-          t.width(path: 'imageWidth')
-          t.height(path: 'imageHeight')
-          t.color_space(path: 'colorSpace')
-          t.profile_name(path: 'iccProfileName')
-          t.profile_version(path: 'iccProfileVersion')
-          t.orientation(path: 'orientation')
-          t.color_map(path: 'colorMap')
-          t.image_producer(path: 'imageProducer')
-          t.capture_device(path: 'captureDevice')
-          t.scanning_software(path: 'scanningSoftwareName')
-          t.exif_version(path: 'exifVersion')
-          t.gps_timestamp(path: 'gpsTimeStamp')
-          t.latitude(path: 'gpsDestLatitude')
-          t.longitude(path: 'gpsDestLongitude')
-        end
-        t.text do
-          t.character_set(path: 'charset')
-          t.markup_basis(path: 'markupBasis')
-          t.markup_language(path: 'markupLanguage')
-        end
-        t.audio do
-          t.duration(path: 'duration')
-          t.bit_depth(path: 'bitDepth')
-          t.sample_rate(path: 'sampleRate')
-          t.channels(path: 'channels')
-          t.data_format(path: 'dataFormatType')
-          t.offset(path: 'offset')
-        end
-        t.video do
-          t.width(path: 'imageWidth')
-          t.height(path: 'imageHeight')
-          t.duration(path: 'duration')
-          t.sample_rate(path: 'sampleRate')
-          t.audio_sample_rate(path: 'audioSampleRate')
-          t.frame_rate(path: 'frameRate')
-        end
-      end
-      # fits_version needs a different name than it's target node since they're at the same level
-      t.fits_version(proxy: [:fits, :fits_v])
-      t.format_label(proxy: [:identification, :identity, :format_label])
-      # Can't use .mime_type because it's already defined for this dcoument so method missing won't work.
-      t.file_mime_type(proxy: [:identification, :identity, :mime_type])
-      t.exif_tool_version(proxy: [:identification, :identity, :tool, :exif_tool_version])
-      t.file_size(proxy: [:fileinfo, :file_size])
-      t.last_modified(proxy: [:fileinfo, :last_modified])
-      t.filename(proxy: [:fileinfo, :filename])
-      t.original_checksum(proxy: [:fileinfo, :original_checksum])
-      t.date_created(proxy: [:fileinfo, :date_created])
-      t.rights_basis(proxy: [:fileinfo, :rights_basis])
-      t.copyright_basis(proxy: [:fileinfo, :copyright_basis])
-      t.copyright_note(proxy: [:fileinfo, :copyright_note])
-      t.well_formed(proxy: [:filestatus, :well_formed])
-      t.valid(proxy: [:filestatus, :valid])
-      t.filestatus_message(proxy: [:filestatus, :status_message])
-      t.file_title(proxy: [:metadata, :document, :file_title])
-      t.file_author(proxy: [:metadata, :document, :file_author])
-      t.page_count(proxy: [:metadata, :document, :page_count])
-      t.file_language(proxy: [:metadata, :document, :file_language])
-      t.word_count(proxy: [:metadata, :document, :word_count])
-      t.character_count(proxy: [:metadata, :document, :character_count])
-      t.paragraph_count(proxy: [:metadata, :document, :paragraph_count])
-      t.line_count(proxy: [:metadata, :document, :line_count])
-      t.table_count(proxy: [:metadata, :document, :table_count])
-      t.graphics_count(proxy: [:metadata, :document, :graphics_count])
-      t.byte_order(proxy: [:metadata, :image, :byte_order])
-      t.compression(proxy: [:metadata, :image, :compression])
-      t.width(proxy: [:metadata, :image, :width])
-      t.video_width(proxy: [:metadata, :video, :width])
-      t.height(proxy: [:metadata, :image, :height])
-      t.video_height(proxy: [:metadata, :video, :height])
-      t.color_space(proxy: [:metadata, :image, :color_space])
-      t.profile_name(proxy: [:metadata, :image, :profile_name])
-      t.profile_version(proxy: [:metadata, :image, :profile_version])
-      t.orientation(proxy: [:metadata, :image, :orientation])
-      t.color_map(proxy: [:metadata, :image, :color_map])
-      t.image_producer(proxy: [:metadata, :image, :image_producer])
-      t.capture_device(proxy: [:metadata, :image, :capture_device])
-      t.scanning_software(proxy: [:metadata, :image, :scanning_software])
-      t.exif_version(proxy: [:metadata, :image, :exif_version])
-      t.gps_timestamp(proxy: [:metadata, :image, :gps_timestamp])
-      t.latitude(proxy: [:metadata, :image, :latitude])
-      t.longitude(proxy: [:metadata, :image, :longitude])
-      t.character_set(proxy: [:metadata, :text, :character_set])
-      t.markup_basis(proxy: [:metadata, :text, :markup_basis])
-      t.markup_language(proxy: [:metadata, :text, :markup_language])
-      t.audio_duration(proxy: [:metadata, :audio, :duration])
-      t.video_duration(proxy: [:metadata, :video, :duration])
-      t.bit_depth(proxy: [:metadata, :audio, :bit_depth])
-      t.audio_sample_rate(proxy: [:metadata, :audio, :sample_rate])
-      t.video_sample_rate(proxy: [:metadata, :video, :sample_rate])
-      t.video_audio_sample_rate(proxy: [:metadata, :video, :audio_sample_rate])
-      t.channels(proxy: [:metadata, :audio, :channels])
-      t.data_format(proxy: [:metadata, :audio, :data_format])
-      t.offset(proxy: [:metadata, :audio, :offset])
-      t.frame_rate(proxy: [:metadata, :video, :frame_rate])
+    attr_accessor :ng_xml
+
+    PROXIED_TERMS = %i(
+      fits_version format_label file_mime_type exif_tool_version file_size
+      date_modified filename original_checksum date_created rights_basis
+      copyright_basis copyright_note well_formed valid filestatus_message
+      file_title file_author page_count file_language word_count
+      character_count paragraph_count line_count table_count graphics_count
+      byte_order compression width video_width video_track_width height
+      video_height video_track_height color_space profile_name profile_version
+      orientation color_map image_producer capture_device scanning_software
+      exif_version gps_timestamp latitude longitude character_set markup_basis
+      markup_language audio_duration video_duration bit_depth audio_sample_rate
+      video_sample_rate video_audio_sample_rate channels data_format offset
+      frame_rate audio_bit_rate video_bit_rate track_frame_rate aspect_ratio
+    ).freeze
+
+    def proxied_term_hash
+      PROXIED_TERMS.map { |t| [t, send(t)] }.to_h
+    end
+
+    def self.terminology
+      struct = Struct.new(:proxied_term).new
+      terminology = Struct.new(:terms)
+      terminology.new(PROXIED_TERMS.map { |t| [t, struct] }.to_h)
+    end
+
+    #  t.fits_version(proxy: [:fits, :fits_v])
+    def fits_version
+      ng_xml.css("fits").map { |n| n['version'].text }
+    end
+
+    # t.format_label(proxy: [:identification, :identity, :format_label])
+    def format_label
+      ng_xml.css("fits > identification > identity").map { |n| n['format'] }
+    end
+
+    # Can't use .mime_type because it's already defined for this document so method missing won't work.
+    # t.file_mime_type(proxy: [:identification, :identity, :mime_type])
+    def file_mime_type
+      # Sometimes, FITS reports the mimetype attribute as a comma-separated string.
+      # All terms are arrays and, in this case, there is only one element, so scan the first.
+      ng_xml.css("fits > identification > identity").map { |n| n['mimetype'].split(',').first }
+    end
+
+    # t.exif_tool_version(proxy: [:identification, :identity, :tool, :exif_tool_version])
+    def exif_tool_version
+      ng_xml.css("fits > identification > identity > tool[toolname='Exiftool']").map { |n| n['toolversion'] }
+    end
+
+    # t.file_size(proxy: [:fileinfo, :file_size])
+    def file_size
+      ng_xml.css("fits > fileinfo > size").map(&:text)
+    end
+
+    # t.date_modified(proxy: [:fileinfo, :last_modified])
+    def date_modified
+      ng_xml.css("fits > fileinfo > lastmodified[toolname='Exiftool']").map(&:text)
+    end
+
+    #  t.filename(proxy: [:fileinfo, :filename])
+    def filename
+      ng_xml.css("fits > fileinfo > filename").map(&:text)
+    end
+
+    # t.original_checksum(proxy: [:fileinfo, :original_checksum])
+    def original_checksum
+      ng_xml.css("fits > fileinfo > md5checksum").map(&:text)
+    end
+
+    # t.date_created(proxy: [:fileinfo, :date_created])
+    def date_created
+      ng_xml.css("fits > fileinfo > created[toolname='Exiftool']").map(&:text)
+    end
+
+    # t.rights_basis(proxy: [:fileinfo, :rights_basis])
+    def rights_basis
+      ng_xml.css("fits > fileinfo > rightsBasis").map(&:text)
+    end
+
+    # t.copyright_basis(proxy: [:fileinfo, :copyright_basis])
+    def copyright_basis
+      ng_xml.css("fits > fileinfo > copyrightBasis").map(&:text)
+    end
+
+    # t.copyright_basis(proxy: [:fileinfo, :copyright_note])
+    def copyright_note
+      ng_xml.css("fits > fileinfo > copyrightNote").map(&:text)
+    end
+
+    # t.well_formed(proxy: [:filestatus, :well_formed])
+    def well_formed
+      ng_xml.css("fits > filestatus > well-formed").map(&:text)
+    end
+
+    # t.valid(proxy: [:filestatus, :valid])
+    def valid
+      ng_xml.css("fits > filestatus > valid").map(&:text)
+    end
+
+    # t.filestatus_message(proxy: [:filestatus, :status_message])
+    def filestatus_message
+      ng_xml.css("fits > filestatus > message").map(&:text)
+    end
+
+    # t.file_title(proxy: [:metadata, :document, :file_title])
+    def file_title
+      ng_xml.css("fits > metadata > document > title").map(&:text)
+    end
+
+    # t.file_author(proxy: [:metadata, :document, :file_author])
+    def file_author
+      ng_xml.css("fits > metadata > document > author").map(&:text)
+    end
+
+    # t.file_language(proxy: [:metadata, :document, :file_language])
+    def file_language
+      ng_xml.css("fits > metadata > document > language").map(&:text)
+    end
+
+    # t.page_count(proxy: [:metadata, :document, :page_count])
+    def page_count
+      ng_xml.css("fits > metadata > document > pageCount").map(&:text)
+    end
+
+    # t.word_count(proxy: [:metadata, :document, :word_count])
+    def word_count
+      ng_xml.css("fits > metadata > document > wordCount").map(&:text)
+    end
+
+    # t.character_count(proxy: [:metadata, :document, :character_count])
+    def character_count
+      ng_xml.css("fits > metadata > document > characterCount").map(&:text)
+    end
+
+    # t.paragraph_count(proxy: [:metadata, :document, :paragraph_count])
+    def paragraph_count
+      ng_xml.css("fits > metadata > document > paragraphCount").map(&:text)
+    end
+
+    # t.line_count(proxy: [:metadata, :document, :line_count])
+    def line_count
+      ng_xml.css("fits > metadata > document > lineCount").map(&:text)
+    end
+
+    # t.table_count(proxy: [:metadata, :document, :table_count])
+    def table_count
+      ng_xml.css("fits > metadata > document > tableCount").map(&:text)
+    end
+
+    # t.graphics_count(proxy: [:metadata, :document, :graphics_count])
+    def graphics_count
+      ng_xml.css("fits > metadata > document > graphicsCount").map(&:text)
+    end
+
+    # t.byte_order(proxy: [:metadata, :image, :byte_order])
+    def byte_order
+      ng_xml.css("fits > metadata > image > byteOrder").map(&:text)
+    end
+
+    # t.compression(proxy: [:metadata, :image, :compression])
+    def compression
+      ng_xml.css("fits > metadata > image > compressionScheme").map(&:text)
+    end
+
+    # t.width(proxy: [:metadata, :image, :width])
+    def width
+      ng_xml.css("fits > metadata > image > imageWidth").map(&:text)
+    end
+
+    # t.height(proxy: [:metadata, :image, :height])
+    def height
+      ng_xml.css("fits > metadata > image > imageHeight").map(&:text)
+    end
+
+    # t.video_width(proxy: [:metadata, :video, :width])
+    def video_width
+      ng_xml.css("fits > metadata > video > imageWidth").map(&:text)
+    end
+
+    # t.video_height(proxy: [:metadata, :video, :height])
+    def video_height
+      ng_xml.css("fits > metadata > video > imageHeight").map(&:text)
+    end
+
+    # for fits 1
+    # t.video_track_width(proxy: [:metadata, :video, :track, :width])
+    def video_track_width
+      ng_xml.css("fits > metadata > video > track[type='video'] > width").map(&:text)
+    end
+
+    # for fits 1
+    # t.video_track_height(proxy: [:metadata, :video, :track, :height])
+    def video_track_height
+      ng_xml.css("fits > metadata > video > track[type='video'] > height").map(&:text)
+    end
+
+    #  t.color_space(proxy: [:metadata, :image, :color_space])
+    def color_space
+      ng_xml.css("fits > metadata > image > colorSpace").map(&:text)
+    end
+
+    # t.profile_name(proxy: [:metadata, :image, :profile_name])
+    def profile_name
+      ng_xml.css("fits > metadata > image > iccProfileName").map(&:text)
+    end
+
+    # t.profile_version(proxy: [:metadata, :image, :profile_version])
+    def profile_version
+      ng_xml.css("fits > metadata > image > iccProfileVersion").map(&:text)
+    end
+
+    # t.orientation(proxy: [:metadata, :image, :orientation])
+    def orientation
+      ng_xml.css("fits > metadata > image > orientation").map(&:text)
+    end
+
+    # t.color_map(proxy: [:metadata, :image, :color_map])
+    def color_map
+      ng_xml.css("fits > metadata > image > colorMap").map(&:text)
+    end
+
+    # t.image_producer(proxy: [:metadata, :image, :image_producer])
+    def image_producer
+      ng_xml.css("fits > metadata > image > imageProducer").map(&:text)
+    end
+
+    # t.capture_device(proxy: [:metadata, :image, :capture_device])
+    def capture_device
+      ng_xml.css("fits > metadata > image > captureDevice").map(&:text)
+    end
+
+    # t.scanning_software(proxy: [:metadata, :image, :scanning_software])
+    def scanning_software
+      ng_xml.css("fits > metadata > image > scanningSoftwareName").map(&:text)
+    end
+
+    # t.exif_version(proxy: [:metadata, :image, :exif_version])
+    def exif_version
+      ng_xml.css("fits > metadata > image > exifVersion[toolname='Exiftool']").map(&:text)
+    end
+
+    # t.gps_timestamp(proxy: [:metadata, :image, :gps_timestamp])
+    def gps_timestamp
+      ng_xml.css("fits > metadata > image > gpsTimeStamp").map(&:text)
+    end
+
+    # t.latitude(proxy: [:metadata, :image, :latitude])
+    def latitude
+      ng_xml.css("fits > metadata > image > gpsDestLatitude").map(&:text)
+    end
+
+    # t.longitude(proxy: [:metadata, :image, :longitude])
+    def longitude
+      ng_xml.css("fits > metadata > image > gpsDestLongitude").map(&:text)
+    end
+
+    # t.character_set(proxy: [:metadata, :text, :character_set])
+    def character_set
+      ng_xml.css("fits > metadata > text > charset").map(&:text)
+    end
+
+    # t.markup_basis(proxy: [:metadata, :text, :markup_basis])
+    def markup_basis
+      ng_xml.css("fits > metadata > text > markupBasis").map(&:text)
+    end
+
+    # t.markup_language(proxy: [:metadata, :text, :markup_language])
+    def markup_language
+      ng_xml.css("fits > metadata > text > markupLanguage").map(&:text)
+    end
+
+    # t.audio_duration(proxy: [:metadata, :audio, :duration])
+    def audio_duration
+      ng_xml.css("fits > metadata > audio > duration").map(&:text)
+    end
+
+    # t.bit_depth(proxy: [:metadata, :audio, :bit_depth])
+    def bit_depth
+      ng_xml.css("fits > metadata > audio > bitDepth").map(&:text)
+    end
+
+    # t.audio_sample_rate(proxy: [:metadata, :audio, :sample_rate])
+    def audio_sample_rate
+      ng_xml.css("fits > metadata > audio > sampleRate").map(&:text)
+    end
+
+    # t.channels(proxy: [:metadata, :audio, :channels])
+    def channels
+      ng_xml.css("fits > metadata > audio > channels").map(&:text)
+    end
+
+    # t.data_format(proxy: [:metadata, :audio, :data_format])
+    def data_format
+      ng_xml.css("fits > metadata > audio > dataFormatType").map(&:text)
+    end
+
+    # t.offset(proxy: [:metadata, :audio, :offset])
+    def offset
+      ng_xml.css("fits > metadata > audio > offset").map(&:text)
+    end
+
+    # t.audio_bit_rate(proxy: [:metadata, :audio, :bit_rate])
+    def audio_bit_rate
+      ng_xml.css("fits > metadata > audio > bitRate").map(&:text)
+    end
+
+    # t.video_duration(proxy: [:metadata, :video, :duration])
+    def video_duration
+      ng_xml.css("fits > metadata > video > duration").map(&:text)
+    end
+
+    # t.video_sample_rate(proxy: [:metadata, :video, :sample_rate])
+    def video_sample_rate
+      ng_xml.css("fits > metadata > video > sampleRate").map(&:text)
+    end
+
+    # t.video_audio_sample_rate(proxy: [:metadata, :video, :audio_sample_rate])
+    def video_audio_sample_rate
+      ng_xml.css("fits > metadata > video > audioSampleRate").map(&:text)
+    end
+
+    # t.frame_rate(proxy: [:metadata, :video, :frame_rate])
+    def frame_rate
+      ng_xml.css("fits > metadata > video > frameRate").map(&:text)
+    end
+
+    # t.video_bit_rate(proxy: [:metadata, :video, :bit_rate])
+    def video_bit_rate
+      ng_xml.css("fits > metadata > video > bitRate").map(&:text)
+    end
+
+    # t.track_frame_rate(proxy: [:metadata, :video, :track, :frame_rate])
+    def track_frame_rate
+      ng_xml.css("fits > metadata > video > track[type='video'] > frameRate").map(&:text)
+    end
+
+    # t.aspect_ratio(proxy: [:metadata, :video, :track, :aspect_ratio])
+    def aspect_ratio
+      ng_xml.css("fits > metadata > video > track[type='video'] > aspectRatio").map(&:text)
+    end
+
+    # Cleanup phase; ugly name to avoid collisions.
+    # The send construct here is required to fix up values because the setters
+    # are not defined, but rather applied with method_missing.
+    def __cleanup__
+      # Add any other scrubbers here; don't return any particular value
+      nil
     end
 
     def self.xml_template