From f196bf7bc2f5e6f61f86a0f41a4ee9ce061b8b31 Mon Sep 17 00:00:00 2001 From: Kevin Sawicki Date: Mon, 16 May 2011 12:44:31 -0700 Subject: Refactor package and class names for issue connector. Change-Id: Icfaf402936f0ebd2527ac9ba2fbbeb135a58e5c4 Signed-off-by: Kevin Sawicki --- org.eclipse.mylyn.github.core/META-INF/MANIFEST.MF | 5 +- .../org/eclipse/mylyn/github/internal/GitHub.java | 163 -------- .../mylyn/github/internal/GitHubException.java | 74 ---- .../github/internal/GitHubRepositoryConnector.java | 410 ------------------- .../github/internal/GitHubTaskAttributeMapper.java | 57 --- .../github/internal/GitHubTaskAttributes.java | 190 --------- .../github/internal/GitHubTaskDataHandler.java | 447 -------------------- .../mylyn/github/internal/GitHubTaskOperation.java | 76 ---- .../eclipse/mylyn/github/internal/Messages.java | 98 ----- .../eclipse/mylyn/github/internal/QueryUtils.java | 78 ---- .../mylyn/github/internal/messages.properties | 23 -- .../eclipse/mylyn/internal/github/core/GitHub.java | 163 ++++++++ .../internal/github/core/GitHubException.java | 74 ++++ .../mylyn/internal/github/core/Messages.java | 41 ++ .../mylyn/internal/github/core/QueryUtils.java | 78 ++++ .../github/core/gist/GistAttachmentHandler.java | 2 +- .../internal/github/core/gist/GistConnector.java | 2 +- .../github/core/gist/GistTaskDataHandler.java | 6 +- .../internal/github/core/issue/IssueAttribute.java | 190 +++++++++ .../github/core/issue/IssueAttributeMapper.java | 57 +++ .../internal/github/core/issue/IssueConnector.java | 413 +++++++++++++++++++ .../internal/github/core/issue/IssueOperation.java | 76 ++++ .../github/core/issue/IssueTaskDataHandler.java | 449 +++++++++++++++++++++ .../mylyn/internal/github/core/issue/Messages.java | 86 ++++ .../internal/github/core/issue/messages.properties | 19 + .../mylyn/internal/github/core/messages.properties | 4 + .../GitHubRepositoryConnectorUIHeadlessTest.java | 9 +- org.eclipse.mylyn.github.ui/META-INF/MANIFEST.MF | 4 +- org.eclipse.mylyn.github.ui/plugin.xml | 8 +- .../github/ui/internal/CreateGistHandler.java | 119 ------ .../mylyn/github/ui/internal/CreateGistJob.java | 66 --- .../github/ui/internal/GistNotificationPopup.java | 77 ---- .../ui/internal/GitHubRepositoryConnectorUI.java | 173 -------- .../ui/internal/GitHubRepositoryQueryPage.java | 436 -------------------- .../ui/internal/GitHubRepositorySettingsPage.java | 204 ---------- .../github/ui/internal/GitHubTaskEditorPage.java | 71 ---- .../ui/internal/GitHubTaskEditorPageFactory.java | 62 --- .../ui/internal/ImportRepositoriesWizard.java | 2 +- .../github/ui/internal/IssueAttributePart.java | 244 ----------- .../ui/internal/IssueLabelAttributeEditor.java | 250 ------------ .../mylyn/github/ui/internal/IssueSummaryPart.java | 235 ----------- .../eclipse/mylyn/github/ui/internal/Messages.java | 87 ---- .../ui/internal/RepositorySelectionWizardPage.java | 2 +- .../mylyn/github/ui/internal/messages.properties | 29 -- .../internal/github/ui/gist/CreateGistHandler.java | 120 ++++++ .../internal/github/ui/gist/CreateGistJob.java | 67 +++ .../github/ui/gist/GistNotificationPopup.java | 78 ++++ .../github/ui/gist/GistTaskEditorPage.java | 2 +- .../github/ui/issue/IssueAttributePart.java | 244 +++++++++++ .../internal/github/ui/issue/IssueConnectorUi.java | 173 ++++++++ .../github/ui/issue/IssueLabelAttributeEditor.java | 251 ++++++++++++ .../github/ui/issue/IssueRepositoryQueryPage.java | 435 ++++++++++++++++++++ .../ui/issue/IssueRepositorySettingsPage.java | 204 ++++++++++ .../internal/github/ui/issue/IssueSummaryPart.java | 237 +++++++++++ .../github/ui/issue/IssueTaskEditorPage.java | 71 ++++ .../ui/issue/IssueTaskEditorPageFactory.java | 63 +++ .../mylyn/internal/github/ui/issue/Messages.java | 113 ++++++ .../internal/github/ui/issue/messages.properties | 28 ++ 58 files changed, 3757 insertions(+), 3688 deletions(-) delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHub.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubException.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubRepositoryConnector.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubTaskAttributeMapper.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubTaskAttributes.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubTaskDataHandler.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubTaskOperation.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/Messages.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/QueryUtils.java delete mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/messages.properties create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/GitHub.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/GitHubException.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/Messages.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/QueryUtils.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/IssueAttribute.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/IssueAttributeMapper.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/IssueConnector.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/IssueOperation.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/IssueTaskDataHandler.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/Messages.java create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/issue/messages.properties create mode 100644 org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/internal/github/core/messages.properties delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/CreateGistHandler.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/CreateGistJob.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/GistNotificationPopup.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/GitHubRepositoryConnectorUI.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/GitHubRepositoryQueryPage.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/GitHubRepositorySettingsPage.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/GitHubTaskEditorPage.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/GitHubTaskEditorPageFactory.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/IssueAttributePart.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/IssueLabelAttributeEditor.java delete mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/github/ui/internal/IssueSummaryPart.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/gist/CreateGistHandler.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/gist/CreateGistJob.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/gist/GistNotificationPopup.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueAttributePart.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueConnectorUi.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueLabelAttributeEditor.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueRepositoryQueryPage.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueRepositorySettingsPage.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueSummaryPart.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueTaskEditorPage.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/IssueTaskEditorPageFactory.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/Messages.java create mode 100644 org.eclipse.mylyn.github.ui/src/org/eclipse/mylyn/internal/github/ui/issue/messages.properties diff --git a/org.eclipse.mylyn.github.core/META-INF/MANIFEST.MF b/org.eclipse.mylyn.github.core/META-INF/MANIFEST.MF index 1721cbc7..f03adb37 100644 --- a/org.eclipse.mylyn.github.core/META-INF/MANIFEST.MF +++ b/org.eclipse.mylyn.github.core/META-INF/MANIFEST.MF @@ -6,8 +6,9 @@ Bundle-SymbolicName: org.eclipse.mylyn.github.core;singleton:=true Bundle-Version: 0.1.0.qualifier Bundle-Vendor: %providerName Bundle-RequiredExecutionEnvironment: J2SE-1.5 -Export-Package: org.eclipse.mylyn.github.internal;x-friends:="org.eclipse.mylyn.github.ui", - org.eclipse.mylyn.internal.github.core.gist;x-friends:="org.eclipse.mylyn.github.ui" +Export-Package: org.eclipse.mylyn.internal.github.core;x-friends:="org.eclipse.mylyn.github.ui", + org.eclipse.mylyn.internal.github.core.gist;x-friends:="org.eclipse.mylyn.github.ui", + org.eclipse.mylyn.internal.github.core.issue;x-friends:="org.eclipse.mylyn.github.ui" Require-Bundle: org.eclipse.core.runtime;bundle-version="3.5.0", org.eclipse.mylyn.tasks.core;bundle-version="3.2.0", org.eclipse.mylyn.commons.net;bundle-version="3.2.0", diff --git a/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHub.java b/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHub.java deleted file mode 100644 index cf2e1330..00000000 --- a/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHub.java +++ /dev/null @@ -1,163 +0,0 @@ -/******************************************************************************* - * Copyright (c) 2011 Red Hat and others. - * All rights reserved. This program and the accompanying materials - * are made available under the terms of the Eclipse Public License v1.0 - * which accompanies this distribution, and is available at - * http://www.eclipse.org/legal/epl-v10.html - * - * Contributors: - * David Green - initial contribution - * Christian Trutz - initial contribution - * Chris Aniszczyk - initial contribution - *******************************************************************************/ -package org.eclipse.mylyn.github.internal; - -import java.util.regex.Pattern; - -import org.eclipse.core.runtime.ILog; -import org.eclipse.core.runtime.IStatus; -import org.eclipse.core.runtime.Platform; -import org.eclipse.core.runtime.Status; -import org.eclipse.egit.github.core.Repository; - -/** - * GitHub class - */ -public class GitHub { - - /** BUNDLE_ID */ - public static final String BUNDLE_ID = "org.eclipse.mylyn.github.core"; - - /** CONNECTOR_KIND */ - public static final String CONNECTOR_KIND = "github"; - - /** HTTP_WWW_GITHUB_ORG */ - public static final String HTTP_WWW_GITHUB_ORG = "http://www.github.org"; - - /** HTTP_GITHUB_COM */ - public static final String HTTP_GITHUB_COM = "http://github.com"; - - /** URL_PATTERN */ - public static final Pattern URL_PATTERN = Pattern.compile("(?:" - + Pattern.quote(HTTP_WWW_GITHUB_ORG) + "|" - + Pattern.quote(HTTP_GITHUB_COM) + ")/([^/]+)/([^/]+)"); - - /** - * Create status of severity with message - * - * @param severity - * @param message - * @return status - */ - public static IStatus createStatus(int severity, String message) { - return new Status(severity, BUNDLE_ID, message); - } - - /** - * Create status of severity with message and throwable - * - * @param severity - * @param message - * @param e - * @return status - */ - public static IStatus createStatus(int severity, String message, Throwable e) { - return new Status(severity, BUNDLE_ID, message, e); - } - - /** - * Create error status from message - * - * @param message - * @return status - */ - public static IStatus createErrorStatus(String message) { - return createStatus(IStatus.ERROR, message); - } - - /** - * Create error status from message and throwable - * - * @param message - * @param t - * @return status - */ - public static IStatus createErrorStatus(String message, Throwable t) { - return createStatus(IStatus.ERROR, message, t); - } - - /** - * Create error status from throwable - * - * @param e - * @return status - */ - public static IStatus createErrorStatus(Throwable e) { - return createStatus(IStatus.ERROR, - "Unexpected error: " + e.getLocalizedMessage(), e); - } - - /** - * Get log - * - * @return log - */ - public static ILog getLog() { - return Platform.getLog(Platform.getBundle(BUNDLE_ID)); - } - - /** - * Log message and throwable as error status - * - * @param message - * @param t - */ - public static void logError(String message, Throwable t) { - getLog().log(createErrorStatus(message, t)); - } - - /** - * Log throwable as error status - * - * @param t - */ - public static void logError(Throwable t) { - getLog().log(createErrorStatus(t.getMessage(), t)); - } - - /** - * Get repository for url - * - * @param repositoryUrl - * @return repository or null if not present in url - */ - public static Repository getRepository(String repositoryUrl) { - return Repository.createFromUrl(repositoryUrl); - } - - /** - * Create url with github.com host - * - * @param user - * @param project - * @return url - * - * @see #createGitHubUrlAlternate(String, String) - */ - public static String createGitHubUrl(String user, String project) { - return HTTP_GITHUB_COM + '/' + user + '/' + project; - } - - /** - * Create url with github.org host - * - * @param user - * @param project - * @return url - * - * @see #createGitHubUrl(String, String) - */ - public static String createGitHubUrlAlternate(String user, String project) { - return HTTP_WWW_GITHUB_ORG + '/' + user + '/' + project; - } -} diff --git a/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubException.java b/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubException.java deleted file mode 100644 index 6ff1a908..00000000 --- a/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubException.java +++ /dev/null @@ -1,74 +0,0 @@ -/******************************************************************************* - * Copyright (c) 2011 GitHub Inc. - * All rights reserved. This program and the accompanying materials - * are made available under the terms of the Eclipse Public License v1.0 - * which accompanies this distribution, and is available at - * http://www.eclipse.org/legal/epl-v10.html - * - * Contributors: - * Kevin Sawicki (GitHub Inc.) - initial API and implementation - *******************************************************************************/ -package org.eclipse.mylyn.github.internal; - -import java.io.IOException; -import java.text.MessageFormat; -import java.util.List; - -import org.eclipse.egit.github.core.FieldError; -import org.eclipse.egit.github.core.RequestError; -import org.eclipse.egit.github.core.client.RequestException; - -/** - * GitHub exception that wraps and formats a {@link RequestException} - */ -public class GitHubException extends IOException { - - /** - * - */ - private static final long serialVersionUID = -1456910662911777231L; - - /** - * Create GitHub exception from {@link RequestException} - * - * @param cause - */ - public GitHubException(RequestException cause) { - super(cause); - } - - public String getMessage() { - RequestError error = ((RequestException) getCause()).getError(); - StringBuilder message = new StringBuilder(error.getMessage()); - List errors = error.getErrors(); - if (errors != null && errors.size() > 0) { - message.append(':'); - for (FieldError fieldError : errors) - message.append(' ').append(format(fieldError)).append(','); - message.deleteCharAt(message.length() - 1); - } - return message.toString(); - } - - private String format(FieldError error) { - String code = error.getCode(); - String value = error.getValue(); - String field = error.getField(); - String resource = error.getResource(); - if (FieldError.CODE_INVALID.equals(code)) - if (value != null) - return MessageFormat - .format(Messages.FieldError_InvalidFieldWithValue, - value, field); - else - return MessageFormat.format(Messages.FieldError_InvalidField, - field, value); - else if (FieldError.CODE_MISSING_FIELD.equals(code)) - return MessageFormat - .format(Messages.FieldError_MissingField, field); - else - return MessageFormat.format(Messages.FieldError_ResourceError, - field, resource); - } - -} diff --git a/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubRepositoryConnector.java b/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubRepositoryConnector.java deleted file mode 100644 index 24f688b5..00000000 --- a/org.eclipse.mylyn.github.core/src/org/eclipse/mylyn/github/internal/GitHubRepositoryConnector.java +++ /dev/null @@ -1,410 +0,0 @@ -/******************************************************************************* - * Copyright (c) 2011 Red Hat and others. - * All rights reserved. This program and the accompanying materials - * are made available under the terms of the Eclipse Public License v1.0 - * which accompanies this distribution, and is available at - * http://www.eclipse.org/legal/epl-v10.html - * - * Contributors: - * David Green - initial contribution - * Christian Trutz - initial contribution - * Chris Aniszczyk - initial contribution - *******************************************************************************/ -package org.eclipse.mylyn.github.internal; - -import java.io.IOException; -import java.util.Collections; -import java.util.Date; -import java.util.HashMap; -import java.util.LinkedList; -import java.util.List; -import java.util.Map; -import java.util.regex.Matcher; -import java.util.regex.Pattern; - -import org.eclipse.core.runtime.Assert; -import org.eclipse.core.runtime.CoreException; -import org.eclipse.core.runtime.IProgressMonitor; -import org.eclipse.core.runtime.IStatus; -import org.eclipse.core.runtime.Status; -import org.eclipse.egit.github.core.Comment; -import org.eclipse.egit.github.core.Issue; -import org.eclipse.egit.github.core.Label; -import org.eclipse.egit.github.core.Milestone; -import org.eclipse.egit.github.core.Repository; -import org.eclipse.egit.github.core.client.GitHubClient; -import org.eclipse.egit.github.core.client.RequestException; -import org.eclipse.egit.github.core.service.IssueService; -import org.eclipse.egit.github.core.service.LabelService; -import org.eclipse.egit.github.core.service.MilestoneService; -import org.eclipse.egit.github.core.util.LabelComparator; -import org.eclipse.mylyn.commons.net.AuthenticationCredentials; -import org.eclipse.mylyn.commons.net.AuthenticationType; -import org.eclipse.mylyn.commons.net.Policy; -import org.eclipse.mylyn.tasks.core.AbstractRepositoryConnector; -import org.eclipse.mylyn.tasks.core.IRepositoryQuery; -import org.eclipse.mylyn.tasks.core.ITask; -import org.eclipse.mylyn.tasks.core.TaskRepository; -import org.eclipse.mylyn.tasks.core.data.AbstractTaskDataHandler; -import org.eclipse.mylyn.tasks.core.data.TaskAttribute; -import org.eclipse.mylyn.tasks.core.data.TaskData; -import org.eclipse.mylyn.tasks.core.data.TaskDataCollector; -import org.eclipse.mylyn.tasks.core.data.TaskMapper; -import org.eclipse.mylyn.tasks.core.sync.ISynchronizationSession; - -/** - * GitHub connector. - */ -public class GitHubRepositoryConnector extends AbstractRepositoryConnector { - - /** - * GitHub kind. - */ - public static final String KIND = GitHub.CONNECTOR_KIND; - - /** - * Create client for repository - * - * @param repository - * @return client - */ - public static GitHubClient createClient(TaskRepository repository) { - GitHubClient client = new GitHubClient(); - AuthenticationCredentials credentials = repository - .getCredentials(AuthenticationType.REPOSITORY); - if (credentials != null) - client.setCredentials(credentials.getUserName(), - credentials.getPassword()); - return client; - } - - /** - * GitHub specific {@link AbstractTaskDataHandler}. - */ - private final GitHubTaskDataHandler taskDataHandler; - - private final Map> repositoryLabels = Collections - .synchronizedMap(new HashMap>()); - - private final Map> repositoryMilestones = Collections - .synchronizedMap(new HashMap>()); - - /** - * Create GitHub issue repository connector - */ - public GitHubRepositoryConnector() { - taskDataHandler = new GitHubTaskDataHandler(this); - } - - /** - * Refresh labels for repository - * - * @param repository - * @return labels - * @throws CoreException - */ - public List