Add BagProfile (#67)

* Initial BagItProfile work

* More testing

* More tests

* Track serializations

* Load and validate profiles

* Return file path in archive type exception

* Windows test

* Clean up extension handling

* Add more profile tests

* Fixes and more profile tests

* Always check for Allow-Fetch before Require-Fetch, so the reverse test is unnecessary

* Fix callable in array_filter

* Move json into tests

* More tests

* Add methods to add profile to bag and some tests

* Allow other bag-info tag options

* Test bag is valid before creating a package

* Read BagIt Profile identifiers from bag-info.txt and validate

Additional tests

Handle long lines without a space to break at

* Add a change log

* Some tidying

* Correct PIPEWAIT and add test for trait

CHANGELOG.md

@@ -0,0 +1,44 @@
+# Change Log
+
+The purpose of this document is to provide a list of changes included in a release under the covers that
+may have an impact on the user. This is not a comprehensive list of all changes, but hopefully catches most and 
+provides a reasoning for the changes.
+
+## v5.0.0
+
+### Added
+
+#### BagIt Profile support. 
+     
+You can now add BagIt Profile(s) to a newly created bag and/or they will be downloaded and parsed when validating an
+existing bag, assuming the profile is available at the URL specified in the bag-info.txt file.
+
+Profiles are validated against the [BagIt Profile Specification (v1.4.0)](https://bagit-profiles.github.io/bagit-profiles-specification/)
+and profile rules are enforced when validating a bag (`$bag->isValid()`) and any errors are displayed in the `$bag->getErrors()` array.
+
+To add a profile to a bag you can use either:
+- `$bag->addProfileByJson($jsonString)` - To add a profile from a JSON string.
+- `$bag->addProfileByURL($url)` - To add a profile from a URL.
+
+Profiles are stored internally using their `BagIt-Profile-Identifier` as a key. You can only add a profile once
+per identifier. If you try to add a profile with the same identifier it will be ignored.
+
+To remove a profile you can use `$bag->removeBagProfile($profileIdentifier)` to remove a profile.
+
+You can also use `$bag->clearAllProfiles()` to remove all profiles from a bag.
+
+#### Package command validates the bag
+
+Previous versions allowed you to package without validating the bag. Now the package command will validate the bag
+before packaging. If the bag is not valid the package command will fail with a `BagItException::class` being thrown.
+
+This is due to the addition of BagIt Profile support, if you add a profile to a bag we want to ensure you do not package
+an invalid bag.
+
+TODO: Validate the serialization being attempted during package validation.
+
+### Removed
+
+#### PHP 7 Support
+
+This library now requires PHP 8.0 or higher, while PHP 8.0 is already end of life we will support it for the time being.

README.md

@@ -26,6 +26,8 @@ Features:
 * Create an archive (zip, tar, tar.gz, tgz, tar.bz2)
 * In-place upgrade of bag from v0.97 to v1.0
 
+## [Change Log](./CHANGELOG.md)
+
 ## Installation
 
 **Composer**
@@ -169,12 +171,33 @@ if ($bag->hasBagInfoTag('contact-name')) {
     $bag->removeBagInfoTag('contact-name');
 }
 
-// Write bagit support files (manifests, bag-info, etc)
-$bag->update();
-
 // Write the bag to the specified path and filename using the expected archiving method.
 $bag->package('./archive.tar.bz2');
+```
+
+#### Using a BagIt Profile
+```php
+require_once './vendor/autoload.php';
+
+use \whikloj\BagItTools\Bag;
+
+$dir = "./newbag";
+
+// Create new bag as directory $dir
+$bag = Bag::create($dir);
+// Add a profile by URL
+$bag->addBagProfileByURL("https://some.example.com/bagit-profile.json");
+// or add a profile by JSON
+$bag->addBagProfileByJson(file_get_contents("path/to/bagit-profile.json"));
+
+// Add a file
+$bag->addFile('../README.md', 'data/documentation/myreadme.md');
+
+// Add another algorithm
+$bag->addAlgorithm('sha1');
 
+// Write the bag to the specified path and filename using the expected archiving method.
+$bag->package('./archive.tar.bz2');
 ```
 
 ## Maintainer
@@ -184,9 +207,3 @@ $bag->package('./archive.tar.bz2');
 ## License
 
 [MIT](./LICENSE)
-
-## Development
-
-To-Do:
-
-* CLI interface to handle simple bag CRUD (CReate/Update/Delete) functions.

composer.json

@@ -22,8 +22,7 @@
     "symfony/console": "^5.4"
   },
   "require-dev": {
-    "phpunit/phpunit": "^9.0",
-    "sebastian/phpcpd": "^6.0",
+    "phpunit/phpunit": "^9.6",
     "squizlabs/php_codesniffer": "^3.5",
     "donatj/mock-webserver": "^2.6",
     "phpstan/phpstan": "^1.4"
@@ -46,15 +45,15 @@
       "php -d xdebug.mode=profile -d xdebug.output_dir=mytracedir/ -d xdebug.start_with_request=yes -d xdebug.use_compression=true ./vendor/bin/phpunit"
     ],
     "check": [
-      "./vendor/bin/phpcs --standard=PSR12 src tests",
-      "./vendor/bin/phpcpd --suffix='.php' src"
+      "./vendor/bin/phpcs --standard=PSR12 src tests"
     ],
     "phpunit": [
       "phpdbg -qrr ./vendor/bin/phpunit -d memory_limit=-1 --verbose --testsuite BagIt"
     ],
     "test": [
       "@check",
-      "@phpunit"
+      "@phpunit",
+      "@phpstan"
     ]
   },
   "config": {

phpunit.xml.dist

@@ -9,7 +9,7 @@
          processIsolation="false"
          stopOnFailure="false"
          bootstrap="./vendor/autoload.php"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd">
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd">
   <coverage processUncoveredFiles="true">
     <include>
       <directory suffix=".php">./src</directory>