Hide java project setting files by default.

README.md

@@ -80,6 +80,7 @@ The following settings are supported:
 * `java.import.gradle.enabled` : Enable/disable the Gradle importer.
 * `java.import.maven.enabled` : Enable/disable the Maven importer.
 * `java.autobuild.enabled` : Enable/disable the 'auto build'.
+* `java.maxConcurrentBuilds`: Set max simultaneous project builds.
 * `java.completion.favoriteStaticMembers` : Defines a list of static members or types with static members.
 * `java.completion.importOrder` : Defines the sorting order of import statements.
 * `java.progressReports.enabled` : [Experimental] Enable/disable progress reports from background processes on the server.
@@ -91,8 +92,8 @@ The following settings are supported:
 * `java.completion.guessMethodArguments` : When set to true, method arguments are guessed when a method is selected from as list of code assist proposals.
 * `java.completion.enabled` : Enable/disable code completion support.
 
-*New in 0.36.0:*
-* `java.maxConcurrentBuilds`: Set max simultaneous project builds.
+*New in 0.38.0:*
+* `java.configuration.excludeProjectSettingsFiles`: Exclude the extension generated project setting files(.classpath, .project, .settings, .factorypath) from the file explorer. When set to false, it will not revert the changes made on the setting.json file(s).
 
 
 Troubleshooting

package.json

@@ -89,6 +89,12 @@
           "description": "Specifies the severity of the message when the classpath is incomplete for a Java file",
           "scope": "window"
         },
+        "java.configuration.excludeProjectSettingsFiles": {
+          "type": "boolean",
+          "default": true,
+          "description": "Exclude the extension generated project setting files from the file explorer. When set false, it will not revert the changes made on the setting.json file(s).",
+          "scope": "window"
+        },
         "java.configuration.updateBuildConfiguration": {
           "type": [
             "string"

src/extension.ts

@@ -14,8 +14,9 @@ SourceAttachmentRequest, SourceAttachmentResult, SourceAttachmentAttribute } fro
 import { ExtensionAPI } from './extension.api';
 import * as buildpath from './buildpath';
 import * as net from 'net';
+import { getJavaConfiguration } from './utils';
+import { onConfigurationChange, excludeProjectSettingsFiles } from './settings';
 
-let oldConfig;
 let lastStatus;
 let languageClient: LanguageClient;
 let jdtEventEmitter = new EventEmitter<Uri>();
@@ -82,7 +83,6 @@ export function activate(context: ExtensionContext): Promise<ExtensionAPI> {
 				item.command = Commands.OPEN_OUTPUT;
 				let progressBar = window.createStatusBarItem(StatusBarAlignment.Left, Number.MIN_VALUE+1);
 
-				oldConfig = getJavaConfiguration();
 				let serverOptions;
 				let port = process.env['SERVER_PORT'];
 				if (!port) {
@@ -301,6 +301,11 @@ export function activate(context: ExtensionContext): Promise<ExtensionAPI> {
 						}
 					};
 					workspace.registerTextDocumentContentProvider('jdt', provider);
+					if (workspace.workspaceFolders) {
+						workspace.workspaceFolders.map((folder) => {
+							excludeProjectSettingsFiles(folder.uri);
+						});
+					}
 				});
 
 				let cleanWorkspaceExists = fs.existsSync( path.join(workspacePath,  cleanWorkspaceFileName));
@@ -435,34 +440,6 @@ function isJavaConfigFile(path: String) {
 	return path.endsWith('pom.xml') || path.endsWith('.gradle');
 }
 
-function onConfigurationChange() {
-	return workspace.onDidChangeConfiguration(params => {
-		let newConfig = getJavaConfiguration();
-		if (hasJavaConfigChanged(oldConfig, newConfig)) {
-			let msg = 'Java Language Server configuration changed, please restart VS Code.';
-			let action = 'Restart Now';
-			let restartId = Commands.RELOAD_WINDOW;
-			oldConfig = newConfig;
-			window.showWarningMessage(msg, action).then((selection) => {
-				if (action === selection) {
-					commands.executeCommand(restartId);
-				}
-			});
-		}
-	});
-}
-
-function hasJavaConfigChanged(oldConfig, newConfig) {
-	return hasConfigKeyChanged('home', oldConfig, newConfig)
-		|| hasConfigKeyChanged('jdt.ls.vmargs', oldConfig, newConfig)
-		|| hasConfigKeyChanged('progressReports.enabled', oldConfig, newConfig);
-}
-
-function hasConfigKeyChanged(key, oldConfig, newConfig) {
-	return oldConfig.get(key) !== newConfig.get(key);
-}
-
-
 function getTempWorkspace() {
 	return path.resolve(os.tmpdir(), 'vscodesws_' + makeRandomHexString(5));
 }
