Skip to content

Port away from deprecated KMessageBox Yes/No API

Friedrich W. H. Kossebau requested to merge work/kossebau/portyesno into master

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 to LoadCorruptQuery::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 named reallyDelete 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.

Merge request reports