This repository was archived by the owner on Jan 4, 2020. It is now read-only.
修复setInc setDec无法处理与关键字重名的字段的bug#439
Open
wangxiyu191 wants to merge 1 commit intotop-think:masterfrom
Open
Conversation
setfield时给field加上``
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
setfield时给field加上``
由于tp对set参数的处理很粗暴,等号后面是直接拼接传入的字符串的,只能从这里修复一下了。
不清楚是不是有数据库不支持真这种写法，不过从update中调用的prasekey的写法来看，似乎问题不大。