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

UserForms EditableFormField Column Clean task error #1332

Closed
2 tasks done
elkebe opened this issue Oct 2, 2024 · 3 comments
Closed
2 tasks done

UserForms EditableFormField Column Clean task error #1332

elkebe opened this issue Oct 2, 2024 · 3 comments

Comments

@elkebe
Copy link

elkebe commented Oct 2, 2024

Module version(s) affected

6.2.9

Description

Affects SS5

When you try to run or queue the job, you get the following error:

Uncaught Exception SilverStripe\ORM\Connect\DatabaseException: "Couldn't run query: SHOW COLUMNS FROM SilverStripe\UserForms\Model\EditableFormField 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 '\UserForms\Model\EditableFormField' at line 1" at /var/www/html/vendor/silverstripe/framework/src/ORM/Connect/DBConnector.php line 63 {"exception":"[object] (SilverStripe\\ORM\\Connect\\DatabaseException(code: 0): Couldn't run query:\n\nSHOW COLUMNS\nFROM SilverStripe\\UserForms\\Model\\EditableFormField\n\nYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '\\UserForms\\Model\\EditableFormField' at line 1 at /var/www/html/vendor/silverstripe/framework/src/ORM/Connect/DBConnector.php:63)"} []

How to reproduce

go to yoursite/dev/tasks
run UserForms EditableFormField Column Clean task

Possible Solution

No response

Additional Context

No response

Validations

  • Check that there isn't already an issue that reports the same bug
  • Double check that your reproduction steps work in a fresh installation of silverstripe/installer (with any code examples you've provided)

PRs

@GuySartorelli
Copy link
Member

Thank you for reporting this, I've submitted a pull request to fix it.

How did you come across this? What was your purpose for running the task? It looks like it was created to support migrating from a very old version of the module (see #119 and #129) so it seems like it shouldn't be needed anymore?

@emteknetnz
Copy link
Member

Linked PR has been merged as released as 6.2.10

@elkebe
Copy link
Author

elkebe commented Oct 17, 2024

@GuySartorelli I was going through all the available tasks in /dev/tasks while testing a pre-release website and came across it... was wondering what it was meant to do :)

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

No branches or pull requests

3 participants