Merge pull request #20 from Art4/Bug_Pagination

Bugfix: Add tests and code for pagination links

Author: Art4

CHANGELOG.md

@@ -5,6 +5,14 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+### Fixed
+
+- **BREAKING**: pagination links moved from `Pagination` to `DocumentLink` and `RelationshipLink`
+
+### Removed
+
+- **BREAKING**: object `Pagination` was removed
+
 ## [0.5] - 2015-10-12
 
 ### Added

docs/README.md

@@ -18,7 +18,6 @@
 * [Document Link object](objects-document-link.md)
 * [Relationship Link object](objects-relationship-link.md)
 * [Error Link object](objects-error-link.md)
-* [Pagination object](objects-pagination.md)
 * [Jsonapi object](objects-jsonapi.md)
 * [Meta object](objects-meta.md)
 

docs/objects-document-link.md

@@ -16,4 +16,7 @@ _[Symbols definition](objects-introduction.md#symbols)_
 --- | ---- | ----- | ----
 ? | self | `string` |
 ? | related | `string` |
-? | pagination | [Pagination object](objects-pagination.md) |
+? | first | - `null`<br />- `string` |
+? | last | - `null`<br />- `string` |
+? | prev | - `null`<br />- `string` |
+? | next | - `null`<br />- `string` |

docs/objects-introduction.md

@@ -34,7 +34,6 @@ All possible objects and their hierarchical structure are listet below.
 1. [Document Link object](objects-document-link.md)
 1. [Relationship Link object](objects-relationship-link.md)
 1. [Error Link object](objects-error-link.md)
-1. [Pagination object](objects-pagination.md)
 1. [Jsonapi object](objects-jsonapi.md)
 1. [Meta object](objects-meta.md)
 

docs/objects-pagination.md

@@ -16,4 +16,7 @@ _[Symbols definition](objects-introduction.md#symbols)_
 --- | ---- | ----- | ----
 # | self | `string` |
 # | related | `string` |
-? | pagination | [Pagination object](objects-pagination.md) | Only exists if the parent relationship object represents a to-many relationship
+? | first | - `null`<br />- `string` | Only exists if the parent relationship object represents a to-many relationship |
+? | last | - `null`<br />- `string` | Only exists if the parent relationship object represents a to-many relationship |
+? | prev | - `null`<br />- `string` | Only exists if the parent relationship object represents a to-many relationship |
+? | next | - `null`<br />- `string` | Only exists if the parent relationship object represents a to-many relationship |

docs/objects-relationship-link.md

@@ -70,12 +70,58 @@ public function __construct($object, FactoryManagerInterface $manager)
 			$this->container->set('related', $object->related);
 		}
 
-		if ( property_exists($object, 'pagination') )
+		// Pagination links
+
+		if ( property_exists($object, 'first') )
+		{
+			if ( ! is_string($object->first) and ! is_null($object->first) )
+			{
+				throw new ValidationException('property "first" has to be a string or null, "' . gettype($object->first) . '" given.');
+			}
+
+			if ( ! is_null($object->first) )
+			{
+				$this->container->set('first', strval($object->first));
+			}
+		}
+
+		if ( property_exists($object, 'last') )
+		{
+			if ( ! is_string($object->last) and ! is_null($object->last) )
+			{
+				throw new ValidationException('property "last" has to be a string or null, "' . gettype($object->last) . '" given.');
+			}
+
+			if ( ! is_null($object->last) )
+			{
+				$this->container->set('last', strval($object->last));
+			}
+		}
+
+		if ( property_exists($object, 'prev') )
 		{
-			$this->container->set('pagination', $this->manager->getFactory()->make(
-				'Pagination',
-				[$object->pagination, $this->manager]
-			));
+			if ( ! is_string($object->prev) and ! is_null($object->prev) )
+			{
+				throw new ValidationException('property "prev" has to be a string or null, "' . gettype($object->prev) . '" given.');
+			}
+
+			if ( ! is_null($object->prev) )
+			{
+				$this->container->set('prev', strval($object->prev));
+			}
+		}
+
+		if ( property_exists($object, 'next') )
+		{
+			if ( ! is_string($object->next) and ! is_null($object->next) )
+			{
+				throw new ValidationException('property "next" has to be a string or null, "' . gettype($object->next) . '" given.');
+			}
+
+			if ( ! is_null($object->next) )
+			{
+				$this->container->set('next', strval($object->next));
+			}
 		}
 	}
 

src/DocumentLink.php

@@ -79,12 +79,58 @@ public function __construct($object, FactoryManagerInterface $manager)
 			$this->container->set('related', strval($object->related));
 		}
 
-		if ( property_exists($object, 'pagination') )
+		// Pagination links
+
+		if ( property_exists($object, 'first') )
+		{
+			if ( ! is_string($object->first) and ! is_null($object->first) )
+			{
+		throw new ValidationException('property "first" has to be a string or null, "' . gettype($object->first) . '" given.');
+			}
+
+			if ( ! is_null($object->first) )
+			{
+				$this->container->set('first', strval($object->first));
+			}
+		}
+
+		if ( property_exists($object, 'last') )
+		{
+			if ( ! is_string($object->last) and ! is_null($object->last) )
+			{
+				throw new ValidationException('property "last" has to be a string or null, "' . gettype($object->last) . '" given.');
+			}
+
+			if ( ! is_null($object->last) )
+			{
+				$this->container->set('last', strval($object->last));
+			}
+		}
+
+		if ( property_exists($object, 'prev') )
 		{
-			$this->container->set('pagination', $this->manager->getFactory()->make(
-				'Pagination',
-				[$object->pagination, $this->manager]
-			));
+			if ( ! is_string($object->prev) and ! is_null($object->prev) )
+			{
+				throw new ValidationException('property "prev" has to be a string or null, "' . gettype($object->prev) . '" given.');
+			}
+
+			if ( ! is_null($object->prev) )
+			{
+				$this->container->set('prev', strval($object->prev));
+			}
+		}
+
+		if ( property_exists($object, 'next') )
+		{
+			if ( ! is_string($object->next) and ! is_null($object->next) )
+			{
+				throw new ValidationException('property "next" has to be a string or null, "' . gettype($object->next) . '" given.');
+			}
+
+			if ( ! is_null($object->next) )
+			{
+				$this->container->set('next', strval($object->next));
+			}
 		}
 	}