[commit-keeper sha=00b12cd] Игнорирование ошибок обновления последней версии #7

New Issue

commit-keeper · 2026-04-27T09:18:08Z

commit-keeper commented

2026-04-27 09:18:08 +00:00

Severity: major
Commit: 00b12cd — upd store
Затронутые места:

service/version.go строки 113–113

В чём проблема

Игнорирование ошибки возвращаемой методом UpdateLatestVersion. В нескольких местах кода (строки 66, 85, 113) результат вызова этого метода просто отбрасывается с помощью оператора _. Это может привести к ситуации, когда версия создана, но документ не обновлен, и пользователь не заметит ошибки записи в БД.

Цитата кода

_ = s.storage.UpdateLatestVersion(docID, newVer.ID)

Как исправить

Вариант 1 — Обрабатывать ошибку и возвращать её вызывающему коду.

В Go принято обрабатывать все ошибки, чтобы избежать непредсказуемого поведения системы.

if err := s.storage.UpdateLatestVersion(docID, newVer.ID); err != nil {
		return nil, err
	}

_{^{Автоматическая проверка commit-keeper · модель gemma4:31b · sha=00b12cd}}

**Severity:** major **Commit:** [`00b12cd`](https://git.gm6.ru/icewind/seadoc/commit/00b12cdf7dbee7b9d54a1075acfdc0666ff6aa30) — upd store **Затронутые места:** - `service/version.go` строки 113–113 ## В чём проблема Игнорирование ошибки возвращаемой методом `UpdateLatestVersion`. В нескольких местах кода (строки 66, 85, 113) результат вызова этого метода просто отбрасывается с помощью оператора `_`. Это может привести к ситуации, когда версия создана, но документ не обновлен, и пользователь не заметит ошибки записи в БД. ## Цитата кода ``` _ = s.storage.UpdateLatestVersion(docID, newVer.ID) ``` ## Как исправить ### Вариант 1 — Обрабатывать ошибку и возвращать её вызывающему коду. В Go принято обрабатывать все ошибки, чтобы избежать непредсказуемого поведения системы. ``` if err := s.storage.UpdateLatestVersion(docID, newVer.ID); err != nil { return nil, err } ``` --- <sub><sup>Автоматическая проверка commit-keeper · модель `gemma4:31b` · `sha=00b12cd`</sup></sub>

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: icewind/seadoc#7