The Move to Islandora Kit is an extensible PHP command-line tool for converting source content and metadata into packages suitable for importing into Islandora (or other digital repository and preservations systems).
Removes the requirement of a SYSTEM section of the config.ini
What's new?
mik required you to have the SYSTEM section just in case your timezone was not set. But conceivably never checked it, it also parsed the settings file twice.
This uses the existing array of settings and only checks for SYSTEM if we need it.
How should this be tested?
I'm going to suggest that if tests run before and after this PR with the same count you should be fine. However my PHPUnit is not letting me run existing tests under PHP 7
But you could try running mik without a SYSTEM section in your config.ini and you should (depending on your PHP installation) get the PHP timezone or 'America/Vancouver' 😄
To help reviewers test your work:
Indicate whether your work requires a smoke test or is covered by PHPUnit tests (see CONTRIBUTING.md for more information).
If your work is covered by PHPUnit tests, indicate how many successful tests and assertions the reviewer should see when they run your tests.
If your work requires a smoke test, provide sample configuration files and data for reviewers.
Be as detailed as possible.
Good testing instructions and sample confiruation files/data help get your PR completed faster.
Should be covered by existing tests, but again I haven't been able to run them. There is no new functionality added in here, just a different way of achieving the same thing.
Additional Notes
Any additional information that you think would be helpful when reviewing this PR.
Example:
Does this change require documentation to be updated? no
Does this change add any new dependencies? no
Could this change impact execution of existing code? no
Github issue: #412
What does this Pull Request do?
Removes the requirement of a
SYSTEM
section of theconfig.ini
What's new?
mik
required you to have theSYSTEM
section just in case your timezone was not set. But conceivably never checked it, it also parsed the settings file twice.This uses the existing array of settings and only checks for SYSTEM if we need it.
How should this be tested?
I'm going to suggest that if tests run before and after this PR with the same count you should be fine. However my PHPUnit is not letting me run existing tests under PHP 7
But you could try running
mik
without aSYSTEM
section in yourconfig.ini
and you should (depending on your PHP installation) get the PHP timezone or 'America/Vancouver' 😄To help reviewers test your work:
Good testing instructions and sample confiruation files/data help get your PR completed faster.
Should be covered by existing tests, but again I haven't been able to run them. There is no new functionality added in here, just a different way of achieving the same thing.
Additional Notes
Any additional information that you think would be helpful when reviewing this PR.
Example:
Interested parties