+
Skip to content

Conversation

cesumilo
Copy link
Contributor

Proposal

I figured out a way to fix columnNameMappers applied to json fields to prevent remapping inner keys. This PR fixes #1089 by lazy loading the columns information of the current Model to be able to identify json columns in a later query.

Important: this introduces an additional query on the Model's first query. I believe it won't affect any users and rather improve the library consistency.

While discussing this PR, I'll add more tests to cover as much cases as possible.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

snakeCaseMappers works differently for objects and FieldExpressions

1 participant

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载