Summary: | allow typing in task data directory field | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | z_Archived | Reporter: | Andrew Overholt <overholt> | ||||||
Component: | Mylyn | Assignee: | Abner Ballardo <modlost> | ||||||
Status: | RESOLVED FIXED | QA Contact: | |||||||
Severity: | enhancement | ||||||||
Priority: | P2 | CC: | modlost, steffen.pingel | ||||||
Version: | 3.1 | Keywords: | bugday, contributed, helpwanted | ||||||
Target Milestone: | 3.4 | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Andrew Overholt
2009-05-13 11:18:06 EDT
Sounds like a good idea. I think most place in platform allow you to type in the directory text boxes. Hi, I want to contribute fixing this bug Go for it :). Created attachment 160130 [details]
Patch fixing bug
Hi everybody,
With this patch the task data directory field is editable. The validations after choosing the directory has been moved when clicking Apply or Ok buttons. Finally if the directory doesn't exists it will try to create it and its parents.
Created attachment 160131 [details]
mylyn/context/zip
Thanks. I'll review later this week. Great patch! I have applied with a minor simplification: checkForExistingTasklist() now returns false if the user cancels instead of storing the response in a field. Noticed a problem when the following steps are executed: 1. Import preferences that include task directory setting 2. Use Restore Defaults on Tasks preferences page This fails with an error that the directory does not exist. The directory should be automatically created instead. Can not reproduce. The workspace directory may have had the wrong permissions. |