Merge branch 'master' of github.com:go-xorm/xorm
This commit is contained in:
commit
90a3d5e271
|
@ -9,6 +9,15 @@ conventions when submitting patches.
|
||||||
* [fork a repo](https://help.github.com/articles/fork-a-repo)
|
* [fork a repo](https://help.github.com/articles/fork-a-repo)
|
||||||
* [creating a pull request ](https://help.github.com/articles/creating-a-pull-request)
|
* [creating a pull request ](https://help.github.com/articles/creating-a-pull-request)
|
||||||
|
|
||||||
|
### Sign your codes with comments
|
||||||
|
```
|
||||||
|
// !<you github id>! your comments
|
||||||
|
|
||||||
|
e.g.,
|
||||||
|
|
||||||
|
// !lunny! this is comments made by lunny
|
||||||
|
```
|
||||||
|
|
||||||
### Patch review
|
### Patch review
|
||||||
|
|
||||||
Help review existing open [pull requests](https://help.github.com/articles/using-pull-requests) by commenting on the code or
|
Help review existing open [pull requests](https://help.github.com/articles/using-pull-requests) by commenting on the code or
|
||||||
|
@ -17,7 +26,7 @@ proposed functionality.
|
||||||
### Bug reports
|
### Bug reports
|
||||||
|
|
||||||
We appreciate any bug reports, but especially ones with self-contained
|
We appreciate any bug reports, but especially ones with self-contained
|
||||||
(doesn't depend on code outside of pq), minimal (can't be simplified
|
(doesn't depend on code outside of xorm), minimal (can't be simplified
|
||||||
further) test cases. It's especially helpful if you can submit a pull
|
further) test cases. It's especially helpful if you can submit a pull
|
||||||
request with just the failing test case (you'll probably want to
|
request with just the failing test case (you'll probably want to
|
||||||
pattern it after the tests in
|
pattern it after the tests in
|
||||||
|
|
|
@ -960,7 +960,8 @@ func (statement *Statement) genGetSql(bean interface{}) (string, []interface{})
|
||||||
if columnStr == "" {
|
if columnStr == "" {
|
||||||
columnStr = statement.genColumnStr()
|
columnStr = statement.genColumnStr()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
statement.attachInSql() // !admpub! fix bug:Iterate func missing "... IN (...)"
|
||||||
return statement.genSelectSql(columnStr), append(statement.Params, statement.BeanArgs...)
|
return statement.genSelectSql(columnStr), append(statement.Params, statement.BeanArgs...)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue