Skip to content

Hide tab-panels if project is on whitelist - #59

Open
sithera wants to merge 1 commit into
MeetMe:masterfrom
sithera:master
Open

Hide tab-panels if project is on whitelist#59
sithera wants to merge 1 commit into
MeetMe:masterfrom
sithera:master

Conversation

@sithera

@sithera sithera commented Jan 22, 2018

Copy link
Copy Markdown

Fixes #58

@gawlik

gawlik commented Jan 22, 2018

Copy link
Copy Markdown

Yes, it would be great to have it in the official release!

}

if (configuration.getUseGerritProjectWhitelist() && ! isGerritProject(arg0.issue()))
isShowing = false;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please follow surrounding code styles: braces around multi-line if blocks, and no space after !

        if (configuration.getUseGerritProjectWhitelist() && !isGerritProject(arg0.issue())) {
            isShowing = false;
        }

Collection<Issue> subtasks = request.issue().getSubTaskObjects();
show = subtasks != null && subtasks.size() > 0;

if (configuration.getUseGerritProjectWhitelist() && ! isGerritProject(request.issue()))

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

}

private boolean isGerritProject(final Issue issue) {
if (issue.getProjectId() == null)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here

}

private boolean isGerritProject(final Issue issue) {
if (issue.getProjectId() == null)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tab-panels are not hidden if project is whitelisted

3 participants