Files
phabricator/resources/sql/patches
epriestley 9125095587 Distinguish between empty and unparsed commits in Diffusion
Summary:
Fixes T3416. Fixes T1733.

  - Adds a flag to the commit table showing whether or not we have parsed it.
  - The flag is set to `0` initially when the commit is discovered.
  - The flag is set to `1` when the changes are parsed.
  - The UI can now use the flag to distinguish between "empty commit" and "commit which we haven't imported changes for yet".
  - Simplify rendering code a little bit.
  - Fix an issue with the Message parser for empty commits.
  - There's a key on the flag so we can do `SELECT * FROM repository_commit WHERE repositoryID = %d AND importStatus = 0 LIMIT 1` soon, to determine if a repository is fully imported or not. This will let us improve the UI (Ref T776, Ref T3217).

Test Plan:
  - Ran `bin/storage upgrade -f`.
  - Created an empty commit.
  - Without the daemons running, ran `bin/repository pull GTEST` and `bin/repository discover GTEST`.
  - Viewed web UI to get the first screenshot ("Still Importing...").
  - Ran the message and change steps with `scripts/repository/reparse.php`.
  - Viewed web UI to get the second screenshot ("Empty Commit").

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T776, T1733, T3416, T3217

Differential Revision: https://secure.phabricator.com/D7428
2013-10-29 15:32:41 -07:00
..
2012-04-30 07:53:53 -07:00
2012-04-30 07:53:53 -07:00
2012-04-30 07:53:53 -07:00
2012-04-30 07:53:53 -07:00
2013-01-24 17:23:05 -08:00
2013-02-15 07:47:14 -08:00
2013-03-20 18:01:52 -07:00
2013-03-28 09:10:34 -07:00
2013-05-03 15:53:36 -07:00
2013-05-03 15:53:36 -07:00
2013-05-03 15:53:36 -07:00
2013-06-28 13:56:16 -07:00
2013-10-22 15:01:06 -07:00
2012-08-01 17:06:04 -07:00
2012-08-16 14:43:03 -07:00
2012-05-03 09:22:52 -07:00
2012-09-13 10:11:14 -07:00
2012-07-19 09:03:10 -07:00
2012-09-11 12:13:20 -07:00
2012-11-05 13:21:24 -08:00
2013-03-15 11:28:43 +00:00
2012-05-03 18:24:30 -07:00
2012-10-08 16:09:20 -07:00