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

Bug 437777

Summary: JavaScript Validator - "Switch case fall-through:" unexternalized
Product: [ECD] Orion Reporter: Kit Lo <kitlo>
Component: JS ToolsAssignee: Project Inbox <orion.client-inbox>
Status: RESOLVED WORKSFORME QA Contact:
Severity: normal    
Priority: P3 CC: Michael_Rennie
Version: 6.0   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Settings - JavaScript Validator.png none

Description Kit Lo CLA 2014-06-19 15:16:49 EDT
Created attachment 244371 [details]
Settings - JavaScript Validator.png

JavaScript Validator - "Switch case fall-through:" unexternalized
Comment 1 Michael Rennie CLA 2014-06-19 15:47:25 EDT
Hmm, something seems fishy here, the pref is externalized, here is the entry in the NLS file:

'noFallthrough' : 'Switch case fall-through', //$NON-NLS-0$  //$NON-NLS-1$

and here is its definition:

{id: "validate_no_fallthrough",  //$NON-NLS-0$
 nameKey: 'noFallthrough',  //$NON-NLS-0$
 type: "number",  //$NON-NLS-0$
 defaultValue: error,
 options: severities
},

perhaps you need to clear your local storage and refresh?
Comment 2 Kit Lo CLA 2014-06-19 16:08:20 EDT
I don't see the string in this snapshot http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/snapshot/bogTag.zip in this file org.eclipse.orion.client.javascript\web\javascript\nls\root\messages.js

Am I looking at the right file? Was the string checked in recently?
Comment 3 Michael Rennie CLA 2014-06-20 09:56:40 EDT
(In reply to Kit Lo from comment #2)
> I don't see the string in this snapshot
> http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/snapshot/bogTag.
> zip in this file
> org.eclipse.orion.client.javascript\web\javascript\nls\root\messages.js
> 
> Am I looking at the right file? Was the string checked in recently?

It was just added this week (bug 429815). The string does appear in the file:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/tree/bundles/org.eclipse.orion.client.javascript/web/javascript/nls/root/messages.js

I guess maybe it didn't make it into that tag?
Comment 4 Michael Rennie CLA 2014-07-08 23:40:00 EDT
(In reply to Kit Lo from comment #2)
> I don't see the string in this snapshot
> http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/snapshot/bogTag.
> zip in this file
> org.eclipse.orion.client.javascript\web\javascript\nls\root\messages.js
> 
> Am I looking at the right file? Was the string checked in recently?

Did you manage to get a build with the externalized string in it? Can we close this?
Comment 5 Kit Lo CLA 2014-07-09 00:30:27 EDT
Yes, I can see the string externalized now. We can resolve this. Thanks!
Comment 6 Michael Rennie CLA 2014-07-09 03:25:16 EDT
(In reply to Kit Lo from comment #5)
> Yes, I can see the string externalized now. We can resolve this. Thanks!

Thanks for reporting back Kit. Closing worksforme.