From c705ae942573984784ef501bf8198f61f5206ddd Mon Sep 17 00:00:00 2001
From: historia <[not public]>
Date: Thu, 9 Jul 2026 05:27:10 -0400
Subject: refactor: simplify yaml, remove support for old scalar fields

---
 internal/admin/static/map.js | 25 +++----------------------
 1 file changed, 3 insertions(+), 22 deletions(-)

(limited to 'internal/admin/static/map.js')

diff --git a/internal/admin/static/map.js b/internal/admin/static/map.js
index 3507e2d..aeb2d57 100644
--- a/internal/admin/static/map.js
+++ b/internal/admin/static/map.js
@@ -1433,11 +1433,9 @@ function renderExitsSection(r) {
     var target = typeof exit === 'object' ? exit.room : exit;
     var cond = (typeof exit === 'object' && exit.condition) ? exit.condition : null;
     var bmsg = (typeof exit === 'object' && exit.blocked_message) ? exit.blocked_message : '';
-    var setFlags = (typeof exit === 'object' && exit.set_flags) ? exit.set_flags : null;
-    var setPlayerFlags = (typeof exit === 'object' && exit.set_player_flags) ? exit.set_player_flags : null;
     var hidden = (typeof exit === 'object' && exit.hidden) ? exit.hidden : false;
     var alwaysBlocked = (typeof exit === 'object' && exit.always_blocked) ? exit.always_blocked : false;
-    var isConditional = !!(cond || bmsg || setFlags || setPlayerFlags || hidden || alwaysBlocked);
+    var isConditional = !!(cond || bmsg || hidden || alwaysBlocked);
     var isBidi = mapData && mapData.links && mapData.links.some(function(l) {
       return l.bidirectional && ((l.from === r.id && l.to === target) || (l.from === target && l.to === r.id));
     });
@@ -1475,16 +1473,9 @@ function editExitCondition(dir) {
   if (typeof exit !== 'object') { exit = {room: exit}; }
   var cond = exit.condition || null;
   var bmsg = exit.blocked_message || '';
-  var setFlags = exit.set_flags || null;
-  var setPlayerFlags = exit.set_player_flags || null;
   var hidden = exit.hidden || false;
   var alwaysBlocked = exit.always_blocked || false;
 
-  var flagsStr = '';
-  if (setFlags) flagsStr = Object.keys(setFlags).map(function(k) { return k + '=' + setFlags[k]; }).join(', ');
-  var pflagsStr = '';
-  if (setPlayerFlags) pflagsStr = Object.keys(setPlayerFlags).map(function(k) { return k + '=' + setPlayerFlags[k]; }).join(', ');
-
   var parsed = parseCondition(cond);
 
   var overlay = document.createElement('div');
@@ -1494,8 +1485,6 @@ function editExitCondition(dir) {
   menu.style.cssText = 'position:fixed;top:50%;left:50%;transform:translate(-50%,-50%);min-width:520px;max-width:90vw;max-height:85vh;overflow-y:auto;overflow-x:hidden;padding:14px;z-index:201';
 
   var h = '<h3 style="margin-bottom:8px;font-size:13px">Exit: ' + esc(dir) + ' &rarr; #' + exit.room + '</h3>';
-  h += '<div class="form-group"><label>Set Flags (key=value, comma separated)</label><input id="exitFlags" value="' + escAttr(flagsStr) + '"></div>';
-  h += '<div class="form-group"><label>Set Player Flags (key=value, comma separated)</label><input id="exitPFlags" value="' + escAttr(pflagsStr) + '"></div>';
   h += '<div class="form-group"><label>Blocked Message</label><textarea id="exitBmsg" rows="2">' + esc(bmsg) + '</textarea></div>';
   h += '<div class="form-group"><label><input type="checkbox" id="exitHidden"' + (hidden ? ' checked' : '') + '> Hidden</label></div>';
   h += '<div class="form-group"><label><input type="checkbox" id="exitAlwaysBlocked"' + (alwaysBlocked ? ' checked' : '') + '> Always Blocked (blocked, shown on map)</label></div>';
@@ -1540,7 +1529,7 @@ function parseCondition(cond) {
 function extractClause(c) {
   var not = c.not === true;
   if (c.player_flag && c.player_flag !== '') return {type: 'player_flag', value: c.player_flag, not: not};
-  if (c.flag && c.flag !== '') return {type: 'flag', value: c.flag, not: not};
+  if (c.global_flag && c.global_flag !== '') return {type: 'global_flag', value: c.global_flag, not: not};
   if (c.has_item && c.has_item !== '') return {type: 'has_item', value: c.has_item, not: not};
   if (c.min_credits && c.min_credits !== 0) return {type: 'min_credits', value: String(c.min_credits), not: not};
   if (c.value && typeof c.value === 'object' && c.value.key) return {type: 'value', value: c.value.key + '=' + (c.value.value || ''), not: not};
@@ -1555,7 +1544,7 @@ function renderConditionClauses(menu) {
   clauses.forEach(function(clause, i) {
     h += '<div class="cond-clause" data-idx="' + i + '" style="display:flex;gap:4px;align-items:center;margin-bottom:4px">';
     h += '<select class="cond-type" onchange="var c=this.parentElement;var m=this.closest(\'.exit-modal\');var idx=parseInt(c.getAttribute(\'data-idx\'));m._condClauses[idx].type=this.value;renderConditionClauses(m)">';
-    ['player_flag','flag','value','has_item','min_credits'].forEach(function(t) {
+    ['player_flag','global_flag','value','has_item','min_credits'].forEach(function(t) {
       h += '<option value="' + t + '"' + (clause.type === t ? ' selected' : '') + '>' + t + '</option>';
     });
     h += '</select>';
@@ -1640,10 +1629,6 @@ function saveExitCondition() {
   exit.hidden = hiddenEl ? hiddenEl.checked : false;
   var alwaysBlockedEl = document.getElementById('exitAlwaysBlocked');
   exit.always_blocked = alwaysBlockedEl ? alwaysBlockedEl.checked : false;
-  var flagsStr = document.getElementById('exitFlags');
-  var pflagsStr = document.getElementById('exitPFlags');
-  exit.set_flags = parseFlagInput(flagsStr ? flagsStr.value : '');
-  exit.set_player_flags = parseFlagInput(pflagsStr ? pflagsStr.value : '');
 
   var clauseEls = menu.querySelectorAll('.cond-clause');
   clauseEls.forEach(function(el) { updateClauseData(el); });
@@ -1682,10 +1667,6 @@ function clearExitCondition() {
   renderConditionClauses(menu);
   var bmsg = document.getElementById('exitBmsg');
   if (bmsg) bmsg.value = '';
-  var ef = document.getElementById('exitFlags');
-  if (ef) ef.value = '';
-  var pf = document.getElementById('exitPFlags');
-  if (pf) pf.value = '';
   var hx = document.getElementById('exitHidden');
   if (hx) hx.checked = false;
   var ab = document.getElementById('exitAlwaysBlocked');
-- 
cgit v1.2.3