Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] multiple-value uuid.NewV4() in single-value context #93

Closed
vborisov opened this issue Nov 6, 2019 · 3 comments
Closed

[BUG] multiple-value uuid.NewV4() in single-value context #93

vborisov opened this issue Nov 6, 2019 · 3 comments
Labels
🐛bug Something isn't working

Comments

@vborisov
Copy link

vborisov commented Nov 6, 2019

Bug Description ../github.com/GoAdminGroup/go-admin/plugins/admin/modules/helper.go:42:19: multiple-value uuid.NewV4() in single-value context

How to reproduction Just use the latest Go distribution with the latest GoAdmin lib.

Expect The project to start.

Versions

  • GoAdmin version: 1.0.6
  • go version go1.13.3 linux/amd64
  • OS Ubuntu 16.04.6 LTS

Others
An easy fix will be changing the Uuid function to:
`

func Uuid() string {
u, _ := uuid.NewV4()
return u.String()
// return uuid.NewV4().String()
}`

@vborisov vborisov added the 🐛bug Something isn't working label Nov 6, 2019
@chenhg5
Copy link
Collaborator

chenhg5 commented Nov 6, 2019

@vborisov In the latest version of go.uuid, the api NewV4() has only one return value.

@vborisov
Copy link
Author

vborisov commented Nov 6, 2019

It's very strange as I'm with the same version of go.uuid and the latest of Golang it generates an error.


func NewV4() (UUID, error)

I'm seeing that there are two values returned - UUID and error.

@kotleta
Copy link

kotleta commented Aug 5, 2022

Hello!

I have the same problem.

go1.16.9 linux/amd64
github.com/GoAdminGroup/go-admin v1.2.23
github.com/satori/go.uuid v1.2.0

go build -mod vendor my-admin
# github.com/GoAdminGroup/go-admin/plugins/admin/modules
vendor/github.com/GoAdminGroup/go-admin/plugins/admin/modules/helper.go:50:19: multiple-value uuid.NewV4() in single-value context
make: *** [my-admin] Error 2

Could you reopen the issue, please?

This was referenced Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants