refactor(configuration): avoid unnecessary configuration event

Not send event when not find folder configuration file, avoid issues
when opening library files out of workspace folder.

Closes #3625

src/configuration/index.ts

@@ -180,21 +180,16 @@ export default class Configurations {
     let u = URI.parse(uri)
     if (u.scheme != 'file') return
     let filepath = u.fsPath
-    let find = false
     for (let [configFile, model] of this.foldConfigurations) {
       let root = path.resolve(configFile, '../..')
       if (isParentFolder(root, filepath, true)) {
-        find = true
         if (this.workspaceConfigFile != configFile) {
+          logger.debug('Change folder configuration to:', configFile)
           this.changeConfiguration(ConfigurationTarget.Workspace, model, configFile)
         }
         break
       }
     }
-    if (this.workspaceConfigFile && find === false) {
-      logger.debug('reset folder configuration to empty')
-      this.changeConfiguration(ConfigurationTarget.Workspace, { contents: {} }, undefined)
-    }
   }
 
   public hasFolderConfiguration(filepath: string): boolean {
@@ -325,7 +320,7 @@ export default class Configurations {
       let folder = findUp('.vim', rootPath)
       if (folder) {
         let file = path.join(folder, CONFIG_FILE_NAME)
-        if (fs.existsSync(file)) {
+        if (!sameFile(file, this.userConfigFile) && fs.existsSync(file)) {
           this.addFolderFile(file)
         }
       }

src/util/fs.ts

@@ -192,7 +192,7 @@ export function readFileLine(fullpath: string, count: number): Promise<string> {
   })
 }
 
-export function sameFile(fullpath: string, other: string): boolean {
+export function sameFile(fullpath: string | null, other: string | null): boolean {
   const caseInsensitive = platform.isWindows || platform.isMacintosh
   if (!fullpath || !other) return false
   if (caseInsensitive) return fullpath.toLowerCase() === other.toLowerCase()