コンマの実装 #15

New Issue

konkon-fox · 2023-10-05T20:09:31+09:00

konkon-fox commented

2023-10-05 20:09:31 +09:00

レスの投稿日時にコンマが欲しいという人を多く見かけますので提案します。
/test/bbs-main.php 16行目の$DATEの定義部分を以下のように変更するのはどうでしょうか。

$DATE = date("Y/m/d H:i:s", $NOWTIME);
↓
$NOWMICROTIME= microtime(true);
$microTime = $NOWMICROTIME - floor($NOWMICROTIME);
$commmaTime = floor($microTime * 100);
$commmaTime = sprintf("%02d", $commmaTime);
$DATE = date("Y/m/d H:i:s", $NOWMICROTIME);
$DATE = $DATE.'.'.$commmaTime;

$NOWTIME自体を変更するべきかもしれませんが、$NOWTIMEは使用されている箇所が多いのに対して$DATEはおそらくDATファイルと!addコマンドくらいにしか使われていないので、変更によるバグを未然に防ぐためにも最小限の変更にしたほうが良いかと思い上記のようなコードとなりました。$NOWTIMEとの誤差が生じますが、$DATEは日時表示用にしか使われていないので許容範囲かと思います。

レスの投稿日時にコンマが欲しいという人を多く見かけますので提案します。 /test/bbs-main.php 16行目の$DATEの定義部分を以下のように変更するのはどうでしょうか。 $DATE = date("Y/m/d H:i:s", $NOWTIME); ↓ $NOWMICROTIME= microtime(true); $microTime = $NOWMICROTIME - floor($NOWMICROTIME); $commmaTime = floor($microTime * 100); $commmaTime = sprintf("%02d", $commmaTime); $DATE = date("Y/m/d H:i:s", $NOWMICROTIME); $DATE = $DATE.'.'.$commmaTime; $NOWTIME自体を変更するべきかもしれませんが、$NOWTIMEは使用されている箇所が多いのに対して$DATEはおそらくDATファイルと!addコマンドくらいにしか使われていないので、変更によるバグを未然に防ぐためにも最小限の変更にしたほうが良いかと思い上記のようなコードとなりました。$NOWTIMEとの誤差が生じますが、$DATEは日時表示用にしか使われていないので許容範囲かと思います。

👍 2

stat2 added the

enhancement

label 2023-10-06 02:12:28 +09:00

haruhi commented

2023-10-06 09:19:16 +09:00

First-time contributor

ご提案ありがとうございますー！
はるひの方で上記のコードを試用させていただきますー

ご提案ありがとうございますー！はるひの方で上記のコードを試用させていただきますー

konkon-fox commented

2023-10-21 20:23:33 +09:00

コンマに関しては管理画面のオプション(無し、2桁、3桁等)で選べようにするのが良いのかな、と考えています。
ほぼお遊び目的なので作業の優先度は低めで…