Code improvement
This commit is contained in:
parent
a6e4092b3b
commit
b9a0be78f7
|
@ -99,10 +99,11 @@ func (session *Session) find(rowsSlicePtr interface{}, condiBean ...interface{})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
var table = session.statement.RefTable
|
var (
|
||||||
|
table = session.statement.RefTable
|
||||||
var addedTableName = (len(session.statement.JoinStr) > 0)
|
addedTableName = (len(session.statement.JoinStr) > 0)
|
||||||
var autoCond builder.Cond
|
autoCond builder.Cond
|
||||||
|
)
|
||||||
if tp == tpStruct {
|
if tp == tpStruct {
|
||||||
if !session.statement.NoAutoCondition && len(condiBean) > 0 {
|
if !session.statement.NoAutoCondition && len(condiBean) > 0 {
|
||||||
var err error
|
var err error
|
||||||
|
@ -111,23 +112,13 @@ func (session *Session) find(rowsSlicePtr interface{}, condiBean ...interface{})
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// !oinume! Add "<col> IS NULL" to WHERE whatever condiBean is given.
|
|
||||||
// See https://gitea.com/xorm/xorm/issues/179
|
|
||||||
if col := table.DeletedColumn(); col != nil && !session.statement.GetUnscoped() { // tag "deleted" is enabled
|
if col := table.DeletedColumn(); col != nil && !session.statement.GetUnscoped() { // tag "deleted" is enabled
|
||||||
var colName = session.engine.Quote(col.Name)
|
|
||||||
if addedTableName {
|
|
||||||
var nm = session.statement.TableName()
|
|
||||||
if len(session.statement.TableAlias) > 0 {
|
|
||||||
nm = session.statement.TableAlias
|
|
||||||
}
|
|
||||||
colName = session.engine.Quote(nm) + "." + colName
|
|
||||||
}
|
|
||||||
|
|
||||||
autoCond = session.statement.CondDeleted(col)
|
autoCond = session.statement.CondDeleted(col)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// if it's a map with Cols but primary key not in column list, we still need the primary key
|
||||||
if isMap && !session.statement.ColumnMap.IsEmpty() {
|
if isMap && !session.statement.ColumnMap.IsEmpty() {
|
||||||
for _, k := range session.statement.RefTable.PrimaryKeys {
|
for _, k := range session.statement.RefTable.PrimaryKeys {
|
||||||
session.statement.ColumnMap.Add(k)
|
session.statement.ColumnMap.Add(k)
|
||||||
|
|
Loading…
Reference in New Issue