-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4698/
-----------------------------------------------------------
Review request for shindig, Stanton Sievers and Dan Dumont.
Summary
-------
AbstractLockedDomainService has both a protected enabled variable as well as a
public isEnabled method. Both of which can be used to determine whether locked
domains is enabled. However the methods in AbstractLockedDomainService and
HashLockedDomainService all use the protected enabled variable so if someone
who extends AbstractLockedDomainService or HashLockedDomainService and
overrides isEnabled it won't actually be honored or used by any of the methods.
This addresses bug SHINDIG-1747.
https://issues.apache.org/jira/browse/SHINDIG-1747
Diffs
-----
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/AbstractLockedDomainService.java
1324741
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/HashLockedDomainService.java
1324741
Diff: https://reviews.apache.org/r/4698/diff
Testing
-------
Unit tests passed
Thanks,
Ryan
