Assign to self causes retain cycle

[georgescumihai]

Using assign with self causes a retain cycle.

ModernMVVM/ModernMVVM/Features/MovieDetails/MovieDetailViewModel.swift

Line 31 in 98a36fc

.assign(to: \.state, on: self)

For more details check this forum post or stackoverflow post

[bednar]

Hi @georgescumihai,

did you find solutions?

Regards

[georgescumihai]

@bednar You can see the solutions in the forum or stack overflow link.

[debisani]

//.assign(to: \.state, on: self)
.sink{[weak self] (value) in self?.state = value}

[DanSkeel]

Look at this solution assignNoRetain
https://forums.swift.org/t/does-assign-to-produce-memory-leaks/29546/9

extension Publisher where Self.Failure == Never {
    public func assignNoRetain<Root>(to keyPath: ReferenceWritableKeyPath<Root, Self.Output>, on object: Root) -> AnyCancellable where Root: AnyObject {
        sink { [weak object] (value) in
            guard let object = object else { return }
            _ = Just(value).assign(to: keyPath, on: object)
        }
    }
}