3.8.3 (#330)

* Убрал забытую локализацию

* Поправил отступ снизу экрана

* Дорабатываю обработку ошибок сервера

* Дефолтный город для фильтра площадок - Москва

Если не определили город, в том числе из-за проблем с локализацией, применяем фильтр площадок по Москве

* Доработал обновление диалогов

* Доработал список прошедших мероприятий

Если при открытии приложения не удалось загрузить список прошедших мероприятий, показываем там список ранее сохраненных мероприятий

* Доработки экрана мероприятий

- Если нет сохраненных прошедших мероприятий, то при отсутствии сети показываем плашку про отсутствие сети вместо пустого экрана на вкладке с прошедшими мероприятиями
- Если нет сети, то не делаем запрос - сразу показываем алерт

* Обрабатываем отстутсвие сети на экране с картой

* Доработки списка диалогов

Ошибки загрузки диалогов отображаем не в алерте, а прямо вместо контента экрана

* Доработал профиль

Загружаем данные профиля при разворачивании приложения по той же логике, что и диалоги - если изменилась фаза приложения, сохраненная в State-свойстве

Author: easydev991

SwiftUI-WorkoutApp.xcodeproj/project.pbxproj

@@ -468,7 +468,7 @@
 					"$(inherited)",
 					"@executable_path/Frameworks",
 				);
-				MARKETING_VERSION = 3.8.2;
+				MARKETING_VERSION = 3.8.3;
 				PRODUCT_BUNDLE_IDENTIFIER = com.FGU.WorkOut;
 				PRODUCT_NAME = WorkoutApp;
 				RUN_CLANG_STATIC_ANALYZER = YES;
@@ -522,7 +522,7 @@
 					"$(inherited)",
 					"@executable_path/Frameworks",
 				);
-				MARKETING_VERSION = 3.8.2;
+				MARKETING_VERSION = 3.8.3;
 				PRODUCT_BUNDLE_IDENTIFIER = com.FGU.WorkOut;
 				PRODUCT_NAME = WorkoutApp;
 				RUN_CLANG_STATIC_ANALYZER = YES;

SwiftUI-WorkoutApp/Libraries/SWNetwork/Sources/SWNetwork/APIError.swift

