refactor(alias): Apply config changes to scoop-alias command

libexec/scoop-alias.ps1

@@ -30,85 +30,87 @@ param(
 $script:config_alias = "alias"
 
 function init_alias_config {
-  $aliases = get_config $script:config_alias
-  if(!$aliases) {
-    $aliases = @{}
-  }
+    $aliases = get_config $script:config_alias
+    if(!$aliases) {
+        $aliases = @{}
+    }
 
-  $aliases
+    return $aliases
 }
 
 function add_alias($name, $command) {
-  if(!$command) {
-    abort "Can't create an empty alias."
-  }
+    if(!$command) {
+        abort "Can't create an empty alias."
+    }
 
-  # get current aliases from config
-  $aliases = init_alias_config
-  if($aliases.containskey($name)) {
-    abort "Alias $name already exists."
-  }
+    # get current aliases from config
+    $aliases = init_alias_config
+    if($aliases.$name) {
+        abort "Alias $name already exists."
+    }
 
-  $alias_file = "scoop-$name"
+    $alias_file = "scoop-$name"
 
-  # generate script
-  $shimdir = shimdir $false
-  $script =
+    # generate script
+    $shimdir = shimdir $false
+    $script =
 @"
 # Summary: $description
 $command
 "@
-  $script | out-file "$shimdir\$alias_file.ps1" -encoding utf8
+    $script | out-file "$shimdir\$alias_file.ps1" -encoding utf8
 
-  # add alias to config
-  $aliases += @{ $name = $alias_file }
-  set_config $script:config_alias $aliases
+    # add alias to config
+    $aliases | Add-Member -MemberType NoteProperty -Name $name -Value $alias_file
+
+    set_config $script:config_alias $aliases
 }
 
 function rm_alias($name) {
-  $aliases = init_alias_config
-  if(!$name) {
-    abort "Which alias should be removed?"
-  }
+    $aliases = init_alias_config
+    if(!$name) {
+        abort "Which alias should be removed?"
+    }
 
-  if($aliases.containskey($name)) {
-    "Removing alias $name..."
+    if($aliases.$name) {
+        "Removing alias $name..."
 
-    rm_shim $aliases.get_item($name) (shimdir $false)
+        rm_shim $aliases.$name (shimdir $false)
 
-    $aliases.remove($name)
-    set_config $script:config_alias $aliases
-  }
-  else { abort "Alias $name doesn't exist." }
+        $aliases.PSObject.Properties.Remove($name)
+        set_config $script:config_alias $aliases
+    } else {
+        abort "Alias $name doesn't exist."
+    }
 }
 
 function list_aliases {
-  $aliases = @()
-
-  (init_alias_config).GetEnumerator() | ForEach-Object {
-    $content = Get-Content (command_path $_.name)
-    $command = ($content | Select-Object -Skip 1).Trim()
-    $summary = (summary $content).Trim()
-
-    $aliases += New-Object psobject -Property @{Name=$_.name; Summary=$summary; Command=$command}
-  }
-
-  if(!$aliases.count) {
-    warn "No aliases founds."
-  }
-  $aliases = $aliases.GetEnumerator() | Sort-Object Name
-  if($verbose) {
-    return $aliases | Select-Object Name, Command, Summary | Format-Table -autosize -wrap
-  } else {
-    return $aliases | Select-Object Name, Command | Format-Table -autosize -hidetablehead -wrap
-  }
+    $aliases = @()
+
+    (init_alias_config).PSObject.Properties.GetEnumerator() | ForEach-Object {
+        $content = Get-Content (command_path $_.Name)
+        $command = ($content | Select-Object -Skip 1).Trim()
+        $summary = (summary $content).Trim()
+
+        $aliases += New-Object psobject -Property @{Name=$_.name; Summary=$summary; Command=$command}
+    }
+
+    if(!$aliases.count) {
+        warn "No aliases founds."
+    }
+    $aliases = $aliases.GetEnumerator() | Sort-Object Name
+    if($verbose) {
+        return $aliases | Select-Object Name, Command, Summary | Format-Table -autosize -wrap
+    } else {
+        return $aliases | Select-Object Name, Command | Format-Table -autosize -hidetablehead -wrap
+    }
 }
 
 switch($opt) {
-  "add" { add_alias $name $command }
-  "rm" { rm_alias $name }
-  "list" { list_aliases }
-  default { my_usage; exit 1 }
+    "add" { add_alias $name $command }
+    "rm" { rm_alias $name }
+    "list" { list_aliases }
+    default { my_usage; exit 1 }
 }
 
 exit 0