@@ -477,9 +454,6 @@ function makeRandomHexString(length) {
 	return result;
 }
 
-function getJavaConfiguration(): WorkspaceConfiguration {
-	return workspace.getConfiguration('java');
-}
 
 async function cleanWorkspace(workspacePath) {
 	const doIt = 'Restart and delete';
@@ -746,4 +720,4 @@ function isPrefix(parentPath: string, childPath: string): boolean {
 	}
 	const relative = path.relative(parentPath, childPath);
 	return !!relative && !relative.startsWith('..') && !path.isAbsolute(relative);
-}
+}

src/settings.ts

@@ -0,0 +1,69 @@
+'use strict';
+
+import { window, Uri, workspace, WorkspaceConfiguration, commands, ConfigurationTarget } from 'vscode';
+import { LanguageClient } from 'vscode-languageclient';
+import { Commands } from './commands';
+import { getJavaConfiguration } from './utils';
+
+
+const DEFAULT_HIDDEN_FILES: string[] = ['**/.classpath', '**/.project', '**/.settings', '**/.factorypath'];
+
+let oldConfig: WorkspaceConfiguration = getJavaConfiguration();
+
+export function onConfigurationChange() {
+	return workspace.onDidChangeConfiguration(params => {
+		if (!params.affectsConfiguration('java')) {
+			return;
+		}
+		let newConfig = getJavaConfiguration();
+		if (hasJavaConfigChanged(oldConfig, newConfig)) {
+			let msg = 'Java Language Server configuration changed, please restart VS Code.';
+			let action = 'Restart Now';
+			let restartId = Commands.RELOAD_WINDOW;
+			oldConfig = newConfig;
+			window.showWarningMessage(msg, action).then((selection) => {
+				if (action === selection) {
+					commands.executeCommand(restartId);
+				}
+			});
+		}
+	});
+}
+
+export function excludeProjectSettingsFiles(workspaceUri: Uri) {
+	const excudedConfig = getJavaConfiguration().get('configuration.excludeProjectSettingsFiles');
+	if (excudedConfig) {
+		const config = workspace.getConfiguration('files', workspaceUri);
+		const excludedValue: Object = config.get('exclude');
+		const needExcludeFiles: Object = {};
+
+		let needUpdate = false;
+		for (const hiddenFiles of DEFAULT_HIDDEN_FILES) {
+			if (!excludedValue.hasOwnProperty(hiddenFiles)) {
+				needExcludeFiles[hiddenFiles] = true;
+				needUpdate = true;
+			}
+		}
+		if (needUpdate) {
+			window.showInformationMessage('Do you want to exclude the VSCode Java project settings files(.classpath, .project. .settings, .factorypath) from the file explorer.', 'Always', 'Workspace', 'Never').then((result) => {
+				if (result === 'Always') {
+					config.update('exclude', needExcludeFiles, ConfigurationTarget.Global);
+				} if (result === 'Workspace') {
+					config.update('exclude', needExcludeFiles, ConfigurationTarget.Workspace);
+				} else if (result === 'Never') {
+					getJavaConfiguration().update('configuration.excludeProjectSettingsFiles', false, ConfigurationTarget.Global);
+				}
+			});
+		}
+	}
+}
+
+function hasJavaConfigChanged(oldConfig: WorkspaceConfiguration, newConfig: WorkspaceConfiguration) {
+	return hasConfigKeyChanged('home', oldConfig, newConfig)
+		|| hasConfigKeyChanged('jdt.ls.vmargs', oldConfig, newConfig)
+		|| hasConfigKeyChanged('progressReports.enabled', oldConfig, newConfig);
+}
+
+function hasConfigKeyChanged(key, oldConfig, newConfig) {
+	return oldConfig.get(key) !== newConfig.get(key);
+}

src/utils.ts

@@ -0,0 +1,7 @@
+'use strict';
+
+import { workspace, WorkspaceConfiguration } from 'vscode';
+
+export function getJavaConfiguration(): WorkspaceConfiguration {
+	return workspace.getConfiguration('java');
+}