Skip to content

Fix error 500 on organization dashboard page#147

Closed
thibaultmeyer wants to merge 1 commit intogo-gitea:masterfrom
thibaultmeyer:fix/err500-org-dashboard
Closed

Fix error 500 on organization dashboard page#147
thibaultmeyer wants to merge 1 commit intogo-gitea:masterfrom
thibaultmeyer:fix/err500-org-dashboard

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