aclindsa/moneygo
1
0
Fork 0
mirror of https://github.com/aclindsa/moneygo.git synced 2025-11-03 18:13:27 -05:00
Code Issues Releases Wiki Activity

Don't leak non-interface type from database module

Browse Source 
This removes the last place I used *db.DbStore instead of store.Store
outside the 'db' package.
This commit is contained in:
Aaron Lindsay
2017-12-10 21:02:17 -05:00
parent d5bea1102d
commit 483adb5c56
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
internal/handlers/handlers.go
View File

@@ -3,7 +3,6 @@ package handlers
import (
"github.com/aclindsa/moneygo/internal/models"
"github.com/aclindsa/moneygo/internal/store"
"github.com/aclindsa/moneygo/internal/store/db"
"log"
"net/http"
"path"
@@ -47,7 +46,7 @@ func (c *Context) LastLevel() bool {
type Handler func(*http.Request, *Context) ResponseWriterWriter
type APIHandler struct {
Store *db.DbStore
Store store.Store
}
func (ah *APIHandler) txWrapper(h Handler, r *http.Request, context *Context) (writer ResponseWriterWriter) {

2
internal/store/db/db.go
View File

@@ -84,7 +84,7 @@ func (db *DbStore) Close() error {
return err
}
func GetStore(dbtype config.DbType, dsn string) (store *DbStore, err error) {
func GetStore(dbtype config.DbType, dsn string) (store store.Store, err error) {
dsn = getDSN(dbtype, dsn)
database, err := sql.Open(dbtype.String(), dsn)
if err != nil {