Skip to content

Commit

Permalink
Merge pull request #14 from gumgum/mediasummary_moduleid
Browse files Browse the repository at this point in the history
Now we update module ids inside media summary
  • Loading branch information
javimol authored May 15, 2019
2 parents 55b7f40 + 8cbfc5d commit d6ff48e
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
7 changes: 6 additions & 1 deletion multivitamin/module/module.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,11 @@ def _update_ids(self):
self._update_property_id(prop)
self._update_module_id(prop)

for video_ann in self.response.media_summary:
for prop in video_ann["props"]:
self._update_property_id(prop)
self._update_module_id(prop)

def _update_property_id(self, prop):
"""Internal method for updating property id in a property
"""
Expand All @@ -133,7 +138,7 @@ def _update_property_id(self, prop):
def _update_module_id(self, prop):
"""Internal method for updating module id in a property
"""
log.debug("Updating module ids")
log.debug("Updating module ids")
if self.module_id_map is None:
return
if prop["module_id"] == 0:
Expand Down
2 changes: 1 addition & 1 deletion multivitamin/version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "1.3.28"
__version__ = "1.3.29"

0 comments on commit d6ff48e

Please sign in to comment.