Port away from deprecated KMessageBox Yes/No API
The new action-oriented KMessageBox API breaks a bit with the Ark code having picked up the Yes & No semantics.
-
LoadCorruptQuery::responseYes()
perhaps should be renamed toLoadCorruptQuery::responseLoad()
in another commit? - There are 3 different code patterns in part.cpp to deal with the sync KMessageBox method call in otherwise similar code logic: assign to a variable named
buttonCode
, assign to a variable namedreallyDelete
and directly compare the returned value to a KMessageBox enum value. Perhaps that should be turned into consistent code patterns in yet another commit.
The current patch just deals with doing the 1:1 KMessageBox API replacement.