diff --git a/object/cert.go b/object/cert.go index 2852404d..4b7b001c 100644 --- a/object/cert.go +++ b/object/cert.go @@ -162,7 +162,7 @@ func UpdateCert(id string, cert *Cert) (bool, error) { if name != cert.Name { err := certChangeTrigger(name, cert.Name) if err != nil { - return false, nil + return false, err } } affected, err := ormer.Engine.ID(core.PK{owner, name}).AllCols().Update(cert) diff --git a/object/organization.go b/object/organization.go index dd3ec14e..d5ad35ac 100644 --- a/object/organization.go +++ b/object/organization.go @@ -189,7 +189,7 @@ func UpdateOrganization(id string, organization *Organization) (bool, error) { if name != organization.Name { err := organizationChangeTrigger(name, organization.Name) if err != nil { - return false, nil + return false, err } } diff --git a/object/provider.go b/object/provider.go index 28650a46..9bc57066 100644 --- a/object/provider.go +++ b/object/provider.go @@ -203,7 +203,7 @@ func UpdateProvider(id string, provider *Provider) (bool, error) { if name != provider.Name { err := providerChangeTrigger(name, provider.Name) if err != nil { - return false, nil + return false, err } } diff --git a/object/role.go b/object/role.go index a1d560e7..9dccbcb2 100644 --- a/object/role.go +++ b/object/role.go @@ -133,7 +133,7 @@ func UpdateRole(id string, role *Role) (bool, error) { if name != role.Name { err := roleChangeTrigger(name, role.Name) if err != nil { - return false, nil + return false, err } } diff --git a/object/user.go b/object/user.go index ab5adb79..c4f09cef 100644 --- a/object/user.go +++ b/object/user.go @@ -568,7 +568,7 @@ func UpdateUserForAllFields(id string, user *User) (bool, error) { if name != user.Name { err := userChangeTrigger(name, user.Name) if err != nil { - return false, nil + return false, err } }