Merge lp:~elachuni/ubuntu-recommender/client-uuid-recommend-me into lp:~canonical-ca-hackers/ubuntu-recommender/ubuntu-recommender-client

[Anthony Lenton]

One way of providing a version of recommend_me that only requires a uuid. I could think of two other alternatives for this:
 - A single method with an "authenticated" bool argument, but if you (the caller) would need to have instantiated the client with an OAuth authorizer already (or set it as an instance variable just before calling the method)
 - A single method that receives an optional "authorizer" argument, so that you could pass in an OAuth authorizer when you go to make the call. I think this would be nicer, but completely inconsistent with other client code out there (and with other authenticated methods on this same class).

The current approach has the downside that it provides two different methods for what translates to a single API call, but I think we'd need to have two call sites already. I imagine the client code will be something like:

    if self.has_a_token:
        authorizer = OAuthAuthorizer(token, secret, bitties, ...)
        api = SoftwareCenterRecommenderAPI(auth=authorizer)
        recommendations = api.recommend_me(self.my_uuid)
    else:
        api = SoftwareCenterRecommenderAPI()
        recommendations = api.anon_recommend_me(self.my_uuid)

Let me know if you think some other alternative would be better!

[Łukasz Czyżykowski (lukasz-czyzykowski)]

LGTM

review: Approve

=== modified file 'sreclient.py'
--- sreclient.py	2012-02-14 13:15:07 +0000
+++ sreclient.py	2012-03-28 20:05:22 +0000
@@ -40,8 +40,15 @@
             scheme=PUBLIC_API_SCHEME)
 
     @oauth_protected
-    def recommend_me(self):
-        return self._get('recommend_me/', scheme=AUTHENTICATED_API_SCHEME)
+    @validate_pattern('uuid', r'[\dabcdef]{32}')
+    def recommend_me(self, uuid):
+        return self._get('recommend_me/%s/' % uuid,
+            scheme=AUTHENTICATED_API_SCHEME)
+
+    @validate_pattern('uuid', r'[\dabcdef]{32}')
+    def anon_recommend_me(self, uuid):
+        return self._get('recommend_me/%s/' % uuid,
+            scheme=AUTHENTICATED_API_SCHEME)
 
     @validate_pattern('pkgname', '[^/]+')
     @returns_json