In Phriction when I have two documents, A and B, and I perform the following steps:
- Delete or move B
- Attempt to move A to B
I get the following exception:
Description
Description
Revisions and Commits
Revisions and Commits
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | epriestley | T9379 Build Phriction v3 | ||
| Resolved | epriestley | T8082 Exception when moving Phriction document into place of old one |
Event Timeline
Comment Actions
This:
https://secure.phabricator.com/differential/diff/30637/
...fixes the immediate issue, but then we hit something deeper down:
...so this isn't trivial to resolve correctly.
Comment Actions
I think fixing this is basically removing the unique constraint on the "slugs" table in Phriction, and then using application logic to make sure that there can be only one active page with a given slug at any time.
- remove unique constraint on "slugs" table in Phriction
- double check that create / move don't let you clobber any non-deleted, non-stub pages
- change loading logic to prefer active documents first, next by most recently updated
- change history page to show the active document first, next by most recently updated
- e.g. "page active" version history followed by "page deleted that was here" version history
- make sure "revert" workflow works right in these delete + move scenarios
Sound right-ish?