Add database schemas and enhance configuration

Add schemas for SlackQuestion, StackExchangeQuestion and StackExchangeUser
Read configuration into map[string]string instead into structs
Add SessionRefresh for commands depending on configuration
Add option to enable or disable functionality of Slack Overflow

Author: mkungla

slack/slack.go

@@ -1,5 +1,7 @@
 package slack
 
+var slack *Client
+
 // Client Slack API Client resource
 type Client struct {
 	channel string
@@ -9,7 +11,8 @@ type Client struct {
 
 // Load slack
 func Load() *Client {
-	return &Client{}
+	slack = &Client{}
+	return slack
 }
 
 // SetToken set Slack API Token

slackoverflow/application.go

@@ -47,11 +47,6 @@ type Application struct {
 func (so *Application) Run() {
 	so.Banner()
 
-	// Check configuration
-	if !so.config.Validate() {
-		so.config.Reconfigure()
-	}
-
 	// Handle call
 	if _, err := parser.Parse(); err != nil {
 		// Failure was fine since -h or --help flag was provided
@@ -103,16 +98,22 @@ func (so *Application) Close(code int) {
 // SessionRefresh refresh session and makes sure that all deps are loaded
 func (so *Application) SessionRefresh() {
 
+	// Check configuration
+	if !so.config.IsConfigured() {
+		Emergency("You must execute 'slackoverflow reconfigure' or correct errors in ~/.slackoverflow/slackoverflow.yaml")
+	}
+
 	// Set Log Level from -v or -d flag default to config.Data.SlackOverflow.LogLevel
 	UpdateLogLevel()
 
 	// Load Slack Client
-	if so.Slack != nil {
+	if so.Slack == nil {
 		// Configure slack
 		so.Slack = slack.Load()
 		so.Slack.SetHost(so.config.Slack.APIHost)
 		so.Slack.SetToken(so.config.Slack.Token)
 		so.Slack.SetChannel(so.config.Slack.Channel)
+		Debug("Slack Client is loaded.")
 	} else {
 		Debug("Slack Client is already loaded.")
 	}
@@ -128,42 +129,49 @@ func (so *Application) SessionRefresh() {
 		Ok("SQLite3 Database loaded: %s", so.databaseFile)
 
 		// Check does the StackQuestion table exist
-		err = so.SQLite3.VerifyTable("StackQuestion")
+		err = so.SQLite3.VerifyTable("StackExchangeQuestion")
 		if err != nil {
-			Emergency("Table StackQuestion: %q", err)
+			Emergency("Table StackExchangeQuestion: %q", err)
 		}
-		Ok("Table: StackQuestion exists.")
+		Ok("Table: StackExchangeQuestion exists.")
 
-		// Check does the StackQuestionLink table exist
-		err = so.SQLite3.VerifyTable("StackQuestionLink")
+		// Check does the SlackQuestion table exist
+		err = so.SQLite3.VerifyTable("SlackQuestion")
 		if err != nil {
-			Emergency("Table StackQuestionLink: %q", err)
+			Emergency("Table SlackQuestion: %q", err)
 		}
-		Ok("Table: StackQuestionLink exists.")
+		Ok("Table: SlackQuestion exists.")
 
-		// Check does the StackUser table exist
-		err = so.SQLite3.VerifyTable("StackUser")
+		// Check does the StackExchangeUser table exist
+		err = so.SQLite3.VerifyTable("StackExchangeUser")
 		if err != nil {
-			Emergency("Table StackUser: %q", err)
+			Emergency("Table StackExchangeUser: %q", err)
 		}
-		Ok("Table: StackQuestionLink exists.")
-
+		Ok("Table: StackExchangeUser exists.")
 	} else {
-		Ok("Table: StackUser exists.")
+		Ok("SQLite3 Database is already loaded.")
 	}
 
 	// Load Stack Exchange Client
 	// Load Slack Client
-	if so.StackExchange != nil {
+	if so.StackExchange == nil {
 		// Configure slack
 		so.StackExchange = stackexchange.Load()
-		so.StackExchange.SetHost(so.config.StackExchange.APIVersion)
+		so.StackExchange.SetHost(so.config.StackExchange.APIHost)
+		so.StackExchange.SetAPIVersion(so.config.StackExchange.APIVersion)
+		so.StackExchange.SetAPIKey(so.config.StackExchange.Key)
 
+		Debug("Stack Exchange Client is loaded.")
 	} else {
-		Debug("Slack Client is already loaded.")
+		Debug("Stack Exchange Client is already loaded.")
 	}
 }
 
+// Debugging Either is debugging enabled or not
+func (so *Application) Debugging() bool {
+	return so.logLevel == 100
+}
+
 // Start session
 func Start() *Application {
 	gid := os.Getegid()

slackoverflow/cmd-config.go

@@ -12,6 +12,9 @@ type cmdConfig struct{}
 
 func (a *cmdConfig) Execute(args []string) error {
 
+	// Refresh the session before running this command and make sure that Slack Overflow is configured
+	slackoverflow.SessionRefresh()
+
 	selfConfig := std.NewTable("SlackOverflow Configuration", " ")
 	selfConfig.AddRow("Log Level", slackoverflow.config.SlackOverflow.LogLevel)
 	selfConfig.AddRow("Number of Questions to watch", strconv.Itoa(slackoverflow.config.SlackOverflow.Watch))
@@ -43,8 +46,8 @@ func (a *cmdConfig) Execute(args []string) error {
 
 	stackexchange.AddRow("Key", slackoverflow.config.StackExchange.Key)
 
-	stackexchange.AddRow("Site", slackoverflow.config.StackExchange.Parameters.Site)
-	stackexchange.AddRow("Tagged", slackoverflow.config.StackExchange.Parameters.Tagged)
+	stackexchange.AddRow("Site", slackoverflow.config.StackExchange.SearchAdvanced["site"])
+	stackexchange.AddRow("Tagged", slackoverflow.config.StackExchange.SearchAdvanced["tagged"])
 	stackexchange.Print()
 
 	return nil

slackoverflow/cmd-reconfigure.go

@@ -8,9 +8,9 @@ type cmdReconfigure struct{}
 
 func (a *cmdReconfigure) Execute(args []string) error {
 
-	doReconfigure := std.AskForConfirmation("This will overwrite current configuration, Are you sure you want to continue?")
+	doReconfigure := std.AskForConfirmation("Start interactive confguration?")
 	if doReconfigure {
-		slackoverflow.config.Reconfigure()
+		slackoverflow.config.ReconfigureAll()
 		Ok("Reconfiguration done")
 	} else {
 		Warning("Reconfiguration canceled!")

slackoverflow/cmd-restart.go

@@ -3,3 +3,11 @@ package slackoverflow
 // slackoverflow restart
 // Restart SlackOverflow daemon.
 type cmdRestart struct{}
+
+func (a *cmdRestart) Execute(args []string) error {
+
+	// Refresh the session before running this command and make sure that Slack Overflow is configured
+	slackoverflow.SessionRefresh()
+
+	return nil
+}

slackoverflow/cmd-slack.go

@@ -9,7 +9,10 @@ type cmdSlackChannels struct{}
 
 // Execute
 func (slack *cmdSlackChannels) Execute(args []string) error {
-	// Refresh the session before running this command
+
+	// Refresh the session before running this command and make sure that Slack Overflow is configured
+	slackoverflow.SessionRefresh()
+
 	slackoverflow.Slack.ListChannels()
 	return nil
 }

slackoverflow/cmd-start.go

@@ -3,3 +3,11 @@ package slackoverflow
 // slackoverflow start
 // Start SlackOverflow daemon.
 type cmdStart struct{}
+
+func (a *cmdStart) Execute(args []string) error {
+
+	// Refresh the session before running this command and make sure that Slack Overflow is configured
+	slackoverflow.SessionRefresh()
+
+	return nil
+}

slackoverflow/cmd-watch.go

@@ -3,3 +3,11 @@ package slackoverflow
 // slackoverflow watch
 // SlackOverflow .
 type cmdWatch struct{}
+
+func (a *cmdWatch) Execute(args []string) error {
+
+	// Refresh the session before running this command and make sure that Slack Overflow is configured
+	slackoverflow.SessionRefresh()
+
+	return nil
+}