Deprecate CurrentTimeUTC #870
Conversation
|
Thank you for stepping up. Currently the comments do not enable quick replacement. Please try to put yourself in the shoes of someone who wrote a plugin three years ago and since then has only programmed websites with vue or similar. The comment needs to enable them to adjust the code without investigating, otherwise it will never be possible to turn this into deprecation for removal. There are already modified plugins in use. If it’s not trivial (in the strictest sense) to replace the deprecation, removing the deprecated version will still break plugins in the next decade. |
bertm
force-pushed
the
deprecate-current-time-utc
branch
from
1218e76
to
c91ec6f
Compare
|
I would not expect plugin developers to need this level of hand-holding in finding alternatives for these (until now mostly undocumented) methods. I've now updated the deprecation notices with verbatim copies of the new implementations provided in #866 to allow for a copy-paste migration path. |
bertm
force-pushed
the
deprecate-current-time-utc
branch
from
c91ec6f
to
09f1fce
Compare
The Java 8 Date-Time APIs provide superior alternatives for the methods provided in this class.
Only get() and getInMillis() were used within Fred. These methods can trivially be replaced by new Date() and System.currentTimeMillis() respectively.
bertm
force-pushed
the
deprecate-current-time-utc
branch
from
09f1fce
to
b53b937
Compare
|
Updated the commits to have GPG signatures. |
|
@bertm merged — thank you! And welcome back! It’s great to see you again ❤️ |
Follow-up on discussion in #866