nextcloud / nextcloud/talk-android

Migrate from usage of APIs under java.util when dealing with date and time.

Open
#4,274 2 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

1. to develop enhancement technical debt
Dominant language
Kotlin
Stars
739
Forks
321
Avg merge
14h 59m
Merged PRs (30d)
151

Description

We should remove the usage of APIs related date and time from java.util package. Why? See below ->

From oracle's site link :

Why do we need a new date and time library?

A long-standing bugbear of Java developers has been the inadequate support for the date and time use cases of ordinary developers.

In order to address these problems and provide better support in the JDK core, a new date and time API, which is free of these problems, has been designed for Java SE 8.

This stackoverflow answer explains it well too.

Also, not only from above I have personally encountered bugs and weird outputs. Also, less intuitive meaning more chances of introducing bugs while maintaining or writing new code.

So, I make the case to migrate from this to some other solution like APIs from java.time which in fact was introduced to overcome the shortcomings of above.

Let me know @mahibi

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Search the project for date and time API usage from java.util and identify the affected call sites before comparing them with java.time alternatives. Done means the relevant legacy date/time usages have been migrated without changing behavior and the project’s existing checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, kotlin
Domain
mobile
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.