| Summary: | Bug with translation of the "description" property | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [RT] RAP | Reporter: | Igor Pavlenko <dopperst> | ||||||||||||
| Component: | Workbench | Assignee: | Project Inbox <rap-inbox> | ||||||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||||||
| Severity: | normal | ||||||||||||||
| Priority: | P3 | ||||||||||||||
| Version: | 1.2 | ||||||||||||||
| Target Milestone: | 1.3 M1 | ||||||||||||||
| Hardware: | All | ||||||||||||||
| OS: | All | ||||||||||||||
| Whiteboard: | |||||||||||||||
| Attachments: |
|
||||||||||||||
|
Description
Igor Pavlenko
Created attachment 140218 [details]
Path to fix the bug
Look a the org.eclipse.core.internal.registry.ConfigurationElement from org.eclipse.rap.equinox.registry patch-fragment.
Look at the ConfigurationElement#getAttribute(String) method. It is reimplemented to support localization on demand.
Look at the ConfigurationElement#getValue() method. I think it also should be reimplemented (as in my patch).
It is important, for example, for the org.eclipse.ui.internal.dialogs.WorkbenchWizardElement.getDescription() method (it use the "description" property)
Igor, thanks for the patch. But I still didn't get the point what "description" you're refering to. Can you give me a concrete example where the current implemention fails? I'd really like to get these fixes in - just want to be sure for what they're for ;-) Created attachment 142338 [details]
Extensions page in plugin.xml editor
Created attachment 142339 [details]
Wizard without patch
Created attachment 142340 [details]
Wizard with patch
Created attachment 142341 [details]
Wizards description value in plugin.xml value
Look at the attached images for details Great, thanks Igor! Will take a look at this during M1. Thanks for the patch Igor. Applied to CVS HEAD with some minor modifications (added comment). |