Moves suppressed msgs from globalState to settings

eamodio · eamodio · commit 7862c7ccecb8 · 2017-11-21T22:03:17.000-05:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,13 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](http://keepachangelog.com/) and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## [Unreleased]
+### Added
+- Adds `gitlens.advanced.messages` setting to specify which messages should be suppressed
+
+### Changed
+- Changes from using `globalState` to use `gitlens.advanced.messages` setting for message suppression - provides more control and avoids strange intermittent with `globalState`
+
 ## [6.1.2] - 2017-11-21
 ### Fixed
 - Fixes [#207](https://github.com/eamodio/vscode-gitlens/issues/207) - Applying and deleting stashes suddenly stopped working
diff --git a/README.md b/README.md
@@ -463,6 +463,7 @@ GitLens is highly customizable and provides many configuration settings to allow
 |`gitlens.advanced.git`|Specifies the git path to use
 |`gitlens.advanced.repositorySearchDepth`|Specifies how many folders deep to search for repositories
 |`gitlens.advanced.menus`|Specifies which commands will be added to which menus
+|`gitlens.advanced.messages`|Specifies which messages should be suppressed
 |`gitlens.advanced.caching.enabled`|Specifies whether git output will be cached
 |`gitlens.advanced.caching.maxLines`|Specifies the threshold for caching larger documents
 |`gitlens.advanced.maxQuickHistory`|Specifies the maximum number of QuickPick history entries to show
diff --git a/package.json b/package.json
@@ -974,6 +974,55 @@
                     "description": "Specifies which commands will be added to which menus",
                     "scope": "window"
                 },
+                "gitlens.advanced.messages": {
+                    "type": "object",
+                    "default": {
+                        "suppressCommitHasNoPreviousCommitWarning": false,
+                        "suppressCommitNotFoundWarning": false,
+                        "suppressFileNotUnderSourceControlWarning": false,
+                        "suppressGitVersionWarning": false,
+                        "suppressLineUncommittedWarning": false,
+                        "suppressNoRepositoryWarning": false,
+                        "suppressUpdateNotice": false,
+                        "suppressWelcomeNotice": false
+                    },
+                    "properties": {
+                        "suppressCommitHasNoPreviousCommitWarning": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressCommitNotFoundWarning": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressFileNotUnderSourceControlWarning": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressGitVersionWarning": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressLineUncommittedWarning": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressNoRepositoryWarning": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressUpdateNotice": {
+                            "type": "boolean",
+                            "default": false
+                        },
+                        "suppressWelcomeNotice": {
+                            "type": "boolean",
+                            "default": false
+                        }
+                    },
+                    "description": "Specifies which messages should be suppressed",
+                    "scope": "window"
+                },
                 "gitlens.advanced.quickPick.closeOnFocusOut": {
                     "type": "boolean",
                     "default": true,
diff --git a/src/commands.ts b/src/commands.ts
@@ -87,7 +87,7 @@ export function configureCommands(
     context.subscriptions.push(new Commands.ToggleFileBlameCommand(annotationController));
     context.subscriptions.push(new Commands.ToggleFileRecentChangesCommand(annotationController));
     context.subscriptions.push(new Commands.ToggleLineBlameCommand(currentLineController));
-    context.subscriptions.push(new Commands.ResetSuppressedWarningsCommand(context));
+    context.subscriptions.push(new Commands.ResetSuppressedWarningsCommand());
     context.subscriptions.push(new Commands.ShowLastQuickPickCommand());
     context.subscriptions.push(new Commands.ShowQuickBranchHistoryCommand(git));
     context.subscriptions.push(new Commands.ShowQuickCurrentBranchHistoryCommand(git));
diff --git a/src/commands/resetSuppressedWarnings.ts b/src/commands/resetSuppressedWarnings.ts
@@ -1,20 +1,15 @@
 'use strict';
-import { Objects } from '../system';
-import { ExtensionContext } from 'vscode';
+import { ConfigurationTarget } from 'vscode';
 import { Command, Commands } from './common';
-import { SuppressedKeys } from '../messages';
+import { configuration } from '../configuration';
 
 export class ResetSuppressedWarningsCommand extends Command {
 
-    constructor(
-        private readonly context: ExtensionContext
-    ) {
+    constructor() {
         super(Commands.ResetSuppressedWarnings);
     }
 
     async execute() {
-        for (const key of Objects.values(SuppressedKeys)) {
-            await this.context.globalState.update(key, undefined);
-        }
+        await configuration.update(configuration.name('advanced')('messages').value, undefined, ConfigurationTarget.Global);
     }
 }
diff --git a/src/configuration.ts b/src/configuration.ts
@@ -1,5 +1,5 @@
 'use strict';
-import { ConfigurationChangeEvent, Event, EventEmitter, ExtensionContext, Uri, workspace } from 'vscode';
+import { ConfigurationChangeEvent, ConfigurationTarget, Event, EventEmitter, ExtensionContext, Uri, workspace } from 'vscode';
 import { FileAnnotationType } from './annotations/annotationController';
 import { ExtensionKey } from './constants';
 import { LineAnnotationType } from './currentLineController';
@@ -90,6 +90,16 @@ export interface IAdvancedConfig {
             remote: boolean;
         };
     };
+    messages: {
+        suppressCommitHasNoPreviousCommitWarning: boolean,
+        suppressCommitNotFoundWarning: boolean,
+        suppressFileNotUnderSourceControlWarning: boolean,
+        suppressGitVersionWarning: boolean,
+        suppressLineUncommittedWarning: boolean,
+        suppressNoRepositoryWarning: boolean,
+        suppressUpdateNotice: boolean,
+        suppressWelcomeNotice: boolean
+    };
     quickPick: {
         closeOnFocusOut: boolean;
     };
@@ -363,45 +373,45 @@ const emptyConfig: IConfig = {
         file: {
             gutter: {
                 format: '',
-    dateFormat: null,
-    compact: false,
-    heatmap: {
+                dateFormat: null,
+                compact: false,
+                heatmap: {
                     enabled: false,
-        location: 'left'
-    },
-    hover: {
+                    location: 'left'
+                },
+                hover: {
                     details: false,
-        changes: false,
-        wholeLine: false
-    }
+                    changes: false,
+                    wholeLine: false
+                }
             },
-    hover: {
+            hover: {
                 details: false,
-        changes: false,
-        heatmap: {
+                changes: false,
+                heatmap: {
                     enabled: false
-        }
+                }
             },
-    recentChanges: {
+            recentChanges: {
                 hover: {
                     details: false,
-        changes: false
-    }
+                    changes: false
+                }
             }
         },
-    line: {
+        line: {
             hover: {
                 details: false,
-        changes: false
-    },
-        trailing: {
+                changes: false
+            },
+            trailing: {
                 format: '',
-            dateFormat: null,
-            hover: {
+                dateFormat: null,
+                hover: {
                     details: false,
-                changes: false,
-                wholeLine: false
-            }
+                    changes: false,
+                    wholeLine: false
+                }
             }
         }
     },
@@ -472,9 +482,9 @@ const emptyConfig: IConfig = {
         codeLens: {
             unsavedChanges: {
                 recentChangeAndAuthors: '',
-        recentChangeOnly: '',
-        authorsOnly: ''
-    }
+                recentChangeOnly: '',
+                authorsOnly: ''
+            }
         }
     },
     theme: themeDefaults,
@@ -484,16 +494,16 @@ const emptyConfig: IConfig = {
     advanced: {
         caching: {
             enabled: false,
-        maxLines: 0
-    },
+            maxLines: 0
+        },
         git: '',
         maxQuickHistory: 0,
         menus: {
             explorerContext: {
                 fileDiff: false,
-            history: false,
-            remote: false
-        },
+                history: false,
+                remote: false
+            },
             editorContext: {
                 blame: false,
                 copy: false,
@@ -516,6 +526,16 @@ const emptyConfig: IConfig = {
                 remote: false
             }
         },
+        messages: {
+            suppressCommitHasNoPreviousCommitWarning: false,
+            suppressCommitNotFoundWarning: false,
+            suppressFileNotUnderSourceControlWarning: false,
+            suppressGitVersionWarning: false,
+            suppressLineUncommittedWarning: false,
+            suppressNoRepositoryWarning: false,
+            suppressUpdateNotice: false,
+            suppressWelcomeNotice: false
+        },
         quickPick: {
             closeOnFocusOut: false
         },
@@ -563,6 +583,10 @@ export class Configuration {
     name<K extends keyof IConfig>(name: K) {
         return Functions.propOf(emptyConfig, name);
     }
+
+    update(section: string, value: any, target: ConfigurationTarget) {
+        return workspace.getConfiguration(ExtensionKey).update(section, value, target);
+    }
 }
 
 export const configuration = new Configuration();
diff --git a/src/extension.ts b/src/extension.ts
@@ -1,7 +1,8 @@
 'use strict';
-import { ExtensionContext, extensions, languages, window, workspace } from 'vscode';
+import { Objects } from './system';
+import { ConfigurationTarget, ExtensionContext, extensions, languages, window, workspace } from 'vscode';
 import { AnnotationController } from './annotations/annotationController';
-import { Configuration, IConfig } from './configuration';
+import { configuration, Configuration, IConfig } from './configuration';
 import { ApplicationInsightsKey, CommandContext, ExtensionKey, GlobalState, QualifiedExtensionId, setCommandContext } from './constants';
 import { CodeLensController } from './codeLensController';
 import { configureCommands } from './commands';
@@ -12,14 +13,13 @@ import { GitRevisionCodeLensProvider } from './gitRevisionCodeLensProvider';
 import { GitContextTracker, GitService } from './gitService';
 import { Keyboard } from './keyboard';
 import { Logger } from './logger';
-import { Messages, SuppressedKeys } from './messages';
+import { Messages, SuppressedMessages } from './messages';
 import { Telemetry } from './telemetry';
 
 // this method is called when your extension is activated
 export async function activate(context: ExtensionContext) {
     Configuration.configure(context);
     Logger.configure(context);
-    Messages.configure(context);
     Telemetry.configure(ApplicationInsightsKey);
 
     const gitlens = extensions.getExtension(QualifiedExtensionId)!;
@@ -49,8 +49,11 @@ export async function activate(context: ExtensionContext) {
     telemetryContext['git.version'] = gitVersion;
     Telemetry.setContext(telemetryContext);
 
+    const previousVersion = context.globalState.get<string>(GlobalState.GitLensVersion);
+
+    await migrateSettings(context, previousVersion);
     notifyOnUnsupportedGitVersion(context, gitVersion);
-    notifyOnNewGitLensVersion(context, gitlensVersion);
+    notifyOnNewGitLensVersion(context, gitlensVersion, previousVersion);
 
     await context.globalState.update(GlobalState.GitLensVersion, gitlensVersion);
 
@@ -87,14 +90,50 @@ export async function activate(context: ExtensionContext) {
 // this method is called when your extension is deactivated
 export function deactivate() { }
 
-async function notifyOnNewGitLensVersion(context: ExtensionContext, version: string) {
-    if (context.globalState.get(SuppressedKeys.UpdateNotice, false)) return;
+const migration = {
+    major: 6,
+    minor: 1,
+    patch: 2
+};
 
-    const previousVersion = context.globalState.get<string>(GlobalState.GitLensVersion);
+async function migrateSettings(context: ExtensionContext, previousVersion: string | undefined) {
+    if (previousVersion === undefined) return;
+
+    const [major, minor, patch] = previousVersion.split('.');
+    if (parseInt(major, 10) >= migration.major && parseInt(minor, 10) >= migration.minor && parseInt(patch, 10) >= migration.patch) return;
+
+    try {
+        const section = configuration.name('advanced')('messages').value;
+        const messages: { [key: string]: boolean } = configuration.get(section);
+
+        let migrated = false;
+
+        for (const m of Objects.values(SuppressedMessages)) {
+            const suppressed = context.globalState.get<boolean>(m);
+            if (suppressed === undefined) continue;
+
+            migrated = true;
+            messages[m] = suppressed;
+
+            context.globalState.update(m, undefined);
+        }
+
+        if (!migrated) return;
+
+        await configuration.update(section, messages, ConfigurationTarget.Global);
+    }
+    catch (ex) {
+        Logger.error(ex, 'migrateSettings');
+    }
+}
+
+async function notifyOnNewGitLensVersion(context: ExtensionContext, version: string, previousVersion: string | undefined) {
+    if (configuration.get<boolean>(configuration.name('advanced')('messages')(SuppressedMessages.UpdateNotice).value)) return;
 
     if (previousVersion === undefined) {
         Logger.log(`GitLens first-time install`);
         await Messages.showWelcomeMessage();
+
         return;
     }
 
diff --git a/src/messages.ts b/src/messages.ts
diff --git a/src/views/explorerCommands.ts b/src/views/explorerCommands.ts
