Differential D10012

Allow authors to edit their tasks
Needs RevisionPublic
Actions

Authored by mikaye on Jul 22 2014, 12:30 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Summary

Currently, only owners of tasks can edit them by default. It also make sense if author could also edit it...

Test Plan

Set up a new Phabricator server, created 2 users, one of the users owned task from the other one, author can edit its task as expected.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 1831
Build 1832: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

mikaye updated this revision to Diff 24063.Jul 22 2014, 12:30 PM

mikaye retitled this revision from to Allow authors to edit their tasks.

mikaye updated this object.

mikaye edited the test plan for this revision. (Show Details)

mikaye set the repository for this revision to rP Phabricator.

mikaye added a project: Maniphest.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJul 22 2014, 12:30 PM

Herald added a subscriber: epriestley. · View Herald Transcript

Harbormaster completed remote builds in B1831: Diff 24063.Jul 22 2014, 12:32 PM

We don't want to give authors permanent authority over tasks. For example, on this install, authors may be random members of the public. If they report a security issue, we might want to lock the task down and prevent them from opening it back up.

This revision now requires changes to proceed.Jul 22 2014, 1:28 PM

See T5681 for some more discussion and another approach to this.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

storage/

ManiphestTask.php

12 lines

Diff 24063

View Options

src/applications/maniphest/storage/ManiphestTask.php

Allow authors to edit their tasksNeeds RevisionPublicActions