http://gwt-code-reviews.appspot.com/1427801/diff/1/user/src/com/google/web/bindery/requestfactory/server/LocatorServiceLayer.java
File
user/src/com/google/web/bindery/requestfactory/server/LocatorServiceLayer.java
(right):

http://gwt-code-reviews.appspot.com/1427801/diff/1/user/src/com/google/web/bindery/requestfactory/server/LocatorServiceLayer.java#newcode55
user/src/com/google/web/bindery/requestfactory/server/LocatorServiceLayer.java:55:
ServiceLocator locator = createServiceLocator(locatorType);
This should be getTop().createServiceLocator(...) or you won't be able
to "override" it in another ServiceLayerDecorator.

Maybe add a test to prove it's working as intended? (but beware of the
ServiceLayerCache in unit-tests! don't re-use existing services or the
cache will mess up with your test every once in a while)

http://gwt-code-reviews.appspot.com/1427801/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to