Page MenuHomePhabricator
Differential D16237

Make repository URI creation work regardless of "repository" transaction order
ClosedPublic
Actions

Authored by epriestley on Jul 5 2016, 11:40 PM.
Tags
None
Referenced Files
F19869154: D16237.id39057.diff
Sun, Mar 15, 5:13 AM
F19825888: D16237.diff
Sun, Mar 8, 3:39 PM
F19819344: D16237.diff
Fri, Mar 6, 5:36 AM
F19819342: D16237.diff
Fri, Mar 6, 5:36 AM
F19655319: D16237.id39057.diff
Feb 6 2026, 5:09 PM
F19358935: D16237.id39055.diff
Dec 26 2025, 8:29 AM
F18869252: D16237.id39055.diff
Nov 4 2025, 10:41 AM
F18868467: D16237.id.diff
Nov 4 2025, 6:48 AM
View All Files
Subscribers
None

Details

Reviewers
avivey
chad
Maniphest Tasks
T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input
Commits
rP921d56efb0f0: Make repository URI creation work regardless of "repository" transaction order
Summary

Fixes T11276. This feels slightly iffy (we attachRepository() here, and also when applying the TYPE_REPOSITORY transaction) but simpler than trying to reorder things.

Test Plan

Created a repository URI with transactions in ["uri", "repository"] order.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 39055.Jul 5 2016, 11:40 PM
epriestley retitled this revision from to Make repository URI creation work regardless of "repository" transaction order.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, avivey.
epriestley added a task: T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.
epriestley added inline comments.Jul 5 2016, 11:41 PM
src/applications/diffusion/editor/DiffusionURIEditor.php
126

When processing in the other order, we run this part first so the code works.

avivey accepted this revision.Jul 5 2016, 11:44 PM
avivey edited edge metadata.

probably fine

This revision is now accepted and ready to land.Jul 5 2016, 11:44 PM
Closed by commit rP921d56efb0f0: Make repository URI creation work regardless of "repository" transaction order (authored by epriestley, committed by epriestley). · Explain WhyJul 5 2016, 11:45 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in rPa5b26104f6eb: Fix an issue with creating new Repository URIs via the Web UI.Jul 9 2016, 12:56 PM
epriestley mentioned this in rP46ceba728fee: (stable) Fix an issue with creating new Repository URIs via the Web UI.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
editor/
5 lines

Diff 39057

View Options

src/applications/diffusion/editor/DiffusionURIEditor.php

Loading...
Phabricator · Built by Phacility