Skip to content

Fix error 500 on organization dashboard page #147

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

Closed
wants to merge 1 commit into from
Closed

Fix error 500 on organization dashboard page #147

wants to merge 1 commit into from

Conversation

thibaultmeyer
Copy link
Contributor

This pul request fix an error 500 on organization dashboard page. When an organization don't contains any repositories, the count SQL query return nothing.

@thibaultmeyer thibaultmeyer added this to the 1.0.0 milestone Nov 11, 2016
@lunny
Copy link
Member

lunny commented Nov 11, 2016

I will send a PR to fix that.

@lunny
Copy link
Member

lunny commented Nov 11, 2016

is this fixed the same as #149?

@lunny
Copy link
Member

lunny commented Nov 11, 2016

It's not the same.

@lunny
Copy link
Member

lunny commented Nov 11, 2016

This should be a bug of xorm. I will fix it and we should not change gitea's code.

@thibaultmeyer
Copy link
Contributor Author

thibaultmeyer commented Nov 11, 2016

@lunny if you can change the behavior of count() on XORM to return 0 when zero rows has been returned by the query. It could be nice yeah.

@lunny
Copy link
Member

lunny commented Nov 11, 2016

Yes. I have. #150 changed like you said.

@thibaultmeyer
Copy link
Contributor Author

Nice, you got my LGTM, just need one more to merge

@lunny
Copy link
Member

lunny commented Nov 11, 2016

Yes. So you can close this PR.

@tboerger tboerger removed this from the 1.0.0 milestone Nov 12, 2016
@thibaultmeyer thibaultmeyer deleted the fix/err500-org-dashboard branch November 12, 2016 10:58
ethantkoenig pushed a commit to ethantkoenig/gitea that referenced this pull request Feb 15, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
@delvh delvh added issue/not-a-bug The reported issue is the intended behavior or the problem is not inside Gitea and removed reviewed/invalid labels Oct 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/not-a-bug The reported issue is the intended behavior or the problem is not inside Gitea
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants