Accept subreddit objects in multi add_srs/del_srs.

This commit is contained in:
Max Goodman
2013-05-02 19:45:58 -07:00
parent cf70e146c1
commit 7a7f1b9311
2 changed files with 8 additions and 5 deletions

View File

@@ -140,7 +140,7 @@ class MultiApiController(RedditController, OAuth2ResourceController):
"""Add a subreddit to a multi."""
try:
multi.add_srs({sr._id: {}})
multi.add_srs({sr: {}})
except TooManySubredditsException as e:
raise RedditError('MULTI_TOO_MANY_SUBREDDITS', code=409)
else:
@@ -156,5 +156,5 @@ class MultiApiController(RedditController, OAuth2ResourceController):
)
def DELETE_multi_subreddit(self, multi, sr):
"""Remove a subreddit from a multi."""
multi.del_srs(sr._id)
multi.del_srs(sr)
multi._commit()

View File

@@ -1416,9 +1416,12 @@ class LabeledMulti(tdb_cassandra.Thing, MultiReddit):
@classmethod
def sr_props_to_columns(cls, sr_props):
sr_columns = {cls.SR_PREFIX + str(sr_id): json.dumps(props)
for sr_id, props in sr_props.iteritems()}
return sr_columns
columns = {}
for sr_id, props in sr_props.iteritems():
if isinstance(sr_id, Subreddit):
sr_id = sr_id._id
columns[cls.SR_PREFIX + str(sr_id)] = json.dumps(props)
return columns
@classmethod
def columns_to_sr_props(cls, columns):