From dfd16f99b8b0ba098bc52a806653f19fbbc84fcb Mon Sep 17 00:00:00 2001
From: Simone Gotti <simone.gotti@gmail.com>
Date: Tue, 14 May 2019 18:05:17 +0200
Subject: [PATCH] projectgroupsettings: implement update project group

---
 src/components/projectgroupsettings.vue | 75 ++++++++++++++++++++++++-
 src/util/data.js                        | 18 ++++++
 2 files changed, 90 insertions(+), 3 deletions(-)

diff --git a/src/components/projectgroupsettings.vue b/src/components/projectgroupsettings.vue
index c3aec50..8febc91 100644
--- a/src/components/projectgroupsettings.vue
+++ b/src/components/projectgroupsettings.vue
@@ -1,5 +1,34 @@
 <template>
   <div>
+    <nav class="panel">
+      <p class="panel-heading">Project Group Settings</p>
+      <div class="panel-block is-block">
+        <div v-if="!isRootProjectGroup" class="field">
+          <label class="label">Project Group Name</label>
+          <div class="control">
+            <input class="input" type="text" placeholder="Text input" v-model="projectGroup.name">
+          </div>
+        </div>
+        <div class="field">
+          <div class="control">
+            <label class="checkbox">
+              <input type="checkbox" v-model="projectGroupIsPrivate">
+              Private
+            </label>
+          </div>
+        </div>
+
+        <div class="field is-grouped">
+          <div class="control">
+            <button class="button is-primary" @click="updateProjectGroup()">Update</button>
+          </div>
+        </div>
+
+        <div v-if="updateProjectGroupError" class="message is-danger">
+          <div class="message-body">{{ updateProjectGroupError }}</div>
+        </div>
+      </div>
+    </nav>
     <nav class="panel">
       <p class="panel-heading">Variables</p>
       <div class="panel-block is-block">
@@ -51,7 +80,12 @@
 </template>
 
 <script>
-import { fetchVariables, deleteProjectGroup } from "@/util/data.js";
+import {
+  fetchProjectGroup,
+  fetchVariables,
+  updateProjectGroup,
+  deleteProjectGroup
+} from "@/util/data.js";
 
 import { projectGroupLink } from "@/util/link.js";
 
@@ -67,7 +101,10 @@ export default {
   },
   data() {
     return {
+      updateProjectGroupError: null,
       deleteProjectGroupError: null,
+      projectGroup: null,
+      projectGroupIsPrivate: false,
       variables: [],
       allvariables: [],
       projectGroupNameToDelete: ""
@@ -91,8 +128,32 @@ export default {
   },
   methods: {
     resetErrors() {
+      this.updateProjectGroupError = null;
       this.deleteProjectGroupError = null;
     },
+    async updateProjectGroup() {
+      this.resetErrors();
+
+      let projectgroupref = [
+        this.ownertype,
+        this.ownername,
+        ...this.projectgroupref
+      ].join("/");
+
+      let visibility = "public";
+      if (this.projectGroupIsPrivate) {
+        visibility = "private";
+      }
+      let { error } = await updateProjectGroup(
+        projectgroupref,
+        this.projectGroup.name,
+        visibility
+      );
+      if (error) {
+        this.updateProjectGroupError = error;
+        return;
+      }
+    },
     async deleteProjectGroup() {
       let projectgroupref = [
         this.ownertype,
@@ -123,11 +184,19 @@ export default {
       ...this.projectgroupref
     ].join("/");
 
-    let { data, error } = await fetchVariables(
+    let { data, error } = await fetchProjectGroup(projectgroupref);
+    if (error) {
+      this.$store.dispatch("setError", error);
+      return;
+    }
+    this.projectGroup = data;
+    this.projectGroupIsPrivate = this.projectGroup.visibility == "private";
+
+    ({ data, error } = await fetchVariables(
       "projectgroup",
       projectgroupref,
       false
-    );
+    ));
     if (error) {
       this.$store.dispatch("setError", error);
       return;
diff --git a/src/util/data.js b/src/util/data.js
index bfabe14..74f1ac4 100644
--- a/src/util/data.js
+++ b/src/util/data.js
@@ -78,6 +78,12 @@ export async function fetchUser(username) {
     return await fetch(apiurl(path));
 }
 
+export async function fetchProjectGroup(projectgroupref) {
+    let path = "/projectgroups/" + encodeURIComponent(projectgroupref)
+
+    return await fetch(apiurl(path));
+}
+
 export async function fetchProjectGroupSubgroups(projectgroupref) {
     let path = "/projectgroups/" + encodeURIComponent(projectgroupref)
     path += "/subgroups";
@@ -209,6 +215,18 @@ export async function createProjectGroup(parentref, name, visibility) {
     return await fetch(apiurl(path), init)
 }
 
+export async function updateProjectGroup(projectgroupref, name, visibility) {
+    let path = "/projectgroups/" + encodeURIComponent(projectgroupref)
+    let init = {
+        method: "PUT",
+        body: JSON.stringify({
+            name: name,
+            visibility: visibility,
+        })
+    }
+    return await fetch(apiurl(path), init)
+}
+
 export async function createProject(parentref, name, visibility, remotesourcename, remoterepopath) {
     let path = "/projects"
     let init = {