diff --git a/plugins/tasks/tasks-api/src/com/intellij/tasks/Comment.java b/plugins/tasks/tasks-api/src/com/intellij/tasks/Comment.java index bf67b94d4584..fd2b19a503e7 100644 --- a/plugins/tasks/tasks-api/src/com/intellij/tasks/Comment.java +++ b/plugins/tasks/tasks-api/src/com/intellij/tasks/Comment.java @@ -34,4 +34,15 @@ public abstract class Comment { @Nullable public abstract Date getDate(); + + public void appendTo(StringBuilder builder) { + builder.append("
"); + if (getAuthor() != null) { + builder.append("Author: ").append(getAuthor()).append("
"); + } + if (getDate() != null) { + builder.append("Date: ").append(getDate()).append("
"); + } + builder.append(getText()).append("
"); + } } diff --git a/plugins/tasks/tasks-core/src/com/intellij/tasks/doc/TaskDocumentationProvider.java b/plugins/tasks/tasks-core/src/com/intellij/tasks/doc/TaskDocumentationProvider.java index 6a298c9fade7..c1dd539b947b 100644 --- a/plugins/tasks/tasks-core/src/com/intellij/tasks/doc/TaskDocumentationProvider.java +++ b/plugins/tasks/tasks-core/src/com/intellij/tasks/doc/TaskDocumentationProvider.java @@ -59,23 +59,12 @@ public class TaskDocumentationProvider extends AbstractDocumentationProvider imp builder.append("Description:
").append(task.getDescription()); } for (Comment comment : task.getComments()) { - addComment(builder, comment); + comment.appendTo(builder); } builder.append(""); return builder.toString(); } - private static void addComment(final StringBuilder builder, final Comment comment) { - builder.append("
"); - if (comment.getAuthor() != null) { - builder.append("Author: ").append(comment.getAuthor()).append("
"); - } - if (comment.getDate() != null) { - builder.append("Date: ").append(comment.getDate()).append("
"); - } - builder.append(comment.getText()).append("
"); - } - @Override public PsiElement getDocumentationElementForLookupItem(PsiManager psiManager, Object object, PsiElement element) { return object instanceof Task ? new TaskPsiElement(psiManager, (Task)object): null;