This check was correct. Without this check the notification was showing on startup for already full torrents after checking. The idea is that we should only display the notification when the torrent actually finishes (for the first time). If the torrent is already marked as a seed in TorrentPersistentData, then it has already finished in the past.

If the notification does not show for newly finished torrents, then maybe we mark the torrent as a seed to early in TorrentPersistentData?