fix: shortcut modify failed

modify failed when new shortcut accels(Supper+X) contains old shortcut accels(Supper)
requestRestore is only emitted when "press" is false, to handle self-conflicts
pms: Bug-289101

src/plugin-keyboard/operation/keyboardcontroller.cpp

@@ -29,11 +29,6 @@ KeyboardController::KeyboardController(QObject *parent)
 
         ShortcutInfo *conflict = m_shortcutModel->getInfo(shortcut);
 
-        if (conflict && conflict == current && conflict->accels == current->accels) {
-            Q_EMIT requestRestore();
-            return;
-        }
-
         if (!press) {
             if (shortcut.isEmpty()) {
                 Q_EMIT requestRestore();
@@ -45,8 +40,13 @@ KeyboardController::KeyboardController(QObject *parent)
                 return;
             }
 
+            // have conflict
             if (conflict) {
-                // have conflict
+                // self conflict
+                if (conflict == current && conflict->accels == current->accels) {
+                    Q_EMIT requestRestore();
+                    return;
+                }
 
                 auto conflictName = QString("<font color=\"red\">%1</font>").arg(conflict->name);
                 QString text = KeyboardController::tr("This shortcut conflicts with [%1]").arg(conflictName);