chihlasm/resolutionflow
1
0
Fork 0
Code Issues 23 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: keep is_public and visibility in sync on updates

Browse Source 
When visibility changes to 'public', is_public=True. When it changes away
from 'public', is_public=False. When is_public is set via TreeUpdate,
visibility column is updated to match.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
chihlasm
2026-02-24 03:36:23 -05:00
parent 8545a123ab
commit 55d329e07b
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
backend/app/api/endpoints/trees.py
View File

@@ -629,6 +629,13 @@ async def update_tree(
for field, value in update_data.items(): for field, value in update_data.items():
setattr(tree, field, value) setattr(tree, field, value)
# Keep visibility and is_public in sync
if tree_data.is_public is not None:
if tree_data.is_public and tree.visibility not in ('public',):
tree.visibility = 'public'
elif not tree_data.is_public and tree.visibility == 'public':
tree.visibility = 'team' # downgrade from public to team
# Increment version if tree structure changed # Increment version if tree structure changed
if "tree_structure" in update_data: if "tree_structure" in update_data:
tree.version += 1 tree.version += 1
@@ -1074,6 +1081,7 @@ async def update_tree_visibility(
# Update visibility # Update visibility
old_visibility = tree.visibility old_visibility = tree.visibility
tree.visibility = visibility_data.visibility tree.visibility = visibility_data.visibility
tree.is_public = (visibility_data.visibility == 'public')
await log_audit(db, current_user.id, "tree.visibility.update", "tree", tree.id, await log_audit(db, current_user.id, "tree.visibility.update", "tree", tree.id,
{"tree_name": tree.name, "old_visibility": old_visibility, {"tree_name": tree.name, "old_visibility": old_visibility,