Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.
Mylyn/Refactoring
Contents
For RC0
- [Rob] Offline externalization in xml form
- RepositoryTaskAttribute no longer holds options
- Remove name and hidden property from RepositoryTaskAttribute
- Remove all methods from AbstractAttributeFactory except for createAttribute() and getDateForAttributeType() [default implementation should return attribute.getDate()?]
- Attributes need to be created by connectors
- [Rob] bug#176513
- [Rob] bug#161734
- [Steffen] AbstractRepositoryTaskEditor ui factory implementation
- [Steffen] bug#173011 - specifically: add IProgressMonitor parameter to IO methods
Related Bug Reports
- 160389 [api] change how offline task data is cached to disk and refactor attribute factory
- 176513: [api] add support for requesting multiple task data in single request
https://bugs.eclipse.org/bugs/show_bug.cgi?id=176513
Requirements
- Persistence
- Provide a generic API for storing task attributes
- Configuration extraction
- Presenation api
API
Changes to current implementation:
- RepositoryTaskData becomes a generic storage API
- Excpected attributes and data types are specified through Java interfaces
Task Data
ITaskDataManager { TaskDataState getTaskDataState(String repositoryUrl, String id); void saveTaskDataState(TaskDataState taskState); void refactorRepositoryUrl(String oldUrl, String newUrl); void removeTaskDataState(String repositoryUrl, String id); void saveNewTaskDataState(TaskDataState newTaskDataState); // (rfc) Sets unique new id Set<TaskDataState> getNewTaskDataState(String repositoryUrl); // (rfc) }
- TaskDataState created via ITaskDataHandler.buildTaskDataState(RepositoryTaskData newData, RepositoryTaskData oldData, Set<RepositoryTaskAttribute> edits);
- TaskDataState is an unmodifiable object but not final.
TaskDataState { public TaskDataState(RepositoryTaskData newTaskData, RepositoryTaskData oldTaskData, Set<RepositoryTaskAttribute> edits); // Constructor sets values but does not do computation public init(); // Perform constructor specific initialization (i.e. calc changed attributes etc) RepositoryTaskData newTaskData; RepositoryTaskData oldTaskData; Set<RepositoryTaskAttribute> edits; isStateModified(); hasIncomingChanges(); hasChanged(RepositoryTaskAttribute attribute); Set<RepositoryTaskAttribute> getChanged(); void discardEdits(String repositoryUrl, String id); }
Task Attributes
Attributes are stored in RepositoryTaskData objects which manages all offline data for a task. For simplicity attributes can have properties (key/value pairs) but not have child attributes. To easily store and retrieve a list of related attributes such as attachments or comment attributes have a type (e.g. "attachment", "comment", "custom").
RepositoryTaskData { RepositoryTaskAttributeMapper mapper; RepositoryTaskAttribute createAttribute(String id, String type); RepositoryTaskAttribute getAttribute(String id); RepositoryTaskAttribute[] getAttributes(String type); }
RepositoryTaskAttribute { // IMemento style String id; // unique ID void setDateValue(Date date); Date getDateValue(); ... // support for String, int, float, boolean }
To map from predefined attribute keys that are used by the editor to access information about changed attributes each task data object holds on to a mapper (former AbstractAttributeFactory):
RepositoryTaskAttributeMapper { String mapToAttributeID(String mylarID); }
Mylar Task Model
Make this model explicit and define it in terms of Java interfaces. Only these interfaces are used by the editor to set/get values. Default implementation of proxy classes are provided for mapping to underlying task data objects.
Accessor methods should mirror fields available on AbstractRepositoryTask. (This could be separate from the offline storage so connectors without a TaskDataHandler could provide attachment support.)
public interface IRepositoryTask { String getDescription(); void setDescription(String description); ... }
public class RepositoryTaskProxy { RepositoryTaskProxy(RepositoryTaskData taskData); String getDescription() { taskData.getAttribute(RepositoryTaskData.DESCRIPTION).getStringValue(); } void setDescription(String description) { taskData.getAttribute(RepositoryTaskData.DESCRIPTION).setStringValue(description); } void createAttributes() { taskData.createAttribute(RepositoryTaskData.DESCRIPTION); } ... }
IRepositoryTaskComment { String getId(); // allows to map to underlying attribute? String getAuthor(); Date getCreated(); ... }
IRepositoryTaskAttachment { String getFilename() String getDescription() ... }
IRepositoryTaskOperation { ... }
Instances of the interfaces are provided by ITaskDataHandler:
ITaskDataHandler { IRepositoryTask getTask(RepositoryTaskData); IRepositoryTaskAttachment[] getAttachments(RepositoryTaskData); IRepositoryTaskComment[] getComments(RepositoryTaskData); }
Editor/UI
The editor should only hold on to a RepositoryTaskData object. Connectors provide a factory for UI representation of attributes:
AbstractConnectorUi { abstract ITaskUiFactory getTaskUiFactory() }
ITaskUiFactory { String getLabel(RepositoryTaskAttribute attr) Control createEditor(RepositoryTaskAttribute attr, Composite parent) void addFieldEditors(Set<RepositoryTaskAttribute>, Composite parent); // reponsible for layout/order String getToolTip(RepositoryTaskAttribute attr) }
Mylar provides default implementations to create an editor for an attribute:
MylarEditorFactory { Control createComboEditor(RepositoryTaskAttribute attr, Composite parent, String[] options); }