Paths

Table of Contentst

Diffusion Phabricator 0e8ed0c61684

Desactivate subtask when logged out.
Concern Raised0e8ed0c61684
Actions

Description

Desactivate subtask when logged out.

Summary: Fixes T9592.

Test Plan: Log out ! Navigates to a task. See the add button grey-ed out !

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin

Maniphest Tasks: T9592

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

Details

Auditors

tycho.tatitscheff

Provenance

tycho.tatitscheff	Authored on
epriestley	Committed on Oct 18 2015, 11:22 AM
epriestley	Pushed on Oct 18 2015, 11:22 AM

Reviewer

Blessed Reviewers

Differential Revision

D14299: Desactivate subtask when logged out.

Parents

rP92a626fc1ca3: Add a basic list view for repository operations

Branches

Unknown

Tags

Unknown

Tasks

T9592: "Create subtask" is not greyed out for logged-out users

Build Status

Buildable 8329
Build 9551: Run Core Tests

Event Timeline

epriestley committed rP0e8ed0c61684: Desactivate subtask when logged out. (authored by tycho <tycho.tatitscheff+artemis@gadz.org>).Oct 18 2015, 11:22 AM

epriestley added a task: T9592: "Create subtask" is not greyed out for logged-out users.

Harbormaster completed building B8329: rP0e8ed0c61684: Desactivate subtask when logged out..Oct 18 2015, 11:22 AM

@epriestley : I just see I made a typo I typo this when reindenting !

/src/applications/maniphest/controller/ManiphestTaskDetailController.php
421	I typo this when reindenting ! Should be : ->setIcon('fa-level-down') without the extra `)`
423	and here an extra `)`

vlada mentioned this in T9596: Call to undefined method PhabricatorActionListView::setDisabled().Oct 18 2015, 7:36 PM

Changes (1)

Path

Size

src/

applications/

maniphest/

controller/

ManiphestTaskDetailController.php

rP0e8ed0c61684

View Options

src/applications/maniphest/controller/ManiphestTaskDetailController.php