lessql icon indicating copy to clipboard operation
lessql copied to clipboard

Cannot have a table name with 'List' at the end or this line removes it from the table name

Open jgerk opened this issue 8 years ago • 2 comments

https://github.com/morris/lessql/blob/338966185fc052e7ee769360d19950cf90c0fd42/src/LessQL/Database.php#L54

jgerk avatar Mar 28 '18 16:03 jgerk

How many times would it happen that you need a table name ending with 'List'?

A solution would be to make this configurable:

$name = preg_replace('/'.preg_quote($this->list_suffix, '/') .'$/', '', $name);

Langmans avatar Dec 03 '18 20:12 Langmans

Shouldn't "_list" suffix or "list" (lower-case) work? SQL is case insensitive. @rubenvincenten that could work, however think this is a very niche edge case...

morris avatar May 03 '19 10:05 morris