fix(lua): StudioCollab глотал 5-ый параметр language в upsertScript
Корень бага переключателя Lua: StudioCollab.js переопределял scene.upsertScript с 4-мя аргументами (id,code,target,name) и не передавал переданный 5-ый language в оригинал. Поэтому при смене языка в UI language пропадал и переключатель оставался на JS. Исправлено: - Обёртка scene.upsertScript = function (id, code, target, name, language) - Наследие collab-операции с language в sendOp и в _applyRemoteOp Диагностические console.log убраны. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
parent
71d2f2db83
commit
d41051edf9
@ -348,13 +348,19 @@ function ScriptEditor({ value, onSave, onRunSolo, isSoloRunning, scriptId, targe
|
||||
onLanguageChange(lang, nextCode);
|
||||
return;
|
||||
}
|
||||
// Код не пустой — показываем кастомную модалку
|
||||
// Код не пустой — показываем кастомную модалку.
|
||||
// ВАЖНО: lang захвачен через map'a, но localCode и onLanguageChange
|
||||
// надо взять из ref'ов на момент confirm, иначе stale closure.
|
||||
const targetLang = lang;
|
||||
setConfirmState({
|
||||
title: `Сменить язык на ${lang === 'lua' ? 'Lua' : 'JavaScript'}?`,
|
||||
title: `Сменить язык на ${targetLang === 'lua' ? 'Lua' : 'JavaScript'}?`,
|
||||
message: `Код останется как есть — синтаксис прежнего языка перестанет подсвечиваться, но текст не исчезнет. Можно переключиться обратно в любой момент.`,
|
||||
confirmLabel: `Сменить на ${lang === 'lua' ? 'Lua' : 'JS'}`,
|
||||
confirmLabel: `Сменить на ${targetLang === 'lua' ? 'Lua' : 'JS'}`,
|
||||
cancelLabel: 'Отмена',
|
||||
onConfirm: () => onLanguageChange(lang, localCode),
|
||||
onConfirm: () => {
|
||||
// Берём актуальное значение из ref (не stale closure)
|
||||
onLanguageChange(targetLang, localCodeRef.current);
|
||||
},
|
||||
});
|
||||
}}
|
||||
style={{
|
||||
|
||||
@ -170,8 +170,8 @@ export class StudioCollab {
|
||||
sc.__collabScriptsPatched = true;
|
||||
if (typeof sc.upsertScript === 'function') {
|
||||
const origUpsert = sc.upsertScript.bind(sc);
|
||||
sc.upsertScript = function (id, code, target, name) {
|
||||
const r = origUpsert(id, code, target, name);
|
||||
sc.upsertScript = function (id, code, target, name, language) {
|
||||
const r = origUpsert(id, code, target, name, language);
|
||||
if (!self._applyingRemote) {
|
||||
// id может быть сгенерён внутри upsertScript, если был null —
|
||||
// достаём фактический из _scripts (последний с этим code).
|
||||
@ -188,6 +188,7 @@ export class StudioCollab {
|
||||
code: rec.code,
|
||||
target: rec.target ?? null,
|
||||
name: rec.name ?? null,
|
||||
language: rec.language ?? 'js',
|
||||
});
|
||||
}
|
||||
}
|
||||
@ -433,7 +434,7 @@ export function applyRemoteOp(scene, op) {
|
||||
// Создание/редактирование скрипта у соавтора. _applyingRemote уже
|
||||
// выставлен (см. _applyRemoteOp) → обёртка upsertScript не зашлёт
|
||||
// эхо обратно. _onSceneChange внутри обновит React-панели.
|
||||
scene.upsertScript?.(op.id, op.code, op.target ?? null, op.name ?? null);
|
||||
scene.upsertScript?.(op.id, op.code, op.target ?? null, op.name ?? null, op.language ?? undefined);
|
||||
scene._onCollabScriptsChange?.();
|
||||
return;
|
||||
case 'scriptRemove':
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user