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

	F14059358: D11229.diff
	Sun, Nov 17, 7:16 PM

	F14048361: D11229.id26963.diff
	Thu, Nov 14, 7:37 AM

	F14046083: D11229.diff
	Wed, Nov 13, 6:02 PM

	F14045626: D11229.id26963.diff
	Wed, Nov 13, 7:35 AM

	F14045513: D11229.id26963.diff
	Wed, Nov 13, 5:28 AM

	F14040854: D11229.id.diff
	Mon, Nov 11, 1:48 PM

	F14038923: D11229.id26963.diff
	Mon, Nov 11, 2:49 AM

	F14038842: D11229.id26959.diff
	Mon, Nov 11, 2:25 AM

Subscribers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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
231–242	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.

...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

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

Path

Size

src/

applications/

project/

query/

PhabricatorProjectSearchEngine.php

17 lines

Diff 26963

src/applications/project/query/PhabricatorProjectSearchEngine.php

Loading...