оптимизация, рефакторинг, исправления (#62)

* префикс для profile режима

* рефакторинг: блоки, оптимизация: кнопка "наверх"

- удалены УтИлИтЫ

- добавлен снэк для уведомления о необходимости авторизации

* дашборд: билдер не нужен

* переработка dashboard в публикациях

- размер текста счетчиков

- положение табов

- фикс: на иконке трекера область бэйджа была некликабельной

* переработан скелетон для изображения, там-тут по-мелочи

- репаинт для кнопки "читают сейчас"

- репаинт для лоадера

* оказывается linter на const не работал

* линтеру не понравилось

* фикс: дублирование ключей

* фикс плейсхолдера, свой плейсхолдер для компаний и хабов

* виджет для html у публикации в ленте

* не создаем блок закладок если неавторизован

* рефакторинг тем, карточки, пересказ: путь до пакета, удалены лишние файлы

* заменен устаревший метод перевода материал цвета в hex

* фикс: card - не применялся цвет указанный извне

* UnmodifiableListView во всех моделях, фикс отметки в подписках

отметки в подписках (ui): если отмечал как прочитанное, затем листал
список и возвращался - то ранее просмотренные отображались как не просмотренные

чуть подправлена верстка в подписках, чтобы не менялся размер карточки при отметке

rethrow ошибок в кубитах

* построже

* фикс: не парсился тип публикации при переходе по ссылке

потерялась запятая между элементами

* рефакторинг: расширение для роутера

* избавились от множественных "on" хендлеров в блоках

* выпилен AnimatedBuilder из оберток, таббар в публикациях поверх списка

* удалены некоторые константы

* трекер: при "прочтении" отправлять только непрочитанные

* публикации в трекере: фикс цвета лоадера, размер кнопок, фикс размера карточки

* фикс: общий скролл для табов в трекере

* подписки в трекере: фиксированный размер карточки

* цвет для отключенных элементов, стиль FilledButton

* трекер, публикации: переписано с использованием стрима

- список больше не перезагружается при отметке как прочитанные
или удалении из трекера

- удалены лишние модели ответов

* трекер, подписки: ключи для виджетов

* рефакторинг исключений

* фикс: переход по внешним ссылкам

* отметка публикации локально

* трекер, уведомления: переписано с использованием стрима

отдельный блок для пометки как прочитанное

* залочен масштаб шрифта

* bloc как отдельный слой

* рефакторинг виджета списка публикаций

* фикс: не запускалось на веб-платформе, убраны адаптивные виджеты

* рефакторинг экрана поиска, чекбоксов

* язык интерфейса: перевод на чипы

* выпилена подписка на языки, и их передача по всей цепочке

* http клиент: работа с куками, подстановка языков в куки и в параметры

разделение интерцепторов

* переписан scrollcubit и виджет кнопки "наверх"

* рефакторинг логина и авторизации

- рефакторинг http клиента
- completer для csrf interceptor
- допилен менеджер cookie:
теперь все взаимодействие с куками происходит с помощью него
- переделан кубит авторизации: использование стрима
- кубит и виджет входа: использование cookie manager
- фикс: не получалось добавить статьи в избранное после
входа в приложение

* скрипт для build_runner

* ключи для заголовков и кук

Author: iska9der

.vscode/tasks.json

@@ -18,20 +18,16 @@
       "detail": ""
     },
     {
-      "type": "flutter",
-      "command": "flutter",
+      "type": "shell",
+      "command": "bash",
       "args": [
-        "pub",
-        "run",
-        "build_runner",
-        "build",
-        "--delete-conflicting-outputs"
+        "'./scripts/runner.sh'"
       ],
       "problemMatcher": [
         "$dart-build_runner"
       ],
       "group": "build",
-      "label": "build_runner build",
+      "label": "build_runner: build",
       "detail": ""
     }
   ]

analysis_options.yaml

@@ -15,7 +15,8 @@ linter:
     use_super_parameters: true
     use_key_in_widget_constructors: true
     eol_at_end_of_file: true
-    # always_declare_return_types: true
+    prefer_const_constructors: true
+    always_declare_return_types: true
 
 analyzer:
   exclude:

android/app/build.gradle

@@ -90,6 +90,11 @@ android {
             applicationIdSuffix ".debug"
             minifyEnabled false
         }
+
+        profile {
+            applicationIdSuffix ".debug"
+            minifyEnabled false
+        }
     }
 }
 

lib/bloc/company/company_cubit.dart

@@ -0,0 +1,40 @@
+import 'package:equatable/equatable.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+
+import '../../data/exception/exception.dart';
+import '../../data/model/company/company.dart';
+import '../../data/model/language/language.dart';
+import '../../data/repository/repository.dart';
+
+part 'company_state.dart';
+
+class CompanyCubit extends Cubit<CompanyState> {
+  CompanyCubit(String alias, {required CompanyRepository repository})
+    : _repository = repository,
+      super(CompanyState(alias: alias));
+
+  final CompanyRepository _repository;
+
+  void fetchCard() async {
+    CompanyCard card = state.card;
+
+    try {
+      if (card.isEmpty) {
+        emit(state.copyWith(status: CompanyStatus.loading));
+
+        card = await _repository.fetchCard(state.alias);
+      }
+
+      emit(state.copyWith(status: CompanyStatus.success, card: card));
+    } catch (error, stackTrace) {
+      emit(
+        state.copyWith(
+          status: CompanyStatus.failure,
+          error: error.parseException('Не удалось получить профиль компании'),
+        ),
+      );
+
+      super.onError(error, stackTrace);
+    }
+  }
+}

