fix(import): don't use java-style iterator before first element

Fix #4962

src/widget/form/addfriendform.cpp

@@ -245,17 +245,19 @@ void AddFriendForm::onImportOpenClicked()
     }
 
     contactsToImport = QString::fromUtf8(contactFile.readAll()).split('\n');
-    QMutableListIterator<QString> it(contactsToImport);
     qDebug() << "Import list:";
-    while (it.hasNext()) {
+    for (auto it = contactsToImport.begin(); it != contactsToImport.end();) {
-        const QString id = it.value().trimmed();
+        const QString id = it->trimmed();
-        const bool valid = !id.isEmpty() && checkIsValidId(id);
+        if (checkIsValidId(id)) {
-        if (valid) {
+            *it = id;
-            it.value() = id;
+            qDebug() << *it;
+            ++it;
         } else {
-            it.remove();
+            if (!id.isEmpty()) {
+                qDebug() << "Invalid ID:" << *it;
+            }
+            it = contactsToImport.erase(it);
         }
-        qDebug() << it.next();
     }
 
     if (contactsToImport.isEmpty()) {