feat: allow per-user settings and store them in database

internal/database/database.go

@@ -39,8 +39,16 @@ type GetBookmarksOptions struct {
 
 // GetAccountsOptions is options for fetching accounts from database.
 type GetAccountsOptions struct {
-	Keyword string
-	Owner   bool
+	Keyword       string
+	Owner         bool
+	Showid        bool
+	Listmode      bool
+	Hidethumbnail bool
+	Hideexcerpt   bool
+	Nightmode     bool
+	Keepmetadata  bool
+	Usearchive    bool
+	Makepublic    bool
 }
 
 // DB is interface for accessing and manipulating data in database.
@@ -66,6 +74,9 @@ type DB interface {
 	// SaveAccount saves new account in database
 	SaveAccount(ctx context.Context, a model.Account) error
 
+	// SaveSettings saves settings for specific user in database
+	SaveSettings(ctx context.Context, a model.Account) error
+
 	// GetAccounts fetch list of account (without its password) with matching keyword.
 	GetAccounts(ctx context.Context, opts GetAccountsOptions) ([]model.Account, error)
 

internal/database/migrations/sqlite/0001_initial.up.sql

@@ -3,6 +3,7 @@ CREATE TABLE IF NOT EXISTS account(
     username TEXT NOT NULL,
     password TEXT NOT NULL,
     owner INTEGER NOT NULL DEFAULT 0,
+    configures TEXT NOT NULL,
     CONSTRAINT account_PK PRIMARY KEY(id),
     CONSTRAINT account_username_UNIQUE UNIQUE(username)
 );

internal/database/sqlite.go

@@ -649,11 +649,28 @@ func (db *SQLiteDatabase) SaveAccount(ctx context.Context, account model.Account
 
 		// Insert account to database
 		_, err = tx.Exec(`INSERT INTO account
-		(username, password, owner) VALUES (?, ?, ?)
+		(username, password, owner, configures) VALUES (?, ?, ?, ?)
 		ON CONFLICT(username) DO UPDATE SET
 		password = ?, owner = ?`,
-			account.Username, hashedPassword, account.Owner,
-			hashedPassword, account.Owner)
+			account.Username, hashedPassword, account.Owner, account.Configures,
+			hashedPassword, account.Owner, account.Configures)
+		return errors.WithStack(err)
+	}); err != nil {
+		return errors.WithStack(err)
+	}
+
+	return nil
+}
+
+// SaveSettings update settings for specific account  in database. Returns error if any happened.
+func (db *SQLiteDatabase) SaveSettings(ctx context.Context, account model.Account) error {
+	if err := db.withTx(ctx, func(tx *sqlx.Tx) error {
+		// TODO: be sure configures format is correct before change that in database
+		// Update account configures in database for specific user
+		_, err := tx.Exec(`UPDATE account
+	   SET configures = ?
+	   WHERE username = ?`,
+			account.Configures, account.Username)
 		return errors.WithStack(err)
 	}); err != nil {
 		return errors.WithStack(err)
@@ -666,7 +683,7 @@ func (db *SQLiteDatabase) SaveAccount(ctx context.Context, account model.Account
 func (db *SQLiteDatabase) GetAccounts(ctx context.Context, opts GetAccountsOptions) ([]model.Account, error) {
 	// Create query
 	args := []interface{}{}
-	query := `SELECT id, username, owner FROM account WHERE 1`
+	query := `SELECT id, username, owner, configures FROM account WHERE 1`
 
 	if opts.Keyword != "" {
 		query += " AND username LIKE ?"
@@ -694,7 +711,7 @@ func (db *SQLiteDatabase) GetAccounts(ctx context.Context, opts GetAccountsOptio
 func (db *SQLiteDatabase) GetAccount(ctx context.Context, username string) (model.Account, bool, error) {
 	account := model.Account{}
 	if err := db.GetContext(ctx, &account, `SELECT
-		id, username, password, owner FROM account WHERE username = ?`,
+		id, username, password, owner, configures FROM account WHERE username = ?`,
 		username,
 	); err != nil {
 		return account, false, errors.WithStack(err)

internal/model/model.go

@@ -30,8 +30,9 @@ type Bookmark struct {
 
 // Account is person that allowed to access web interface.
 type Account struct {
-	ID       int    `db:"id"       json:"id"`
-	Username string `db:"username" json:"username"`
-	Password string `db:"password" json:"password,omitempty"`
-	Owner    bool   `db:"owner"    json:"owner"`
+	ID         int    `db:"id"                   json:"id"`
+	Username   string `db:"username"             json:"username"`
+	Password   string `db:"password"             json:"password,omitempty"`
+	Owner      bool   `db:"owner"                json:"owner"`
+	Configures string `db:"configures"           json:"configures"`
 }

internal/view/index.html

@@ -92,6 +92,7 @@
 								return response;
 							}).then(() => {
 								localStorage.removeItem("shiori-account");
+								localStorage.removeItem("shiori-setting");
 								document.cookie = `session-id=; Path=${new URL(document.baseURI).pathname}; Expires=Thu, 01 Jan 1970 00:00:00 GMT;`;
 								location.href = new URL("login", document.baseURI);
 							}).catch(err => {
@@ -173,4 +174,4 @@
 	</script>
 </body>
 
-</html>
+</html>

internal/view/js/page/setting.js

@@ -85,16 +85,40 @@ export default {
 		saveSetting() {
 			this.$emit("setting-changed", {
 				showId: this.appOptions.showId,
-				listMode: this.appOptions.listMode,
-				hideThumbnail: this.appOptions.hideThumbnail,
-				hideExcerpt: this.appOptions.hideExcerpt,
-				nightMode: this.appOptions.nightMode,
-				keepMetadata: this.appOptions.keepMetadata,
-				useArchive: this.appOptions.useArchive,
-				makePublic: this.appOptions.makePublic,
-			});
-		},
-		loadAccounts() {
+                listMode: this.appOptions.listMode,
+                hideThumbnail: this.appOptions.hideThumbnail,
+                hideExcerpt: this.appOptions.hideExcerpt,
+                nightMode: this.appOptions.nightMode,
+                keepMetadata: this.appOptions.keepMetadata,
+                useArchive: this.appOptions.useArchive,
+                makePublic: this.appOptions.makePublic,
+            });
+            const request = {
+                username: this.activeAccount.username,
+                configures: JSON.stringify(this.appOptions)
+            };
+            // TODO: DO i need loading page? if no remove this.dialog.loading = false
+            fetch(new URL("api/accountssettings", document.baseURI), {
+                method: "put",
+                body: JSON.stringify(request),
+                headers: {
+                    "Content-Type": "application/json",
+                },
+            }).then(response => {
+                if (!response.ok) throw response;
+                return response;
+            }).then(() => {
+                this.dialog.loading = false;
+                this.dialog.visible = false;
+            }).catch(err => {
+                this.dialog.loading = false;
+                this.getErrorMessage(err).then(msg => {
+                    this.showErrorDialog(msg);
+                })
+            });
+
+        },
+        loadAccounts() {
 			if (this.loading) return;
 
 			this.loading = true;
@@ -155,11 +179,23 @@ export default {
 						this.showErrorDialog("Password does not match");
 						return;
 					}
+                    const defaultconfigures = {
+                        showId: false,
+                        listMode: false,
+                        hideThumbnail: false,
+                        hideExcerpt: false,
+                        nightMode: false,
+                        keepMetadata: false,
+                        useArchive: false,
+                        makePublic: false
+                    };
+                    data.configures = JSON.stringify(defaultconfigures);
 
 					var request = {
 						username: data.username,
 						password: data.password,
 						owner: !data.visitor,
+                        configures: data.configures,
 					}
 
 					this.dialog.loading = true;
@@ -301,4 +337,4 @@ export default {
 	mounted() {
 		this.loadAccounts();
 	}
-}
+}

internal/view/login.html

@@ -109,6 +109,8 @@
 
 						// Save account data
 						localStorage.setItem("shiori-account", JSON.stringify(json.account));
+                        const configures = JSON.parse(json.account.configures);
+                        localStorage.setItem("shiori-setting", JSON.stringify(configures));
 
 						// Go to destination page
 						var currentUrl = new Url,

internal/webserver/handler-api.go

@@ -106,8 +106,9 @@ func (h *handler) apiLogin(w http.ResponseWriter, r *http.Request, ps httprouter
 
 	if len(accounts) == 0 && request.Username == "shiori" && request.Password == "gopher" {
 		genSession(model.Account{
-			Username: "shiori",
-			Owner:    true,
+			Username:   "shiori",
+			Owner:      true,
+			Configures: "{\"showId\":false,\"listMode\":false,\"hideThumbnail\":false,\"hideExcerpt\":false,\"nightMode\":false,\"keepMetadata\":false,\"useArchive\":false,\"makePublic\":false}",
 		}, time.Hour)
 		return
 	}
@@ -846,6 +847,40 @@ func (h *handler) apiUpdateAccount(w http.ResponseWriter, r *http.Request, ps ht
 	fmt.Fprint(w, 1)
 }
 
+// apiUpdateSettings is handler for PUT /api/accounts
+func (h *handler) apiUpdateSettings(w http.ResponseWriter, r *http.Request, ps httprouter.Params) {
+	ctx := r.Context()
+
+	// Make sure session still valid
+	err := h.validateSession(r)
+	checkError(err)
+
+	// Decode request
+	request := struct {
+		Username   string `json:"username"`
+		Configures string `json:"configures"`
+	}{}
+
+	err = json.NewDecoder(r.Body).Decode(&request)
+	checkError(err)
+
+	// Get existing account data from database
+	account, exist, err := h.DB.GetAccount(ctx, request.Username)
+	checkError(err)
+
+	if !exist {
+		panic(fmt.Errorf("username doesn't exist"))
+	}
+
+	// Save new password to database
+	account.Configures = request.Configures
+	//	account.Owner = request.Owner
+	err = h.DB.SaveSettings(ctx, account)
+	checkError(err)
+
+	fmt.Fprint(w, 1)
+}
+
 // apiDeleteAccount is handler for DELETE /api/accounts
 func (h *handler) apiDeleteAccount(w http.ResponseWriter, r *http.Request, ps httprouter.Params) {
 	ctx := r.Context()

internal/webserver/server.go

@@ -202,6 +202,7 @@ func ServeApp(cfg Config) error {
 
 	router.GET(jp("/api/accounts"), withLogging(hdl.apiGetAccounts))
 	router.PUT(jp("/api/accounts"), withLogging(hdl.apiUpdateAccount))
+	router.PUT(jp("/api/accountssettings"), withLogging(hdl.apiUpdateSettings))
 	router.POST(jp("/api/accounts"), withLogging(hdl.apiInsertAccount))
 	router.DELETE(jp("/api/accounts"), withLogging(hdl.apiDeleteAccount))