Merge branch 'oplog_skipped_even_if_OPLOG_is_True_#1095'

AUTHORS

@@ -64,6 +64,7 @@ Patches and Contributions
 - Henrique Barroso
 - Huan Di
 - Hugo Larcher
+- Hung Le
 - James Stewart
 - Jaroslav Semančík
 - Javier Gonel

CHANGES

@@ -8,6 +8,7 @@ Development
 
 Version 0.8
 ~~~~~~~~~~~
+- Fix: OPLOG skipped even if ``OPLOG = True``. Closes 1074 (Hung Le).
 - Fix: Cannot define default projection and request specific field. Closes
   #1036 (DHuan).
 - Dev: pin testfixtures to v5.x as latest releases break on Python 2.6.

eve/methods/common.py

@@ -1234,7 +1234,7 @@ def oplog_push(resource, document, op, id=None):
         'r' = resource endpoint,
         'o' = operation performed,
         'i' = unique id of the document involved,
-        'pi' = client IP,
+        'ip' = client IP,
         'c' = changes
 
     config.LAST_UPDATED, config.LAST_CREATED and AUTH_FIELD are not being
@@ -1258,9 +1258,10 @@ def oplog_push(resource, document, op, id=None):
 
     .. versionadded:: 0.5
     """
+
     if not config.OPLOG \
             or op not in config.OPLOG_METHODS\
-            or resource in config.URLS[resource]:
+            or resource not in config.URLS:
         return
 
     resource_def = config.DOMAIN[resource]

eve/tests/methods/common.py

@@ -653,10 +653,12 @@ def oplog_callback(resource, entries):
         self.assertTrue('customvalue' in oplog_entry['extra']['customfield'])
 
     def test_post_oplog(self):
-        r = self.test_client.post(self.known_resource_url,
-                                  data=json.dumps(self.data),
-                                  headers=self.headers,
-                                  environ_base={'REMOTE_ADDR': '127.0.0.1'})
+        r = self.test_client.post(
+            self.different_resource_url,
+            data=json.dumps({'username': 'test', 'ref':
+                             '1234567890123456789012345' }),
+            headers=self.headers, environ_base={'REMOTE_ADDR': '127.0.0.1'})
+
         r, status = self.oplog_get()
         self.assert200(status)
         self.assertEqual(len(r['_items']), 1)