Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 193430 - Trac tasks accessed in web mode do not display browser tab anymore
Summary: Trac tasks accessed in web mode do not display browser tab anymore
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 2.0   Edit
Assignee: Steffen Pingel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-19 21:53 EDT by Steffen Pingel CLA
Modified: 2007-06-20 04:30 EDT (History)
0 users

See Also:


Attachments
mylar/context/zip (5.71 KB, application/octet-stream)
2007-06-20 01:56 EDT, Steffen Pingel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Steffen Pingel CLA 2007-06-19 21:53:47 EDT
When I open a web task I don't see a browser widget or straight forward button to open the task in a browser.
Comment 1 Mik Kersten CLA 2007-06-19 22:02:57 EDT
What do you mean by web tasks, those created by the generic web repository connector or just tasks that have their URL set?
Comment 2 Steffen Pingel CLA 2007-06-19 22:08:25 EDT
Tasks from the web connector seem to work but not Trac tasks that are accessed in web mode and do not have task data. I can look into it.
Comment 3 Mik Kersten CLA 2007-06-20 00:58:51 EDT
What you will probably want to do is copy what I did for the WebConnectorUi, which now contributes a browser editor.
Comment 4 Steffen Pingel CLA 2007-06-20 01:34:52 EDT
While trying to figure out how to do this I ran into strange code in TaskEditor.addPages():

	IEditorPart editor = factory.createEditor(this, getEditorInput());
	IEditorInput input = task != null ? factory.createEditorInput(task) : getEditorInput();

This creates the editor from the factory using a TaskEditorInput created by TaskUiUtil, then creates a new RepositoryTaskEditorInput through the connector specific factory and initializes the editor with that input. I would expect the input to be created first and to be passed to factory.createEditor()?
Comment 5 Mik Kersten CLA 2007-06-20 01:42:50 EDT
Yes, that is a bit weird and begs fixing. I forget now why I did it that way, but I'm guessing that they only two bits of information needed to create the editor are the task and whether or not it is a new task.
Comment 6 Steffen Pingel CLA 2007-06-20 01:56:40 EDT
For Trac it requires extra checking of the editor input. Looking at the API of AbstractTaskEditorInput I would have anticipated different behavior from the framework. I had to read the implementation of TaskEditor to understand what was going on and to get this right.

We should consider making BrowserEditorInput, WebTaskEditorFactory and BrowserFormPage API since all plug-ins displaying web tasks will need that.

Comment 7 Steffen Pingel CLA 2007-06-20 01:56:42 EDT
Created attachment 71831 [details]
mylar/context/zip
Comment 8 Mik Kersten CLA 2007-06-20 02:57:32 EDT
Steffen: very good point, please file a bug for that.
Comment 9 Steffen Pingel CLA 2007-06-20 04:30:43 EDT
Created bug 193472.