From 48052f8a701f6ebe374f240d272059c6ed073d89 Mon Sep 17 00:00:00 2001
From: Mario de Frutos <mario.defrutos@cartodb.com>
Date: Fri, 6 Nov 2015 17:49:11 +0100
Subject: [PATCH] Increment every time a row is georeferenced successfully

---
 .../cartodb_geocoder/quota_service.py         | 26 ++++++++++++-------
 1 file changed, 17 insertions(+), 9 deletions(-)

diff --git a/server/lib/python/cartodb_geocoder/cartodb_geocoder/quota_service.py b/server/lib/python/cartodb_geocoder/cartodb_geocoder/quota_service.py
index 070b4bd..dab3f2f 100644
--- a/server/lib/python/cartodb_geocoder/cartodb_geocoder/quota_service.py
+++ b/server/lib/python/cartodb_geocoder/cartodb_geocoder/quota_service.py
@@ -6,35 +6,43 @@ class QuotaService:
 
     GEOCODING_QUOTA_KEY = "geocoding_quota"
 
-    def __init__(self, logger, user_id, transaction_id):
+    def __init__(self, logger, user_id, transaction_id, redis_host='localhost', redis_port=6379, redis_db = 5):
         self.logger = logger
         self.user_id = user_id
         self.transaction_id = transaction_id
+        self.redis_host = redis_host
+        self.redis_port = redis_port
+        self.redis_db = redis_db
         self.redis_conn = self.__get_redis_connection()
 
     def check_user_quota(self):
-        """ Get the user quota and add it to redis in order to cache it """
-        # TODO: Check if the redis key geocoder::user_id::tx_id exists:
-        #       a) If exists check the quota
+        """ Check if the current user quota surpasses the current quota """
+        # TODO We need to add the hard/soft limit flag for the geocoder
         user_quota = self.get_user_quota()
         current_used = self.get_current_used_quota()
+        self.logger.debug("User quota: {0} --- Current used quota: {1}".format(user_quota, current_used))
         return True if (current_used + 1) < user_quota else False
 
     def get_user_quota(self):
-        return self.redis_conn.hget(self.__get_user_redis_key(), self.GEOCODING_QUOTA_KEY)
+        # Check for exceptions or redis timeout
+        user_quota = self.redis_conn.hget(self.__get_user_redis_key(), self.GEOCODING_QUOTA_KEY)
+        return int(user_quota)
 
     def get_current_used_quota(self):
-        # TODO: Check if exist geocoder:user_id:year_month , if yes sum up all of it
+        """ Recover the used quota for the user in the current month """
+        # Check for exceptions or redis timeout
         current_used = 0
         for _, value in self.redis_conn.hscan_iter(self.__get_month_redis_key()):
             current_used += int(value)
         return current_used
 
-    def increment_georeference_use(self):
-        pass
+    def increment_georeference_use(self, amount=1):
+        """ Increment the geocoder use in 1 """
+        # TODO Manage exceptions or timeout
+        self.redis_conn.hincrby(self.__get_month_redis_key(), self.transaction_id,amount)
 
     def __get_redis_connection(self):
-        pool = redis.ConnectionPool(host='localhost', port=6379, db=5)
+        pool = redis.ConnectionPool(host=self.redis_host, port=self.redis_port, db=self.redis_db)
         return redis.Redis(connection_pool=pool)
 
     def __get_month_redis_key(self):