@@ -13,26 +13,45 @@ public enum APIError: Error, LocalizedError, Equatable {
     case decodingError
     case notConnectedToInternet
 
-    init(_ error: ErrorResponse, _ code: Int?) {
-        if code == 401 {
-            self = .invalidCredentials
-        } else if let message = error.message {
-            self = .customError(code: code ?? 404, message: message)
-        } else if let array = error.errors {
-            let message = array.joined(separator: ",\n")
-            self = .customError(code: code ?? 404, message: message)
-        } else {
-            self.init(with: error.realCode)
+    init(_ error: ErrorResponse, _ statusCode: Int?) {
+        let serverCode = error.makeRealCode(statusCode: statusCode)
+
+        // Приоритет 1: Кастомные сообщения из ErrorResponse
+        if let message = error.realMessage {
+            self = .customError(
+                code: serverCode,
+                message: message.trimmingCharacters(in: .whitespacesAndNewlines)
+            )
+            return
+        }
+
+        // Приоритет 2: Обработка через код (если есть валидный код)
+        if serverCode != 0 {
+            self.init(with: serverCode)
+            return
         }
+
+        // Приоритет 3: Стандартный код из статуса
+        if let statusCode {
+            self.init(with: statusCode)
+            return
+        }
+
+        // Если все варианты исчерпаны
+        self = .unknown
     }
 
     init(with code: Int?) {
+        guard let code else {
+            self = .unknown
+            return
+        }
         switch code {
-        case 400: self = .badRequest
+        case 400, 402, 403, 405 ... 412, 414 ... 499: self = .badRequest
         case 401: self = .invalidCredentials
         case 404: self = .notFound
         case 413: self = .payloadTooLarge
-        case 500: self = .serverError
+        case 500 ... 599: self = .serverError
         default: self = .unknown
         }
     }

SwiftUI-WorkoutApp/Libraries/SWNetwork/Sources/SWNetwork/ErrorResponse.swift

@@ -1,15 +1,43 @@
 import Foundation
 
 struct ErrorResponse: Codable {
-    let errors: [String]?
+    let errors: [String]
     let name, message: String?
-    let code, status: Int?
+    let code, status: Int
 
-    var realCode: Int {
-        if let code, code != 0 {
-            code
+    init(from decoder: Decoder) throws {
+        let container = try decoder.container(keyedBy: CodingKeys.self)
+        self.errors = try container.decodeIfPresent([String].self, forKey: .errors) ?? []
+        self.name = try container.decodeIfPresent(String.self, forKey: .name)
+        self.message = try container.decodeIfPresent(String.self, forKey: .message)
+        self.code = try container.decodeIfPresent(Int.self, forKey: .code) ?? 0
+        self.status = try container.decodeIfPresent(Int.self, forKey: .status) ?? 0
+    }
+
+    init(
+        errors: [String] = [],
+        name: String? = nil,
+        message: String? = nil,
+        code: Int = 0,
+        status: Int = 0
+    ) {
+        self.errors = errors
+        self.name = name
+        self.message = message
+        self.code = code
+        self.status = status
+    }
+
+    var realMessage: String? {
+        if let message {
+            message
         } else {
-            status ?? 0
+            errors.isEmpty ? nil : errors.joined(separator: ", ")
         }
     }
+
+    func makeRealCode(statusCode: Int?) -> Int {
+        let realCode = code != 0 ? code : status
+        return realCode != 0 ? realCode : (statusCode ?? 0)
+    }
 }

SwiftUI-WorkoutApp/Libraries/SWNetwork/Tests/SWNetworkTests/APIErrorTests.swift

@@ -2,60 +2,96 @@
 import Testing
 
 struct APIErrorTests {
-    @Test
-    func badRequest() {
-        let error = APIError(with: 400)
-        #expect(error == .badRequest)
+    private static var badRequestCodes: [Int] {
+        [400, 402, 403] + Array(405 ... 412) + Array(414 ... 499)
     }
 
-    @Test
-    func invalidCredentials() {
-        let error = APIError(with: 401)
-        #expect(error == .invalidCredentials)
-    }
+    private static let serverErrorCodes = Array(500 ... 599)
 
-    @Test
-    func notFound() {
-        let error = APIError(with: 404)
-        #expect(error == .notFound)
+    @Test(arguments: badRequestCodes)
+    func badRequest(code: Int) {
+        let error = APIError(with: code)
+        #expect(error == .badRequest)
     }
 
-    @Test
-    func payloadTooLarge() {
-        let error = APIError(with: 413)
-        #expect(error == .payloadTooLarge)
+    @Test(arguments: serverErrorCodes)
+    func serverError(code: Int) {
+        let error = APIError(with: code)
+        #expect(error == .serverError)
     }
 
     @Test
-    func serverError() {
-        let error = APIError(with: 500)
-        #expect(error == .serverError)
+    func errorInitializationByCode() {
+        let testCases: [(Int?, APIError)] = [
+            (401, .invalidCredentials),
+            (404, .notFound),
+            (413, .payloadTooLarge),
+            (nil, .unknown),
+            (999, .unknown)
+        ]
+        for (code, expected) in testCases {
+            let error = APIError(with: code)
+            #expect(error == expected)
+        }
     }
 
     @Test
     func customErrorWithMessage() {
         let errorResponse = ErrorResponse(
-            errors: nil,
-            name: nil,
             message: "Непредвиденная ошибка",
-            code: nil,
-            status: nil
+            code: 0,
+            status: 0
         )
         let error = APIError(errorResponse, 123)
         #expect(error.errorDescription == "123, Непредвиденная ошибка")
     }
 
     @Test
     func customErrorWithErrorsArray() {
-        let errorResponse = ErrorResponse(errors: ["Ошибка 1", "Ошибка 2"], name: nil, message: nil, code: nil, status: nil)
+        let errorResponse = ErrorResponse(errors: ["Ошибка 1", "Ошибка 2"], code: 0, status: 0)
         let error = APIError(errorResponse, nil)
-        #expect(error.errorDescription == "404, Ошибка 1,\nОшибка 2")
+        #expect(error.errorDescription == "0, Ошибка 1, Ошибка 2")
     }
 
     @Test
     func unknownError() {
-        let errorResponse = ErrorResponse(errors: nil, name: nil, message: nil, code: nil, status: nil)
+        let errorResponse = ErrorResponse(code: 0, status: 0)
         let error = APIError(errorResponse, nil)
         #expect(error == .unknown)
     }
+
+    @Test
+    func customErrorPriorities() {
+        // Сообщение имеет приоритет над errors
+        let case1 = ErrorResponse(
+            errors: ["Error1", "Error2"],
+            message: "Priority Message",
+            code: 400,
+            status: 0
+        )
+        let error1 = APIError(case1, nil)
+        #expect(error1.errorDescription == "400, Priority Message")
+
+        // Errors используются, если message отсутствует
+        let case2 = ErrorResponse(
+            errors: ["Error1", "Error2"],
+            code: 0,
+            status: 500
+        )
+        let error2 = APIError(case2, nil)
+        #expect(error2.errorDescription == "500, Error1, Error2")
+    }
+
+    @Test
+    func specialCombinations() {
+        // Кастомная ошибка с кодом, соответствующим стандартному
+        let response1 = ErrorResponse(message: "Custom Credentials", code: 401, status: 0)
+        let error1 = APIError(response1, nil)
+        #expect(error1 == .customError(code: 401, message: "Custom Credentials"))
+
+        // Пустой массив errors
+        let response2 = ErrorResponse(errors: [], code: 500, status: 0)
+        let error2 = APIError(response2, nil)
+        #expect(error2 == .serverError)
+    }
 }

SwiftUI-WorkoutApp/Resources/Localizable.xcstrings

@@ -3013,23 +3013,6 @@
         }
       }
     },
-    "Поддержать разработчика" : {
-      "extractionState" : "manual",
-      "localizations" : {
-        "en" : {
-          "stringUnit" : {
-            "state" : "translated",
-            "value" : "Support the developer"
-          }
-        },
-        "ru" : {
-          "stringUnit" : {
-            "state" : "translated",
-            "value" : "Поддержать разработчика"
-          }
-        }
-      }
-    },
     "Поделиться приложением" : {
       "extractionState" : "manual",
       "localizations" : {

SwiftUI-WorkoutApp/Screens/Events/EventsListScreen.swift

@@ -6,6 +6,7 @@ import SWUtils
 
 /// Экран со списком мероприятий
 struct EventsListScreen: View {
+    @Environment(\.isNetworkConnected) private var isNetworkConnected
     @EnvironmentObject private var tabViewModel: TabViewModel
     @EnvironmentObject private var defaults: DefaultsService
     @State private var futureEvents = [EventResponse]()
@@ -18,6 +19,14 @@ struct EventsListScreen: View {
     @State private var showEventCreationSheet = false
     @State private var showEventCreationRule = false
     private let pastEventStorage = PastEventStorage()
+    private var currentEventList: [EventResponse] {
+        switch selectedEventType {
+        case .future:
+            futureEvents
+        case .past:
+            pastEvents.isEmpty ? pastEventStorage.savedPastEvents : pastEvents
+        }
+    }
 
     var body: some View {
         NavigationView {
@@ -62,7 +71,7 @@ private extension EventsListScreen {
             Icons.Regular.refresh.view
         }
         .opacity(
-            showEmptyView && !DeviceOSVersionChecker.iOS16Available ? 1 : 0
+            !isLoading && !DeviceOSVersionChecker.iOS16Available ? 1 : 0
         )
         .disabled(isLoading)
     }
@@ -78,26 +87,37 @@ private extension EventsListScreen {
         .padding(.horizontal)
     }
 
+    @ViewBuilder
     var emptyView: some View {
-        EmptyContentView(
-            mode: .events,
-            action: {
-                if canAddEvent {
-                    showEventCreationSheet.toggle()
-                } else {
-                    goToMap()
+        switch selectedEventType {
+        case .future:
+            EmptyContentView(
+                mode: .events,
+                action: {
+                    if canAddEvent {
+                        showEventCreationSheet.toggle()
+                    } else {
+                        goToMap()
+                    }
                 }
-            }
-        )
-        .opacity(showEmptyView ? 1 : 0)
+            )
+            .opacity(futureEvents.isEmpty && !isLoading ? 1 : 0)
+        case .past:
+            let showView = pastEvents.isEmpty
+                && pastEventStorage.savedPastEvents.isEmpty
+                && !isLoading
+                && !isNetworkConnected
+            NoConnectionView()
+                .opacity(showView ? 1 : 0)
+        }
     }
 
     func goToMap() { tabViewModel.selectTab(.map) }
 
     var eventsList: some View {
         ScrollView {
             LazyVStack(spacing: 12) {
-                ForEach(selectedEventType == .future ? futureEvents : pastEvents) { event in
+                ForEach(currentEventList) { event in
                     Button {
                         selectedEvent = event
                     } label: {
@@ -111,7 +131,7 @@ private extension EventsListScreen {
                     .accessibilityIdentifier("EventViewCell")
                 }
             }
-            .padding([.top, .horizontal])
+            .padding()
         }
         .opacity(isLoading ? 0 : 1)
         .sheet(item: $selectedEvent) { event in
@@ -159,11 +179,8 @@ private extension EventsListScreen {
         defaults.hasParks && defaults.isAuthorized
     }
 
-    var showEmptyView: Bool {
-        selectedEventType == .future && futureEvents.isEmpty && !isLoading
-    }
-
     func askForEvents(refresh: Bool = false) async {
+        guard !SWAlert.shared.presentNoConnection(isNetworkConnected) else { return }
         let hasFutureEvents = selectedEventType == .future && !futureEvents.isEmpty
         let hasPastEvents = selectedEventType == .past && !pastEvents.isEmpty
         if isLoading && !refresh