# Summary
A simple module that provides a block showing the current time.
Features
Time zone can be configured as: Site time zone, user time zone, local time zone and custom time zone.
Date format can be configured using the preconfigured "short", "medium" and "long" date types, as well as any custom date types.
The clock can be continuously updated via JavaScript
# Project URL
https://www.drupal.org/project/clock
# Where is the code?
https://www.drupal.org/sandbox/laxmang/2831894
# Estimated completion date
Dec 2016
# Dependencies
# Who's doing the port?
https://www.drupal.org/u/laxmanghavte
# What help do they need?
# D8 roadmap
Convert existing d7 module to d8
# Background and reference information
Comments
Comment #2
laxman.ghavte CreditAttribution: laxman.ghavte as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commentedPlease review, Drupal 8 module code in sandbox
https://www.drupal.org/sandbox/laxmang/2831894
Comment #3
sourabh.singhal CreditAttribution: sourabh.singhal as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commentedHi laxman.ghavte,
There are some issues in your code which was detected on "https://pareview.sh" site. For your reference here is the URL for PA Review Code Result.
Thanks
Sourdrup
Comment #4
laxman.ghavte CreditAttribution: laxman.ghavte as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commentedMost code formatting issue is fixed. Please review.
Comment #5
laxman.ghavte CreditAttribution: laxman.ghavte as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commentedComment #6
sourabh.singhal CreditAttribution: sourabh.singhal as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commentedHi Laxman,
Still showing some warnings. Please check
FILE: /root/repos/pareviewsh/pareview_temp/src/Plugin/Block/ClockBlock.php
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES
--------------------------------------------------------------------------
112 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
150 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
195 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
201 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
202 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
249 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
252 | WARNING | \Drupal calls should be avoided in classes, use
| | dependency injection instead
--------------------------------------------------------------------------
Thanks
Sourdrup
Comment #7
mmjvb CreditAttribution: mmjvb as a volunteer commentedWith a dev release Status should be Needs work. Once tagged as alpha, beta, or rc it can be Needs review or RTBC.
Use the issue queue of the project itself for support issues. This issue is about the D8 port, see D8CPT description for usage of Status.
Comment #8
vipul tulse CreditAttribution: vipul tulse at Clarion Technologies commentedHello,
Can I fix those issue of \Drupal call avoided
Comment #9
vipul tulse CreditAttribution: vipul tulse commentedComment #10
fdefeyter@gmail.com CreditAttribution: fdefeyter@gmail.com commentedI installed it but no block appears...
I finally put code in a block and it works.
But I can't get the current day...
Comment #11
fdefeyter@gmail.com CreditAttribution: fdefeyter@gmail.com commentedand put the timezone in htacess