Revert check for incomplete parentheses

[emirb]

Make sure these boxes checked before submitting your pull request.

• Do only one thing
• No API-breaking changes
• New code/logic commented & tested

For significant changes like big bug fixes, new features, please open an issue to make an agreement on an implementation design/plan first before starting it.

What did this pull request do?

This reverts commit e3cc5ea from https://github.com/jinzhu/gorm/pull/2519 due to breaking of valid queries.

[codecov-io]

Codecov Report

Merging #2674 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2674      +/-   ##
==========================================
- Coverage   79.57%   79.49%   -0.09%     
==========================================
  Files          24       24              
  Lines        3472     3458      -14     
==========================================
- Hits         2763     2749      -14     
  Misses        609      609              
  Partials      100      100

Impacted Files	Coverage Δ
scope.go	87% <ø> (-0.22%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c98e7d...17debe1. Read the comment docs.

[depado]

Related issue: #2561

[jinzhu]

Thank you, sorry for the issue.

[Jonathan34]

@jinzhu are you going to provide a version with a fix for this CVE https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-15562 ?
I can revert to 1.9.10 but i see some interesting bugs fixed in .11 and i might be impacted by the reason this was reverted...
Thanks!

[jinzhu]

Hi @Jonathan34

It's hard to make this one correct, suggest you don't accept user's input as the select params.

[mathieupost]

@Jonathan34
If you replace db.First(&m, id) by db.First(&m, "id = ?", id) you're safe.