Good point. Will change.


On Tue, Feb 25, 2014 at 5:42 PM, marcel-blonk <notifications@github.com>wrote:

> In
> src/main/java/org/sagebionetworks/bridge/webapp/servlet/BridgeRequest.java:
>
> >  		}
> >  		return false;
> >  	}
> > +
> > +	private boolean isBridgeAdmin() {
> > +		if (!isUserAuthenticated()) {
> > +			return false;
> > +		}
> > +		if (getBridgeUser().isBridgeAdmin() != null) {
> > +			return getBridgeUser().isBridgeAdmin();
> > +		}
> > +		try {
> > +			SynapseClient client = getBridgeUser().getSynapseClient();
> > +			Team team = client.getTeam(BOOTSTRAP_TEAM.BRIDGE_ADMINISTRATORS.getId());
>
> The id used in getTeamMembershipStatus is the same as the bootstrap id. No
> need for the getTeam() here.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/Sage-Bionetworks/Bridge/pull/93/files#r10065242>
> .
>