Merge pull request hound-search#240 from bschlenk/reponames

Remove lowercasing of repo names

Author: kellegous

api/api.go

@@ -101,10 +101,10 @@ func parseAsBool(v string) bool {
 }
 
 func parseAsRepoList(v string, idx map[string]*searcher.Searcher) []string {
-	v = strings.TrimSpace(strings.ToLower(v))
+	v = strings.TrimSpace(v)
 	var repos []string
 	if v == "*" {
-		for repo, _ := range idx {
+		for repo := range idx {
 			repos = append(repos, repo)
 		}
 		return repos

cmds/houndd/main.go

@@ -79,7 +79,7 @@ func makeTemplateData(cfg *config.Config) (interface{}, error) {
 
 	res := map[string]*config.Repo{}
 	for name, repo := range cfg.Repos {
-		res[strings.ToLower(name)] = repo
+		res[name] = repo
 	}
 
 	b, err := json.Marshal(res)

searcher/searcher.go

@@ -10,7 +10,6 @@ import (
 	"os"
 	"path/filepath"
 	"runtime"
-	"strings"
 	"sync"
 	"time"
 
@@ -267,10 +266,6 @@ func init() {
 // occurred and no other return values are valid. If an error occurs that is specific
 // to a particular searcher, that searcher will not be present in the searcher map and
 // will have an error entry in the error map.
-//
-// NOTE: The keys in the searcher map will be normalized to lower case, but not such
-// transformation will be done on the error map to make it easier to match those errors
-// back to the original repo name.
 func MakeAll(cfg *config.Config) (map[string]*Searcher, map[string]error, error) {
 	errs := map[string]error{}
 	searchers := map[string]*Searcher{}
@@ -290,7 +285,7 @@ func MakeAll(cfg *config.Config) (map[string]*Searcher, map[string]error, error)
 			continue
 		}
 
-		searchers[strings.ToLower(name)] = s
+		searchers[name] = s
 	}
 
 	if err := refs.removeUnclaimed(); err != nil {

ui/assets/js/hound.js

@@ -105,7 +105,6 @@ var Model = {
     var all = this.repos,
         seen = {};
     return repos.filter(function(repo) {
-      repo = repo.toLowerCase();
       var valid = all[repo] && !seen[repo];
       seen[repo] = true;
       return valid;
@@ -131,7 +130,7 @@ var Model = {
       var data = JSON.parse(ModelData),
           repos = {};
       for (var name in data) {
-        repos[name.toLowerCase()] = data[name];
+        repos[name] = data[name];
       }
       this.repos = repos;
       next();