Recipe Name:
Migrate Date after to isAfter
Description:
Migrate Date after to isAfter
Level:
warning
Language:
  • java
Tags:
  • java.time
  • java.util.Date
  • quality
Documentation

Migrate from java.util.Date and java.util.Calendar to Java Time

Prior to the Java SE 8 release, the Java date and time mechanism was provided by the java.util.Date, java.util.Calendar, and java.util.TimeZone classes, as well as their subclasses, such as java.util.GregorianCalendar.

These classes had several drawbacks, including:

  • The Calendar class was not type safe.
  • Because the classes were mutable, they could not be used in multithreaded applications.
  • Bugs in application code were common due to the unusual numbering of months and the lack of type safety.

Perhaps you have legacy code that uses the java.util date and time classes and you would like to take advantage of the java.time functionality with minimal changes to your code.

Examples

Before 
    Calendar calendar = new GregorianCalendar(2020, 05, 10, 10, 30);
    calendar.add(Calendar.YEAR, 1);
After 
    LocalDateTime calendar = LocalDateTime.of(2020, 05, 10, 10, 30);
    calendar.plusYears(1);
References
Recipe 
id: scw:date:is-after
version: 10
metadata:
  name: Migrate Date after to isAfter
  shortDescription: Migrate Date after to isAfter
  level: warning
  language: java
  enabled: true
  comment: "This recipe is designed to match on broken code, the method that is being searched for does not actually exist on the type.\n\nWhen migrating 'after' to the java.time equivalent, the type must first be migrated to a java.time\nThis recipe then searches for the java.time variables, and finishes the migration by converting it to the \nequivalent java.time isAfter method."
  descriptionFile: Java/Date-Calendar/descriptions/date-calendar.html
  tags: java.time;java.util.Date;quality
search:
  methodcall:
    name: after
    anyOf:
    - type: java.time.LocalDateTime
    - type: java.time.LocalDate
    - type: java.time.ZonedDateTime
    - type: java.time.OffsetDateTime
availableFixes:
- name: Change to isAfter
  actions:
  - rewrite:
      to: '{{{ qualifier.name }}}.isAfter({{{ arguments.0.name }}})'