diff options
-rw-r--r-- | models/log.go (renamed from modules/log/xorm.go) | 52 | ||||
-rw-r--r-- | models/models.go | 5 | ||||
-rw-r--r-- | modules/setting/log.go | 3 |
3 files changed, 25 insertions, 35 deletions
diff --git a/modules/log/xorm.go b/models/log.go index c08b7fd7d5..4994545c5f 100644 --- a/modules/log/xorm.go +++ b/models/log.go @@ -2,11 +2,13 @@ // Use of this source code is governed by a MIT-style // license that can be found in the LICENSE file. -package log +package models import ( "fmt" + "code.gitea.io/gitea/modules/log" + "github.com/go-xorm/core" ) @@ -14,80 +16,72 @@ import ( type XORMLogBridge struct { showSQL bool level core.LogLevel + logger *log.Logger } -var ( - // XORMLogger the logger for xorm - XORMLogger *XORMLogBridge -) - -// InitXORMLogger inits a log bridge for xorm -func InitXORMLogger(showSQL bool) { - XORMLogger = &XORMLogBridge{ +// NewXORMLogger inits a log bridge for xorm +func NewXORMLogger(showSQL bool) core.ILogger { + return &XORMLogBridge{ showSQL: showSQL, + logger: log.GetLogger("xorm"), } } -// GetGiteaLevel returns the minimum Gitea logger level -func (l *XORMLogBridge) GetGiteaLevel() Level { - return GetLogger("xorm").GetLevel() -} - // Log a message with defined skip and at logging level -func (l *XORMLogBridge) Log(skip int, level Level, format string, v ...interface{}) error { - return GetLogger("xorm").Log(skip+1, level, format, v...) +func (l *XORMLogBridge) Log(skip int, level log.Level, format string, v ...interface{}) error { + return l.logger.Log(skip+1, level, format, v...) } // Debug show debug log func (l *XORMLogBridge) Debug(v ...interface{}) { - l.Log(2, DEBUG, fmt.Sprint(v...)) + l.Log(2, log.DEBUG, fmt.Sprint(v...)) } // Debugf show debug log func (l *XORMLogBridge) Debugf(format string, v ...interface{}) { - l.Log(2, DEBUG, format, v...) + l.Log(2, log.DEBUG, format, v...) } // Error show error log func (l *XORMLogBridge) Error(v ...interface{}) { - l.Log(2, ERROR, fmt.Sprint(v...)) + l.Log(2, log.ERROR, fmt.Sprint(v...)) } // Errorf show error log func (l *XORMLogBridge) Errorf(format string, v ...interface{}) { - l.Log(2, ERROR, format, v...) + l.Log(2, log.ERROR, format, v...) } // Info show information level log func (l *XORMLogBridge) Info(v ...interface{}) { - l.Log(2, INFO, fmt.Sprint(v...)) + l.Log(2, log.INFO, fmt.Sprint(v...)) } // Infof show information level log func (l *XORMLogBridge) Infof(format string, v ...interface{}) { - l.Log(2, INFO, format, v...) + l.Log(2, log.INFO, format, v...) } // Warn show warning log func (l *XORMLogBridge) Warn(v ...interface{}) { - l.Log(2, WARN, fmt.Sprint(v...)) + l.Log(2, log.WARN, fmt.Sprint(v...)) } // Warnf show warnning log func (l *XORMLogBridge) Warnf(format string, v ...interface{}) { - l.Log(2, WARN, format, v...) + l.Log(2, log.WARN, format, v...) } // Level get logger level func (l *XORMLogBridge) Level() core.LogLevel { - switch l.GetGiteaLevel() { - case TRACE, DEBUG: + switch l.logger.GetLevel() { + case log.TRACE, log.DEBUG: return core.LOG_DEBUG - case INFO: + case log.INFO: return core.LOG_INFO - case WARN: + case log.WARN: return core.LOG_WARNING - case ERROR, CRITICAL: + case log.ERROR, log.CRITICAL: return core.LOG_ERR } return core.LOG_OFF diff --git a/models/models.go b/models/models.go index 352c07e0c3..c7e58737ed 100644 --- a/models/models.go +++ b/models/models.go @@ -15,7 +15,6 @@ import ( "path/filepath" "strings" - "code.gitea.io/gitea/modules/log" "code.gitea.io/gitea/modules/setting" // Needed for the MySQL driver @@ -261,7 +260,7 @@ func NewTestEngine(x *xorm.Engine) (err error) { } x.SetMapper(core.GonicMapper{}) - x.SetLogger(log.XORMLogger) + x.SetLogger(NewXORMLogger(!setting.ProdMode)) x.ShowSQL(!setting.ProdMode) return x.StoreEngine("InnoDB").Sync2(tables...) } @@ -276,7 +275,7 @@ func SetEngine() (err error) { x.SetMapper(core.GonicMapper{}) // WARNING: for serv command, MUST remove the output to os.stdout, // so use log file to instead print to stdout. - x.SetLogger(log.XORMLogger) + x.SetLogger(NewXORMLogger(setting.LogSQL)) x.ShowSQL(setting.LogSQL) return nil } diff --git a/modules/setting/log.go b/modules/setting/log.go index cd2c5cc8c1..43a28309d2 100644 --- a/modules/setting/log.go +++ b/modules/setting/log.go @@ -295,8 +295,5 @@ func NewXORMLogService(disableConsole bool) { Cfg.Section("log").Key("XORM").MustString(",") generateNamedLogger("xorm", options) - log.InitXORMLogger(LogSQL) - } else { - log.InitXORMLogger(false) } } |