diff --git a/models/packages/package_file.go b/models/packages/package_file.go index 77cd163b23..69401eee3e 100644 --- a/models/packages/package_file.go +++ b/models/packages/package_file.go @@ -125,7 +125,6 @@ func DeleteFilesByPackageID(ctx context.Context, packageID int64) error { // DeleteFilesByVersionID deletes all files of a specific version func DeleteFilesByVersionID(ctx context.Context, versionID int64) error { - _, err := db.GetEngine(ctx).Where("version_id = ?", versionID).Delete(&PackageFile{}) return err } diff --git a/routers/web/user/package.go b/routers/web/user/package.go index f59a25d352..5eeb989899 100644 --- a/routers/web/user/package.go +++ b/routers/web/user/package.go @@ -531,7 +531,7 @@ func PackageVersionDelete(ctx *context.Context) { func DownloadPackageFile(ctx *context.Context) { pf, err := packages_model.GetFileForVersionByID(ctx, ctx.Package.Descriptor.Version.ID, ctx.PathParamInt64("fileid")) if err != nil { - if err == packages_model.ErrPackageFileNotExist { + if errors.Is(err, packages_model.ErrPackageFileNotExist) { ctx.NotFound(err) } else { ctx.ServerError("GetFileForVersionByID", err)