Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 381679

Summary: Plug-ins localization confusion
Product: [ECD] Orion Reporter: Szymon Brandys <Szymon.Brandys>
Component: ClientAssignee: Malgorzata Janczarska <malgorzata.tomczyk>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: simon_kaegi, susan
Version: 0.5   
Target Milestone: 0.5 RC1   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Szymon Brandys CLA 2012-06-05 06:21:04 EDT
See gitplugin.html for instance:

provider.registerServiceProvider("orion.navigate.command", {}, {
				name: "Git Status",
				id: "eclipse.git.status",
				tooltip: "Go to Git Status",
				nls: "git/nls/gitmessages",
				validationProperties: [
					{source: "Git:StatusLocation", variableName: "GitStatusLocation"},
					{source: "Directory", match: true}
				],
				uriTemplate: "{OrionHome}/git/git-status2.html#{GitStatusLocation}",
				forceSingleItem: true
			});
			
With the recent change in localization "Git Status" and "Go to Git Status" are no more messages, but keys in the message file pointed in nls property. That's confusing. Either the property should be nameKey/tooltipKey or we should have name: message["Git Status"] as in other places.
Comment 1 Malgorzata Janczarska CLA 2012-06-05 08:35:04 EDT
> Either the property should be nameKey/tooltipKey or we should have
> name: message["Git Status"] as in other places.
We can't have message["Git Status"], because that would require to localize messages on the plugin site and we are moving it to the client. nameKey/tooltipKey should be fine though.
Comment 2 Malgorzata Janczarska CLA 2012-06-06 09:48:30 EDT
Done.
This page describes how to globalize plugins now:
http://wiki.eclipse.org/Orion/Internationalization#Plugins
Comment 3 Szymon Brandys CLA 2012-06-06 13:39:50 EDT
I had to revert the fix since 'Related' pages stopped working. Reverting commit is 94fd854b815c2ba9e7e9e6f53ec9ae92b90d36cf.
Comment 4 Malgorzata Janczarska CLA 2012-06-06 14:33:22 EDT
(In reply to comment #3)
> I had to revert the fix since 'Related' pages stopped working. Reverting commit
> is 94fd854b815c2ba9e7e9e6f53ec9ae92b90d36cf.
Sorry about that, corrected.