Homec4science

When users choose a default project icon, make a permanent file

Authored by epriestley <git@epriestley.com> on Jul 11 2016, 17:20.

Description

When users choose a default project icon, make a permanent file

Summary:
Fixes T10907. As written, this workflow will incorrectly reuse a temporary file if one exists.

Instead, make a new permanent file.

(Storage is still shared, so this usually will not actually create a copy of the file's data.)

Test Plan:

  • Set a project's icon by clicking first button in "Use Picture" row.
  • Before patch: temporary image was reused.
  • After patch: new permanent file is generated.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10907

Differential Revision: https://secure.phabricator.com/D16270

Details

Committed
epriestley <git@epriestley.com>Jul 11 2016, 18:24
Pushed
aubortJan 31 2017, 17:16
Parents
rPH553c335fbdf8: Ignore unreachable commits when testing if a repository has imported
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH830f3eb8f803: When users choose a default project icon, make a permanent file (authored by epriestley <git@epriestley.com>).Jul 11 2016, 18:24