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.

Sign up for GitHub

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

Jump to bottom

not using quoted_table_name causes MySQL syntax error with SQL reserved words as table names #1

Closed
joeykid6 opened this issue May 1, 2009 · 1 comment
Closed

not using quoted_table_name causes MySQL syntax error with SQL reserved words as table names #1

joeykid6 opened this issue May 1, 2009 · 1 comment

Comments

@joeykid6
Copy link

joeykid6 commented May 1, 2009

Use of self.class.table_name in awesome_nested_set.rb lines:
292, 313, 325, 364, 365, 370, 420 (only tested with line 292, but I assume it's the same in all instances)

Throws MySQL syntax error 1064 when SQL reserved words are used as table names.

I believe it should be replaced with quoted_table_name to add backticks to SQL table names.
@bkeepers
Copy link
Contributor

bkeepers commented May 1, 2009

Properly quote table names. Closed by 4617c90

matenia pushed a commit to matenia/awesome_nested_set that referenced this issue Jul 24, 2011
Merged pull request collectiveidea#1 from mguymon/master.
beaaaa7 
Support for saving without validation on rebuild!
parndt pushed a commit that referenced this issue Jan 11, 2022
@peterberkenbosch @andrewhampton
Add support for Rails 7 (#460)
0272e84 
* Add support for Rails 7
* Add Rails 7 to github action CI flow
* Update dependency to rake ~> 12
* Fix specs and rework database loading (#1)

The older version of combustion used by awesome_nested_set doesn't
appear to be compatible with rails 7, and the latest version of
combustion no longer has an API that lets you pass in a config.

So this commit drops the dependency on combustion and just uses the
rails task for database setup and teardown.

Co-authored-by: Andrew Hampton <andrewwh@gmail.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bkeepers @joeykid6