chore: deprecate prev reply, videopage

Closes #579

Signed-off-by: bggRGjQaUbCoE <githubaccount56556@proton.me>

lib/common/widgets/list_sheet.dart

@@ -142,7 +142,6 @@ class _ListSheetContentState extends CommonSlidePageState<ListSheetContent>
       }();
     }
     WidgetsBinding.instance.addPostFrameCallback((_) {
-      if (GStorage.collapsibleVideoPage) {
       if (mounted) {
         setState(() {
           _isInit = false;
@@ -153,11 +152,6 @@ class _ListSheetContentState extends CommonSlidePageState<ListSheetContent>
           } catch (_) {}
         });
       }
-      } else {
-        try {
-          itemScrollController[_index].jumpTo(index: currentIndex);
-        } catch (_) {}
-      }
     });
   }
 
@@ -295,7 +289,7 @@ class _ListSheetContentState extends CommonSlidePageState<ListSheetContent>
 
   @override
   Widget build(BuildContext context) {
-    if (GStorage.collapsibleVideoPage && _isInit) {
+    if (_isInit) {
       return CustomScrollView(
         physics: const NeverScrollableScrollPhysics(),
       );

lib/common/widgets/loading_widget.dart

@@ -1,5 +1,4 @@
 import 'package:PiliPlus/common/widgets/http_error.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:flutter/material.dart';
 
 Widget get loadingWidget => Center(child: CircularProgressIndicator());
@@ -19,27 +18,3 @@ Widget scrollErrorWidget({errMsg, callback}) => CustomScrollView(
         )
       ],
     );
-
-Widget replyErrorWidget(context, isSliver, errMsg, onReload) => HttpError(
-      isSliver: isSliver,
-      errMsg:
-          '${errMsg.startsWith('gRPC Error') ? '如无法加载评论：\n关闭代理\n或设置中关闭使用gRPC加载评论\n\n' : ''}$errMsg',
-      callback: onReload,
-      extraWidget: errMsg.startsWith('gRPC Error') && GlobalData().grpcReply
-          ? FilledButton.tonal(
-              onPressed: () {
-                GlobalData().grpcReply = false;
-                onReload();
-              },
-              style: ButtonStyle(
-                backgroundColor: WidgetStateProperty.resolveWith((states) {
-                  return Theme.of(context).colorScheme.primary.withAlpha(20);
-                }),
-              ),
-              child: Text(
-                '暂时关闭gRPC加载评论',
-                style: TextStyle(color: Theme.of(context).colorScheme.primary),
-              ),
-            )
-          : null,
-    );

lib/http/reply.dart

@@ -20,6 +20,7 @@ class ReplyHttp {
   static RegExp replyRegExp =
       RegExp(GStorage.banWordForReply, caseSensitive: false);
 
+  @Deprecated('Use replyListGrpc instead')
   static Future<LoadingState> replyList({
     required bool isLogin,
     required int oid,
@@ -214,6 +215,7 @@ class ReplyHttp {
     return false;
   }
 
+  @Deprecated('Use replyReplyListGrpc instead')
   static Future<LoadingState> replyReplyList({
     required bool isLogin,
     required int oid,

lib/main.dart

@@ -16,7 +16,6 @@ import 'package:hive/hive.dart';
 import 'package:PiliPlus/common/widgets/custom_toast.dart';
 import 'package:PiliPlus/http/init.dart';
 import 'package:PiliPlus/models/common/color_type.dart';
-import 'package:PiliPlus/pages/video/detail/index.dart';
 import 'package:PiliPlus/router/app_pages.dart';
 import 'package:PiliPlus/pages/main/view.dart';
 import 'package:PiliPlus/services/service_locator.dart';
@@ -218,7 +217,6 @@ class MyApp extends StatelessWidget {
           ),
           navigatorObservers: [
             FlutterSmartDialog.observer,
-            VideoDetailPage.routeObserver,
             VideoDetailPageV.routeObserver,
             MainApp.routeObserver,
           ],

lib/pages/common/common_collapse_slide_page.dart

@@ -13,7 +13,6 @@ abstract class CommonCollapseSlidePageState<T extends CommonCollapseSlidePage>
   @override
   void initState() {
     super.initState();
-    if (GStorage.collapsibleVideoPage) {
     WidgetsBinding.instance.addPostFrameCallback((_) {
       if (mounted) {
         setState(() {
@@ -22,11 +21,10 @@ abstract class CommonCollapseSlidePageState<T extends CommonCollapseSlidePage>
       }
     });
   }
-  }
 
   @override
   Widget build(BuildContext context) {
-    if (GStorage.collapsibleVideoPage && _isInit) {
+    if (_isInit) {
       return CustomScrollView(
         physics: const NeverScrollableScrollPhysics(),
       );

lib/pages/common/reply_controller.dart

@@ -69,7 +69,6 @@ abstract class ReplyController extends CommonController {
 
   @override
   bool customHandleResponse(Success response) {
-    if (GlobalData().grpcReply) {
     MainListReply replies = response.response;
     if (cursor == null) {
       count.value = replies.subjectControl.count.toInt();
@@ -87,39 +86,7 @@ abstract class ReplyController extends CommonController {
         replies.cursor.isEnd ||
         replies.replies.length >= count.value;
     loadingState.value = LoadingState.success(replies);
-    } else {
+
-      List<ReplyItemModel> replies = response.response.replies;
-      if (isLogin.not) {
-        nextOffset = response.response.cursor.paginationReply.nextOffset ?? '';
-      }
-      count.value = isLogin.not
-          ? response.response.cursor.allCount
-          : response.response.page.count ?? 0;
-      if (replies.isEmpty) {
-        isEnd = true;
-      }
-      if (currentPage == 1) {
-        if (response.response.upper.top != null) {
-          final bool flag = response.response.topReplies.any(
-              (ReplyItemModel reply) =>
-                  reply.rpid != response.response.upper.top.rpid) as bool;
-          if (flag) {
-            replies.insert(0, response.response.upper.top);
-            hasUpTop = true;
-          }
-        }
-        if ((response.response.topReplies as List?)?.isNotEmpty == true) {
-          replies.insertAll(0, response.response.topReplies);
-          hasUpTop = true;
-        }
-      } else if (loadingState.value is Success) {
-        replies.insertAll(0, (loadingState.value as Success).response.replies);
-      }
-      if (isEnd.not && replies.length >= count.value) {
-        isEnd = true;
-      }
-      loadingState.value = LoadingState.success(response.response);
-    }
     return true;
   }
 
@@ -149,15 +116,12 @@ abstract class ReplyController extends CommonController {
     int index = 0,
     ReplyType? replyType,
   }) {
-    dynamic key = oid ??
+    dynamic key = oid ?? replyItem.oid + replyItem.id;
-        replyItem.oid +
-            (GlobalData().grpcReply ? replyItem.id : replyItem.rpid);
     Navigator.of(context)
         .push(
       GetDialogRoute(
         pageBuilder: (buildContext, animation, secondaryAnimation) {
-          return GlobalData().grpcReply
+          return ReplyPage(
-              ? ReplyPage(
             oid: oid ?? replyItem.oid.toInt(),
             root: oid != null ? 0 : replyItem.id.toInt(),
             parent: oid != null ? 0 : replyItem.id.toInt(),
@@ -169,19 +133,6 @@ abstract class ReplyController extends CommonController {
             onSave: (reply) {
               savedReplies[key] = reply;
             },
-                )
-              : ReplyPage(
-                  oid: oid ?? replyItem.oid,
-                  root: oid != null ? 0 : replyItem.rpid,
-                  parent: oid != null ? 0 : replyItem.rpid,
-                  replyType: replyItem != null
-                      ? ReplyType.values[replyItem.type.toInt()]
-                      : replyType,
-                  replyItem: replyItem,
-                  initialValue: savedReplies[key],
-                  onSave: (reply) {
-                    savedReplies[key] = reply;
-                  },
           );
         },
         transitionDuration: const Duration(milliseconds: 500),
@@ -204,7 +155,6 @@ abstract class ReplyController extends CommonController {
       (res) {
         if (res != null) {
           savedReplies[key] = null;
-          if (GlobalData().grpcReply) {
           ReplyInfo replyInfo = Utils.replyCast(res);
           MainListReply response = loadingState.value is Success
               ? (loadingState.value as Success).response
@@ -236,46 +186,12 @@ abstract class ReplyController extends CommonController {
               mid: replyInfo.mid.toInt(),
             );
           }
-          } else {
-            ReplyData response = loadingState.value is Success
-                ? (loadingState.value as Success).response
-                : ReplyData();
-            response.replies ??= <ReplyItemModel>[];
-            ReplyItemModel replyInfo = ReplyItemModel.fromJson(res, '');
-            if (oid != null) {
-              response.replies?.insert(hasUpTop ? 1 : 0, replyInfo);
-            } else {
-              response.replies?[index].replies ??= <ReplyItemModel>[];
-              response.replies?[index].replies?.add(replyInfo);
-            }
-            count.value += 1;
-            loadingState.value = LoadingState.success(response);
-            if (enableCommAntifraud && context.mounted) {
-              checkReply(
-                context: context,
-                oid: oid ?? replyItem.oid,
-                rpid: replyItem?.rpid,
-                replyType: replyItem?.type.toInt() ??
-                    replyType?.index ??
-                    ReplyType.video.index,
-                replyId: replyInfo.rpid ?? 0,
-                message: replyInfo.content?.message ?? '',
-                //
-                root: replyInfo.root,
-                parent: replyInfo.parent,
-                ctime: replyInfo.ctime,
-                pictures: replyInfo.content?.pictures,
-                mid: replyInfo.mid,
-              );
-            }
-          }
         }
       },
     );
   }
 
   onMDelete(rpid, frpid) {
-    if (GlobalData().grpcReply) {
     MainListReply response = (loadingState.value as Success).response;
     if (frpid == null) {
       response.replies.removeWhere((item) {
@@ -284,8 +200,7 @@ abstract class ReplyController extends CommonController {
     } else {
       response.replies.map((item) {
         if (item.id == frpid) {
-            return item
+          return item..replies.removeWhere((reply) => reply.id.toInt() == rpid);
-              ..replies.removeWhere((reply) => reply.id.toInt() == rpid);
         } else {
           return item;
         }
@@ -293,23 +208,6 @@ abstract class ReplyController extends CommonController {
     }
     count.value -= 1;
     loadingState.value = LoadingState.success(response);
-    } else {
-      ReplyData response = (loadingState.value as Success).response;
-      response.replies = frpid == null
-          ? response.replies?.where((item) => item.rpid != rpid).toList()
-          : response.replies?.map((item) {
-              if (item.rpid == frpid) {
-                return item
-                  ..replies = item.replies
-                      ?.where((reply) => reply.rpid != rpid)
-                      .toList();
-              } else {
-                return item;
-              }
-            }).toList();
-      count.value -= 1;
-      loadingState.value = LoadingState.success(response);
-    }
   }
 
   void onCheckReply(context, item) {

lib/pages/dynamics/detail/controller.dart

@@ -3,7 +3,6 @@ import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/models/common/reply_type.dart';
 import 'package:PiliPlus/models/dynamics/result.dart';
 import 'package:PiliPlus/pages/common/reply_controller.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/id_utils.dart';
 import 'package:PiliPlus/utils/storage.dart';
 import 'package:get/get.dart';
@@ -47,8 +46,7 @@ class DynamicDetailController extends ReplyController {
   }
 
   @override
-  Future<LoadingState> customGetData() => GlobalData().grpcReply
+  Future<LoadingState> customGetData() => ReplyHttp.replyListGrpc(
-      ? ReplyHttp.replyListGrpc(
         type: type,
         oid: oid,
         cursor: CursorReq(
@@ -56,14 +54,5 @@ class DynamicDetailController extends ReplyController {
           mode: mode.value,
         ),
         antiGoodsReply: antiGoodsReply,
-        )
-      : ReplyHttp.replyList(
-          isLogin: isLogin,
-          oid: oid,
-          nextOffset: nextOffset,
-          type: type,
-          sort: sortType.value.index,
-          page: currentPage,
-          antiGoodsReply: antiGoodsReply,
       );
 }

lib/pages/dynamics/detail/view.dart

@@ -8,10 +8,8 @@ import 'package:PiliPlus/http/constants.dart';
 import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/models/common/reply_sort_type.dart';
 import 'package:PiliPlus/pages/dynamics/repost_dyn_panel.dart';
-import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item.dart';
 import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item_grpc.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/storage.dart';
 import 'package:PiliPlus/utils/utils.dart';
 import 'package:easy_debounce/easy_throttle.dart';
@@ -170,7 +168,7 @@ class _DynamicDetailPageState extends State<DynamicDetailPage>
   void replyReply(context, replyItem, id, isTop) {
     EasyThrottle.throttle('replyReply', const Duration(milliseconds: 500), () {
       int oid = replyItem.oid.toInt();
-      int rpid = GlobalData().grpcReply ? replyItem.id.toInt() : replyItem.rpid;
+      int rpid = replyItem.id.toInt();
       Widget replyReplyPage(
               [bool automaticallyImplyLeading = true,
               VoidCallback? onDispose]) =>
@@ -813,8 +811,7 @@ class _DynamicDetailPageState extends State<DynamicDetailPage>
                       ),
                     );
                   } else {
-                    return GlobalData().grpcReply
+                    return ReplyItemGrpc(
-                        ? ReplyItemGrpc(
                       replyItem: loadingState.response.replies[index],
                       showReplyRow: true,
                       replyLevel: '1',
@@ -829,31 +826,11 @@ class _DynamicDetailPageState extends State<DynamicDetailPage>
                         );
                       },
                       onDelete: _dynamicDetailController.onMDelete,
-                            isTop:
+                      isTop: _dynamicDetailController.hasUpTop && index == 0,
-                                _dynamicDetailController.hasUpTop && index == 0,
                       upMid: loadingState.response.subjectControl.upMid,
                       callback: _getImageCallback,
-                            onCheckReply: (item) => _dynamicDetailController
+                      onCheckReply: (item) =>
-                                .onCheckReply(context, item),
+                          _dynamicDetailController.onCheckReply(context, item),
-                          )
-                        : ReplyItem(
-                            replyItem: loadingState.response.replies[index],
-                            showReplyRow: true,
-                            replyLevel: '1',
-                            replyReply: (replyItem, id, isTop) =>
-                                replyReply(context, replyItem, id, isTop),
-                            replyType: ReplyType.values[replyType],
-                            onReply: () {
-                              _dynamicDetailController.onReply(
-                                context,
-                                replyItem: loadingState.response.replies[index],
-                                index: index,
-                              );
-                            },
-                            onDelete: _dynamicDetailController.onMDelete,
-                            callback: _getImageCallback,
-                            onCheckReply: (item) => _dynamicDetailController
-                                .onCheckReply(context, item),
                     );
                   }
                 },
@@ -863,11 +840,9 @@ class _DynamicDetailPageState extends State<DynamicDetailPage>
           : HttpError(
               callback: _dynamicDetailController.onReload,
             ),
-      Error() => replyErrorWidget(
+      Error() => errorWidget(
-          context,
+          errMsg: loadingState.errMsg,
-          true,
+          callback: _dynamicDetailController.onReload,
-          loadingState.errMsg,
-          _dynamicDetailController.onReload,
         ),
       LoadingState() => throw UnimplementedError(),
     };

lib/pages/html/controller.dart

@@ -5,7 +5,6 @@ import 'package:PiliPlus/http/video.dart';
 import 'package:PiliPlus/models/dynamics/result.dart';
 import 'package:PiliPlus/pages/common/reply_controller.dart';
 import 'package:PiliPlus/pages/mine/controller.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/storage.dart';
 import 'package:PiliPlus/utils/url_utils.dart';
 import 'package:flutter/material.dart';
@@ -88,8 +87,7 @@ class HtmlRenderController extends ReplyController {
 
   @override
   Future<LoadingState> customGetData() {
-    return GlobalData().grpcReply
+    return ReplyHttp.replyListGrpc(
-        ? ReplyHttp.replyListGrpc(
       type: type,
       oid: oid.value,
       cursor: CursorReq(
@@ -97,15 +95,6 @@ class HtmlRenderController extends ReplyController {
         mode: mode.value,
       ),
       antiGoodsReply: antiGoodsReply,
-          )
-        : ReplyHttp.replyList(
-            isLogin: isLogin,
-            oid: oid.value,
-            nextOffset: nextOffset,
-            type: type,
-            sort: sortType.value.index,
-            page: currentPage,
-            antiGoodsReply: antiGoodsReply,
     );
   }
 }

lib/pages/html/view.dart

@@ -8,10 +8,8 @@ import 'package:PiliPlus/http/constants.dart';
 import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/models/common/reply_sort_type.dart';
 import 'package:PiliPlus/pages/dynamics/repost_dyn_panel.dart';
-import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item.dart';
 import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item_grpc.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/storage.dart';
 import 'package:PiliPlus/utils/utils.dart';
 import 'package:easy_debounce/easy_throttle.dart';
@@ -167,7 +165,7 @@ class _HtmlRenderPageState extends State<HtmlRenderPage>
   void replyReply(context, replyItem, id, isTop) {
     EasyThrottle.throttle('replyReply', const Duration(milliseconds: 500), () {
       int oid = replyItem.oid.toInt();
-      int rpid = GlobalData().grpcReply ? replyItem.id.toInt() : replyItem.rpid;
+      int rpid = replyItem.id.toInt();
       Widget replyReplyPage(
               [bool automaticallyImplyLeading = true,
               VoidCallback? onDispose]) =>
@@ -786,8 +784,7 @@ class _HtmlRenderPageState extends State<HtmlRenderPage>
                     ),
                   );
                 } else {
-                  return GlobalData().grpcReply
+                  return ReplyItemGrpc(
-                      ? ReplyItemGrpc(
                     replyItem: loadingState.response.replies[index],
                     showReplyRow: true,
                     replyLevel: '1',
@@ -807,25 +804,6 @@ class _HtmlRenderPageState extends State<HtmlRenderPage>
                     callback: _getImageCallback,
                     onCheckReply: (item) =>
                         _htmlRenderCtr.onCheckReply(context, item),
-                        )
-                      : ReplyItem(
-                          replyItem: loadingState.response.replies[index],
-                          showReplyRow: true,
-                          replyLevel: '1',
-                          replyReply: (replyItem, id, isTop) =>
-                              replyReply(context, replyItem, id, isTop),
-                          replyType: ReplyType.values[type],
-                          onReply: () {
-                            _htmlRenderCtr.onReply(
-                              context,
-                              replyItem: loadingState.response.replies[index],
-                              index: index,
-                            );
-                          },
-                          onDelete: _htmlRenderCtr.onMDelete,
-                          callback: _getImageCallback,
-                          onCheckReply: (item) =>
-                              _htmlRenderCtr.onCheckReply(context, item),
                   );
                 }
               },
@@ -833,11 +811,9 @@ class _HtmlRenderPageState extends State<HtmlRenderPage>
           : HttpError(
               callback: _htmlRenderCtr.onReload,
             ),
-      Error() => replyErrorWidget(
+      Error() => errorWidget(
-          context,
+          errMsg: loadingState.errMsg,
-          true,
+          callback: _htmlRenderCtr.onReload,
-          loadingState.errMsg,
-          _htmlRenderCtr.onReload,
         ),
       LoadingState() => throw UnimplementedError(),
     };

lib/pages/main/controller.dart

@@ -2,13 +2,11 @@ import 'dart:async';
 
 import 'package:PiliPlus/grpc/grpc_repo.dart';
 import 'package:PiliPlus/http/api.dart';
-import 'package:PiliPlus/http/common.dart';
 import 'package:PiliPlus/http/init.dart';
 import 'package:PiliPlus/pages/dynamics/view.dart';
 import 'package:PiliPlus/pages/home/view.dart';
 import 'package:PiliPlus/pages/media/view.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/utils.dart';
 import 'package:get/get.dart';
 import 'package:flutter/material.dart';
@@ -165,19 +163,11 @@ class MainController extends GetxController {
     if (!isLogin.value || dynIndex == -1) {
       return;
     }
-    if (GlobalData().grpcReply) {
     await GrpcRepo.dynRed().then((res) {
       if (res['status']) {
         setCount(res['data']);
       }
     });
-    } else {
-      await CommonHttp.unReadDynamic().then((res) {
-        if (res['status']) {
-          setCount(res['data']);
-        }
-      });
-    }
   }
 
   void setCount([int count = 0]) async {

lib/pages/setting/widgets/model.dart

@@ -1515,21 +1515,6 @@ List<SettingsModel> get extraSettings => [
           );
         },
       ),
-      SettingsModel(
-        settingsType: SettingsType.sw1tch,
-        title: '使用gRPC加载评论',
-        subtitle: '如无法加载评论，可关闭\n非gRPC楼中楼无法定位评论、按热度/时间排序、查看对话',
-        leading: SizedBox(
-          height: 24,
-          width: 24,
-          child: Icon(MdiIcons.google, size: 20),
-        ),
-        setKey: SettingBoxKey.grpcReply,
-        defaultVal: true,
-        onChanged: (value) {
-          GlobalData().grpcReply = value;
-        },
-      ),
       SettingsModel(
         settingsType: SettingsType.sw1tch,
         title: '显示视频分段信息',
@@ -2072,16 +2057,6 @@ List<SettingsModel> get extraSettings => [
         setKey: SettingBoxKey.antiGoodsReply,
         defaultVal: false,
       ),
-      SettingsModel(
-        settingsType: SettingsType.sw1tch,
-        title: '使用可折叠的播放页面',
-        leading: const Icon(Icons.video_settings),
-        setKey: SettingBoxKey.collapsibleVideoPage,
-        defaultVal: true,
-        onChanged: (value) {
-          GStorage.collapsibleVideoPage = value;
-        },
-      ),
       SettingsModel(
         settingsType: SettingsType.sw1tch,
         title: '侧滑关闭二级评论页面',

lib/pages/video/detail/controller.dart

@@ -146,10 +146,6 @@ class VideoDetailController extends GetxController
             ? 'horizontal'
             : 'vertical'
         : 'horizontal';
-    if (GStorage.collapsibleVideoPage.not) {
-      this.direction.value = direction;
-      return;
-    }
     if (scrollCtr.hasClients.not) {
       videoHeight = direction == 'vertical' ? maxVideoHeight : minVideoHeight;
       this.direction.value = direction;
@@ -1548,11 +1544,9 @@ class VideoDetailController extends GetxController
   @override
   void onClose() {
     tabCtr.dispose();
-    if (GStorage.collapsibleVideoPage) {
     scrollCtr.removeListener(scrollListener);
     scrollCtr.dispose;
     animationController.dispose();
-    }
     super.onClose();
   }
 
@@ -1561,7 +1555,7 @@ class VideoDetailController extends GetxController
     videoUrl = null;
     audioUrl = null;
 
-    if (GStorage.collapsibleVideoPage && scrollRatio.value != 0) {
+    if (scrollRatio.value != 0) {
       scrollRatio.refresh();
     }
 

lib/pages/video/detail/reply/controller.dart

@@ -1,9 +1,7 @@
 import 'package:PiliPlus/grpc/app/main/community/reply/v1/reply.pb.dart';
 import 'package:PiliPlus/http/loading_state.dart';
-import 'package:PiliPlus/models/common/reply_type.dart';
 import 'package:PiliPlus/pages/common/reply_controller.dart';
 import 'package:PiliPlus/http/reply.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/id_utils.dart';
 import 'package:fixnum/fixnum.dart' as $fixnum;
 import 'package:flutter/material.dart';
@@ -38,23 +36,13 @@ class VideoReplyController extends ReplyController
   }
 
   @override
-  Future<LoadingState> customGetData() => GlobalData().grpcReply
+  Future<LoadingState> customGetData() => ReplyHttp.replyListGrpc(
-      ? ReplyHttp.replyListGrpc(
         oid: aid,
         cursor: CursorReq(
           next: cursor?.next ?? $fixnum.Int64(0),
           mode: mode.value,
         ),
         antiGoodsReply: antiGoodsReply,
-        )
-      : ReplyHttp.replyList(
-          isLogin: isLogin,
-          oid: aid,
-          nextOffset: nextOffset,
-          type: ReplyType.video.index,
-          sort: sortType.value.index,
-          page: currentPage,
-          antiGoodsReply: antiGoodsReply,
       );
 
   @override

lib/pages/video/detail/reply/view.dart

@@ -4,11 +4,8 @@ import 'package:PiliPlus/common/widgets/refresh_indicator.dart';
 import 'package:PiliPlus/common/widgets/http_error.dart';
 import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/models/common/reply_sort_type.dart';
-import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item.dart';
 import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item_grpc.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
-import 'package:PiliPlus/utils/storage.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/rendering.dart';
 import 'package:get/get.dart';
@@ -75,7 +72,6 @@ class _VideoReplyPanelState extends State<VideoReplyPanel>
   @override
   void didUpdateWidget(VideoReplyPanel oldWidget) {
     super.didUpdateWidget(oldWidget);
-    if (GStorage.collapsibleVideoPage) {
     _videoReplyController.showFab();
     if (widget.needController != false) {
       _videoReplyController.scrollController.addListener(listener);
@@ -83,7 +79,6 @@ class _VideoReplyPanelState extends State<VideoReplyPanel>
       _videoReplyController.scrollController.removeListener(listener);
     }
   }
-  }
 
   @override
   void dispose() {
@@ -240,8 +235,7 @@ class _VideoReplyPanelState extends State<VideoReplyPanel>
                       ),
                     );
                   } else {
-                    return GlobalData().grpcReply
+                    return ReplyItemGrpc(
-                        ? ReplyItemGrpc(
                       replyItem: loadingState.response.replies[index],
                       showReplyRow: true,
                       replyLevel: replyLevel,
@@ -261,29 +255,8 @@ class _VideoReplyPanelState extends State<VideoReplyPanel>
                       onViewImage: widget.onViewImage,
                       onDismissed: widget.onDismissed,
                       callback: widget.callback,
-                            onCheckReply: (item) => _videoReplyController
+                      onCheckReply: (item) =>
-                                .onCheckReply(context, item),
+                          _videoReplyController.onCheckReply(context, item),
-                          )
-                        : ReplyItem(
-                            replyItem: loadingState.response.replies[index],
-                            showReplyRow: true,
-                            replyLevel: replyLevel,
-                            replyReply: widget.replyReply,
-                            replyType: ReplyType.video,
-                            onReply: () {
-                              _videoReplyController.onReply(
-                                context,
-                                replyItem: loadingState.response.replies[index],
-                                index: index,
-                              );
-                            },
-                            onDelete: _videoReplyController.onMDelete,
-                            onViewImage: widget.onViewImage,
-                            onDismissed: widget.onDismissed,
-                            getTag: () => heroTag,
-                            callback: widget.callback,
-                            onCheckReply: (item) => _videoReplyController
-                                .onCheckReply(context, item),
                     );
                   }
                 },
@@ -294,11 +267,9 @@ class _VideoReplyPanelState extends State<VideoReplyPanel>
               errMsg: '还没有评论',
               callback: _videoReplyController.onReload,
             ),
-      Error() => replyErrorWidget(
+      Error() => errorWidget(
-          context,
+          errMsg: loadingState.errMsg,
-          true,
+          callback: _videoReplyController.onReload,
-          loadingState.errMsg,
-          _videoReplyController.onReload,
         ),
       LoadingState() => throw UnimplementedError(),
     };

lib/pages/video/detail/reply/widgets/reply_item.dart

@@ -26,6 +26,7 @@ import 'package:PiliPlus/utils/utils.dart';
 import 'zan.dart';
 import 'package:html/parser.dart' show parse;
 
+@Deprecated('Use ReplyItemGrpc instead')
 class ReplyItem extends StatelessWidget {
   const ReplyItem({
     super.key,

lib/pages/video/detail/reply_new/reply_page.dart

@@ -5,7 +5,6 @@ import 'package:PiliPlus/main.dart';
 import 'package:PiliPlus/pages/common/common_publish_page.dart';
 import 'package:PiliPlus/pages/emote/view.dart';
 import 'package:PiliPlus/pages/video/detail/reply_new/toolbar_icon_button.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/storage.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_smart_dialog/flutter_smart_dialog.dart';
@@ -275,7 +274,7 @@ class _ReplyPageState extends CommonPublishPageState<ReplyPage> {
       root: widget.root!,
       parent: widget.parent!,
       message: widget.replyItem != null && widget.replyItem.root != 0
-          ? ' 回复 @${GlobalData().grpcReply ? widget.replyItem.member.name : widget.replyItem.member.uname} : $message'
+          ? ' 回复 @${widget.replyItem.member.name} : $message'
           : message,
       pictures: pictures,
       syncToDynamic: _syncToDynamic.value,

lib/pages/video/detail/reply_reply/controller.dart

@@ -1,8 +1,6 @@
 import 'package:PiliPlus/grpc/app/main/community/reply/v1/reply.pb.dart';
 import 'package:PiliPlus/http/loading_state.dart';
-import 'package:PiliPlus/models/video/reply/item.dart';
 import 'package:PiliPlus/pages/common/reply_controller.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/id_utils.dart';
 import 'package:PiliPlus/utils/storage.dart';
 import 'package:flutter/material.dart';
@@ -61,7 +59,6 @@ class VideoReplyReplyController extends ReplyController
 
   @override
   bool customHandleResponse(Success response) {
-    if (GlobalData().grpcReply) {
     dynamic replies = response.response;
     // reply2Reply // isDialogue.not
     if (replies is DetailListReply) {
@@ -124,8 +121,7 @@ class VideoReplyReplyController extends ReplyController
         isEnd = true;
       }
     } else {
-        if (replies.cursor.isEnd ||
+      if (replies.cursor.isEnd || replies.root.replies.length >= count.value) {
-            replies.root.replies.length >= count.value) {
         isEnd = true;
       }
     }
@@ -134,24 +130,7 @@ class VideoReplyReplyController extends ReplyController
     } else {
       loadingState.value = LoadingState.success(replies.root.replies);
     }
-    } else {
+
-      if (response.response.root != null) {
-        firstFloor = response.response.root;
-        hasRoot = true;
-      }
-      List<ReplyItemModel> replies = response.response.replies;
-      count.value = response.response.page.count;
-      if (replies.isEmpty) {
-        isEnd = true;
-      }
-      if (currentPage != 1 && loadingState.value is Success) {
-        replies.insertAll(0, (loadingState.value as Success).response);
-      }
-      if (replies.length >= response.response.page.count) {
-        isEnd = true;
-      }
-      loadingState.value = LoadingState.success(replies);
-    }
     return true;
   }
 
@@ -168,8 +147,7 @@ class VideoReplyReplyController extends ReplyController
           ),
           antiGoodsReply: antiGoodsReply,
         )
-      : GlobalData().grpcReply
+      : ReplyHttp.replyReplyListGrpc(
-          ? ReplyHttp.replyReplyListGrpc(
           type: replyType.index,
           oid: oid,
           root: rpid,
@@ -179,14 +157,6 @@ class VideoReplyReplyController extends ReplyController
             mode: mode.value,
           ),
           antiGoodsReply: antiGoodsReply,
-            )
-          : ReplyHttp.replyReplyList(
-              isLogin: isLogin,
-              oid: oid,
-              root: rpid,
-              pageNum: currentPage,
-              type: replyType.index,
-              antiGoodsReply: antiGoodsReply,
         );
 
   @override

lib/pages/video/detail/reply_reply/view.dart

@@ -2,13 +2,10 @@ import 'package:PiliPlus/common/widgets/loading_widget.dart';
 import 'package:PiliPlus/common/widgets/refresh_indicator.dart';
 import 'package:PiliPlus/grpc/app/main/community/reply/v1/reply.pb.dart';
 import 'package:PiliPlus/http/loading_state.dart';
-import 'package:PiliPlus/models/video/reply/item.dart';
 import 'package:PiliPlus/pages/common/common_slide_page.dart';
-import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item.dart';
 import 'package:PiliPlus/pages/video/detail/reply/widgets/reply_item_grpc.dart';
 import 'package:PiliPlus/pages/video/detail/reply_new/reply_page.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/utils.dart';
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
@@ -181,8 +178,7 @@ class _VideoReplyReplyPanelState
                           _videoReplyReplyController.loadingState.value, index);
                     } else if (firstFloor != null) {
                       if (index == 0) {
-                        return GlobalData().grpcReply
+                        return ReplyItemGrpc(
-                            ? ReplyItemGrpc(
                           replyItem: firstFloor,
                           replyLevel: '2',
                           showReplyRow: false,
@@ -196,25 +192,8 @@ class _VideoReplyReplyPanelState
                           onViewImage: widget.onViewImage,
                           onDismissed: widget.onDismissed,
                           callback: _getImageCallback,
-                                onCheckReply: (item) =>
+                          onCheckReply: (item) => _videoReplyReplyController
-                                    _videoReplyReplyController.onCheckReply(
+                              .onCheckReply(context, item),
-                                        context, item),
-                              )
-                            : ReplyItem(
-                                replyItem: firstFloor,
-                                replyLevel: '2',
-                                showReplyRow: false,
-                                replyType: widget.replyType,
-                                needDivider: false,
-                                onReply: () {
-                                  _onReply(firstFloor, -1);
-                                },
-                                onViewImage: widget.onViewImage,
-                                onDismissed: widget.onDismissed,
-                                callback: _getImageCallback,
-                                onCheckReply: (item) =>
-                                    _videoReplyReplyController.onCheckReply(
-                                        context, item),
                         );
                       } else if (index == 1) {
                         return Divider(
@@ -273,7 +252,6 @@ class _VideoReplyReplyPanelState
                     )
                   : const SizedBox.shrink(),
             ),
-            if (GlobalData().grpcReply)
             SizedBox(
               height: 35,
               child: TextButton.icon(
@@ -285,8 +263,7 @@ class _VideoReplyReplyPanelState
                 ),
                 label: Obx(
                   () => Text(
-                      _videoReplyReplyController.mode.value ==
+                    _videoReplyReplyController.mode.value == Mode.MAIN_LIST_HOT
-                              Mode.MAIN_LIST_HOT
                         ? '按热度'
                         : '按时间',
                     style: TextStyle(
@@ -333,7 +310,7 @@ class _VideoReplyReplyPanelState
 
   void _onReply(dynamic item, int index) {
     dynamic oid = item?.oid.toInt();
-    dynamic root = GlobalData().grpcReply ? item?.id.toInt() : item?.rpid;
+    dynamic root = item?.id.toInt();
     dynamic key = oid + root;
 
     Navigator.of(context)
@@ -371,7 +348,6 @@ class _VideoReplyReplyPanelState
         .then((res) {
       if (res != null) {
         _savedReplies[key] = null;
-        if (GlobalData().grpcReply) {
         ReplyInfo replyInfo = Utils.replyCast(res);
         List list = _videoReplyReplyController.loadingState.value is Success
             ? (_videoReplyReplyController.loadingState.value as Success)
@@ -399,33 +375,6 @@ class _VideoReplyReplyPanelState
             mid: replyInfo.mid.toInt(),
           );
         }
-        } else {
-          List list = _videoReplyReplyController.loadingState.value is Success
-              ? (_videoReplyReplyController.loadingState.value as Success)
-                  .response
-              : <ReplyItemModel>[];
-          ReplyItemModel replyInfo = ReplyItemModel.fromJson(res, '');
-          list.insert(index + 1, replyInfo);
-          _videoReplyReplyController.count.value += 1;
-          _videoReplyReplyController.loadingState.value =
-              LoadingState.success(list);
-          if (_videoReplyReplyController.enableCommAntifraud && mounted) {
-            _videoReplyReplyController.checkReply(
-              context: context,
-              oid: oid,
-              rpid: root,
-              replyType: widget.replyType.index,
-              replyId: replyInfo.rpid ?? 0,
-              message: replyInfo.content?.message ?? '',
-              //
-              root: replyInfo.root,
-              parent: replyInfo.parent,
-              ctime: replyInfo.ctime,
-              pictures: replyInfo.content?.pictures,
-              mid: replyInfo.mid,
-            );
-          }
-        }
       }
     });
   }
@@ -489,19 +438,16 @@ class _VideoReplyReplyPanelState
             return _replyItem(loadingState.response[index], index);
           }
         }(),
-      Error() => replyErrorWidget(
+      Error() => errorWidget(
-          context,
+          errMsg: loadingState.errMsg,
-          false,
+          callback: _videoReplyReplyController.onReload,
-          loadingState.errMsg,
-          _videoReplyReplyController.onReload,
         ),
       LoadingState() => throw UnimplementedError(),
     };
   }
 
   Widget _replyItem(replyItem, index) {
-    return GlobalData().grpcReply
+    return ReplyItemGrpc(
-        ? ReplyItemGrpc(
       replyItem: replyItem,
       replyLevel: widget.isDialogue ? '3' : '2',
       showReplyRow: false,
@@ -511,8 +457,7 @@ class _VideoReplyReplyPanelState
       },
       onDelete: (rpid, frpid) {
         List list =
-                  (_videoReplyReplyController.loadingState.value as Success)
+            (_videoReplyReplyController.loadingState.value as Success).response;
-                      .response;
         list = list.where((item) => item.id != rpid).toList();
         _videoReplyReplyController.count.value -= 1;
         _videoReplyReplyController.loadingState.value =
@@ -537,29 +482,6 @@ class _VideoReplyReplyPanelState
       callback: _getImageCallback,
       onCheckReply: (item) =>
           _videoReplyReplyController.onCheckReply(context, item),
-          )
-        : ReplyItem(
-            replyItem: replyItem,
-            replyLevel: '2',
-            showReplyRow: false,
-            replyType: widget.replyType,
-            onReply: () {
-              _onReply(replyItem, index);
-            },
-            onDelete: (rpid, frpid) {
-              List list =
-                  (_videoReplyReplyController.loadingState.value as Success)
-                      .response;
-              list.removeWhere((item) => item.rpid == rpid);
-              _videoReplyReplyController.count.value -= 1;
-              _videoReplyReplyController.loadingState.value =
-                  LoadingState.success(list);
-            },
-            onViewImage: widget.onViewImage,
-            onDismissed: widget.onDismissed,
-            callback: _getImageCallback,
-            onCheckReply: (item) =>
-                _videoReplyReplyController.onCheckReply(context, item),
     );
   }
 

lib/pages/video/detail/view.dart

@@ -6,12 +6,9 @@ import 'package:PiliPlus/common/constants.dart';
 import 'package:PiliPlus/common/widgets/list_sheet.dart';
 import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/main.dart';
-import 'package:PiliPlus/models/bangumi/info.dart';
 import 'package:PiliPlus/models/common/reply_type.dart';
 import 'package:PiliPlus/pages/bangumi/introduction/widgets/intro_detail.dart'
     as bangumi;
-import 'package:PiliPlus/pages/video/detail/introduction/widgets/intro_detail.dart'
-    as video;
 import 'package:PiliPlus/pages/video/detail/introduction/widgets/page.dart';
 import 'package:PiliPlus/pages/video/detail/introduction/widgets/season.dart';
 import 'package:PiliPlus/pages/video/detail/member/controller.dart';
@@ -21,7 +18,6 @@ import 'package:PiliPlus/pages/video/detail/view_point/view_points_page.dart';
 import 'package:PiliPlus/pages/video/detail/widgets/ai_detail.dart';
 import 'package:PiliPlus/utils/download.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/id_utils.dart';
 import 'package:PiliPlus/utils/utils.dart';
 import 'package:auto_orientation/auto_orientation.dart';
@@ -53,6 +49,7 @@ import '../../../services/shutdown_timer_service.dart';
 import 'widgets/header_control.dart';
 import 'package:PiliPlus/common/widgets/spring_physics.dart';
 
+@Deprecated('Use VideoDetailPageV instead')
 class VideoDetailPage extends StatefulWidget {
   const VideoDetailPage({super.key});
 
@@ -1755,7 +1752,7 @@ class _VideoDetailPageState extends State<VideoDetailPage>
   void replyReply(replyItem, id, isTop) {
     EasyThrottle.throttle('replyReply', const Duration(milliseconds: 500), () {
       int oid = replyItem.oid.toInt();
-      int rpid = GlobalData().grpcReply ? replyItem.id.toInt() : replyItem.rpid;
+      int rpid = replyItem.id.toInt();
       videoDetailController.childKey.currentState?.showBottomSheet(
         backgroundColor: Colors.transparent,
         (context) => VideoReplyReplyPanel(
@@ -1785,14 +1782,9 @@ class _VideoDetailPageState extends State<VideoDetailPage>
     videoDetailController.childKey.currentState?.showBottomSheet(
       shape: const RoundedRectangleBorder(),
       backgroundColor: themeData.colorScheme.surface,
-      (context) => videoDetail is BangumiInfoModel
+      (context) => bangumi.IntroDetail(
-          ? bangumi.IntroDetail(
         bangumiDetail: videoDetail,
         videoTags: videoTags,
-            )
-          : video.IntroDetail(
-              videoDetail: videoDetail,
-              videoTags: videoTags,
       ),
     );
   }

lib/pages/video/detail/view_v.dart

@@ -7,12 +7,9 @@ import 'package:PiliPlus/common/constants.dart';
 import 'package:PiliPlus/common/widgets/list_sheet.dart';
 import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/main.dart';
-import 'package:PiliPlus/models/bangumi/info.dart';
 import 'package:PiliPlus/models/common/reply_type.dart';
 import 'package:PiliPlus/pages/bangumi/introduction/widgets/intro_detail.dart'
     as bangumi;
-import 'package:PiliPlus/pages/video/detail/introduction/widgets/intro_detail.dart'
-    as video;
 import 'package:PiliPlus/pages/video/detail/introduction/widgets/page.dart';
 import 'package:PiliPlus/pages/video/detail/introduction/widgets/season.dart';
 import 'package:PiliPlus/pages/video/detail/member/controller.dart';
@@ -22,7 +19,6 @@ import 'package:PiliPlus/pages/video/detail/view_point/view_points_page.dart';
 import 'package:PiliPlus/pages/video/detail/widgets/ai_detail.dart';
 import 'package:PiliPlus/utils/download.dart';
 import 'package:PiliPlus/utils/extension.dart';
-import 'package:PiliPlus/utils/global_data.dart';
 import 'package:PiliPlus/utils/id_utils.dart';
 import 'package:PiliPlus/utils/utils.dart';
 import 'package:auto_orientation/auto_orientation.dart';
@@ -2206,7 +2202,7 @@ class _VideoDetailPageVState extends State<VideoDetailPageV>
   void replyReply(replyItem, id, isTop) {
     EasyThrottle.throttle('replyReply', const Duration(milliseconds: 500), () {
       int oid = replyItem.oid.toInt();
-      int rpid = GlobalData().grpcReply ? replyItem.id.toInt() : replyItem.rpid;
+      int rpid = replyItem.id.toInt();
       videoDetailController.childKey.currentState?.showBottomSheet(
         backgroundColor: Colors.transparent,
         (context) => VideoReplyReplyPanel(
@@ -2235,14 +2231,9 @@ class _VideoDetailPageVState extends State<VideoDetailPageV>
   showIntroDetail(videoDetail, videoTags) {
     videoDetailController.childKey.currentState?.showBottomSheet(
       backgroundColor: Colors.transparent,
-      (context) => videoDetail is BangumiInfoModel
+      (context) => bangumi.IntroDetail(
-          ? bangumi.IntroDetail(
         bangumiDetail: videoDetail,
         videoTags: videoTags,
-            )
-          : video.IntroDetail(
-              videoDetail: videoDetail,
-              videoTags: videoTags,
       ),
     );
   }

lib/utils/global_data.dart

@@ -1,8 +1,6 @@
 class GlobalData {
   int imgQuality = 10;
 
-  bool grpcReply = true;
-
   int replyLengthLimit = 6;
 
   // 私有构造函数

lib/utils/storage.dart

@@ -264,9 +264,6 @@ class GStorage {
   static double get subtitleFontScaleFS =>
       setting.get(SettingBoxKey.subtitleFontScaleFS, defaultValue: 1.5);
 
-  static bool get grpcReply =>
-      setting.get(SettingBoxKey.grpcReply, defaultValue: true);
-
   static bool get showViewPoints =>
       setting.get(SettingBoxKey.showViewPoints, defaultValue: true);
 
@@ -402,9 +399,6 @@ class GStorage {
   static bool get expandDynLivePanel => GStorage.setting
       .get(SettingBoxKey.expandDynLivePanel, defaultValue: false);
 
-  static bool collapsibleVideoPage = GStorage.setting
-      .get(SettingBoxKey.collapsibleVideoPage, defaultValue: true);
-
   static bool slideDismissReplyPage = GStorage.setting
       .get(SettingBoxKey.slideDismissReplyPage, defaultValue: Platform.isIOS);
 
@@ -529,7 +523,6 @@ class GStorage {
     // 设置全局变量
     GlobalData()
       ..imgQuality = defaultPicQa
-      ..grpcReply = grpcReply
       ..replyLengthLimit = replyLengthLimit;
   }
 
@@ -658,7 +651,6 @@ class SettingBoxKey {
       checkDynamic = 'checkDynamic',
       dynamicPeriod = 'dynamicPeriod',
       schemeVariant = 'schemeVariant',
-      grpcReply = 'grpcReply',
       showViewPoints = 'showViewPoints',
       showRelatedVideo = 'showRelatedVideo',
       showVideoReply = 'showVideoReply',
@@ -708,7 +700,6 @@ class SettingBoxKey {
       antiGoodsReply = 'antiGoodsReply',
       expandDynLivePanel = 'expandDynLivePanel',
       springDescription = 'springDescription',
-      collapsibleVideoPage = 'collapsibleVideoPage',
       enableHttp2 = 'enableHttp2',
       slideDismissReplyPage = 'slideDismissReplyPage',
       showFSActionItem = 'showFSActionItem',

lib/utils/utils.dart

@@ -233,7 +233,7 @@ class Utils {
   }) {
     if (off) {
       Get.offNamed(
-        '${GStorage.collapsibleVideoPage ? '/videoV' : '/video'}?$page',
+        '/videoV?$page',
         arguments: arguments,
         id: id,
         preventDuplicates: preventDuplicates,
@@ -241,7 +241,7 @@ class Utils {
       );
     } else {
       Get.toNamed(
-        '${GStorage.collapsibleVideoPage ? '/videoV' : '/video'}?$page',
+        '/videoV?$page',
         arguments: arguments,
         id: id,
         preventDuplicates: preventDuplicates,