Summary: | [TabbedProperties] TabbedPropertyRegistry logs error when no contributors are available | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Danail Branekov <danail.branekov> | ||||
Component: | IDE | Assignee: | Anthony Hunter <ahunter.eclipse> | ||||
Status: | REOPENED --- | QA Contact: | |||||
Severity: | minor | ||||||
Priority: | P3 | ||||||
Version: | 3.4.1 | ||||||
Target Milestone: | --- | ||||||
Hardware: | PC | ||||||
OS: | Windows Vista | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Danail Branekov
2008-10-31 09:07:15 EDT
By design this is an error. You say you are providing a tabbed property sheet page so your workbench part must implement ITabbedPropertySheetPageContributor. You then say that your contributor id is "my.dummy.contributor" but do not provide a propertyContributor extension for it. In your view, simply define the propertyContributor extension without tabs and sections and you are done. Tabs and sections can go in other plug-ins. Hi Anthony, From my design point of view it is pretty fine to have a view which implements the ITabbedPropertySheetPageContributor. If there will be contributors to it or not might be up to other tools. For example, in my case, I am developing something like a framework (which contains a view) which expects other plugins to contribute stuff to it. Therefore this "framework" provides quite a lot of entry points (one of which is the tabbed property feature) for contributors to plug in. The framework itself does not care to provide properties at all - it simply needs to provide functionality to its contributors. As there might be no contributors (the user missed to install them or has simply disabled them) you would get this error entry. Defining a dummy contributor would be simply a workaround and would simply hide the issue. Regards, Danail ok, we cannot remove the message to the log, so you are just requesting it be an INFO rather than ERROR? Yes, this is my request. The reason for it is that in our local infrastructure we have an automated test which opens all perspectives and views and verifies that there are error entries in the log file. If this test failed, quality management makes our lives miserable :) Thanks for understanding! This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. |