lib/bloc/company/company_list_cubit.dart

@@ -0,0 +1,55 @@
+import 'package:equatable/equatable.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+
+import '../../data/exception/exception.dart';
+import '../../data/model/company/company.dart';
+import '../../data/model/list_response_model.dart';
+import '../../data/repository/repository.dart';
+
+part 'company_list_state.dart';
+
+class CompanyListCubit extends Cubit<CompanyListState> {
+  CompanyListCubit({
+    required CompanyRepository repository,
+    required LanguageRepository languageRepository,
+  }) : _repository = repository,
+       super(const CompanyListState());
+
+  final CompanyRepository _repository;
+
+  void fetch() async {
+    if (state.status == CompanyListStatus.loading ||
+        !state.isFirstFetch && state.isLastPage) {
+      return;
+    }
+
+    emit(state.copyWith(status: CompanyListStatus.loading));
+
+    try {
+      final response = await _repository.fetchAll(page: state.page);
+
+      var newList = state.list.copyWith(
+        pagesCount: response.pagesCount,
+        ids: [...state.list.ids, ...response.ids],
+        refs: [...state.list.refs, ...response.refs],
+      );
+
+      emit(
+        state.copyWith(
+          status: CompanyListStatus.success,
+          list: newList,
+          page: state.page + 1,
+        ),
+      );
+    } catch (error, stackTrace) {
+      emit(
+        state.copyWith(
+          status: CompanyListStatus.failure,
+          error: error.parseException('Не удалось получить список компаний'),
+        ),
+      );
+
+      super.onError(error, stackTrace);
+    }
+  }
+}

lib/presentation/page/services/company/cubit/company_list_state.dart renamed to lib/bloc/company/company_list_state.dart

@@ -1,23 +1,18 @@
 import 'package:equatable/equatable.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 
-import '../../../../../data/exception/exception.dart';
-import '../../../../../data/model/hub/hub.dart';
-import '../../../../../data/repository/repository.dart';
+import '../../data/exception/exception.dart';
+import '../../data/model/hub/hub.dart';
+import '../../data/repository/repository.dart';
 
 part 'hub_state.dart';
 
 class HubCubit extends Cubit<HubState> {
-  HubCubit(
-    String alias, {
-    required HubRepository repository,
-    required LanguageRepository languageRepository,
-  }) : _repository = repository,
-       _languageRepository = languageRepository,
-       super(HubState(alias: alias));
+  HubCubit(String alias, {required HubRepository repository})
+    : _repository = repository,
+      super(HubState(alias: alias));
 
   final HubRepository _repository;
-  final LanguageRepository _languageRepository;
 
   void fetchProfile() async {
     HubProfile profile = state.profile;
@@ -26,11 +21,7 @@ class HubCubit extends Cubit<HubState> {
       if (profile.isEmpty) {
         emit(state.copyWith(status: HubStatus.loading));
 
-        profile = await _repository.fetchProfile(
-          state.alias,
-          langUI: _languageRepository.ui,
-          langArticles: _languageRepository.articles,
-        );
+        profile = await _repository.fetchProfile(state.alias);
       }
 
       emit(state.copyWith(status: HubStatus.success, profile: profile));

lib/presentation/page/services/company/cubit/company_state.dart renamed to lib/bloc/company/company_state.dart

@@ -0,0 +1,54 @@
+import 'package:equatable/equatable.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+
+import '../../data/exception/exception.dart';
+import '../../data/model/hub/hub.dart';
+import '../../data/model/list_response_model.dart';
+import '../../data/repository/repository.dart';
+
+part 'hub_list_state.dart';
+
+class HubListCubit extends Cubit<HubListState> {
+  HubListCubit({required HubRepository repository})
+    : _repository = repository,
+      super(const HubListState());
+
+  final HubRepository _repository;
+
+  void fetch() async {
+    if (state.status == HubListStatus.loading ||
+        !state.isFirstFetch && state.isLastPage) {
+      return;
+    }
+
+    emit(state.copyWith(status: HubListStatus.loading));
+
+    try {
+      final response = await _repository.fetchAll(page: state.page);
+
+      var newList = state.list.copyWith(
+        ids: [...state.list.ids, ...response.ids],
+        pagesCount: response.pagesCount,
+        refs: [...state.list.refs, ...response.refs],
+      );
+
+      emit(
+        state.copyWith(
+          status: HubListStatus.success,
+          list: newList,
+          page: state.page + 1,
+        ),
+      );
+    } catch (error, stackTrace) {
+      const fallbackMessage = 'Не удалось получить список хабов';
+      emit(
+        state.copyWith(
+          status: HubListStatus.failure,
+          error: error.parseException(fallbackMessage),
+        ),
+      );
+
+      super.onError(error, stackTrace);
+    }
+  }
+}