Page MenuHomePhabricator

[lib] add local id field to existing reports

Authored by kamil on Mon, May 22, 6:41 AM.



This is probably not necessary as I don't think that anyone has some persisted, not sent reports, but adding this for clarity.

Depends on D7910

Test Plan

Make sure that there are some reports in store (e.g. throw in keyserver endpoint and create some inconsistencies). Then verify that id was added.

Diff Detail

rCOMM Comm
Lint Not Applicable
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
30–37 ↗(On Diff #26779)

this function will be re-used in next diff

kamil published this revision for review.Tue, May 23, 1:37 AM
tomek added inline comments.
35 ↗(On Diff #26779)

Should we do this only for reports which don't have the id? I guess in the migration it doesn't matter, but maybe for other usages of this function will?

This revision is now accepted and ready to land.Tue, May 23, 3:14 AM

move code directly to migration

35 ↗(On Diff #26779)

it turned out that this was a bad assumption - this function will not be used in more places.

Also, because id will be mandatory function argument type is not proper - I'll put this code directly into migration.