Page MenuHomePhabricator
Differential D11229

Projects - display "members" link in disabled style if user can't edit project
ClosedPublic
Actions

Authored by btrahan on Jan 5 2015, 11:24 PM.
Tags
None
Referenced Files
F13819848: D11229.diff
Fri, Sep 20, 4:44 PM
Unknown Object (File)
Tue, Sep 3, 8:59 PM
Unknown Object (File)
Sat, Aug 31, 1:57 PM
Unknown Object (File)
Sat, Aug 31, 3:42 AM
Unknown Object (File)
Wed, Aug 28, 1:07 AM
Unknown Object (File)
Mon, Aug 26, 1:29 PM
Unknown Object (File)
Sun, Aug 25, 12:45 PM
Unknown Object (File)
Aug 21 2024, 6:58 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Maniphest Tasks
T6862: Grey out 'Edit Members' link if User does not have Can Edit on project
Commits
Restricted Diffusion Commit
rPad1da6ec5ed3: Projects - display "members" link in disabled style if user can't edit project
Summary

Fixes T6862.

Test Plan

viewed a project list and saw disabled-style "Members" links as appropos

Diff Detail

Repository
rP Phabricator
Branch
T6862
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3599
Build 3607: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 26959.Jan 5 2015, 11:24 PM
btrahan added a task: T6862: Grey out 'Edit Members' link if User does not have Can Edit on project.
btrahan retitled this revision from to Projects - display "members" link in disabled style if user can't edit project.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added reviewers: chad, epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 5 2015, 11:24 PM
btrahan added inline comments.Jan 5 2015, 11:25 PM
src/applications/project/query/PhabricatorProjectSearchEngine.php
226–237

the other thing I could do here is make "workflow" true in this case, which would make this a nice dialogue and not a full, page re-load 404 nice dialogue.

btrahan updated this revision to Diff 26960.Jan 5 2015, 11:29 PM

...do the conditional workflow; seems like a more-standard / better behavior.

chad accepted this revision.Jan 5 2015, 11:32 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 5 2015, 11:32 PM
btrahan updated this revision to Diff 26962.Jan 5 2015, 11:39 PM
btrahan edited edge metadata.

move data loading to before the foreach loop (re-run tests and still good) this change alleviates performance concern

btrahan updated this object.Jan 5 2015, 11:39 PM
Closed by commit rPad1da6ec5ed3: Projects - display "members" link in disabled style if user can't edit project (authored by btrahan, committed by btrahan). · Explain WhyJan 5 2015, 11:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
project/
query/
8 lines

Diff 26959

View Options

src/applications/project/query/PhabricatorProjectSearchEngine.php

Loading...
Phabricator · Built by Phacility