Add better diagnostics

core/document.cpp

@@ -2722,12 +2722,14 @@ void Document::sign( const Okular::Annotation* pWhichAnnotation )
             // Sign it!
             if (!d->m_generator->sign( pWhichAnnotation, currentDocument().path() ))
             {
-                KMessageBox::error( nullptr, i18n("Could not sign '%1'. Invalid password or cannot write",
-                                                  currentDocument().fileName() ) );
+                KMessageBox::error( d->m_widget,
+                                    i18nc("%1 is a filename",
+                                          "Could not sign '%1'. Invalid password or cannot write",
+                                         currentDocument().fileName() ) );
             }
-                // no need to - slotAttemptReload() gets called via
-                //dirty handler anyway:
-                //swapBackingFile(currentDocument().path(), currentDocument());
+            // no need to - slotAttemptReload() gets called via
+            //dirty handler anyway:
+            //swapBackingFile(currentDocument().path(), currentDocument());
         }
     }
 }

part/pageviewannotator.cpp

@@ -340,8 +340,14 @@ public:
             if (resok)
             {
                 bool passok = false;
-                QString title = i18n("Enter password to unlock certificate %1");
-                QString pass = QInputDialog::getText(nullptr, i18n( "Enter password" ), title.replace("%1", cert),QLineEdit::Password ,QString(), &passok);
+                QString title = i18n("Enter password to unlock certificate %1",
+                                     cert);
+                QString pass = QInputDialog::getText(nullptr,
+                                                     i18n( "Enter password" ),
+                                                     title,
+                                                     QLineEdit::Password,
+                                                     QString(),
+                                                     &passok);
 
                 if (passok)
                 {