Change lots of keys to IDs for length.
This commit is contained in:
2
api.py
2
api.py
@@ -225,7 +225,7 @@ class APIWrapper(webapp2.RequestHandler):
|
|||||||
@session.session_required
|
@session.session_required
|
||||||
def post(self):
|
def post(self):
|
||||||
profile_str = str(
|
profile_str = str(
|
||||||
models.Client.profile.get_value_for_datastore(self.client))
|
models.Client.profile.get_value_for_datastore(self.client).id())
|
||||||
ret = {
|
ret = {
|
||||||
'status': 'ok',
|
'status': 'ok',
|
||||||
'profile': profile_str,
|
'profile': profile_str,
|
||||||
|
|||||||
@@ -145,7 +145,7 @@ class Subject(db.Model):
|
|||||||
elif subject['readable_only_by'] == 'me':
|
elif subject['readable_only_by'] == 'me':
|
||||||
readable_only_by = Client.profile.get_value_for_datastore(client)
|
readable_only_by = Client.profile.get_value_for_datastore(client)
|
||||||
else:
|
else:
|
||||||
readable_only_by = db.Key(subject['readable_only_by'])
|
readable_only_by = db.Key.from_path('Profile', int(subject['readable_only_by']))
|
||||||
else:
|
else:
|
||||||
readable_only_by = None
|
readable_only_by = None
|
||||||
|
|
||||||
@@ -155,7 +155,7 @@ class Subject(db.Model):
|
|||||||
elif subject['writable_only_by'] == 'me':
|
elif subject['writable_only_by'] == 'me':
|
||||||
writable_only_by = Client.profile.get_value_for_datastore(client)
|
writable_only_by = Client.profile.get_value_for_datastore(client)
|
||||||
else:
|
else:
|
||||||
writable_only_by = db.Key(subject['writable_only_by'])
|
writable_only_by = db.Key.from_path('Profile', int(subject['writable_only_by']))
|
||||||
else:
|
else:
|
||||||
writable_only_by = None
|
writable_only_by = None
|
||||||
|
|
||||||
@@ -349,13 +349,13 @@ class Subject(db.Model):
|
|||||||
if readable_only_by == Profile.ADMIN_KEY:
|
if readable_only_by == Profile.ADMIN_KEY:
|
||||||
ret['readable_only_by'] = 'admin'
|
ret['readable_only_by'] = 'admin'
|
||||||
else:
|
else:
|
||||||
ret['readable_only_by'] = str(readable_only_by)
|
ret['readable_only_by'] = str(readable_only_by.id())
|
||||||
writable_only_by = Subject.writable_only_by.get_value_for_datastore(self)
|
writable_only_by = Subject.writable_only_by.get_value_for_datastore(self)
|
||||||
if writable_only_by:
|
if writable_only_by:
|
||||||
if writable_only_by == Profile.ADMIN_KEY:
|
if writable_only_by == Profile.ADMIN_KEY:
|
||||||
ret['writable_only_by'] = 'admin'
|
ret['writable_only_by'] = 'admin'
|
||||||
else:
|
else:
|
||||||
ret['writable_only_by'] = str(writable_only_by)
|
ret['writable_only_by'] = str(writable_only_by.id())
|
||||||
return ret
|
return ret
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
@@ -480,7 +480,7 @@ class Message(db.Model):
|
|||||||
return {
|
return {
|
||||||
'event_type': 'message',
|
'event_type': 'message',
|
||||||
'id': self.id_,
|
'id': self.id_,
|
||||||
'sender': str(Message.sender.get_value_for_datastore(self)),
|
'sender': str(Message.sender.get_value_for_datastore(self).id()),
|
||||||
'subject': self.parent().ToDict(),
|
'subject': self.parent().ToDict(),
|
||||||
'created': self.created,
|
'created': self.created,
|
||||||
'sender_message_id': self.sender_message_id,
|
'sender_message_id': self.sender_message_id,
|
||||||
@@ -502,8 +502,8 @@ class Pin(db.Model):
|
|||||||
def ToEvent(self, event_type='pin'):
|
def ToEvent(self, event_type='pin'):
|
||||||
return {
|
return {
|
||||||
'event_type': event_type,
|
'event_type': event_type,
|
||||||
'id': str(self.key()),
|
'id': str(self.key().id()),
|
||||||
'sender': str(Pin.sender.get_value_for_datastore(self)),
|
'sender': str(Pin.sender.get_value_for_datastore(self).id()),
|
||||||
'subject': self.parent().ToDict(),
|
'subject': self.parent().ToDict(),
|
||||||
'created': self.created,
|
'created': self.created,
|
||||||
'sender_message_id': self.sender_message_id,
|
'sender_message_id': self.sender_message_id,
|
||||||
|
|||||||
@@ -71,7 +71,7 @@ def session_required(handler):
|
|||||||
self.request_json['client_id'], self.verified_google_user)
|
self.request_json['client_id'], self.verified_google_user)
|
||||||
logging.info('Client: %s', self.client.key().name())
|
logging.info('Client: %s', self.client.key().name())
|
||||||
logging.info('Profile: %s',
|
logging.info('Profile: %s',
|
||||||
models.Client.profile.get_value_for_datastore(self.client))
|
models.Client.profile.get_value_for_datastore(self.client).id())
|
||||||
if old_profile:
|
if old_profile:
|
||||||
self.client.profile.MergeFrom(old_profile)
|
self.client.profile.MergeFrom(old_profile)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user