[think-model-sqlite] Default values surrounded by single quotes about thinkjs HOT 3 CLOSED

Thank you for your feedback, I will try to reproduce and confirm this problem in the future. The solution you provided is great. If possible, you can submit a PR according to this idea. After I confirm the problem, I can merge it directly.

from thinkjs.

Many thanks.

I realize that a simpler fix for lib/schema.js (line 75) is to ignore item.dflt_value and set item.default to the empty string '', as done in think-model-mysql and think-model-postgresql.

It results in letting the native SQLite driver set the default value, which might be a more desirable behavior, and avoids parsing the SQLite syntax.

I plan to submit a PR with this solution.

from thinkjs.

It has been confirmed locally that there is indeed this problem, thank you very much.

Your issue and problem-solving ideas are very clear and correct, and your understanding of ThinkJS is also in place

[email protected] released, thanks for your feedback and fix commits.

from thinkjs.