Skip to content

Commit 1680a55

Browse files
author
jrivard@gmail.com
committed
fix spotbugs errors in rest-test-servlet
1 parent 7b59661 commit 1680a55

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

rest-test-service/src/main/java/password/pwm/resttest/ExternalMacroServlet.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
import java.io.IOException;
3535
import java.io.InputStream;
3636
import java.io.PrintWriter;
37-
import java.util.Date;
37+
import java.time.Instant;
3838

3939
@WebServlet(
4040
name = "NewUserServlet",
@@ -59,7 +59,7 @@ protected void doPost( final HttpServletRequest req, final HttpServletResponse r
5959
final String[] messageContent = body.split( "=" );
6060
final String message = messageContent[messageContent.length - 1];
6161
final String username = message.split( "\\+" )[0];
62-
final Date currentDate = new Date();
62+
final Instant currentDate = Instant.now();
6363
final SmsPostResponseBody messageBody = new SmsPostResponseBody( message, currentDate );
6464

6565
instance.addToMap( username, messageBody );

rest-test-service/src/main/java/password/pwm/resttest/SmsPostResponseBody.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,14 @@
2222

2323
package password.pwm.resttest;
2424

25-
import java.util.Date;
25+
import java.time.Instant;
2626

2727
public class SmsPostResponseBody
2828
{
29-
private static String messageContent;
30-
private static Date date;
29+
private String messageContent;
30+
private Instant date;
3131

32-
public SmsPostResponseBody( final String message, final Date date )
32+
public SmsPostResponseBody( final String message, final Instant date )
3333
{
3434
final String[] strings = message.split( "&" );
3535
this.messageContent = strings[strings.length - 1];
@@ -42,7 +42,7 @@ public SmsPostResponseBody( final String message )
4242
this.messageContent = strings[strings.length - 1];
4343
}
4444

45-
public SmsPostResponseBody( final Date date )
45+
public SmsPostResponseBody( final Instant date )
4646
{
4747
this.date = date;
4848
this.messageContent = "";
@@ -63,12 +63,12 @@ public void setMessageContent( final String messageContent )
6363
this.messageContent = messageContent;
6464
}
6565

66-
public Date getDate()
66+
public Instant getDate()
6767
{
6868
return date;
6969
}
7070

71-
public void setDate( final Date date )
71+
public void setDate( final Instant date )
7272
{
7373
this.date = date;
7474
}

rest-test-service/src/main/java/password/pwm/resttest/SmsResponse.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ public SmsPostResponseBody getRecentFromMap( final String username )
7777
int mostRecentIndex = 0;
7878
for ( int i = 0; i < userMessages.size(); i++ )
7979
{
80-
if ( userMessages.get( i ).getDate().getTime() > userMessages.get( mostRecentIndex ).getDate().getTime() )
80+
if ( userMessages.get( i ).getDate().toEpochMilli() > userMessages.get( mostRecentIndex ).getDate().toEpochMilli() )
8181
{
8282
mostRecentIndex = i;
8383
}

0 commit comments

Comments
 (0)