paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-19 22:19:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

actions: Fix type-incorrect custom profile field value assignment.

Browse Source 
The old assignment is incorrect - field_value.value is a TextField() and
should always be a string. This didn't strictly break anything, because
django converts the value to a string when .save()ing to the db, but
field_value.value persists as a non-string for the rest of this
codepath. After fixing this, the small codeblock in
notify_user_update_custom_profile_data handling conversion of
field_value.value to a string becomes redundant.

We're assured that we're not breaking event format by the test
test_custom_profile_field_data_events in test_events.py.
This commit is contained in:
Mateusz Mandera
2021-09-19 19:49:01 +02:00
committed by Tim Abbott
parent c4edda016f
commit 91ea21a3fc
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
zerver/lib/actions.py
View File

@@ -7727,11 +7727,8 @@ def try_reorder_realm_custom_profile_fields(realm: Realm, order: List[int]) -> N
def notify_user_update_custom_profile_data(
user_profile: UserProfile, field: Dict[str, Union[int, str, List[int], None]]
) -> None:
data = dict(id=field["id"])
if field["type"] == CustomProfileField.USER:
data["value"] = orjson.dumps(field["value"]).decode()
else:
data["value"] = field["value"]
data = dict(id=field["id"], value=field["value"])
if field["rendered_value"]:
data["rendered_value"] = field["rendered_value"]
payload = dict(user_id=user_profile.id, custom_profile_field=data)
@@ -7756,7 +7753,7 @@ def do_update_user_custom_profile_data_if_changed(
# to a string for the comparison in this if.
continue
field_value.value = custom_profile_field["value"]
field_value.value = str(custom_profile_field["value"])
if field_value.field.is_renderable():
field_value.rendered_value = render_stream_description(
str(custom_profile_field["value"])