aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Muré <batolettre@gmail.com>2019-12-10 20:13:49 +0100
committerMichael Muré <batolettre@gmail.com>2019-12-10 20:13:49 +0100
commitda6591e4807d6aec5f230e8a1efad1e2ef686608 (patch)
tree51ef192a080fd02a1230fd7cb7e8566710e006d2
parentf1ed857cbd3a253d77b31c0c896fdc4ade40844f (diff)
downloadgit-bug-da6591e4807d6aec5f230e8a1efad1e2ef686608.tar.gz
cmd: "user create" only assign the user identity if not set
-rw-r--r--cache/repo_cache.go4
-rw-r--r--commands/user_create.go9
-rw-r--r--identity/identity.go10
3 files changed, 22 insertions, 1 deletions
diff --git a/cache/repo_cache.go b/cache/repo_cache.go
index ec4cf436..90a489c8 100644
--- a/cache/repo_cache.go
+++ b/cache/repo_cache.go
@@ -877,6 +877,10 @@ func (c *RepoCache) GetUserIdentity() (*IdentityCache, error) {
return cached, nil
}
+func (c *RepoCache) IsUserIdentitySet() (bool, error) {
+ return identity.IsUserIdentitySet(c.repo)
+}
+
// NewIdentity create a new identity
// The new identity is written in the repository (commit)
func (c *RepoCache) NewIdentity(name string, email string) (*IdentityCache, error) {
diff --git a/commands/user_create.go b/commands/user_create.go
index 88cc94de..15b9767e 100644
--- a/commands/user_create.go
+++ b/commands/user_create.go
@@ -53,11 +53,18 @@ func runUserCreate(cmd *cobra.Command, args []string) error {
return err
}
- err = backend.SetUserIdentity(id)
+ set, err := backend.IsUserIdentitySet()
if err != nil {
return err
}
+ if !set {
+ err = backend.SetUserIdentity(id)
+ if err != nil {
+ return err
+ }
+ }
+
_, _ = fmt.Fprintln(os.Stderr)
fmt.Println(id.Id())
diff --git a/identity/identity.go b/identity/identity.go
index ed8e39e0..cd47c1b7 100644
--- a/identity/identity.go
+++ b/identity/identity.go
@@ -261,6 +261,16 @@ func GetUserIdentity(repo repository.Repo) (*Identity, error) {
return i, nil
}
+// IsUserIdentitySet say if the user has set his identity
+func IsUserIdentitySet(repo repository.Repo) (bool, error) {
+ configs, err := repo.LocalConfig().ReadAll(identityConfigKey)
+ if err != nil {
+ return false, err
+ }
+
+ return len(configs) == 1, nil
+}
+
func (i *Identity) AddVersion(version *Version) {
i.versions = append(i.versions, version)
}