fix(dav): upload --replace не заменяет при серверной конвертации .xls→.xlsx (дубликаты) #82
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: eSlider/go-onlyoffice#82
Reference in New Issue
Block a user
Проблема
oo dav upload FOLDER LOCAL.xls --replaceзаливает локальный.xls, но OnlyOfficeконвертирует его в
.xlsxпри загрузке.UploadToFolderReplacingдедуплицирует поstem|ext(.xls) черезDeleteFilesByDedupKeyи не находит сохранённый.xlsx,поэтому повторный upload создаёт второй файл.
Проверено live на
My Documents/Banks/Caixa/extractos/:id 3799 + 3887(ES29…) иid 3800 + 3888(ES87…) — по два одинаковых.xlsxна каждыйповторный
caixa publish.Что сделать
--replaceдолжен матчить существующие файлы по stem с учётом сервернойконвертации: legacy ext (
.xls/.doc/.ppt) ↔ OOXML (.xlsx/.docx/.pptx); либо длядокументов дедуплицировать по stem.
(
UpdateFile) вместо delete+upload — id стабилен, нет окна без файла..xlsне плодит дубликаты.Acceptance
oo dav upload <folder> f.xls --replace→ в папке один файл на stem (без дублей);oo dav lsподтверждает один файл; поведение прочих форматов не сломано;go test -race ./...,go vet ./...,gofmt -l .— чисто.Оценка: S
SE: взял в работу. Ветка
fix/dav-replace-convert#82отmain(worktree). Причина:UploadToFolderReplacingматчит поstem|extчерезFindFilesByDedupKey, а OO конвертирует.xls→.xlsxпри upload — совпадения нет. План TDD: (1) offline unit-тесты на conversion-aware матчинг (.xls↔.xlsx) и план замены; (2) при совпадении по stem —UpdateFile(id стабилен) с fallback delete+upload; (3) без регрессий для pdf/xlsx. Live-проверка: в process env нетONLYOFFICE_*— сделаю offline; если поднимутся креды, прогоню на временном проекте.Отчёт SE
Ветка:
fix/dav-replace-convert#82(отmain,74e72d0) → PR #83 #83Причина
OO конвертирует legacy-бинарные форматы при upload (
.xls/.doc/.ppt→.xlsx/.docx/.pptx).UploadToFolderReplacingматчил точно поstem|ext(FindFilesByDedupKey), поэтому повторныйoo dav upload FOLDER f.xls --replaceне находил сохранённыйf.xlsx→ каждый повтор создавал второй файл (live: 3799+3887, 3800+3888).Стратегия дедупа
EquivalentUploadExt/FindFilesByStemExt— матч по stem + equivalence расширений: legacy.xls/.doc/.ppt↔ OOXML.xlsx/.docx/.pptx; для прочих форматов поведение прежнее (pdf,xlsxсовпадают точно).planUploadReplacement— по листингу папки выбирает выживающий файл (самый свежий) и лишние дубликаты.UploadToFolderReplacing:UpdateFileпо существующему id (id стабилен, нет окна удаления) + удаление лишних дублей;UpdateFileне поддержан порталом → fallback на conversion-awareDeleteFilesByStemExt+ upload (один файл на stem в любом случае).AssertNoFileConflict(--no-replace) тоже conversion-aware:.xlsтеперь конфликтует с существующим.xlsx.DeleteFilesByStemExt; старыйDeleteFilesByDedupKeyне тронут (используетсяreplace-in).Тесты
files_replace_test.go(offline, pure):EquivalentUploadExt(симметрия, кейсы),FindFilesByStemExt(конверсия + точный матч, pdf/xlsx не сломаны),planUploadReplacement(fresh/update/collapse), регрессияTestPlanUploadReplacementRepeatedXLS— повторный.xlsдаёт update существующего id, не создание второго.go vet ./...,go test -race -shuffle=on -count=1 ./...,gofmt -l,go mod tidy(без диффа), сборкаcmd/ooиcmd/office— всё чисто.Live
В process env нет
ONLYOFFICE_*(в.envне заглядывал), поэтому live-прогон не выполнялся — только offline-тесты. CI-workflow репо — GitHub Actions, на Gitea прогонов нет (po runпусто); эквивалентные команды прогнал локально.Открытый вопрос / риск
UpdateFileпри кросформатном (конвертируемом) обновлении: нужно live-подтвердить, чтоPUT /files/{id}/updateс payload.xlsконвертирует его в.xlsxв рамках существующего id. Error-fallback покрыт, но молчаливая порча формата — нет. Плюс:UpdateFileкопит версии (см. докстрингreplace-inпро «clean version / shared file id») — для дефолтного upsert это осознанный размен по формулировке issue («предпочтительно UpdateFile, id стабилен»). Прошу подтвердить приоритет при live-проверке.Не мержу — жду ревью PO.