Page MenuHomePhabricator
Create Task
Maniphest T7127

Saving new path list when deletes occur is erroneous at best
Closed, ResolvedPublic
Actions

Description

To reproduce:

  • create a pack with 3 paths "x", "y", "z"
  • edit the package by deleting the "y", then adding it back (so its now the 3rd path and you have "x", "z", "y"), then save
  • BUG - observe path list of just "x", "z", EXPECT "x", "z", "y" or at least all three paths in the list

Did some light debugging while building D11652 and I believe this is a problem in the javascript layer - the server ends up getting back "x", "z" as the set of paths. (It might even be an issue with the core multi row manager...)

Revisions and Commits

Restricted Differential RevisionRestricted Diffusion Commit

Related Objects
Search...

Event Timeline

btrahan created this task.Feb 3 2015, 7:35 PM
btrahan raised the priority of this task from to Low.
btrahan updated the task description. (Show Details)
btrahan added a project: Owners.
btrahan added a subscriber: btrahan.
btrahan updated the task description. (Show Details)Feb 3 2015, 7:38 PM
epriestley moved this task from Backlog to Dark Ages on the Owners board.May 26 2015, 7:11 PM
epriestley added a parent task: T8320: Bring Owners out of the dark ages.
epriestley added a revision: Restricted Differential Revision.May 27 2015, 1:29 AM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.May 27 2015, 5:30 PM
epriestley added a commit: Restricted Diffusion Commit.
Phabricator · Built by Phacility