more cleanup

pkg/db/db.go

@@ -52,9 +52,6 @@ func New(dir string) (db *DB, err error) {
 	}
 
 	_, err = db.CreatePage(core.IndexPage, core.WikiSection, welcome, defUser, core.Public)
-	if err != nil && err != store.ErrKeyNotFound {
-		return
-	}
 	return
 }
 

pkg/db/page.go

@@ -13,11 +13,12 @@ import (
 const pagePrefix = "page/"
 
 var (
-	errPageNotFound      = store.ErrKeyNotFound
-	errPermissionDenied  = errors.New("db: permission denied")
-	errInvalidPermission = errors.New("db: invalid permission")
-	errPrivateWikiPage   = errors.New("db: private wiki pages are not allowed")
-	errDeleteIndexPage   = errors.New("db: index page can not be deleted")
+	errPageNotFound       = store.ErrKeyNotFound
+	errPermissionDenied   = errors.New("db: permission denied")
+	errInvalidPermission  = errors.New("db: invalid permission")
+	errPrivateWikiPage    = errors.New("db: private wiki pages are not allowed")
+	errNonPublicIndexPage = errors.New("db: internal/private index page is not allowed")
+	errDeleteIndexPage    = errors.New("db: index page can not be deleted")
 )
 
 func (db *DB) GetAllPages(username string) (pages core.Pages) {
@@ -77,7 +78,9 @@ func (db *DB) CreatePage(title, section, markdown, username string, p core.Permi
 		}
 	}
 	st := render.StoreTitle(section, render.Title(title))
-	if _, err = db.getPage(st, username); err != nil && err != store.ErrKeyNotFound {
+	_, err = db.getPage(st, username)
+	//if err == nil || (err != nil && err != store.ErrKeyNotFound) {
+	if err != store.ErrKeyNotFound {
 		return
 	}
 
@@ -113,6 +116,11 @@ func (db *DB) UpdatePage(title, section, markdown, username string, p core.Permi
 		err = errPrivateWikiPage
 		return
 	}
+	if section == core.WikiSection && title == core.IndexPage &&
+		(p == core.Internal || p == core.Private) {
+		err = errNonPublicIndexPage
+		return
+	}
 	st := render.StoreTitle(section, title)
 	page, err := db.getPage(st, username)
 	if err != nil {

pkg/log/log.go

@@ -13,8 +13,8 @@ var (
 	debug = false
 )
 
-func InitLogger(dir, file string, debug bool) {
-	debug = debug
+func InitLogger(dir, file string, d bool) {
+	debug = d
 	if file == "-" {
 		return
 	}
@@ -40,7 +40,7 @@ func Fatal(v ...interface{}) {
 
 func Debug(v ...interface{}) {
 	if debug {
-		log.Println(append([]interface{}{"debug:"}, v...))
+		log.Println(append([]interface{}{"debug:"}, v...)...)
 	}
 }
 

pkg/store/boltstore.go

@@ -18,22 +18,21 @@ type BoltStore struct {
 	db *bolt.DB
 }
 
-func NewBoltStore(file string, m Marshaler) (bs *BoltStore, err error) {
+func NewBoltStore(file string, m Marshaler) (Store, error) {
 	if m == nil {
 		m = NewGOB()
 	}
 	db, err := bolt.Open(file, fileMode, nil)
 	if err != nil {
-		return
+		return nil, err
 	}
 	if err = db.Update(func(tx *bolt.Tx) (err error) {
 		_, err = tx.CreateBucketIfNotExists([]byte(defaultBoltBucket))
 		return
 	}); err != nil {
-		return
+		return nil, err
 	}
-	bs = &BoltStore{m, db}
-	return
+	return &BoltStore{m, db}, nil
 }
 
 type dump struct {