samvera-nesting_indexer 1.0.0 → 1.0.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/.github/CODE_OF_CONDUCT.md +36 -0
- data/.github/CONTRIBUTING.md +169 -0
- data/.github/ISSUE_TEMPLATE.md +20 -0
- data/.github/PULL_REQUEST_TEMPLATE.md +26 -0
- data/.github/SUPPORT.md +3 -0
- data/.rubocop.yml +1 -0
- data/LICENSE +203 -14
- data/README.md +43 -13
- data/lib/generators/samvera/nesting_indexer/install_generator.rb +41 -0
- data/lib/generators/samvera/nesting_indexer/templates/README +15 -0
- data/lib/generators/samvera/nesting_indexer/templates/samvera-nesting_indexer_initializer.rb +24 -0
- data/lib/samvera/nesting_indexer.rb +8 -1
- data/lib/samvera/nesting_indexer/adapters/in_memory_adapter.rb +8 -0
- data/lib/samvera/nesting_indexer/configuration.rb +1 -2
- data/lib/samvera/nesting_indexer/documents.rb +11 -1
- data/lib/samvera/nesting_indexer/relationship_reindexer.rb +1 -2
- data/lib/samvera/nesting_indexer/repository_reindexer.rb +1 -2
- data/lib/samvera/nesting_indexer/semver_assistant.rb +19 -0
- data/lib/samvera/nesting_indexer/version.rb +1 -1
- data/samvera-nesting_indexer.gemspec +10 -11
- metadata +49 -40
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: cda0810a46d78a31bbf3dfe29b0bc43f184657c0
|
4
|
+
data.tar.gz: c0d79f4628040a28de9735b358223d6a648759cf
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 3aceaba25afffb5c2f7776e7d0df1f6fc6bcd7de1b23b6acad1cbd2413f0f0decb7192b8cc15fc0e0e1c264bf8c9e3b2429544ce5f60aaaa162608f1d6df992c
|
7
|
+
data.tar.gz: 66ae2cdd1470fef272535138656a672c625de02419a992bee823fb7342a5ecdd97c421f7a585167023c5881afb569c9f4a574917a296f5ba009cdf5e73e04d0b
|
@@ -0,0 +1,36 @@
|
|
1
|
+
The Samvera community is dedicated to providing a welcoming and
|
2
|
+
positive experience for all its members, whether they are at a formal
|
3
|
+
gathering, in a social setting, or taking part in activities online.
|
4
|
+
The Samvera community welcomes participation from people all over the
|
5
|
+
world and these members bring with them a wide variety of
|
6
|
+
professional, personal and social backgrounds; whatever these may be,
|
7
|
+
we treat colleagues with dignity and respect.
|
8
|
+
|
9
|
+
Community members communicate primarily in English, though for many of
|
10
|
+
them this is not their native language. We therefore strive to express
|
11
|
+
ourselves simply and clearly remembering that unnecessary use of
|
12
|
+
jargon and slang will be a barrier to understanding for many of our
|
13
|
+
colleagues. We are sensitive to the fact that the international
|
14
|
+
nature of the community means that we span many different social norms
|
15
|
+
around language and behaviour and we strive to conduct ourselves,
|
16
|
+
online and in person, in ways that are unlikely to cause offence.
|
17
|
+
|
18
|
+
Samvera conversations are often information-rich and intended to
|
19
|
+
generate discussion and debate. We discuss ideas from a standpoint of
|
20
|
+
mutual respect and reasoned argument.
|
21
|
+
|
22
|
+
Community members work together to promote a respectful and safe
|
23
|
+
community. In the event that someone’s conduct is causing offence or
|
24
|
+
distress, Samvera has a detailed
|
25
|
+
[Anti-Harassment Policy and Protocol](https://wiki.duraspace.org/display/hydra/Anti-Harassment+Policy)
|
26
|
+
which can be applied to address the problem. The first step in dealing
|
27
|
+
with any serious misconduct is to contact a local meeting organizer,
|
28
|
+
the
|
29
|
+
[Samvera community helpers](https://wiki.duraspace.org/display/hydra/Hydra+Community+Helpers)
|
30
|
+
([email](mailto:helpers@projecthydra.org)), a community member you
|
31
|
+
trust, or the
|
32
|
+
[Samvera Steering Group](https://wiki.duraspace.org/display/hydra/Samvera+Steering+Group+membership)
|
33
|
+
immediately; at Samvera events, these people can be identified by
|
34
|
+
distinctive name badges. The
|
35
|
+
[Policy and Protocol](https://wiki.duraspace.org/display/hydra/Anti-Harassment+Policy)
|
36
|
+
should be consulted for fuller details.
|
@@ -0,0 +1,169 @@
|
|
1
|
+
# How to Contribute
|
2
|
+
|
3
|
+
We want your help to make Samvera great. There are a few guidelines
|
4
|
+
that we need contributors to follow so that we can have a chance of
|
5
|
+
keeping on top of things.
|
6
|
+
|
7
|
+
## Code of Conduct
|
8
|
+
|
9
|
+
The Samvera Community is dedicated to providing a welcoming and positive
|
10
|
+
experience for all its members, whether they are at a formal gathering, in
|
11
|
+
a social setting, or taking part in activities online. Please see our
|
12
|
+
[Code of Conduct](CODE_OF_CONDUCT.md) for more information.
|
13
|
+
|
14
|
+
## Samvera Community Intellectual Property Licensing and Ownership
|
15
|
+
|
16
|
+
All code contributors must have an Individual Contributor License Agreement
|
17
|
+
(iCLA) on file with the Samvera Steering Group. If the contributor works for
|
18
|
+
an institution, the institution must have a Corporate Contributor License
|
19
|
+
Agreement (cCLA) on file.
|
20
|
+
|
21
|
+
https://wiki.duraspace.org/display/hydra/Hydra+Project+Intellectual+Property+Licensing+and+Ownership
|
22
|
+
|
23
|
+
You should also add yourself to the `CONTRIBUTORS.md` file in the root of the project.
|
24
|
+
|
25
|
+
## Contribution Tasks
|
26
|
+
|
27
|
+
* Reporting Issues
|
28
|
+
* Making Changes
|
29
|
+
* Documenting Code
|
30
|
+
* Committing Changes
|
31
|
+
* Submitting Changes
|
32
|
+
* Reviewing and Merging Changes
|
33
|
+
|
34
|
+
### Reporting Issues
|
35
|
+
|
36
|
+
* Make sure you have a [GitHub account](https://github.com/signup/free)
|
37
|
+
* Submit a [Github issue](./issues) by:
|
38
|
+
* Clearly describing the issue
|
39
|
+
* Provide a descriptive summary
|
40
|
+
* Explain the expected behavior
|
41
|
+
* Explain the actual behavior
|
42
|
+
* Provide steps to reproduce the actual behavior
|
43
|
+
|
44
|
+
### Making Changes
|
45
|
+
|
46
|
+
* Fork the repository on GitHub
|
47
|
+
* Create a topic branch from where you want to base your work.
|
48
|
+
* This is usually the master branch.
|
49
|
+
* To quickly create a topic branch based on master; `git branch fix/master/my_contribution master`
|
50
|
+
* Then checkout the new branch with `git checkout fix/master/my_contribution`.
|
51
|
+
* Please avoid working directly on the `master` branch.
|
52
|
+
* You may find the [hub suite of commands](https://github.com/defunkt/hub) helpful
|
53
|
+
* Make sure you have added sufficient tests and documentation for your changes.
|
54
|
+
* Test functionality with RSpec; Test features / UI with Capybara.
|
55
|
+
* Run _all_ the tests to assure nothing else was accidentally broken.
|
56
|
+
|
57
|
+
### Documenting Code
|
58
|
+
|
59
|
+
* All new public methods, modules, and classes should include inline documentation in [YARD](http://yardoc.org/).
|
60
|
+
* Documentation should seek to answer the question "why does this code exist?"
|
61
|
+
* Document private / protected methods as desired.
|
62
|
+
* If you are working in a file with no prior documentation, do try to document as you gain understanding of the code.
|
63
|
+
* 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.
|
64
|
+
* This work greatly increases the usability of the code base and supports the on-ramping of new committers.
|
65
|
+
* We will all be understanding of one another's time constraints in this area.
|
66
|
+
* YARD examples:
|
67
|
+
* [Hydra::Works::RemoveGenericFile](https://github.com/projecthydra-labs/hydra-works/blob/master/lib/hydra/works/services/generic_work/remove_generic_file.rb)
|
68
|
+
* [ActiveTriples::LocalName::Minter](https://github.com/ActiveTriples/active_triples-local_name/blob/master/lib/active_triples/local_name/minter.rb)
|
69
|
+
* [Getting started with YARD](http://www.rubydoc.info/gems/yard/file/docs/GettingStarted.md)
|
70
|
+
|
71
|
+
### Committing changes
|
72
|
+
|
73
|
+
* Make commits of logical units.
|
74
|
+
* Your commit should include a high level description of your work in HISTORY.textile
|
75
|
+
* Check for unnecessary whitespace with `git diff --check` before committing.
|
76
|
+
* Make sure your commit messages are [well formed](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).
|
77
|
+
* 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)
|
78
|
+
|
79
|
+
```
|
80
|
+
Present tense short summary (50 characters or less)
|
81
|
+
|
82
|
+
More detailed description, if necessary. It should be wrapped to 72
|
83
|
+
characters. Try to be as descriptive as you can, even if you think that
|
84
|
+
the commit content is obvious, it may not be obvious to others. You
|
85
|
+
should add such description also if it's already present in bug tracker,
|
86
|
+
it should not be necessary to visit a webpage to check the history.
|
87
|
+
|
88
|
+
Include Closes #<issue-number> when relavent.
|
89
|
+
|
90
|
+
Description can have multiple paragraphs and you can use code examples
|
91
|
+
inside, just indent it with 4 spaces:
|
92
|
+
|
93
|
+
class PostsController
|
94
|
+
def index
|
95
|
+
respond_to do |wants|
|
96
|
+
wants.html { render 'index' }
|
97
|
+
end
|
98
|
+
end
|
99
|
+
end
|
100
|
+
|
101
|
+
You can also add bullet points:
|
102
|
+
|
103
|
+
- you can use dashes or asterisks
|
104
|
+
|
105
|
+
- also, try to indent next line of a point for readability, if it's too
|
106
|
+
long to fit in 72 characters
|
107
|
+
```
|
108
|
+
|
109
|
+
* Make sure you have added the necessary tests for your changes.
|
110
|
+
* Run _all_ the tests to assure nothing else was accidentally broken.
|
111
|
+
* When you are ready to submit a pull request
|
112
|
+
|
113
|
+
### Submitting Changes
|
114
|
+
|
115
|
+
[Detailed Walkthrough of One Pull Request per Commit](http://ndlib.github.io/practices/one-commit-per-pull-request/)
|
116
|
+
|
117
|
+
* Read the article ["Using Pull Requests"](https://help.github.com/articles/using-pull-requests) on GitHub.
|
118
|
+
* Make sure your branch is up to date with its parent branch (i.e. master)
|
119
|
+
* `git checkout master`
|
120
|
+
* `git pull --rebase`
|
121
|
+
* `git checkout <your-branch>`
|
122
|
+
* `git rebase master`
|
123
|
+
* It is a good idea to run your tests again.
|
124
|
+
* If you've made more than one commit take a moment to consider whether squashing commits together would help improve their logical grouping.
|
125
|
+
* [Detailed Walkthrough of One Pull Request per Commit](http://ndlib.github.io/practices/one-commit-per-pull-request/)
|
126
|
+
* `git rebase --interactive master` ([See Github help](https://help.github.com/articles/interactive-rebase))
|
127
|
+
* Squashing your branch's changes into one commit is "good form" and helps the person merging your request to see everything that is going on.
|
128
|
+
* Push your changes to a topic branch in your fork of the repository.
|
129
|
+
* Submit a pull request from your fork to the project.
|
130
|
+
|
131
|
+
### Reviewing and Merging Changes
|
132
|
+
|
133
|
+
We adopted [Github's Pull Request Review](https://help.github.com/articles/about-pull-request-reviews/) for our repositories.
|
134
|
+
Common checks that may occur in our repositories:
|
135
|
+
|
136
|
+
1. Travis CI - where our automated tests are running
|
137
|
+
2. Hound CI - where we check for style violations
|
138
|
+
3. Approval Required - Github enforces at least one person approve a pull request. Also, all reviewers that have chimed in must approve.
|
139
|
+
4. CodeClimate - is our code remaining healthy (at least according to static code analysis)
|
140
|
+
|
141
|
+
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.
|
142
|
+
|
143
|
+
*Example: Carolyn submits a pull request, Justin reviews the pull request and approves. However, Justin is still waiting on other checks (Travis CI is usually the culprit), so he does not merge the pull request. Eventually, all of the checks pass. At this point, Carolyn or anyone else may merge the pull request.*
|
144
|
+
|
145
|
+
#### Things to Consider When Reviewing
|
146
|
+
|
147
|
+
First, the person contributing the code is putting themselves out there. Be mindful of what you say in a review.
|
148
|
+
|
149
|
+
* Ask clarifying questions
|
150
|
+
* State your understanding and expectations
|
151
|
+
* Provide example code or alternate solutions, and explain why
|
152
|
+
|
153
|
+
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:
|
154
|
+
|
155
|
+
* Does the commit message explain what is going on?
|
156
|
+
* Does the code changes have tests? _Not all changes need new tests, some changes are refactorings_
|
157
|
+
* Do new or changed methods, modules, and classes have documentation?
|
158
|
+
* Does the commit contain more than it should? Are two separate concerns being addressed in one commit?
|
159
|
+
* Does the description of the new/changed specs match your understanding of what the spec is doing?
|
160
|
+
* Did the Travis tests complete successfully?
|
161
|
+
|
162
|
+
If you are uncertain, bring other contributors into the conversation by assigning them as a reviewer.
|
163
|
+
|
164
|
+
# Additional Resources
|
165
|
+
|
166
|
+
* [General GitHub documentation](http://help.github.com/)
|
167
|
+
* [GitHub pull request documentation](http://help.github.com/send-pull-requests/)
|
168
|
+
* [Pro Git](http://git-scm.com/book) is both a free and excellent book about Git.
|
169
|
+
* [A Git Config for Contributing](http://ndlib.github.io/practices/my-typical-per-project-git-config/)
|
@@ -0,0 +1,20 @@
|
|
1
|
+
### Descriptive summary
|
2
|
+
|
3
|
+
Include what version of Hyrax relates to this issue (7.x, 6.x, HEAD, etc.) if appropriate, and any relevant tracebacks if you're reporting a bug.
|
4
|
+
|
5
|
+
### Rationale
|
6
|
+
|
7
|
+
Provide the rationale or user story that describes "why" this issue should be addressed. Especially if this is a new feature or significant change to the existing implementation.
|
8
|
+
|
9
|
+
### Expected behavior
|
10
|
+
|
11
|
+
### Actual behavior
|
12
|
+
|
13
|
+
### Steps to reproduce the behavior
|
14
|
+
|
15
|
+
1. Do this
|
16
|
+
1. Then do this...
|
17
|
+
|
18
|
+
### Related work
|
19
|
+
|
20
|
+
Link to related tickets or prior related work here.
|
@@ -0,0 +1,26 @@
|
|
1
|
+
Present tense short summary (50 characters or less)
|
2
|
+
|
3
|
+
Fixes #issuenumber ; refs #issuenumber
|
4
|
+
|
5
|
+
More detailed description, if necessary. Try to be as descriptive as
|
6
|
+
you can: even if you think that the PR content is obvious, it may not
|
7
|
+
be obvious to others. Include tracebacks if helpful, and be sure to
|
8
|
+
call out any bits of the PR that may be work-in-progress.
|
9
|
+
|
10
|
+
Description can have multiple paragraphs and you can use code examples
|
11
|
+
inside:
|
12
|
+
|
13
|
+
``` ruby
|
14
|
+
class PostsController
|
15
|
+
def index
|
16
|
+
respond_with Post.limit(10)
|
17
|
+
end
|
18
|
+
end
|
19
|
+
```
|
20
|
+
|
21
|
+
Changes proposed in this pull request:
|
22
|
+
*
|
23
|
+
*
|
24
|
+
*
|
25
|
+
|
26
|
+
@samvera/hyrax-code-reviewers
|
data/.github/SUPPORT.md
ADDED
@@ -0,0 +1,3 @@
|
|
1
|
+
If you would like to report an issue with Hyrax, first do search [the list of issues](https://github.com/samvera/samvera-nesting_indexer/issues/) to see if someone else has already reported it, and then feel free to [create a new issue](https://github.com/samvera/samvera-nesting_indexer/issues/new).
|
2
|
+
|
3
|
+
If you have questions or need help, please email [the Samvera community tech list](mailto:samvera-tech@googlegroups.com) or stop by the #dev channel in [the Samvera community Slack team](https://wiki.duraspace.org/pages/viewpage.action?pageId=87460391#Getintouch!-Slack).
|
data/.rubocop.yml
CHANGED
data/LICENSE
CHANGED
@@ -1,14 +1,203 @@
|
|
1
|
-
|
2
|
-
|
3
|
-
|
4
|
-
|
5
|
-
|
6
|
-
|
7
|
-
|
8
|
-
|
9
|
-
|
10
|
-
|
11
|
-
|
12
|
-
|
13
|
-
|
14
|
-
|
1
|
+
Apache License
|
2
|
+
Version 2.0, January 2004
|
3
|
+
http://www.apache.org/licenses/
|
4
|
+
|
5
|
+
TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
|
6
|
+
|
7
|
+
1. Definitions.
|
8
|
+
|
9
|
+
"License" shall mean the terms and conditions for use, reproduction,
|
10
|
+
and distribution as defined by Sections 1 through 9 of this document.
|
11
|
+
|
12
|
+
"Licensor" shall mean the copyright owner or entity authorized by
|
13
|
+
the copyright owner that is granting the License.
|
14
|
+
|
15
|
+
"Legal Entity" shall mean the union of the acting entity and all
|
16
|
+
other entities that control, are controlled by, or are under common
|
17
|
+
control with that entity. For the purposes of this definition,
|
18
|
+
"control" means (i) the power, direct or indirect, to cause the
|
19
|
+
direction or management of such entity, whether by contract or
|
20
|
+
otherwise, or (ii) ownership of fifty percent (50%) or more of the
|
21
|
+
outstanding shares, or (iii) beneficial ownership of such entity.
|
22
|
+
|
23
|
+
"You" (or "Your") shall mean an individual or Legal Entity
|
24
|
+
exercising permissions granted by this License.
|
25
|
+
|
26
|
+
"Source" form shall mean the preferred form for making modifications,
|
27
|
+
including but not limited to software source code, documentation
|
28
|
+
source, and configuration files.
|
29
|
+
|
30
|
+
"Object" form shall mean any form resulting from mechanical
|
31
|
+
transformation or translation of a Source form, including but
|
32
|
+
not limited to compiled object code, generated documentation,
|
33
|
+
and conversions to other media types.
|
34
|
+
|
35
|
+
"Work" shall mean the work of authorship, whether in Source or
|
36
|
+
Object form, made available under the License, as indicated by a
|
37
|
+
copyright notice that is included in or attached to the work
|
38
|
+
(an example is provided in the Appendix below).
|
39
|
+
|
40
|
+
"Derivative Works" shall mean any work, whether in Source or Object
|
41
|
+
form, that is based on (or derived from) the Work and for which the
|
42
|
+
editorial revisions, annotations, elaborations, or other modifications
|
43
|
+
represent, as a whole, an original work of authorship. For the purposes
|
44
|
+
of this License, Derivative Works shall not include works that remain
|
45
|
+
separable from, or merely link (or bind by name) to the interfaces of,
|
46
|
+
the Work and Derivative Works thereof.
|
47
|
+
|
48
|
+
"Contribution" shall mean any work of authorship, including
|
49
|
+
the original version of the Work and any modifications or additions
|
50
|
+
to that Work or Derivative Works thereof, that is intentionally
|
51
|
+
submitted to Licensor for inclusion in the Work by the copyright owner
|
52
|
+
or by an individual or Legal Entity authorized to submit on behalf of
|
53
|
+
the copyright owner. For the purposes of this definition, "submitted"
|
54
|
+
means any form of electronic, verbal, or written communication sent
|
55
|
+
to the Licensor or its representatives, including but not limited to
|
56
|
+
communication on electronic mailing lists, source code control systems,
|
57
|
+
and issue tracking systems that are managed by, or on behalf of, the
|
58
|
+
Licensor for the purpose of discussing and improving the Work, but
|
59
|
+
excluding communication that is conspicuously marked or otherwise
|
60
|
+
designated in writing by the copyright owner as "Not a Contribution."
|
61
|
+
|
62
|
+
"Contributor" shall mean Licensor and any individual or Legal Entity
|
63
|
+
on behalf of whom a Contribution has been received by Licensor and
|
64
|
+
subsequently incorporated within the Work.
|
65
|
+
|
66
|
+
2. Grant of Copyright License. Subject to the terms and conditions of
|
67
|
+
this License, each Contributor hereby grants to You a perpetual,
|
68
|
+
worldwide, non-exclusive, no-charge, royalty-free, irrevocable
|
69
|
+
copyright license to reproduce, prepare Derivative Works of,
|
70
|
+
publicly display, publicly perform, sublicense, and distribute the
|
71
|
+
Work and such Derivative Works in Source or Object form.
|
72
|
+
|
73
|
+
3. Grant of Patent License. Subject to the terms and conditions of
|
74
|
+
this License, each Contributor hereby grants to You a perpetual,
|
75
|
+
worldwide, non-exclusive, no-charge, royalty-free, irrevocable
|
76
|
+
(except as stated in this section) patent license to make, have made,
|
77
|
+
use, offer to sell, sell, import, and otherwise transfer the Work,
|
78
|
+
where such license applies only to those patent claims licensable
|
79
|
+
by such Contributor that are necessarily infringed by their
|
80
|
+
Contribution(s) alone or by combination of their Contribution(s)
|
81
|
+
with the Work to which such Contribution(s) was submitted. If You
|
82
|
+
institute patent litigation against any entity (including a
|
83
|
+
cross-claim or counterclaim in a lawsuit) alleging that the Work
|
84
|
+
or a Contribution incorporated within the Work constitutes direct
|
85
|
+
or contributory patent infringement, then any patent licenses
|
86
|
+
granted to You under this License for that Work shall terminate
|
87
|
+
as of the date such litigation is filed.
|
88
|
+
|
89
|
+
4. Redistribution. You may reproduce and distribute copies of the
|
90
|
+
Work or Derivative Works thereof in any medium, with or without
|
91
|
+
modifications, and in Source or Object form, provided that You
|
92
|
+
meet the following conditions:
|
93
|
+
|
94
|
+
(a) You must give any other recipients of the Work or
|
95
|
+
Derivative Works a copy of this License; and
|
96
|
+
|
97
|
+
(b) You must cause any modified files to carry prominent notices
|
98
|
+
stating that You changed the files; and
|
99
|
+
|
100
|
+
(c) You must retain, in the Source form of any Derivative Works
|
101
|
+
that You distribute, all copyright, patent, trademark, and
|
102
|
+
attribution notices from the Source form of the Work,
|
103
|
+
excluding those notices that do not pertain to any part of
|
104
|
+
the Derivative Works; and
|
105
|
+
|
106
|
+
(d) If the Work includes a "NOTICE" text file as part of its
|
107
|
+
distribution, then any Derivative Works that You distribute must
|
108
|
+
include a readable copy of the attribution notices contained
|
109
|
+
within such NOTICE file, excluding those notices that do not
|
110
|
+
pertain to any part of the Derivative Works, in at least one
|
111
|
+
of the following places: within a NOTICE text file distributed
|
112
|
+
as part of the Derivative Works; within the Source form or
|
113
|
+
documentation, if provided along with the Derivative Works; or,
|
114
|
+
within a display generated by the Derivative Works, if and
|
115
|
+
wherever such third-party notices normally appear. The contents
|
116
|
+
of the NOTICE file are for informational purposes only and
|
117
|
+
do not modify the License. You may add Your own attribution
|
118
|
+
notices within Derivative Works that You distribute, alongside
|
119
|
+
or as an addendum to the NOTICE text from the Work, provided
|
120
|
+
that such additional attribution notices cannot be construed
|
121
|
+
as modifying the License.
|
122
|
+
|
123
|
+
You may add Your own copyright statement to Your modifications and
|
124
|
+
may provide additional or different license terms and conditions
|
125
|
+
for use, reproduction, or distribution of Your modifications, or
|
126
|
+
for any such Derivative Works as a whole, provided Your use,
|
127
|
+
reproduction, and distribution of the Work otherwise complies with
|
128
|
+
the conditions stated in this License.
|
129
|
+
|
130
|
+
5. Submission of Contributions. Unless You explicitly state otherwise,
|
131
|
+
any Contribution intentionally submitted for inclusion in the Work
|
132
|
+
by You to the Licensor shall be under the terms and conditions of
|
133
|
+
this License, without any additional terms or conditions.
|
134
|
+
Notwithstanding the above, nothing herein shall supersede or modify
|
135
|
+
the terms of any separate license agreement you may have executed
|
136
|
+
with Licensor regarding such Contributions.
|
137
|
+
|
138
|
+
6. Trademarks. This License does not grant permission to use the trade
|
139
|
+
names, trademarks, service marks, or product names of the Licensor,
|
140
|
+
except as required for reasonable and customary use in describing the
|
141
|
+
origin of the Work and reproducing the content of the NOTICE file.
|
142
|
+
|
143
|
+
7. Disclaimer of Warranty. Unless required by applicable law or
|
144
|
+
agreed to in writing, Licensor provides the Work (and each
|
145
|
+
Contributor provides its Contributions) on an "AS IS" BASIS,
|
146
|
+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
|
147
|
+
implied, including, without limitation, any warranties or conditions
|
148
|
+
of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
|
149
|
+
PARTICULAR PURPOSE. You are solely responsible for determining the
|
150
|
+
appropriateness of using or redistributing the Work and assume any
|
151
|
+
risks associated with Your exercise of permissions under this License.
|
152
|
+
|
153
|
+
8. Limitation of Liability. In no event and under no legal theory,
|
154
|
+
whether in tort (including negligence), contract, or otherwise,
|
155
|
+
unless required by applicable law (such as deliberate and grossly
|
156
|
+
negligent acts) or agreed to in writing, shall any Contributor be
|
157
|
+
liable to You for damages, including any direct, indirect, special,
|
158
|
+
incidental, or consequential damages of any character arising as a
|
159
|
+
result of this License or out of the use or inability to use the
|
160
|
+
Work (including but not limited to damages for loss of goodwill,
|
161
|
+
work stoppage, computer failure or malfunction, or any and all
|
162
|
+
other commercial damages or losses), even if such Contributor
|
163
|
+
has been advised of the possibility of such damages.
|
164
|
+
|
165
|
+
9. Accepting Warranty or Additional Liability. While redistributing
|
166
|
+
the Work or Derivative Works thereof, You may choose to offer,
|
167
|
+
and charge a fee for, acceptance of support, warranty, indemnity,
|
168
|
+
or other liability obligations and/or rights consistent with this
|
169
|
+
License. However, in accepting such obligations, You may act only
|
170
|
+
on Your own behalf and on Your sole responsibility, not on behalf
|
171
|
+
of any other Contributor, and only if You agree to indemnify,
|
172
|
+
defend, and hold each Contributor harmless for any liability
|
173
|
+
incurred by, or claims asserted against, such Contributor by reason
|
174
|
+
of your accepting any such warranty or additional liability.
|
175
|
+
|
176
|
+
END OF TERMS AND CONDITIONS
|
177
|
+
|
178
|
+
APPENDIX: How to apply the Apache License to your work.
|
179
|
+
|
180
|
+
To apply the Apache License to your work, attach the following
|
181
|
+
boilerplate notice, with the fields enclosed by brackets "{}"
|
182
|
+
replaced with your own identifying information. (Don't include
|
183
|
+
the brackets!) The text should be enclosed in the appropriate
|
184
|
+
comment syntax for the file format. We also recommend that a
|
185
|
+
file or class name and description of purpose be included on the
|
186
|
+
same "printed page" as the copyright notice for easier
|
187
|
+
identification within third-party archives.
|
188
|
+
|
189
|
+
Copyright 2014-2017 University of Notre Dame
|
190
|
+
Additional copyright may be held by others, as reflected in the commit log
|
191
|
+
|
192
|
+
|
193
|
+
Licensed under the Apache License, Version 2.0 (the "License");
|
194
|
+
you may not use this file except in compliance with the License.
|
195
|
+
You may obtain a copy of the License at
|
196
|
+
|
197
|
+
http://www.apache.org/licenses/LICENSE-2.0
|
198
|
+
|
199
|
+
Unless required by applicable law or agreed to in writing, software
|
200
|
+
distributed under the License is distributed on an "AS IS" BASIS,
|
201
|
+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
202
|
+
See the License for the specific language governing permissions and
|
203
|
+
limitations under the License.
|
data/README.md
CHANGED
@@ -6,17 +6,19 @@
|
|
6
6
|
[![Documentation Status](http://inch-ci.org/github/samvera-labs/samvera-nesting_indexer.svg?branch=master)](http://inch-ci.org/github/samvera-labs/samvera-nesting_indexer)
|
7
7
|
[![APACHE 2 License](http://img.shields.io/badge/APACHE2-license-blue.svg)](./LICENSE)
|
8
8
|
|
9
|
-
The Samvera::NestingIndexer gem
|
9
|
+
The Samvera::NestingIndexer gem generates the graph related attributes related to querying lineage of objects. It maps a PreservationDocument (e.g. a Fedora object) to an IndexDocument (e.g. a SOLR document) by mapping a PreservationDocument's direct parents to ancestral paths for the given PreservationDocument.
|
10
10
|
|
11
11
|
* [Background](#background)
|
12
12
|
* [Concepts](#concepts)
|
13
13
|
* [Examples](#examples)
|
14
14
|
* [Adapters](#adapters)
|
15
15
|
* [Considerations](#considerations)
|
16
|
+
* [Public API for Semantic Versioning](#public_api_for_semantic_versioning)
|
17
|
+
* [Installation](#installation)
|
16
18
|
|
17
19
|
## Background
|
18
20
|
|
19
|
-
This
|
21
|
+
This gem began as a sandbox to work through the reindexing strategy for [CurateND Collections](https://github.com/ndlib/curate_nd/issues/420). This gem separates the logic of walking a graph, building the attributes necessary for nested collection maintenance, and persistence concerns. In other words, testing this gem does not require spinning up expensive persistence layers (e.g. SOLR, Fedora, or even an RDBMS).
|
20
22
|
|
21
23
|
### Notation
|
22
24
|
|
@@ -24,25 +26,25 @@ When B is a member of A, I am using the `A ={ B` notation. When C is a member of
|
|
24
26
|
|
25
27
|
## Concepts
|
26
28
|
|
27
|
-
|
29
|
+
While indexing objects, we work with two types of documents:
|
28
30
|
|
29
31
|
1. [PreservationDocument](./lib/samvera/nesting_indexer/documents.rb) - a light-weight representation of a Fedora object
|
30
32
|
2. [IndexDocument](./lib/samvera/nesting_indexer/documents.rb) - a light-weight representation of a SOLR document object
|
31
33
|
|
32
|
-
|
34
|
+
The four attributes to consider for indexing the graph:
|
33
35
|
|
34
36
|
1. id - the unique identifier for a document
|
35
|
-
2. parent_ids - the ids for
|
37
|
+
2. parent_ids - the ids for of the parents of the given document
|
36
38
|
3. pathnames - the paths to traverse from a root document to the given document
|
37
|
-
4. ancestors - the pathnames to each node
|
39
|
+
4. ancestors - the pathnames to each ancestral node of the given node (e.g. pathname to my parent, pathname to my grandparent)
|
38
40
|
|
39
41
|
See [Samvera::NestingIndexer::Documents::IndexDocument](./lib/samvera/nesting_indexer/documents.rb) for further discussion.
|
40
42
|
|
41
|
-
To reindex a single document,
|
43
|
+
To reindex a single document, use the [`Samvera::NestingIndexer.reindex_relationships`](./lib/samvera/nesting_indexer.rb) method.
|
42
44
|
|
43
|
-
To reindex all
|
45
|
+
To reindex all documents, use the [`Samvera::NestingIndexer.reindex_all!`](lib/samvera/nesting_indexer.rb) method. **Warning: This is a very slow process.**
|
44
46
|
|
45
|
-
|
47
|
+
From a node's pathname(s), you can query all of that node's descendants (both direct and indirect).
|
46
48
|
|
47
49
|
## Examples
|
48
50
|
|
@@ -57,7 +59,7 @@ Given the following PreservationDocuments:
|
|
57
59
|
| E | C |
|
58
60
|
| F | D |
|
59
61
|
|
60
|
-
|
62
|
+
When we reindex the above PreservationDocuments, then we will get the following IndexDocuments:
|
61
63
|
|
62
64
|
| PID | Parents | Pathnames | Ancestors |
|
63
65
|
|-----|---------|----------------|------------------|
|
@@ -97,6 +99,7 @@ To best ensure you have implemented the adapter to spec:
|
|
97
99
|
require 'samvera/nesting_indexer/adapters/interface_behavior_spec'
|
98
100
|
RSpec.describe MyCustomAdapter
|
99
101
|
it_behaves_like 'a Samvera::NestingIndexer::Adapter'
|
102
|
+
# Your implementation details here
|
100
103
|
end
|
101
104
|
```
|
102
105
|
|
@@ -139,10 +142,36 @@ Given an up to date index and a document, then it is valid to nest the given doc
|
|
139
142
|
|
140
143
|
For examples of determining if we can nest a document within another document, see the [demonstration of nesting](./spec/features/demonstrating_nesting_spec.rb).
|
141
144
|
|
142
|
-
In implementations, you'll likely want to write
|
145
|
+
In implementations, you'll likely want to write queries that answer:
|
143
146
|
|
144
|
-
*
|
145
|
-
*
|
147
|
+
* Find the valid IDs that I can nest within?
|
148
|
+
* Find the valid IDs in which I can nest within and am not already nested within?
|
149
|
+
|
150
|
+
## Public API for Semantic Versioning
|
151
|
+
|
152
|
+
Any method with `@api public` documentation is part of the semantically versioned API. This means you can assume:
|
153
|
+
|
154
|
+
* the method will exist with the same effective signature (perhaps with changes to defaults)
|
155
|
+
* the method can be called by upstream consumers
|
156
|
+
* deprecations will be announced prior to major version upgrades
|
157
|
+
|
158
|
+
Any module or class with `@api public` documentation is part of the semantically versioned API. This means you can assume:
|
159
|
+
|
160
|
+
* the module or class can be accessed upstream
|
161
|
+
* all public methods are assumed to be `@api public` unless otherwise stated
|
162
|
+
* deprecations will be announced prior to major version upgrades
|
163
|
+
|
164
|
+
## Installation
|
165
|
+
|
166
|
+
Add the following to your Gemfile:
|
167
|
+
|
168
|
+
```ruby
|
169
|
+
gem 'samvera-nesting_indexer'
|
170
|
+
```
|
171
|
+
|
172
|
+
Bundle up your gems (e.g. `bundle`).
|
173
|
+
|
174
|
+
Then run `rails generator install samvera:nesting_indexer:install`.
|
146
175
|
|
147
176
|
## TODO
|
148
177
|
|
@@ -150,3 +179,4 @@ In implementations, you'll likely want to write a queries that answer:
|
|
150
179
|
- [ ] Build methods to allow for fanning out the reindexing. At present, when we reindex a node and its "children", we run that entire process within a single context. Likewise, we run a single process when reindexing EVERYTHING.
|
151
180
|
- [ ] Promote from [samvera-labs](https://github.com/samvera-labs) to [samvera](https://github.com/samvera) via the [promotion process](http://samvera-labs.github.io/promotion.html).
|
152
181
|
- [ ] Write adapter method to assist in guarding against self-ancestry. We could probably expose a base adapter that has the method through use of the other adapter methods.
|
182
|
+
- [ ] Add templates to generate the configuration
|
@@ -0,0 +1,41 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
require 'rails/generators/base'
|
4
|
+
|
5
|
+
module Samvera
|
6
|
+
module NestingIndexer
|
7
|
+
module Generators
|
8
|
+
# Responsible for exposing the install generator (e.g. rails generator install amvera:nesting_indexer:install)
|
9
|
+
class InstallGenerator < Rails::Generators::Base
|
10
|
+
DEFAULT_MAXIMUM_NESTING_DEPTH = 5
|
11
|
+
source_root File.expand_path("../templates", __FILE__)
|
12
|
+
desc "Creates a Samvera::NestingIndexer initializer."
|
13
|
+
class_option(
|
14
|
+
:adapter,
|
15
|
+
aliases: '-a',
|
16
|
+
desc: "The class name of your adapter that implements the Samvera::NestingIndexer::Adapter interface",
|
17
|
+
required: false,
|
18
|
+
type: :string
|
19
|
+
)
|
20
|
+
class_option(
|
21
|
+
:depth,
|
22
|
+
aliases: '-d',
|
23
|
+
default: DEFAULT_MAXIMUM_NESTING_DEPTH,
|
24
|
+
desc: "The maximum nesting depth of documents",
|
25
|
+
required: false,
|
26
|
+
type: :numeric
|
27
|
+
)
|
28
|
+
|
29
|
+
def copy_initializer
|
30
|
+
@adapter_class_name = options.key?(:adapter) ? options[:adapter].classify : ''
|
31
|
+
@maximum_nesting_depth = options.fetch(:depth).to_i
|
32
|
+
template "samvera-nesting_indexer_initializer.rb", "config/initializers/samvera-nesting_indexer_initializer.rb"
|
33
|
+
end
|
34
|
+
|
35
|
+
def show_readme
|
36
|
+
readme "README" if behavior == :invoke
|
37
|
+
end
|
38
|
+
end
|
39
|
+
end
|
40
|
+
end
|
41
|
+
end
|
@@ -0,0 +1,15 @@
|
|
1
|
+
===============================================================================
|
2
|
+
|
3
|
+
Some setup you may need to complete:
|
4
|
+
|
5
|
+
1. Review the initializer and configurations installed in
|
6
|
+
config/initializers/samvera-nesting_indexer_initializer.rb:
|
7
|
+
|
8
|
+
a. adapter
|
9
|
+
b. maximum_nesting_depth
|
10
|
+
c. field names in the solr document
|
11
|
+
|
12
|
+
2. Ensure the defined adapter implements the Samvera::NestingIndexer::Adapter
|
13
|
+
(see ./lib/samvera/nesting_indexer/adapters/interface_behavior_spec.rb)
|
14
|
+
|
15
|
+
===============================================================================
|
@@ -0,0 +1,24 @@
|
|
1
|
+
# rubocop:disable Style/FileName
|
2
|
+
require 'samvera/nesting_indexer'
|
3
|
+
# rubocop:enable Style/FileName
|
4
|
+
|
5
|
+
Samvera::NestingIndexer.configure do |config|
|
6
|
+
# How many layers of nesting are allowed for collections
|
7
|
+
# Given a maximum_nesting_depth of 3 the following will raise an exception:
|
8
|
+
# C1 ={ C2 ={ C3 ={ C4
|
9
|
+
# (e.g. C4 is a member of C3 is a member of C2 is a member of C1)
|
10
|
+
config.maximum_nesting_depth = <%= @maximum_nesting_depth %>
|
11
|
+
|
12
|
+
# The adapter that implements the Samvera::NestingIndexer::Adapter interface
|
13
|
+
<%- if @adapter_class_name.blank? -%>
|
14
|
+
# config.adapter = Hyrax::Adapters::NestingIndexAdapter
|
15
|
+
<%- else -%>
|
16
|
+
config.adapter = <%= @adapter_class_name %>
|
17
|
+
<%- end -%>
|
18
|
+
|
19
|
+
# The field names of the index layer attributes; These are not used by Samvera::NestingIndexer but
|
20
|
+
# are certainly useful as part of implementing a Samvera::NestingIndexer::Adapter interface
|
21
|
+
# config.solr_field_name_for_storing_parent_ids = Solrizer.solr_name('nesting_collection__parent_ids', :symbol)
|
22
|
+
# config.solr_field_name_for_storing_ancestors = Solrizer.solr_name('nesting_collection__ancestors', :symbol)
|
23
|
+
# config.solr_field_name_for_storing_pathnames = Solrizer.solr_name('nesting_collection__pathnames', :symbol)
|
24
|
+
end
|
@@ -6,7 +6,10 @@ require 'samvera/nesting_indexer/documents'
|
|
6
6
|
require 'samvera/nesting_indexer/railtie' if defined?(Rails)
|
7
7
|
|
8
8
|
module Samvera
|
9
|
-
#
|
9
|
+
# @api public
|
10
|
+
#
|
11
|
+
# A container module responsible for exposing public API methods for nested indexing and the
|
12
|
+
# underlying configuration to perform that indexing.
|
10
13
|
module NestingIndexer
|
11
14
|
# @api public
|
12
15
|
# Responsible for reindexing the associated document for the given :id and the descendants of that :id.
|
@@ -86,5 +89,9 @@ module Samvera
|
|
86
89
|
@configuration_block.call(configuration)
|
87
90
|
@configuration_block = nil
|
88
91
|
end
|
92
|
+
|
93
|
+
def self.semantic_version_messages
|
94
|
+
SemverAssistant.messages
|
95
|
+
end
|
89
96
|
end
|
90
97
|
end
|
@@ -9,11 +9,13 @@ module Samvera
|
|
9
9
|
# Defines the interface for interacting with the InMemory layer. It is a reference
|
10
10
|
# implementation that is used throughout tests.
|
11
11
|
module InMemoryAdapter
|
12
|
+
SemverAssistant.removing_from_public_api(context: self, as_of: '2.0.0')
|
12
13
|
extend AbstractAdapter
|
13
14
|
# @api public
|
14
15
|
# @param id [String]
|
15
16
|
# @return Samvera::NestingIndexer::Document::PreservationDocument
|
16
17
|
def self.find_preservation_document_by(id:)
|
18
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
17
19
|
Preservation.find(id)
|
18
20
|
end
|
19
21
|
|
@@ -21,6 +23,7 @@ module Samvera
|
|
21
23
|
# @param id [String]
|
22
24
|
# @return [Samvera::NestingIndexer::Documents::IndexDocument]
|
23
25
|
def self.find_index_document_by(id:)
|
26
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
24
27
|
Index.find(id)
|
25
28
|
end
|
26
29
|
|
@@ -28,6 +31,7 @@ module Samvera
|
|
28
31
|
# @yieldparam id [String] The `id` of the preservation document
|
29
32
|
# @yieldparam parent_ids [String] The ids of the parent objects of this presevation document
|
30
33
|
def self.each_perservation_document_id_and_parent_ids(&block)
|
34
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
31
35
|
Preservation.find_each do |document|
|
32
36
|
block.call(document.id, document.parent_ids)
|
33
37
|
end
|
@@ -41,6 +45,7 @@ module Samvera
|
|
41
45
|
# @param document [Samvera::NestingIndexer::Documents::IndexDocument]
|
42
46
|
# @yield [Samvera::NestingIndexer::Documents::IndexDocument]
|
43
47
|
def self.each_child_document_of(document:, &block)
|
48
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
44
49
|
Index.each_child_document_of(document: document, &block)
|
45
50
|
end
|
46
51
|
|
@@ -50,6 +55,7 @@ module Samvera
|
|
50
55
|
# @param attributes [Hash]
|
51
56
|
# @return [Samvera::NestingIndexer::Documents::PreservationDocument]
|
52
57
|
def self.write_document_attributes_to_preservation_layer(attributes)
|
58
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
53
59
|
Preservation.write_document(attributes)
|
54
60
|
end
|
55
61
|
|
@@ -62,6 +68,7 @@ module Samvera
|
|
62
68
|
# @param deepest_nested_depth [Integer]
|
63
69
|
# @return [Hash] - the attributes written to the indexing layer
|
64
70
|
def self.write_document_attributes_to_index_layer(id:, parent_ids:, ancestors:, pathnames:, deepest_nested_depth:)
|
71
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
65
72
|
Index.write_document(id: id, parent_ids: parent_ids, ancestors: ancestors, pathnames: pathnames, deepest_nested_depth: deepest_nested_depth)
|
66
73
|
end
|
67
74
|
|
@@ -69,6 +76,7 @@ module Samvera
|
|
69
76
|
# @see README.md
|
70
77
|
# @param nesting_document [Samvera::NestingIndexer::Documents::IndexDocument]
|
71
78
|
def self.write_nesting_document_to_index_layer(nesting_document:)
|
79
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
72
80
|
Index.write_to_storage(nesting_document)
|
73
81
|
end
|
74
82
|
|
@@ -3,7 +3,6 @@ require 'samvera/nesting_indexer/exceptions'
|
|
3
3
|
require 'logger'
|
4
4
|
|
5
5
|
module Samvera
|
6
|
-
# :nodoc:
|
7
6
|
module NestingIndexer
|
8
7
|
# @api public
|
9
8
|
# Responsible for the configuration of the Samvera::NestingIndexer
|
@@ -81,6 +80,6 @@ module Samvera
|
|
81
80
|
end
|
82
81
|
end
|
83
82
|
end
|
84
|
-
private_constant :Configuration
|
85
83
|
end
|
86
84
|
end
|
85
|
+
Samvera::NestingIndexer.private_constant :Configuration
|
@@ -1,7 +1,9 @@
|
|
1
1
|
require 'dry-equalizer'
|
2
|
+
require 'samvera/nesting_indexer/semver_assistant'
|
2
3
|
|
3
4
|
module Samvera
|
4
5
|
module NestingIndexer
|
6
|
+
# @api public
|
5
7
|
module Documents
|
6
8
|
ANCESTOR_AND_PATHNAME_DELIMITER = '/'.freeze
|
7
9
|
|
@@ -14,10 +16,12 @@ module Samvera
|
|
14
16
|
@id = keywords.fetch(:id).to_s
|
15
17
|
@parent_ids = Array(keywords.fetch(:parent_ids))
|
16
18
|
end
|
19
|
+
SemverAssistant.removing_from_public_api(context: self, as_of: '2.0.0')
|
17
20
|
|
18
21
|
# @api public
|
19
22
|
# @return String The Fedora object's PID
|
20
23
|
attr_reader :id
|
24
|
+
SemverAssistant.removing_from_public_api(context: "#{self}#id", as_of: '2.0.0')
|
21
25
|
|
22
26
|
# @api public
|
23
27
|
#
|
@@ -26,6 +30,7 @@ module Samvera
|
|
26
30
|
# This does not include grandparents, great-grandparents, etc.
|
27
31
|
# @return Array<String>
|
28
32
|
attr_reader :parent_ids
|
33
|
+
SemverAssistant.removing_from_public_api(context: "#{self}#parent_id", as_of: '2.0.0')
|
29
34
|
end
|
30
35
|
|
31
36
|
# @api public
|
@@ -96,18 +101,23 @@ module Samvera
|
|
96
101
|
#
|
97
102
|
# @return Integer
|
98
103
|
def deepest_nested_depth
|
99
|
-
pathnames.map
|
104
|
+
pathnames.map do |pathname|
|
105
|
+
pathname.split(ANCESTOR_AND_PATHNAME_DELIMITER).count
|
106
|
+
end.max
|
100
107
|
end
|
101
108
|
|
102
109
|
def sorted_parent_ids
|
110
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
103
111
|
parent_ids.sort
|
104
112
|
end
|
105
113
|
|
106
114
|
def sorted_pathnames
|
115
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
107
116
|
pathnames.sort
|
108
117
|
end
|
109
118
|
|
110
119
|
def sorted_ancestors
|
120
|
+
SemverAssistant.removing_from_public_api(context: "#{self.class}##{__method__}", as_of: '2.0.0')
|
111
121
|
ancestors.sort
|
112
122
|
end
|
113
123
|
end
|
@@ -3,7 +3,6 @@ require 'forwardable'
|
|
3
3
|
require 'set'
|
4
4
|
|
5
5
|
module Samvera
|
6
|
-
# Establishing namespace
|
7
6
|
module NestingIndexer
|
8
7
|
# Responsible for reindexing the document associated with the given PID and its descendant documents
|
9
8
|
# @note There is cycle detection via the Samvera::NestingIndexer::Configuration#maximum_nesting_depth counter
|
@@ -153,6 +152,6 @@ module Samvera
|
|
153
152
|
end
|
154
153
|
private_constant :ParentAndPathAndAncestorsBuilder
|
155
154
|
end
|
156
|
-
private_constant :RelationshipReindexer
|
157
155
|
end
|
158
156
|
end
|
157
|
+
Samvera::NestingIndexer.private_constant :RelationshipReindexer
|
@@ -1,7 +1,6 @@
|
|
1
1
|
require 'samvera/nesting_indexer/exceptions'
|
2
2
|
require 'forwardable'
|
3
3
|
module Samvera
|
4
|
-
# Establishing namespace
|
5
4
|
module NestingIndexer
|
6
5
|
# Responsible for reindexing the entire repository
|
7
6
|
# @api private
|
@@ -70,6 +69,6 @@ module Samvera
|
|
70
69
|
raise Exceptions::ReindexingError.new(id, e)
|
71
70
|
end
|
72
71
|
end
|
73
|
-
private_constant :RepositoryReindexer
|
74
72
|
end
|
75
73
|
end
|
74
|
+
Samvera::NestingIndexer.private_constant :RepositoryReindexer
|
@@ -0,0 +1,19 @@
|
|
1
|
+
require 'set'
|
2
|
+
module Samvera
|
3
|
+
module NestingIndexer
|
4
|
+
# @api private
|
5
|
+
# A service object responsible for coordinating declarations of interface changes
|
6
|
+
module SemverAssistant
|
7
|
+
def self.messages
|
8
|
+
@messages ||= Set.new
|
9
|
+
end
|
10
|
+
|
11
|
+
def self.removing_from_public_api(context:, as_of:)
|
12
|
+
message = "As of version #{as_of}, #{context} will be removed from the public API"
|
13
|
+
messages << message
|
14
|
+
ActiveSupport::Deprecation.warn(message, caller[1..-1]) if defined?(ActiveSupport::Deprecation) && !ENV.key?('SKIP_ACTIVE_SUPPORT_DEPRECATION')
|
15
|
+
end
|
16
|
+
end
|
17
|
+
end
|
18
|
+
end
|
19
|
+
Samvera::NestingIndexer.private_constant :SemverAssistant
|
@@ -20,20 +20,19 @@ Gem::Specification.new do |spec|
|
|
20
20
|
spec.required_ruby_version = '~>2.0'
|
21
21
|
|
22
22
|
spec.add_development_dependency "bundler", "~> 1.12"
|
23
|
-
spec.add_development_dependency "
|
24
|
-
spec.add_development_dependency "
|
25
|
-
spec.add_development_dependency "rspec-its"
|
23
|
+
spec.add_development_dependency "byebug"
|
24
|
+
spec.add_development_dependency "codeclimate-test-reporter"
|
26
25
|
spec.add_development_dependency "guard-rspec"
|
27
26
|
spec.add_development_dependency "guard-rubocop"
|
28
|
-
spec.add_development_dependency "terminal-notifier-guard"
|
29
|
-
spec.add_development_dependency "terminal-notifier"
|
30
|
-
spec.add_development_dependency "rubocop"
|
31
|
-
spec.add_development_dependency "simplecov"
|
32
|
-
spec.add_development_dependency "codeclimate-test-reporter"
|
33
27
|
spec.add_development_dependency "json"
|
34
|
-
spec.add_development_dependency "byebug"
|
35
|
-
spec.add_development_dependency "railties"
|
36
|
-
# As a secondary dependency, listen is preventing bundling
|
37
28
|
spec.add_development_dependency "listen", '~> 3.0.8'
|
29
|
+
spec.add_development_dependency "railties"
|
30
|
+
spec.add_development_dependency "rake", "~> 10.0"
|
31
|
+
spec.add_development_dependency "rspec-its"
|
32
|
+
spec.add_development_dependency "rspec", "~> 3.0"
|
33
|
+
spec.add_development_dependency "rubocop"
|
34
|
+
spec.add_development_dependency "simplecov"
|
35
|
+
spec.add_development_dependency "terminal-notifier-guard"
|
36
|
+
spec.add_development_dependency "terminal-notifier"
|
38
37
|
spec.add_dependency "dry-equalizer"
|
39
38
|
end
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: samvera-nesting_indexer
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 1.0.
|
4
|
+
version: 1.0.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Jeremy Friesen
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2018-01-
|
11
|
+
date: 2018-01-16 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: bundler
|
@@ -25,35 +25,35 @@ dependencies:
|
|
25
25
|
- !ruby/object:Gem::Version
|
26
26
|
version: '1.12'
|
27
27
|
- !ruby/object:Gem::Dependency
|
28
|
-
name:
|
28
|
+
name: byebug
|
29
29
|
requirement: !ruby/object:Gem::Requirement
|
30
30
|
requirements:
|
31
|
-
- - "
|
31
|
+
- - ">="
|
32
32
|
- !ruby/object:Gem::Version
|
33
|
-
version: '
|
33
|
+
version: '0'
|
34
34
|
type: :development
|
35
35
|
prerelease: false
|
36
36
|
version_requirements: !ruby/object:Gem::Requirement
|
37
37
|
requirements:
|
38
|
-
- - "
|
38
|
+
- - ">="
|
39
39
|
- !ruby/object:Gem::Version
|
40
|
-
version: '
|
40
|
+
version: '0'
|
41
41
|
- !ruby/object:Gem::Dependency
|
42
|
-
name:
|
42
|
+
name: codeclimate-test-reporter
|
43
43
|
requirement: !ruby/object:Gem::Requirement
|
44
44
|
requirements:
|
45
|
-
- - "
|
45
|
+
- - ">="
|
46
46
|
- !ruby/object:Gem::Version
|
47
|
-
version: '
|
47
|
+
version: '0'
|
48
48
|
type: :development
|
49
49
|
prerelease: false
|
50
50
|
version_requirements: !ruby/object:Gem::Requirement
|
51
51
|
requirements:
|
52
|
-
- - "
|
52
|
+
- - ">="
|
53
53
|
- !ruby/object:Gem::Version
|
54
|
-
version: '
|
54
|
+
version: '0'
|
55
55
|
- !ruby/object:Gem::Dependency
|
56
|
-
name: rspec
|
56
|
+
name: guard-rspec
|
57
57
|
requirement: !ruby/object:Gem::Requirement
|
58
58
|
requirements:
|
59
59
|
- - ">="
|
@@ -67,7 +67,7 @@ dependencies:
|
|
67
67
|
- !ruby/object:Gem::Version
|
68
68
|
version: '0'
|
69
69
|
- !ruby/object:Gem::Dependency
|
70
|
-
name: guard-
|
70
|
+
name: guard-rubocop
|
71
71
|
requirement: !ruby/object:Gem::Requirement
|
72
72
|
requirements:
|
73
73
|
- - ">="
|
@@ -81,7 +81,7 @@ dependencies:
|
|
81
81
|
- !ruby/object:Gem::Version
|
82
82
|
version: '0'
|
83
83
|
- !ruby/object:Gem::Dependency
|
84
|
-
name:
|
84
|
+
name: json
|
85
85
|
requirement: !ruby/object:Gem::Requirement
|
86
86
|
requirements:
|
87
87
|
- - ">="
|
@@ -95,21 +95,21 @@ dependencies:
|
|
95
95
|
- !ruby/object:Gem::Version
|
96
96
|
version: '0'
|
97
97
|
- !ruby/object:Gem::Dependency
|
98
|
-
name:
|
98
|
+
name: listen
|
99
99
|
requirement: !ruby/object:Gem::Requirement
|
100
100
|
requirements:
|
101
|
-
- - "
|
101
|
+
- - "~>"
|
102
102
|
- !ruby/object:Gem::Version
|
103
|
-
version:
|
103
|
+
version: 3.0.8
|
104
104
|
type: :development
|
105
105
|
prerelease: false
|
106
106
|
version_requirements: !ruby/object:Gem::Requirement
|
107
107
|
requirements:
|
108
|
-
- - "
|
108
|
+
- - "~>"
|
109
109
|
- !ruby/object:Gem::Version
|
110
|
-
version:
|
110
|
+
version: 3.0.8
|
111
111
|
- !ruby/object:Gem::Dependency
|
112
|
-
name:
|
112
|
+
name: railties
|
113
113
|
requirement: !ruby/object:Gem::Requirement
|
114
114
|
requirements:
|
115
115
|
- - ">="
|
@@ -123,21 +123,21 @@ dependencies:
|
|
123
123
|
- !ruby/object:Gem::Version
|
124
124
|
version: '0'
|
125
125
|
- !ruby/object:Gem::Dependency
|
126
|
-
name:
|
126
|
+
name: rake
|
127
127
|
requirement: !ruby/object:Gem::Requirement
|
128
128
|
requirements:
|
129
|
-
- - "
|
129
|
+
- - "~>"
|
130
130
|
- !ruby/object:Gem::Version
|
131
|
-
version: '0'
|
131
|
+
version: '10.0'
|
132
132
|
type: :development
|
133
133
|
prerelease: false
|
134
134
|
version_requirements: !ruby/object:Gem::Requirement
|
135
135
|
requirements:
|
136
|
-
- - "
|
136
|
+
- - "~>"
|
137
137
|
- !ruby/object:Gem::Version
|
138
|
-
version: '0'
|
138
|
+
version: '10.0'
|
139
139
|
- !ruby/object:Gem::Dependency
|
140
|
-
name:
|
140
|
+
name: rspec-its
|
141
141
|
requirement: !ruby/object:Gem::Requirement
|
142
142
|
requirements:
|
143
143
|
- - ">="
|
@@ -151,21 +151,21 @@ dependencies:
|
|
151
151
|
- !ruby/object:Gem::Version
|
152
152
|
version: '0'
|
153
153
|
- !ruby/object:Gem::Dependency
|
154
|
-
name:
|
154
|
+
name: rspec
|
155
155
|
requirement: !ruby/object:Gem::Requirement
|
156
156
|
requirements:
|
157
|
-
- - "
|
157
|
+
- - "~>"
|
158
158
|
- !ruby/object:Gem::Version
|
159
|
-
version: '0'
|
159
|
+
version: '3.0'
|
160
160
|
type: :development
|
161
161
|
prerelease: false
|
162
162
|
version_requirements: !ruby/object:Gem::Requirement
|
163
163
|
requirements:
|
164
|
-
- - "
|
164
|
+
- - "~>"
|
165
165
|
- !ruby/object:Gem::Version
|
166
|
-
version: '0'
|
166
|
+
version: '3.0'
|
167
167
|
- !ruby/object:Gem::Dependency
|
168
|
-
name:
|
168
|
+
name: rubocop
|
169
169
|
requirement: !ruby/object:Gem::Requirement
|
170
170
|
requirements:
|
171
171
|
- - ">="
|
@@ -179,7 +179,7 @@ dependencies:
|
|
179
179
|
- !ruby/object:Gem::Version
|
180
180
|
version: '0'
|
181
181
|
- !ruby/object:Gem::Dependency
|
182
|
-
name:
|
182
|
+
name: simplecov
|
183
183
|
requirement: !ruby/object:Gem::Requirement
|
184
184
|
requirements:
|
185
185
|
- - ">="
|
@@ -193,7 +193,7 @@ dependencies:
|
|
193
193
|
- !ruby/object:Gem::Version
|
194
194
|
version: '0'
|
195
195
|
- !ruby/object:Gem::Dependency
|
196
|
-
name:
|
196
|
+
name: terminal-notifier-guard
|
197
197
|
requirement: !ruby/object:Gem::Requirement
|
198
198
|
requirements:
|
199
199
|
- - ">="
|
@@ -207,19 +207,19 @@ dependencies:
|
|
207
207
|
- !ruby/object:Gem::Version
|
208
208
|
version: '0'
|
209
209
|
- !ruby/object:Gem::Dependency
|
210
|
-
name:
|
210
|
+
name: terminal-notifier
|
211
211
|
requirement: !ruby/object:Gem::Requirement
|
212
212
|
requirements:
|
213
|
-
- - "
|
213
|
+
- - ">="
|
214
214
|
- !ruby/object:Gem::Version
|
215
|
-
version:
|
215
|
+
version: '0'
|
216
216
|
type: :development
|
217
217
|
prerelease: false
|
218
218
|
version_requirements: !ruby/object:Gem::Requirement
|
219
219
|
requirements:
|
220
|
-
- - "
|
220
|
+
- - ">="
|
221
221
|
- !ruby/object:Gem::Version
|
222
|
-
version:
|
222
|
+
version: '0'
|
223
223
|
- !ruby/object:Gem::Dependency
|
224
224
|
name: dry-equalizer
|
225
225
|
requirement: !ruby/object:Gem::Requirement
|
@@ -241,6 +241,11 @@ executables: []
|
|
241
241
|
extensions: []
|
242
242
|
extra_rdoc_files: []
|
243
243
|
files:
|
244
|
+
- ".github/CODE_OF_CONDUCT.md"
|
245
|
+
- ".github/CONTRIBUTING.md"
|
246
|
+
- ".github/ISSUE_TEMPLATE.md"
|
247
|
+
- ".github/PULL_REQUEST_TEMPLATE.md"
|
248
|
+
- ".github/SUPPORT.md"
|
244
249
|
- ".gitignore"
|
245
250
|
- ".rspec"
|
246
251
|
- ".rubocop.yml"
|
@@ -256,6 +261,9 @@ files:
|
|
256
261
|
- bin/setup
|
257
262
|
- documentation/reindex_relationship.mermaid
|
258
263
|
- documentation/reindex_relationship.mermaid.jpg
|
264
|
+
- lib/generators/samvera/nesting_indexer/install_generator.rb
|
265
|
+
- lib/generators/samvera/nesting_indexer/templates/README
|
266
|
+
- lib/generators/samvera/nesting_indexer/templates/samvera-nesting_indexer_initializer.rb
|
259
267
|
- lib/samvera/nesting_indexer.rb
|
260
268
|
- lib/samvera/nesting_indexer/adapters.rb
|
261
269
|
- lib/samvera/nesting_indexer/adapters/abstract_adapter.rb
|
@@ -267,6 +275,7 @@ files:
|
|
267
275
|
- lib/samvera/nesting_indexer/railtie.rb
|
268
276
|
- lib/samvera/nesting_indexer/relationship_reindexer.rb
|
269
277
|
- lib/samvera/nesting_indexer/repository_reindexer.rb
|
278
|
+
- lib/samvera/nesting_indexer/semver_assistant.rb
|
270
279
|
- lib/samvera/nesting_indexer/version.rb
|
271
280
|
- samvera-nesting_indexer.gemspec
|
272
281
|
homepage: https://github.com/samvera-labs/samvera-nesting_indexer
|