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

Attempting to edit a pool results in "Pool not found." #8

Open
rscullin opened this issue May 14, 2011 · 6 comments
Open

Attempting to edit a pool results in "Pool not found." #8

rscullin opened this issue May 14, 2011 · 6 comments
Assignees

Comments

@rscullin
Copy link

Attempting to edit a pool in the latest git commit ( 81b3ce1 ) results in the error "Pool not found."

This seems to be because of a missing GROUP BY clause in the SQL query starting on line 96 in pool.inc.php, which throws a MySQL error. Adding GROUP BY wp.worker_id after the WHERE clause appears to fix this.

@ghost ghost assigned cdhowie May 14, 2011
@wyze
Copy link
Contributor

wyze commented Jun 23, 2011

I'm wondering if this is still an issue as Ii am not able to duplicate it and can edit pools fine on my end?

@cdhowie
Copy link
Owner

cdhowie commented Jun 23, 2011

@wyze, this issue is specific to some versions of MySQL and not others. Newer versions seem to infer the GROUP BY clause, while older versions require it explicitly.

@edgeman16
Copy link

Doesn't seem to happen to me on MySQL 5.1.56. FYI.

@wyze
Copy link
Contributor

wyze commented Jun 23, 2011

Why not just add this in there then? It won't hurt newer versions of MySQL and will fix older versions. Wouldn't take a performance hit and a very small and simple change to the code.

@rscullin
Copy link
Author

The version of MySQL that's giving me issues is:
mysql Ver 14.12 Distrib 5.0.51a, for debian-linux-gnu (i486) using readline 5.2

@cdhowie
Copy link
Owner

cdhowie commented Jun 28, 2011

I have pushed 83d0672, which might resolve this issue. Can those of you who have had trouble with this query pull down the latest and try again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants