Revert "Fix: prune invalid custom fields (#9224)"

This reverts commit 61cb5103ed96dbe6a4f44129f92616b4c0d99b45.
This commit is contained in:
shamoon 2025-02-25 23:34:29 -08:00
parent 61cb5103ed
commit 4da973f6a5
No known key found for this signature in database
2 changed files with 5 additions and 6 deletions

View File

@ -1136,8 +1136,9 @@ class SavedViewSerializer(OwnedObjectSerializer):
): # i.e. check for 'custom_field_' prefix
field_id = int(re.search(r"\d+", field)[0])
if not CustomField.objects.filter(id=field_id).exists():
# In case the field was deleted, just remove from the list
attrs["display_fields"].remove(field)
raise serializers.ValidationError(
f"Invalid field: {field}",
)
elif field not in SavedView.DisplayFields.values:
raise serializers.ValidationError(
f"Invalid field: {field}",

View File

@ -1911,7 +1911,7 @@ class TestDocumentApi(DirectoriesMixin, DocumentConsumeDelayMixin, APITestCase):
],
)
# Custom field not found, removed from list
# Custom field not found
response = self.client.patch(
f"/api/saved_views/{v1.id}/",
{
@ -1923,9 +1923,7 @@ class TestDocumentApi(DirectoriesMixin, DocumentConsumeDelayMixin, APITestCase):
},
format="json",
)
self.assertEqual(response.status_code, status.HTTP_200_OK)
v1.refresh_from_db()
self.assertNotIn(SavedView.DisplayFields.CUSTOM_FIELD % 99, v1.display_fields)
self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
def test_get_logs(self):
log_data = "test\ntest2\n"