You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by sv...@apache.org on 2018/08/29 16:11:29 UTC

wicket git commit: WICKET-6580 use standard units KB,MB,GB,TB

Repository: wicket
Updated Branches:
  refs/heads/wicket-8.x 778fcef98 -> b171f46d1


WICKET-6580 use standard units KB,MB,GB,TB


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/b171f46d
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/b171f46d
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/b171f46d

Branch: refs/heads/wicket-8.x
Commit: b171f46d1d8eaec0dda3f0c313177a18c1099ca5
Parents: 778fcef
Author: Sven Meier <sv...@apache.org>
Authored: Wed Aug 29 17:36:52 2018 +0200
Committer: Sven Meier <sv...@apache.org>
Committed: Wed Aug 29 17:38:03 2018 +0200

----------------------------------------------------------------------
 .../java/org/apache/wicket/util/lang/Bytes.java |  8 ++++----
 .../org/apache/wicket/util/lang/BytesTest.java  | 10 +++++-----
 .../org/apache/wicket/util/size/BytesTest.java  | 20 ++++++++++----------
 3 files changed, 19 insertions(+), 19 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/b171f46d/wicket-util/src/main/java/org/apache/wicket/util/lang/Bytes.java
----------------------------------------------------------------------
diff --git a/wicket-util/src/main/java/org/apache/wicket/util/lang/Bytes.java b/wicket-util/src/main/java/org/apache/wicket/util/lang/Bytes.java
index 04cdc7d..87476e4 100644
--- a/wicket-util/src/main/java/org/apache/wicket/util/lang/Bytes.java
+++ b/wicket-util/src/main/java/org/apache/wicket/util/lang/Bytes.java
@@ -386,22 +386,22 @@ public final class Bytes extends LongValue
 	{
 		if (terabytes() >= 1.0)
 		{
-			return unitString(terabytes(), "T", locale);
+			return unitString(terabytes(), "TB", locale);
 		}
 
 		if (gigabytes() >= 1.0)
 		{
-			return unitString(gigabytes(), "G", locale);
+			return unitString(gigabytes(), "GB", locale);
 		}
 
 		if (megabytes() >= 1.0)
 		{
-			return unitString(megabytes(), "M", locale);
+			return unitString(megabytes(), "MB", locale);
 		}
 
 		if (kilobytes() >= 1.0)
 		{
-			return unitString(kilobytes(), "K", locale);
+			return unitString(kilobytes(), "KB", locale);
 		}
 
 		return Long.toString(value) + " bytes";

http://git-wip-us.apache.org/repos/asf/wicket/blob/b171f46d/wicket-util/src/test/java/org/apache/wicket/util/lang/BytesTest.java
----------------------------------------------------------------------
diff --git a/wicket-util/src/test/java/org/apache/wicket/util/lang/BytesTest.java b/wicket-util/src/test/java/org/apache/wicket/util/lang/BytesTest.java
index ec2c5c0..bfa6b93 100644
--- a/wicket-util/src/test/java/org/apache/wicket/util/lang/BytesTest.java
+++ b/wicket-util/src/test/java/org/apache/wicket/util/lang/BytesTest.java
@@ -148,11 +148,11 @@ public class BytesTest extends Assert
 	public void testToString()
 	{
 		assertEquals("1 bytes", Bytes.bytes(1).toString());
-		assertEquals("1K", Bytes.bytes(1024).toString());
-		assertEquals("1M", Bytes.bytes(1024 * 1024L).toString());
-		assertEquals("1G", Bytes.bytes(1024 * 1024 * 1024L).toString());
-		assertEquals("1T", Bytes.bytes(1024 * 1024 * 1024 * 1024L).toString());
-		assertEquals("1.5K", Bytes.bytes(1024 * 1.5).toString());
+		assertEquals("1KB", Bytes.bytes(1024).toString());
+		assertEquals("1MB", Bytes.bytes(1024 * 1024L).toString());
+		assertEquals("1GB", Bytes.bytes(1024 * 1024 * 1024L).toString());
+		assertEquals("1TB", Bytes.bytes(1024 * 1024 * 1024 * 1024L).toString());
+		assertEquals("1.5KB", Bytes.bytes(1024 * 1.5).toString());
 
 		assertEquals("1 bytes", Bytes.bytes(1).toString(Locale.GERMAN));
 	}

http://git-wip-us.apache.org/repos/asf/wicket/blob/b171f46d/wicket-util/src/test/java/org/apache/wicket/util/size/BytesTest.java
----------------------------------------------------------------------
diff --git a/wicket-util/src/test/java/org/apache/wicket/util/size/BytesTest.java b/wicket-util/src/test/java/org/apache/wicket/util/size/BytesTest.java
index f0fc62f..3f74671 100644
--- a/wicket-util/src/test/java/org/apache/wicket/util/size/BytesTest.java
+++ b/wicket-util/src/test/java/org/apache/wicket/util/size/BytesTest.java
@@ -65,7 +65,7 @@ public final class BytesTest extends Assert
 	{
 		assertTrue(Bytes.bytes(1024).equals(Bytes.kilobytes(1)));
 		assertTrue(Bytes.bytes(1024 * 1024).equals(Bytes.megabytes(1)));
-		assertTrue("1G".equals(Bytes.gigabytes(1).toString()));
+		assertTrue("1GB".equals(Bytes.gigabytes(1).toString()));
 
 
 		final Bytes b = Bytes.kilobytes(7.3);
@@ -84,8 +84,8 @@ public final class BytesTest extends Assert
 	public void stringOperationsDotLocale() throws StringValueConversionException
 	{
 		Locale.setDefault(Locale.UK);
-		assertTrue("1G".equals(Bytes.gigabytes(1).toString()));
-		assertTrue(Bytes.valueOf("15.5K").bytes() == ((15 * 1024) + 512));
+		assertTrue("1GB".equals(Bytes.gigabytes(1).toString()));
+		assertTrue(Bytes.valueOf("15.5KB").bytes() == ((15 * 1024) + 512));
 
 		final Bytes b = Bytes.kilobytes(7.3);
 
@@ -100,8 +100,8 @@ public final class BytesTest extends Assert
 	public void stringOperationsCommaLocale() throws StringValueConversionException
 	{
 		Locale.setDefault(Locale.GERMANY);
-		assertTrue("1G".equals(Bytes.gigabytes(1).toString()));
-		assertTrue(Bytes.valueOf("15,5K").bytes() == ((15 * 1024) + 512));
+		assertTrue("1GB".equals(Bytes.gigabytes(1).toString()));
+		assertTrue(Bytes.valueOf("15,5KB").bytes() == ((15 * 1024) + 512));
 
 		final Bytes b = Bytes.kilobytes(7.3);
 
@@ -115,14 +115,14 @@ public final class BytesTest extends Assert
 	@Test
 	public void allOperationsExplicitLocale() throws StringValueConversionException
 	{
-		assertTrue("1G".equals(Bytes.gigabytes(1).toString()));
-		assertTrue("1,5G".equals(Bytes.gigabytes(1.5).toString(Locale.GERMAN)));
-		assertTrue("1.5G".equals(Bytes.gigabytes(1.5).toString(Locale.US)));
+		assertTrue("1GB".equals(Bytes.gigabytes(1).toString()));
+		assertTrue("1,5GB".equals(Bytes.gigabytes(1.5).toString(Locale.GERMAN)));
+		assertTrue("1.5GB".equals(Bytes.gigabytes(1.5).toString(Locale.US)));
 
 		final Bytes b = Bytes.kilobytes(7.3);
 		assertEquals(b, Bytes.valueOf(b.toString(Locale.GERMAN), Locale.GERMAN));
 
-		assertTrue(Bytes.valueOf("15,5K", Locale.GERMAN).bytes() == ((15 * 1024) + 512));
-		assertTrue(Bytes.valueOf("15.5K", Locale.US).bytes() == ((15 * 1024) + 512));
+		assertTrue(Bytes.valueOf("15,5KB", Locale.GERMAN).bytes() == ((15 * 1024) + 512));
+		assertTrue(Bytes.valueOf("15.5KB", Locale.US).bytes() == ((15 * 1024) + 512));
 	}
 }