From 5152600ea3434efdcccffbd776ba7aa8f23e2363 Mon Sep 17 00:00:00 2001
From: Frank <frank@wuenscheonline.de>
Date: Sun, 1 Feb 2026 20:55:44 +0100
Subject: [PATCH] User can change a character from private to public and back

---
 backend/character/routes.go                  |   2 +
 backend/router/setup.go                      |   2 +-
 frontend/src/components/CharacterDetails.vue |  21 +++
 frontend/src/components/VisibilityDialog.vue | 181 +++++++++++++++++++
 frontend/src/locales/de                      |  13 ++
 frontend/src/locales/en                      |  13 ++
 6 files changed, 231 insertions(+), 1 deletion(-)
 create mode 100644 frontend/src/components/VisibilityDialog.vue

diff --git a/backend/character/routes.go b/backend/character/routes.go
index bc39338..ab80ee5 100644
--- a/backend/character/routes.go
+++ b/backend/character/routes.go
@@ -10,6 +10,8 @@ func RegisterRoutes(r *gin.RouterGroup) {
 	charGrp.POST("", CreateCharacter)
 	charGrp.GET("/:id", GetCharacter)
 	charGrp.PUT("/:id", UpdateCharacter)
+	charGrp.PATCH("/:id", UpdateCharacter)
+
 	charGrp.DELETE("/:id", DeleteCharacter)
 	charGrp.PUT("/:id/image", UpdateCharacterImage)
 	charGrp.GET("/:id/datasheet-options", GetDatasheetOptions)
diff --git a/backend/router/setup.go b/backend/router/setup.go
index 92d0b42..81385a5 100644
--- a/backend/router/setup.go
+++ b/backend/router/setup.go
@@ -21,7 +21,7 @@ func SetupGin(r *gin.Engine) {
 	r.Use(cors.New(cors.Config{
 		//AllowOrigins:     []string{"http://localhost:3000"}, // Replace with your frontend's URL
 		AllowOrigins:     allowedOrigins,
-		AllowMethods:     []string{"GET", "POST", "PUT", "DELETE"},
+		AllowMethods:     []string{"GET", "POST", "PUT", "PATCH", "DELETE"},
 		AllowHeaders:     []string{"Origin", "Content-Type", "Authorization"},
 		ExposeHeaders:    []string{"Content-Length"},
 		AllowCredentials: true,
diff --git a/frontend/src/components/CharacterDetails.vue b/frontend/src/components/CharacterDetails.vue
index 7e1116f..3ae6682 100644
--- a/frontend/src/components/CharacterDetails.vue
+++ b/frontend/src/components/CharacterDetails.vue
@@ -6,6 +6,9 @@
         <button @click="showExportDialog = true" class="export-button-small" :title="$t('export.title')">
           📄
         </button>
+        <button @click="showVisibilityDialog = true" class="export-button-small" :title="$t('visibility.title')">
+          {{ character.public ? '🌐' : '🔒' }}
+        </button>
         <h2>{{ $t('char') }}: {{ character.name }} ({{ $t(currentView) }})</h2>
       </div>
     </div>
@@ -18,6 +21,15 @@
       @export-success="handleExportSuccess"
     />
 
+    <!-- Visibility Dialog -->
+    <VisibilityDialog 
+      :characterId="id" 
+      :currentVisibility="character.public"
+      :showDialog="showVisibilityDialog"
+      @update:showDialog="showVisibilityDialog = $event"
+      @visibility-updated="handleVisibilityUpdated"
+    />
+
     <!-- Submenu Content -->
     <!-- <div class="character-aspect"> -->
       <component :is="currentView" :character="character" @character-updated="refreshCharacter"/>
@@ -53,6 +65,7 @@
 <script>
 import API from '../utils/api'
 import ExportDialog from "./ExportDialog.vue";
+import VisibilityDialog from "./VisibilityDialog.vue";
 import DatasheetView from "./DatasheetView.vue"; // Component for character stats
 import SkillView from "./SkillView.vue"; // Component for character history
 import WeaponView from "./WeaponView.vue"; // Component for character history
@@ -67,6 +80,7 @@ export default {
   props: ["id"], // Receive the route parameter as a prop
   components: {
     ExportDialog,
+    VisibilityDialog,
     DatasheetView,
     SkillView,
     WeaponView,
@@ -81,6 +95,7 @@ export default {
       currentView: "DatasheetView", // Default view
       lastView: "DatasheetView",
       showExportDialog: false,
+      showVisibilityDialog: false,
       menus: [
         { id: 1, name: "Datasheet", component: "DatasheetView" },
         { id: 2, name: "Skill", component: "SkillView" },
@@ -108,6 +123,12 @@ export default {
       console.log('PDF exported successfully')
     },
     
+    handleVisibilityUpdated(isPublic) {
+      this.character.public = isPublic
+      this.showVisibilityDialog = false
+      console.log('Character visibility updated to:', isPublic ? 'public' : 'private')
+    },
+    
     changeView(view) {
       this.lastView = this.currentView;
       this.currentView = view;
diff --git a/frontend/src/components/VisibilityDialog.vue b/frontend/src/components/VisibilityDialog.vue
new file mode 100644
index 0000000..ecaccf9
--- /dev/null
+++ b/frontend/src/components/VisibilityDialog.vue
@@ -0,0 +1,181 @@
+<template>
+  <div v-if="showDialog" class="modal-overlay" @click.self="closeDialog">
+    <div class="modal-content">
+      <div class="modal-header">
+        <h3>{{ $t('visibility.title') }}</h3>
+        <button @click="closeDialog" class="close-button">&times;</button>
+      </div>
+      <div class="modal-body">
+        <div v-if="isUpdating" class="loading-overlay">
+          <div class="spinner"></div>
+          <p>{{ $t('visibility.updating') }}</p>
+        </div>
+        <div class="form-group">
+          <p>{{ $t('visibility.description') }}</p>
+          <div class="visibility-options">
+            <label class="radio-option">
+              <input 
+                type="radio" 
+                :value="false" 
+                v-model="isPublic"
+                :disabled="isUpdating"
+              >
+              <div class="option-content">
+                <span class="option-label">{{ $t('visibility.private') }}</span>
+                <span class="option-description">{{ $t('visibility.privateDescription') }}</span>
+              </div>
+            </label>
+            <label class="radio-option">
+              <input 
+                type="radio" 
+                :value="true" 
+                v-model="isPublic"
+                :disabled="isUpdating"
+              >
+              <div class="option-content">
+                <span class="option-label">{{ $t('visibility.public') }}</span>
+                <span class="option-description">{{ $t('visibility.publicDescription') }}</span>
+              </div>
+            </label>
+          </div>
+        </div>
+      </div>
+      <div class="modal-footer">
+        <button @click="closeDialog" class="btn-cancel" :disabled="isUpdating">
+          {{ $t('visibility.cancel') }}
+        </button>
+        <button @click="updateVisibility" class="btn-primary" :disabled="isUpdating">
+          <span v-if="!isUpdating">{{ $t('visibility.save') }}</span>
+          <span v-else>{{ $t('visibility.saving') }}</span>
+        </button>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script>
+import API from '../utils/api'
+
+export default {
+  name: "VisibilityDialog",
+  props: {
+    characterId: {
+      type: [String, Number],
+      required: true
+    },
+    currentVisibility: {
+      type: Boolean,
+      default: false
+    },
+    showDialog: {
+      type: Boolean,
+      default: false
+    }
+  },
+  data() {
+    return {
+      isPublic: false,
+      isUpdating: false
+    }
+  },
+  watch: {
+    currentVisibility: {
+      immediate: true,
+      handler(newValue) {
+        this.isPublic = newValue
+      }
+    },
+    showDialog(newValue) {
+      if (newValue) {
+        this.isPublic = this.currentVisibility
+      }
+    }
+  },
+  methods: {
+    closeDialog() {
+      if (!this.isUpdating) {
+        this.$emit('update:showDialog', false)
+      }
+    },
+    
+    async updateVisibility() {
+      this.isUpdating = true
+      
+      try {
+        const token = localStorage.getItem('token')
+        await API.patch(`/api/characters/${this.characterId}`, 
+          { public: this.isPublic },
+          {
+            headers: { Authorization: `Bearer ${token}` }
+          }
+        )
+        
+        this.$emit('visibility-updated', this.isPublic)
+        this.closeDialog()
+      } catch (error) {
+        console.error('Failed to update character visibility:', error)
+        alert(this.$t('visibility.updateError') + ': ' + (error.response?.data?.error || error.message))
+      } finally {
+        this.isUpdating = false
+      }
+    }
+  }
+}
+</script>
+
+<style scoped>
+.visibility-options {
+  display: flex;
+  flex-direction: column;
+  gap: 15px;
+  margin-top: 15px;
+}
+
+.radio-option {
+  display: flex;
+  align-items: flex-start;
+  gap: 12px;
+  padding: 15px;
+  border: 2px solid #dee2e6;
+  border-radius: 8px;
+  cursor: pointer;
+  transition: all 0.2s ease;
+}
+
+.radio-option:hover {
+  border-color: #007bff;
+  background-color: #f8f9fa;
+}
+
+.radio-option input[type="radio"] {
+  margin-top: 3px;
+  cursor: pointer;
+}
+
+.option-content {
+  display: flex;
+  flex-direction: column;
+  gap: 5px;
+  flex: 1;
+}
+
+.option-label {
+  font-weight: 600;
+  font-size: 1.1rem;
+  color: #333;
+}
+
+.option-description {
+  font-size: 0.9rem;
+  color: #666;
+}
+
+.radio-option input[type="radio"]:checked + .option-content .option-label {
+  color: #007bff;
+}
+
+.radio-option:has(input[type="radio"]:checked) {
+  border-color: #007bff;
+  background-color: #e7f3ff;
+}
+</style>
diff --git a/frontend/src/locales/de b/frontend/src/locales/de
index ef61c2f..d0ac7f5 100644
--- a/frontend/src/locales/de
+++ b/frontend/src/locales/de
@@ -513,6 +513,19 @@ export default {
     pleaseWait: 'Bitte warten, dies kann einen Moment dauern',
     popupBlocked: 'Popup wurde blockiert. Bitte erlauben Sie Popups für diese Seite.'
   },
+  visibility: {
+    title: 'Sichtbarkeit ändern',
+    description: 'Legen Sie fest, wer diesen Charakter sehen kann.',
+    private: 'Privat',
+    privateDescription: 'Nur Sie können diesen Charakter sehen',
+    public: 'Öffentlich',
+    publicDescription: 'Alle Benutzer können diesen Charakter sehen',
+    cancel: 'Abbrechen',
+    save: 'Speichern',
+    saving: 'Wird gespeichert...',
+    updating: 'Sichtbarkeit wird aktualisiert...',
+    updateError: 'Fehler beim Aktualisieren der Sichtbarkeit'
+  },
   userManagement: {
     title: 'Benutzerverwaltung',
     loading: 'Lade Benutzer...',
diff --git a/frontend/src/locales/en b/frontend/src/locales/en
index 07217b0..0812e43 100644
--- a/frontend/src/locales/en
+++ b/frontend/src/locales/en
@@ -509,6 +509,19 @@ export default {
     pleaseWait: 'Please wait, this may take a moment',
     popupBlocked: 'Popup was blocked. Please allow popups for this site.'
   },
+  visibility: {
+    title: 'Change Visibility',
+    description: 'Set who can see this character.',
+    private: 'Private',
+    privateDescription: 'Only you can see this character',
+    public: 'Public',
+    publicDescription: 'All users can see this character',
+    cancel: 'Cancel',
+    save: 'Save',
+    saving: 'Saving...',
+    updating: 'Updating visibility...',
+    updateError: 'Failed to update visibility'
+  },
   userManagement: {
     title: 'User Management',
     loading: 'Loading users...',