passepartout-apple/Passepartout/Mac/Menu/VisibilityItem+ViewModel.swift
Davide De Rosa 5627e6c4a9
Address UI race conditions (#229)
* Make some managers concurrency-safe

- IntentsManager: @MainActor, non-shared, continuation

- SSIDReader: @MainActor, continuation

- Reviewer: main queue, non-shared

* Review wrong use of Concurrency framework

There were background thread calls e.g. in VPNToggle, because
ProfileManager was used inside a VPNManager async call.

Annotate @MainActor wherever a Task involves UI.

* Make main managers MainActor

* Apply MainActor to Mac menus

* [ci skip] Update CHANGELOG

* Set MainActor consistently on Mac menu view models
2022-10-13 08:53:50 +02:00

51 lines
1.4 KiB
Swift

//
// VisibilityItem+ViewModel.swift
// Passepartout
//
// Created by Davide De Rosa on 7/3/22.
// Copyright (c) 2022 Davide De Rosa. All rights reserved.
//
// https://github.com/passepartoutvpn
//
// This file is part of Passepartout.
//
// Passepartout is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Passepartout is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Passepartout. If not, see <http://www.gnu.org/licenses/>.
//
import Foundation
import AppKit
extension VisibilityItem {
@MainActor
class ViewModel {
private let transformer: ObservableProcessTransformer
private let utils: LightUtils
init(utils: LightUtils) {
transformer = .shared
self.utils = utils
}
@objc func bringToForeground() {
transformer.bringToForeground()
if transformer.isForeground {
utils.requestScene()
NSApp.activate(ignoringOtherApps: true)
}
}
}
}