- some cleanup

- allowed composer.lock
- added dependency for php >=5.4.0

.gitignore

@@ -1,4 +1,3 @@
 vendor
 .idea
 examples/test.php
-composer.lock

README.md

@@ -21,7 +21,7 @@ BooXtreamClient is available in Packagist, just add it to your composer.json
 
 Alternatively you can just download the package and run ```composer install``` to get the requirements.
 
-The only requirement at the moment is [Guzzle](http://guzzle.readthedocs.org/en/latest/index.html).
+The only requirements at the moment are PHP 5.4 and up and [Guzzle](http://guzzle.readthedocs.org/en/latest/index.html).
 
 If you do not wish to use Composer you will need to fulfill the dependencies on your own.
 

composer.json

@@ -1,20 +1,21 @@
 {
-    "name": "icontact/booxtreamclient",
-    "description": "A simple example client for the BooXtream webservice",
-    "homepage": "https://github.com/BooXtream/php-BooXtreamClient/",
-    "require": {
-        "guzzlehttp/guzzle": "^6.1"
-    },
-    "license": "MIT",
-    "authors": [
-        {
-            "name": "Kevin de Harde",
-            "email": "[email protected]"
-        }
-    ],
-    "autoload": {
-        "psr-4": {
-            "Icontact\\BooXtreamClient\\": "src/"
-        }
+  "name": "icontact/booxtreamclient",
+  "description": "A simple example client for the BooXtream webservice",
+  "homepage": "https://github.com/BooXtream/php-BooXtreamClient/",
+  "require": {
+    "php": ">=5.4.0",
+    "guzzlehttp/guzzle": "^6.1"
+  },
+  "license": "MIT",
+  "authors": [
+    {
+      "name": "Kevin de Harde",
+      "email": "[email protected]"
     }
+  ],
+  "autoload": {
+    "psr-4": {
+      "Icontact\\BooXtreamClient\\": "src/"
+    }
+  }
 }

license.md

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2014, Icontact B.V.
+Copyright (c) 2016, Icontact B.V.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

src/BooXtreamClient.php

@@ -11,6 +11,11 @@
 class BooXtreamClient implements BooXtreamClientInterface {
 	const BASE_URL = 'https://service.booxtream.com';
 
+	/*
+	 * PHP 5.6 would allow us to define this array as a class constant, maybe later.
+	 */
+	private $types = ['xml', 'epub', 'mobi'];
+
 	private $Guzzle;
 	private $authentication;
 	private $type;
@@ -24,26 +29,19 @@ class BooXtreamClient implements BooXtreamClientInterface {
 	 * @param array $authentication
 	 * @param ClientInterface $Guzzle
 	 */
-	public function __construct( $type, Options $Options, array $authentication, ClientInterface $Guzzle  ) {
-		switch ( $type ) {
-			case 'xml':
-			case 'epub':
-			case 'mobi':
-				$this->type = $type;
-				break;
-			default:
-				throw new \InvalidArgumentException( 'invalid type ' . $type );
+	public function __construct( $type, Options $Options, array $authentication, ClientInterface $Guzzle ) {
+		if(!in_array($type, $this->types)) {
+			throw new \InvalidArgumentException( 'invalid type ' . $type );
 		}
 
-		$this->Guzzle         = $Guzzle;
-
-		$this->Options        = $Options;
+		$this->type = $type;
+		$this->Guzzle = $Guzzle;
+		$this->Options = $Options;
 		$this->Options->parseOptions( $this->type === 'xml' );
 
 		$this->authentication = $authentication;
-
-		$this->files          = [ ];
-		$this->storedfiles    = [ ];
+		$this->files       = [ ];
+		$this->storedfiles = [ ];
 	}
 
 	/**