fix: unified date time method on html nodes

In many situations the date time text format is not the same, especially
when in recent 7 days.

Use `dateTime` extension method as a unified way to get the correct date
time.

Refactored callsite:

* Thread last reply time in latest thread page.
* Message publish time in broadcast message page.
* Personal message sent time in personal message page.
* Rate act time in rate log item.
* Thread operate time in operation log item.
* Sales history item time in locked card.
* Post publish tiem in post item.
* Thread latest reply time in forum page.

Author: realth000

lib/extensions/universal_html.dart

@@ -265,18 +265,52 @@ extension GrepExtension on Element {
 
   /// Parse the datetime on current node or first child.
   ///
+  /// In many situations the date time text format is not the same, especially when in recent 7 days.
+  ///
+  /// Use this function as a unified way to get the correct date time.
+  ///
+  /// Example:
+  ///
   /// From current node `<span>`'s title attribute or first child (usually
   /// a span, too)  's title attribute.
   DateTime? dateTime() {
-    if (tagName != 'SPAN') {
-      return null;
+    if (tagName == 'A') {
+      // Use case: last reply time in threads in my thread page.
+      if (children.firstOrNull?.tagName == 'SPAN') {
+        // Recent 7 days.
+        return children.first.attributes['title']?.parseToDateTimeUtc8();
+      }
+
+      // More than 7 days.
+      return nodes.firstOrNull?.text?.trim().parseToDateTimeUtc8();
     }
-    if (classes.contains('xg1')) {
-      final text1 = innerText.parseToDateTimeUtc8();
-      if (text1 != null) {
-        return text1;
+
+    if (tagName == 'SPAN') {
+      if (attributes.containsKey('title')) {
+        // Use case: rate log item in recent 7 days.
+        return attributes['title']!.parseToDateTimeUtc8();
+      }
+
+      // Use case: broadcast message publish time in broadcast page.
+      if (children.firstOrNull?.tagName == 'SPAN') {
+        // Recent 7 days.
+        return children.firstOrNull?.attributes['title']?.parseToDateTimeUtc8();
       }
+
+      // More than 7 days.
+      return innerText.trim().parseToDateTimeUtc8();
+    }
+
+    // Try parse it.
+
+    if (children.firstOrNull?.tagName == 'SPAN' && children.first.attributes.containsKey('title')) {
+      return children.first.attributes['title']!.parseToDateTimeUtc8();
+    }
+
+    if (nodes.firstOrNull?.nodeType == Node.TEXT_NODE) {
+      return nodes.first.text?.trim().parseToDateTimeUtc8();
     }
-    return children.firstOrNull?.attributes['title']?.parseToDateTimeUtc8();
+
+    return null;
   }
 }

lib/features/my_thread/models/my_thread.dart

@@ -94,11 +94,7 @@ class MyThread with MyThreadMappable {
     final latestReplyNode = element.querySelector('td.by');
     final latestReplyAuthorName = latestReplyNode?.querySelector('cite > a')?.firstEndDeepText();
     final latestReplyAuthorUrl = latestReplyNode?.querySelector('cite > a')?.firstHref();
-    final latestReplyTime =
-        // Within 7 days.
-        latestReplyNode?.querySelector('em > a > span')?.attributes['title']?.parseToDateTimeUtc8() ??
-        // More than 7 days ago.
-        latestReplyNode?.querySelector('em > a')?.firstEndDeepText()?.parseToDateTimeUtc8();
+    final latestReplyTime = latestReplyNode?.querySelector('em > a')?.dateTime();
     String? quotedMessage;
     if (element.classes.contains('bw0_all')) {
       quotedMessage = element.nextElementSibling?.querySelector('td.xg1')?.innerText.trim();

lib/features/notification/bloc/broadcast_message_detail_cubit.dart

@@ -1,13 +1,12 @@
 import 'package:bloc/bloc.dart';
 import 'package:dart_mappable/dart_mappable.dart';
 import 'package:tsdm_client/constants/url.dart';
-import 'package:tsdm_client/extensions/string.dart';
+import 'package:tsdm_client/extensions/universal_html.dart';
 import 'package:tsdm_client/features/notification/repository/notification_repository.dart';
 import 'package:tsdm_client/utils/logger.dart';
 import 'package:universal_html/html.dart' as uh;
 
 part 'broadcast_message_detail_cubit.mapper.dart';
-
 part 'broadcast_message_detail_state.dart';
 
 /// Cubit of broadcast message detail page.
@@ -31,10 +30,7 @@ final class BroadcastMessageDetailCubit extends Cubit<BroadcastMessageDetailStat
           (v) {
             final (document, _) = v;
             final infoNode = document.querySelector('div#pm_ul');
-            final datetime =
-                infoNode?.querySelector('dl > dd.ptm > span.xg1')?.innerText.parseToDateTimeUtc8() ??
-                // Recent messages.
-                infoNode?.querySelector('dl > dd.ptm > span.xg1 > span')?.title?.parseToDateTimeUtc8();
+            final datetime = infoNode?.querySelector('dl > dd.ptm > span.xg1')?.dateTime();
             final messageNode = infoNode?.querySelector('dl > dd > p.pm_smry');
             if (datetime == null || messageNode == null) {
               error(

lib/features/notification/models/message.dart

@@ -72,11 +72,7 @@ final class PersonalMessage with PersonalMessageMappable {
     }
 
     final username = element.querySelector('dd:nth-child(3) > a')?.innerText;
-    final lastMessageTime =
-        // Old messages.
-        element.querySelector('dd:nth-child(3) > span.xg1')?.innerText.parseToDateTimeUtc8() ??
-        // Recent messages.
-        element.querySelector('dd:nth-child(3) > span.xg1 > span')?.title?.parseToDateTimeUtc8();
+    final lastMessageTime = element.querySelector('dd:nth-child(3) > span.xg1')?.dateTime();
     final chatUrl = element.querySelector('a#pmlist_${messageId}_a')?.attributes['href']?.unescapeHtml()?.prependHost();
     final message = element
         .querySelector('dd:nth-child(3) > span.xg1')
@@ -139,10 +135,7 @@ final class BroadcastMessage with BroadcastMessageMappable {
       return null;
     }
     final message = infoNode.querySelector('span')?.innerText.trim();
-    final messageTime =
-        infoNode.querySelector('span.xg1')?.innerText.trim().parseToDateTimeUtc8() ??
-        // Less than 7 days
-        infoNode.querySelector('span.xg1 > span')?.title?.parseToDateTimeUtc8();
+    final messageTime = infoNode.querySelector('span.xg1')?.dateTime();
     final redirectUrl = infoNode.querySelector('a')?.attributes['href']?.unescapeHtml()?.prependHost();
     if (message == null || messageTime == null) {
       talker.error(

lib/features/rate/repository/rate_repository.dart

@@ -5,6 +5,7 @@ import 'package:tsdm_client/constants/url.dart';
 import 'package:tsdm_client/exceptions/exceptions.dart';
 import 'package:tsdm_client/extensions/fp.dart';
 import 'package:tsdm_client/extensions/string.dart';
+import 'package:tsdm_client/extensions/universal_html.dart';
 import 'package:tsdm_client/features/rate/models/models.dart';
 import 'package:tsdm_client/instance.dart';
 import 'package:tsdm_client/shared/providers/net_client_provider/net_client_provider.dart';
@@ -55,7 +56,7 @@ final class RateRepository with LoggerMixin {
         final userNode = tds[1].querySelector('a');
         final username = userNode?.innerText.trim();
         final uid = userNode?.attributes['href']?.tryParseAsUri()?.queryParameters['uid'];
-        final time = tds[2].querySelector('span')?.attributes['title']?.parseToDateTimeUtc8();
+        final time = tds[2].dateTime();
         final reason = tds[3].innerText.trim();
 
         if (attrValue == null || username == null || uid == null || time == null) {

lib/features/thread/v1/models/models.dart

@@ -1,10 +1,7 @@
 import 'package:dart_mappable/dart_mappable.dart';
-import 'package:tsdm_client/extensions/string.dart';
 import 'package:tsdm_client/extensions/universal_html.dart';
 import 'package:universal_html/html.dart' as uh;
 
 part 'models.mapper.dart';
-
-part 'thread_breadcrumb.dart';
-
 part 'operation_log_item.dart';
+part 'thread_breadcrumb.dart';

lib/features/thread/v1/models/operation_log_item.dart

@@ -14,9 +14,7 @@ final class OperationLogItem with OperationLogItemMappable {
     }
 
     final username = element.querySelector('a')?.innerText;
-    final time =
-        tds[1].querySelector('span')?.attributes['title']?.parseToDateTimeUtc8() ??
-        tds[1].firstEndDeepText()?.parseToDateTimeUtc8();
+    final time = tds[1].dateTime();
     final action = tds[2].innerText;
     String? duration = tds[3].innerText;
     if (duration.isEmpty) {

lib/shared/models/forum.dart

@@ -144,6 +144,8 @@ final class Forum with ForumMappable {
         element.querySelector('td:nth-child(4) > div') ??
         // 旅行者 theme
         element.querySelector('td:nth-child(3) > div');
+    // We assume that the time text style of latest thread is always in span with title attribute, say it in recent
+    // 7 days.
     final latestThreadTime = latestThreadNode?.querySelector('cite > span')?.attributes['title']?.parseToDateTimeUtc8();
     final latestThreadTimeText = latestThreadNode?.querySelector('cite > span')?.firstEndDeepText();
     final latestThreadUrl = latestThreadNode?.querySelector('a')?.firstHref();

lib/shared/models/normal_thread.dart

@@ -292,11 +292,7 @@ class NormalThread with NormalThreadMappable {
     final threadLastReplyAuthorUrl = threadLastReplyNode?.querySelector('cite > a')?.attributes['href'];
     // We only have username here.
     final threadLastReplyAuthorName = threadLastReplyNode?.querySelector('cite > a')?.firstEndDeepText();
-    final threadLastReplyTime =
-        // Within 7 days.
-        threadLastReplyNode?.querySelector('em > a > span')?.attributes['title']?.parseToDateTimeUtc8() ??
-        // 7 days ago.
-        threadLastReplyNode?.querySelector('em > a')?.firstEndDeepText()?.parseToDateTimeUtc8();
+    final threadLastReplyTime = threadLastReplyNode?.querySelector('em > a')?.dateTime();
 
     if (threadLastReplyAuthorName == null || threadLastReplyAuthorUrl == null || threadLastReplyTime == null) {
       talker.error(

lib/shared/models/post.dart

@@ -180,9 +180,7 @@ class Post with PostMappable {
     // Recent post can grep [publishTime] in the the "title" attribute
     // in first child.
     // Otherwise fallback split time string.
-    final postPublishTime =
-        postPublishTimeNode?.querySelector('span')?.attributes['title']?.parseToDateTimeUtc8() ??
-        postPublishTimeNode?.text?.substring(4).parseToDateTimeUtc8();
+    final postPublishTime = postPublishTimeNode?.dateTime();
     // Sometimes the #postmessage_ID ID does not match postID.
     // e.g. tid=1184238
     // Use div.pcb to match it.