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

Drag and drop in Resource Tree leads to Mysql error #281

Closed
q2apro opened this Issue Oct 7, 2017 · 6 comments

Comments

Projects
None yet
3 participants
@q2apro

q2apro commented Oct 7, 2017

Just used the mouse to drag one resource into a container resource and got an alert that "something went wrong". In the system events I found the following error:

« MODX Parse Error »
Execution of a query to the database failed - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1
SQL > UPDATE mytable.modx_site_content SET menuindex=0 WHERE id =

Probably it is necessary to check if an ID is set, and if not just return?

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Oct 7, 2017

Member

what version of evo and which browser ?

Member

64j commented Oct 7, 2017

what version of evo and which browser ?

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Oct 7, 2017

Evo 1.3.6 and Chrome 60.0.3112.113 (Official Build) (64-bit)

Note: Maybe there was an ajax request during the time I was dragging the resources. Not sure.

q2apro commented Oct 7, 2017

Evo 1.3.6 and Chrome 60.0.3112.113 (Official Build) (64-bit)

Note: Maybe there was an ajax request during the time I was dragging the resources. Not sure.

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Oct 7, 2017

Member

Is the error every time, or occasionally?

Member

64j commented Oct 7, 2017

Is the error every time, or occasionally?

@q2apro

This comment has been minimized.

Show comment
Hide comment
@q2apro

q2apro Oct 7, 2017

It was only occasionally.

q2apro commented Oct 7, 2017

It was only occasionally.

@64j

This comment has been minimized.

Show comment
Hide comment
@64j

64j Oct 7, 2017

Member

very strange, because all checks work

Member

64j commented Oct 7, 2017

very strange, because all checks work

@dmi3yy

This comment has been minimized.

Show comment
Hide comment
@dmi3yy

dmi3yy Jan 10, 2018

tested all ok

dmi3yy commented Jan 10, 2018

tested all ok

@dmi3yy dmi3yy closed this Jan 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment