Skip to content

Don't double emit emitResult()

Marco Martin requested to merge work/mart/installPathChanged into master

setupNotificationsOnJobFinished connects to jobThreadFinished and in there emitResult() is called, so avoid a second connection that would lead to a double emitted signal.

connect to installPathChanged before connecting to jobThreadFinished. Signal order matters here, as some client code relies that the package path() to already be the final "correct" one when the job finishes

together with plasma/plasma-desktop!2535 (merged)

BUG:492557

Edited by Marco Martin

Merge request reports

Loading