Versioning

[elizoller]

GitHub Issue: (Islandora/documentation#740)

What does this Pull Request do?

Bring chullo up to meet the versioning API spec

What's new?

• add interface for new createVersion and getVersions methods
• add method for createVersion - sends a POST request to the timemapURI to create a version in Fedora. optionally sets a specific Memento-Datetime, optionally accepts the content body.
• add method for getVersions - uses timemapURI from header and makes a get request
• add a helper function to extract the timemapURI from the headers of a Fedora resource URI

How should this be tested?

In theory, write a PHP script which calls these methods. In reality, might be easier to use Milliner in an existing Islandora site, something like - Islandora/Crayfish@dev...asulibraries:versioning which will hopefully be the start of a corresponding PR.

Additional Notes:

Fedora Versioning API Spec - https://wiki.duraspace.org/display/FEDORA5x/RESTful+HTTP+API+-+Versioning#RESTfulHTTPAPI-Versioning-BluePOSTCreateanewversionedresource(anewLDPRm)

Interested parties

@Islandora-CLAW/committers

[elizoller]

note: i will work on tests for this also.

[whikloj]

@elizoller this is good work, I have some suggestions but as I haven't tested this if you have reasons for the path you took please let me know.

[elizoller]

I think I got those suggestions above resolved. And added some tests. How does it look?

[whikloj]

@elizoller can you make a change to the .travis.yml and add

branches:
  only:
    - /master/
+   - /^dev/

Travis isn't running because now we are using dev as our "development" branch, is this correct @dannylamb or should we be merging to master?

[elizoller]

Going to look more at the test for createVersion based on https://codecov.io/gh/Islandora-CLAW/chullo/src/48baf212cca7540247eb152725d5525c6906b543/src/FedoraApi.php 😞

[codecov]

Codecov Report

Merging #75 into dev will increase coverage by 2.98%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##                dev    #75      +/-   ##
==========================================
+ Coverage     97.01%   100%   +2.98%     
- Complexity       12     20       +8     
==========================================
  Files             1      1              
  Lines            67     98      +31     
==========================================
+ Hits             65     98      +33     
+ Misses            2      0       -2

Impacted Files	Coverage Δ	Complexity Δ
src/FedoraApi.php	100% <100%> (+2.98%)	20 <8> (+8)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc60e9c...cf3dfff. Read the comment docs.

[dannylamb]

@whikloj merge to dev, please. that's why I set it up as the default branch, just to nudge us in that direction.

[dannylamb]

nice wall of green there @elizoller 🎉

[whikloj]

Nicely done @elizoller