diff --git a/platform/vcs-api/src/com/intellij/openapi/vcs/AbstractVcs.java b/platform/vcs-api/src/com/intellij/openapi/vcs/AbstractVcs.java index 2dccde71dd14..9c8f19d3aa38 100644 --- a/platform/vcs-api/src/com/intellij/openapi/vcs/AbstractVcs.java +++ b/platform/vcs-api/src/com/intellij/openapi/vcs/AbstractVcs.java @@ -21,7 +21,6 @@ import com.intellij.openapi.options.UnnamedConfigurable; import com.intellij.openapi.project.Project; import com.intellij.openapi.util.Pair; import com.intellij.openapi.vcs.annotate.AnnotationProvider; -import com.intellij.openapi.vcs.changes.ChangeListEditHandler; import com.intellij.openapi.vcs.changes.ChangeProvider; import com.intellij.openapi.vcs.changes.CommitExecutor; import com.intellij.openapi.vcs.changes.LocalChangeList; @@ -457,11 +456,6 @@ public abstract class AbstractVcs extends S return null; } - @Nullable - public ChangeListEditHandler getEditHandler() { - return null; - } - public boolean allowsNestedRoots() { return false; } diff --git a/platform/vcs-api/src/com/intellij/openapi/vcs/changes/ChangeListEditHandler.java b/platform/vcs-api/src/com/intellij/openapi/vcs/changes/ChangeListEditHandler.java deleted file mode 100644 index 85e1efb06b7a..000000000000 --- a/platform/vcs-api/src/com/intellij/openapi/vcs/changes/ChangeListEditHandler.java +++ /dev/null @@ -1,29 +0,0 @@ -/* - * Copyright 2000-2009 JetBrains s.r.o. - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -package com.intellij.openapi.vcs.changes; - -/** - * for IDEA changelist name, IDEA changelist comment editing, - * when, for example, under Perforce, comment corresponds to whole Perforce native changelist description, - * while name is only a piece of description - * - * in that case, editing handler should be set for changelist in order to always edit name and comment consistently - */ -public interface ChangeListEditHandler { - String changeCommentOnChangeName(final String name, final String comment); - String changeNameOnChangeComment(final String name, final String comment); - String correctCommentWhenInstalled(final String name, final String comment); -} diff --git a/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ChangeListWorker.java b/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ChangeListWorker.java index bdc657fc95e1..6b3c26b46544 100644 --- a/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ChangeListWorker.java +++ b/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ChangeListWorker.java @@ -299,10 +299,6 @@ public class ChangeListWorker implements ChangeListsWriteOperations { listImpl.setNameImpl(toName); myMap.remove(fromName); myMap.put(toName, list); - final ChangeListEditHandler editHandler = listImpl.getEditHandler(); - if (editHandler != null) { - listImpl.setCommentImpl(editHandler.changeCommentOnChangeName(toName, listImpl.getComment())); - } } return canEdit; } @@ -315,14 +311,6 @@ public class ChangeListWorker implements ChangeListsWriteOperations { if (! Comparing.equal(oldComment, newComment)) { final LocalChangeListImpl listImpl = (LocalChangeListImpl) list; listImpl.setCommentImpl(newComment); - final ChangeListEditHandler editHandler = listImpl.getEditHandler(); - if (editHandler != null) { - listImpl.setNameImpl(editHandler.changeNameOnChangeComment(listImpl.getName(), listImpl.getComment())); - if (! fromName.equals(listImpl.getName())) { - myMap.remove(fromName); - myMap.put(listImpl.getName(), list); - } - } } return oldComment; } diff --git a/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/LocalChangeListImpl.java b/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/LocalChangeListImpl.java index ae815d459e1a..069af2a49b28 100644 --- a/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/LocalChangeListImpl.java +++ b/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/LocalChangeListImpl.java @@ -281,11 +281,6 @@ public class LocalChangeListImpl extends LocalChangeList { return copy; } - @Nullable - public ChangeListEditHandler getEditHandler() { - return null; - } - public void setId(String id) { myId = id; } diff --git a/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/ChangeListChooser.java b/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/ChangeListChooser.java index c36c88f50f10..44e91d88e826 100644 --- a/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/ChangeListChooser.java +++ b/platform/vcs-impl/src/com/intellij/openapi/vcs/changes/ui/ChangeListChooser.java @@ -18,9 +18,7 @@ package com.intellij.openapi.vcs.changes.ui; import com.intellij.openapi.project.Project; import com.intellij.openapi.ui.DialogWrapper; import com.intellij.openapi.vcs.changes.ChangeList; -import com.intellij.openapi.vcs.changes.ChangeListEditHandler; import com.intellij.openapi.vcs.changes.LocalChangeList; -import com.intellij.openapi.vcs.changes.LocalChangeListImpl; import com.intellij.util.NullableConsumer; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; @@ -44,14 +42,6 @@ public class ChangeListChooser extends DialogWrapper { super(project, false); myProject = project; - ChangeListEditHandler handler; - for (ChangeList changelist : changelists) { - handler = ((LocalChangeListImpl)changelist).getEditHandler(); - if (handler != null) { - break; - } - } - myPanel = new ChangeListChooserPanel(myProject, new NullableConsumer() { public void consume(final @Nullable String errorMessage) { setOKActionEnabled(errorMessage == null);