create endpoint for updating task properties

common/json.go

@@ -62,3 +62,17 @@ type TaskJson struct {
 	Members   []Member `json:"members"`
 	Priority  string   `json:"priority"`
 }
+
+type KrPropertiesJson struct {
+	KrName    string      `json:"krName",omitempty`
+	KrBody    string      `json:"krbody",omitempty`
+	Priority  string      `json:"priority",omitempty`
+	Completed interface{} `json:"completed",omitempty`
+}
+
+type TaskPropertiesJson struct {
+	TaskName  string      `json:"taskname",omitempty`
+	TaskBody  string      `json:"taskbody",omitempty`
+	Priority  string      `json:"priority",omitempty`
+	Completed interface{} `json:"completed",omitempty`
+}

handlers/keyResult_update_props.go

@@ -1,6 +1,7 @@
 package handlers
 
 import (
+	"github.com/dmonay/okra/common"
 	"github.com/gin-gonic/gin"
 	"gopkg.in/mgo.v2/bson"
 )
@@ -12,14 +13,7 @@ func (dw *DoWorkResource) UpdateKrProperties(c *gin.Context) {
 	kr := c.Params.ByName("kr")
 	id := bson.ObjectIdHex(treeId)
 
-	type KrPropertiesJson struct {
-		KrName    string      `json:"krName",omitempty`
-		KrBody    string      `json:"krbody",omitempty`
-		Priority  string      `json:"priority",omitempty`
-		Completed interface{} `json:"completed",omitempty`
-	}
-
-	var reqBody KrPropertiesJson
+	var reqBody common.KrPropertiesJson
 	c.Bind(&reqBody)
 	newName := reqBody.KrName
 	newBody := reqBody.KrBody

handlers/start.go

@@ -77,10 +77,11 @@ func Run(cfg common.Config) error {
 	r.POST("/create/objective/:organization", doWorkResource.CreateObjective)
 	r.POST("/update/objective/properties/:organization/:treeid/:objective", doWorkResource.UpdateObjProperties)
 	r.POST("/create/kr/:organization/:objective", doWorkResource.CreateKeyResult)
-	r.POST("/update/objective/properties/:organization/:treeid/:objective/:kr", doWorkResource.UpdateKrProperties)
+	r.POST("/update/kr/properties/:organization/:treeid/:objective/:kr", doWorkResource.UpdateKrProperties)
 
 	// tasks
-	r.POST("/create/kr/:organization/:objective/:kr", doWorkResource.CreateTask)
+	r.POST("/create/task/:organization/:objective/:kr", doWorkResource.CreateTask)
+	r.POST("/update/task/properties/:organization/:treeid/:objective/:kr/:task", doWorkResource.UpdateTaskProperties)
 
 	r.Run(cfg.SvcHost)
 

handlers/task_update_props.go

@@ -0,0 +1,62 @@
+package handlers
+
+import (
+	"github.com/dmonay/okra/common"
+	"github.com/gin-gonic/gin"
+	"gopkg.in/mgo.v2/bson"
+)
+
+func (dw *DoWorkResource) UpdateTaskProperties(c *gin.Context) {
+	org := c.Params.ByName("organization")
+	treeId := c.Params.ByName("treeid")
+	obj := c.Params.ByName("objective")
+	krIndex := c.Params.ByName("kr")
+	taskIndex := c.Params.ByName("task")
+	id := bson.ObjectIdHex(treeId)
+
+	var reqBody common.TaskPropertiesJson
+	c.Bind(&reqBody)
+	newName := reqBody.TaskName
+	newBody := reqBody.TaskBody
+	newStatus := reqBody.Completed
+	newPriority := reqBody.Priority
+
+	// instead of using bson.M, manually create the map
+	// so that we can add fields conditionally
+	myMap := make(map[string]interface{})
+
+	// TEMP workaround. Mongo doesn't support >1 positional operators per query,
+	// so I can't use the index of the nested array inside the objectives document.
+	// Thus I need the index of the array beforehand, and this is currently,
+	// temporarily passed in as the last URL param
+	nameKey := "objectives.$.keyresults." + krIndex + ".tasks." + taskIndex + ".name"
+	bodyKey := "objectives.$.keyresults." + krIndex + ".tasks." + taskIndex + ".body"
+	statusKey := "objectives.$.keyresults." + krIndex + ".tasks." + taskIndex + ".completed"
+	prioritiesKey := "objectives.$.keyresults." + krIndex + ".tasks." + taskIndex + ".priority"
+
+	if newName != "" {
+		myMap[nameKey] = newName
+	}
+
+	if newBody != "" {
+		myMap[bodyKey] = newBody
+	}
+
+	if newStatus != "" {
+		myMap[statusKey] = newStatus
+	}
+
+	if newPriority != "" {
+		myMap[prioritiesKey] = newPriority
+	}
+
+	querier := bson.M{"_id": id, "objectives.id": obj}
+	updateName := bson.M{"$set": myMap}
+	err := dw.mongo.C(org).Update(querier, updateName)
+	if err != nil {
+		CheckErr(err, "Mongo failed to update task's properties", c)
+		return
+	}
+
+	c.JSON(201, SuccessMsg{"Task updated!"})
+}