Add :meth:atomx.models.ScheduledReport.save to update name and emails

Author: dakra

CHANGES.txt

@@ -5,11 +5,15 @@
 - remove ``network`` scope in :meth:`atomx.Atomx.report` and add
   ``network_managed``, ``network_buy``, ``network_sell``
 - Add new models:
-    - :class:`atomx.models.AccountManager` (alias for `User`)
-    - :class:`atomx.models.Banned` (alias for `Reason`)
-    - :class:`atomx.models.PlacementType`
-    - :class:`atomx.models.CampaignDebugReason`
-    - :class:`atomx.models.Visibility`
+
+  * :class:`atomx.models.AccountManager` (alias for `User`)
+  * :class:`atomx.models.Banned` (alias for `Reason`)
+  * :class:`atomx.models.PlacementType`
+  * :class:`atomx.models.CampaignDebugReason`
+  * :class:`atomx.models.Visibility`
+
+- Add :meth:`atomx.models.ScheduledReport.save` to edit ``name`` and ``emails``
+  of a :class:`atomx.models.ScheduledReport`
 
 
 1.4

atomx/__init__.py

@@ -236,9 +236,9 @@ def report(self, scope=None, groups=None, metrics=None, where=None, from_=None,
             self.last_response = r_json
 
         if is_scheduled_report:
-            return models.ScheduledReport(self, query=r.json()['query'], **report)
+            return models.ScheduledReport(self, **report)
 
-        return models.Report(self, query=r.json()['query'], **report)
+        return models.Report(self, **report)
 
     def report_status(self, report):
         """Get the status for a `report`.

atomx/models.py

@@ -196,9 +196,11 @@ class ScheduledReport(object):
     <https://wiki.atomx.com/reporting#scheduling_reports>`_.
     """
 
-    def __init__(self, session, id, query, **kwargs):
+    def __init__(self, session, id, name, emails, query, **kwargs):
         self.session = session
         self.id = id
+        self.name = name
+        self.emails = emails
         self.query = query
 
     def __repr__(self):
@@ -207,9 +209,21 @@ def __repr__(self):
     def __eq__(self, other):
         return self.id == getattr(other, 'id', 'INVALID')
 
+    def save(self, session=None):
+        """Update report `name` and `emails`"""
+
+        session = session or self.session
+        if not session:
+            raise NoSessionError
+        return session.put('report', self.id, {'name': self.name, 'emails': self.emails})
+
     def delete(self, session=None):
         """Delete scheduled report"""
-        return self.session.delete('report', self.id)
+
+        session = session or self.session
+        if not session:
+            raise NoSessionError
+        return session.delete('report', self.id)
 
 
 class Report(object):