[Uml-devel] kdesdk/umbrello/umbrello

[Sebastian Stein]

CVS commit by sstein: 

Give a hint to the user, why the operation was not added. I think with this commit and the previous one by Oliver, bug #73926 is now fixed. Could everybody please test it, so that we can backport the fix???


  M +5 -1      umldoc.cpp   1.127


--- kdesdk/umbrello/umbrello/umldoc.cpp  #1.126:1.127
@@ -895,6 +895,10 @@ UMLOperation* UMLDoc::createOperation(UM
                         delete op;
                         return NULL;
+                } else if (classifier->checkOperationSignature(op->getName(), op->getParmList())) {
+                        KMessageBox::information(0, i18n("An operation with the same name and signature already exists. You can not add it again."));
+                } else {
+                        break;
                 }
-        } while (classifier->checkOperationSignature(op->getName(), op->getParmList()));
+        } while(1);
 
         // operation name is ok, formally add it to the classifier