Comments (11)
I'm thinking this approach meets the need pretty effectively. Perhaps we can leave the manipulator approach on the back burner for now and push the simple one through for expediency?
from mik.
Not yet, see #176.
from mik.
That looks complex! So it's not as simple as, say, creating a new option (say, "migrated_from=(base CDM URL)") and replacing the hard-coded SFU base with the new variable? It feels like it should be pretty simple.
from mik.
Possibly - as long as if that new option was absent, no migrated_from identifier would be added (i.e., add that element is optional). @MarcusBarnes and @jonathangreen what do you think of this simpler approach, as opposed to the manipulator approach I had been pursuing?
from mik.
Yeah, that could work... By default, no migrated_from_uri happens. Here's my thought...
Instead of making include_migrated_from_uri
TRUE or FALSE, make it the base URI for the CDM instance. If it's simpler to keep it required, then allow FALSE or a base URL.
So, if FALSE, keep the same code
If a base URL, follow the code for TRUE but replace http://content.lib.sfu.ca
with the value of include_migrated_from_uri
.
from mik.
The simpler approach would be good for resolving this quicker, both for @bondjimbond but also for other potential parties that may use MIK to move from CDM to Islandora (we may see an increase of this with the continuing work on ISLE). That said, the manipulator approach is still valuable and we should pursue this if there's sufficient community need for this flexibility of this approach.
from mik.
@bondjimbond I'm all for that and thanks for suggesting this simpler approach. I'd like to hear from @jonathangreen before we move ahead with implementing it since he was interested in a solution to this problem too.
@bondjimbond are up able to open a PR for this?
from mik.
Simple solution works for me!
from mik.
Sweet, we're all on board then. Simplest solution wins!
from mik.
@mjordan I can attempt a PR, though I'd prefer you get issue-482 merged first... would make it a lot easier to test.
from mik.
OK, I'm cool with merging issue-482 with proper documentation. Being stretched in a bazillion directions but will get to a PR by end of the day.
from mik.
Related Issues (20)
- regex_replace filter not available? HOT 9
- Postwritehook suggestion: reorganize output by filetype? HOT 3
- CDM toolchains extracting wrong files due to repeating ID numbers HOT 6
- Toolchain: CONTENTdm compound PDFs HOT 33
- CSV Compound Child metadata ignored HOT 7
- Make extensions of page images configurable in CSV Book and Newspaper toolchains HOT 1
- Test using PHP 7.2 and 7.3
- CsvBooks doesn't like periods in filenames HOT 17
- checkconfig doesn't detect missing CSV input file
- Simple Archive toolchain HOT 72
- OAI toolchain error: "Undefined namespace prefix" HOT 1
- OAI toolchain error: "No such file or directory" HOT 1
- OAI-PMH Filegetter only works for DC HOT 11
- Mysterious mapping error? HOT 9
- OAI: If item identifier has special characters, temp metadata filename doesn't match filegetter HOT 4
- Islandora 8: Create a toolchain that produces YAML config files for CSV migrations HOT 1
- CsvCompound fails to create directories HOT 1
- Delimiter problem in CSV Newspapers toolchain? HOT 15
- Mysterious failures in the CSV Newspaper toolchain HOT 8
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from mik.