From eb2163a29e797f6a38e188e0485a74b3ecbd7620 Mon Sep 17 00:00:00 2001 From: Oliver Gierke Date: Sat, 30 Dec 2017 12:00:15 +0100 Subject: [PATCH] DATACMNS-563 - Polishing. Removed obsolete generics. Rearranged some code for better readability. Original pull request: #267. --- .../data/web/PagedResourcesAssembler.java | 11 +++++++---- .../data/web/PagedResourcesAssemblerUnitTests.java | 1 + 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/springframework/data/web/PagedResourcesAssembler.java b/src/main/java/org/springframework/data/web/PagedResourcesAssembler.java index 89cd8468f..0b00f5309 100644 --- a/src/main/java/org/springframework/data/web/PagedResourcesAssembler.java +++ b/src/main/java/org/springframework/data/web/PagedResourcesAssembler.java @@ -1,5 +1,5 @@ /* - * Copyright 2013-2015 the original author or authors. + * Copyright 2013-2017 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -47,6 +47,7 @@ import org.springframework.web.util.UriComponentsBuilder; * @since 1.6 * @author Oliver Gierke * @author Nick Williams + * @author Marcel Overdijk */ public class PagedResourcesAssembler implements ResourceAssembler, PagedResources>> { @@ -290,11 +291,13 @@ public class PagedResourcesAssembler implements ResourceAssembler, Pa * @param page must not be {@literal null}. * @return */ - private PageMetadata asPageMetadata(Page page) { + private PageMetadata asPageMetadata(Page page) { Assert.notNull(page, "Page must not be null!"); - boolean isOneIndexedParameters = pageableResolver.isOneIndexedParameters(); - return new PageMetadata(page.getSize(), isOneIndexedParameters ? page.getNumber() + 1 : page.getNumber(), page.getTotalElements(), page.getTotalPages()); + + int number = pageableResolver.isOneIndexedParameters() ? page.getNumber() + 1 : page.getNumber(); + + return new PageMetadata(page.getSize(), number, page.getTotalElements(), page.getTotalPages()); } private String baseUriOrCurrentRequest() { diff --git a/src/test/java/org/springframework/data/web/PagedResourcesAssemblerUnitTests.java b/src/test/java/org/springframework/data/web/PagedResourcesAssemblerUnitTests.java index 31f668931..2329da83e 100755 --- a/src/test/java/org/springframework/data/web/PagedResourcesAssemblerUnitTests.java +++ b/src/test/java/org/springframework/data/web/PagedResourcesAssemblerUnitTests.java @@ -45,6 +45,7 @@ import org.springframework.web.util.UriComponentsBuilder; * * @author Oliver Gierke * @author Nick Williams + * @author Marcel Overdijk */ public class PagedResourcesAssemblerUnitTests {