Address critical data visibility issues for Event Files and enhance frontend documentation.
This commit resolves the persistent problem where top-level hosted file convenience fields
(e.g., , , ) were
returning as in V3 Event File API responses, even when .
Key changes include:
- Refactored Pydantic model:
- Removed redundant definitions from top-level hosted file convenience fields,
allowing direct mapping from SQL view columns.
- Simplified to focus solely on conditionally loading the nested
object, as top-level fields are now populated directly by Pydantic
from the view.
- Added comprehensive comments to clarify data flow, Pydantic's behavior, and the
expected origin of these convenience fields from SQL views.
- Updated :
- Introduced a new section detailing how to retrieve Event File data, including the
use of to get both top-level convenience fields and a nested
object.
- Clarified all ID references as random string IDs.
- Renumbered the troubleshooting section.
- Copied updated guide to .
- Continued ID Vision compliance audit, ensuring consistent handling of random string IDs
across various core and event models (Account, Address, Contact, DataStore, Event Badge Template).
- Consolidated ID Vision E2E tests and updated related documentation.
- Minor updates to and
to support Event File data retrieval with .
135 lines
5.0 KiB
Python
135 lines
5.0 KiB
Python
import datetime, pytz
|
|
|
|
from typing import Dict, List, Optional, Set, Union, ClassVar
|
|
from pydantic import BaseModel, EmailStr, Field, Json, PrivateAttr, ValidationError, validator, root_validator
|
|
|
|
from app.db_sql import get_id_random, redis_lookup_id_random
|
|
from app.lib_general import log, logging
|
|
|
|
from app.models.common_field_schema import base_fields
|
|
|
|
|
|
# ### BEGIN ### API Data Store Models ### Data_Store_Base() ###
|
|
class Data_Store_Base(BaseModel):
|
|
log.setLevel(logging.INFO) # DEBUG, INFO, WARNING, ERROR, EXCEPTION, CRITICAL
|
|
log.debug(locals())
|
|
|
|
# --- Standardized Vision IDs (Strings) ---
|
|
id: Optional[str] = Field(None, **base_fields['data_store_id_random'])
|
|
data_store_id: Optional[str] = Field(None, **base_fields['data_store_id_random'])
|
|
|
|
account_id: Optional[str] = Field(None, **base_fields['account_id_random'])
|
|
person_id: Optional[str] = Field(None, **base_fields['person_id_random'])
|
|
user_id: Optional[str] = Field(None, **base_fields['user_id_random'])
|
|
|
|
# Standardized Polymorphic Target
|
|
for_type: Optional[str]
|
|
for_id: Optional[str] = Field(**base_fields['obj_id_random'])
|
|
|
|
# --- Standardized Legacy / Internal IDs (Excluded) ---
|
|
id_random: Optional[str] = Field(None, alias='data_store_id_random', exclude=True)
|
|
account_id_random: Optional[str] = Field(None, exclude=True)
|
|
person_id_random: Optional[str] = Field(None, exclude=True)
|
|
user_id_random: Optional[str] = Field(None, exclude=True)
|
|
for_id_random: Optional[str] = Field(None, exclude=True)
|
|
|
|
code: Optional[str]
|
|
name: Optional[str]
|
|
description: Optional[str]
|
|
|
|
type: Optional[str] # html, json, md, text
|
|
|
|
# The JSON fields are case sensitive
|
|
json_str: Optional[Union[Json, None]] = Field(
|
|
alias = 'json',
|
|
)
|
|
|
|
# The text fields are case insensitive
|
|
text: Optional[str]
|
|
|
|
meta_json: Optional[str]
|
|
meta_text: Optional[str]
|
|
|
|
access: Optional[str]
|
|
access_read: Optional[str]
|
|
access_write: Optional[str]
|
|
access_delete: Optional[str]
|
|
|
|
enable: Optional[bool]
|
|
|
|
hide: Optional[bool]
|
|
priority: Optional[bool]
|
|
sort: Optional[int]
|
|
group: Optional[str]
|
|
|
|
notes: Optional[str]
|
|
|
|
created_on: Optional[datetime.datetime] = None
|
|
updated_on: Optional[datetime.datetime] = None
|
|
|
|
_processed_at: datetime.datetime = PrivateAttr(default_factory=datetime.datetime.now)
|
|
|
|
@root_validator(pre=True)
|
|
def map_v3_ids(cls, values):
|
|
"""
|
|
Vision Transformer:
|
|
Map DB keys to clean API keys and strip internal integers.
|
|
"""
|
|
# 0. Scrub stringified NULLs from database
|
|
for k, v in list(values.items()):
|
|
if isinstance(v, str) and v.upper() == 'NULL':
|
|
values[k] = None
|
|
|
|
# 1. Map Primary Object ID
|
|
if rid := values.get('id_random') or values.get('data_store_id_random'):
|
|
values['id'] = rid
|
|
values['data_store_id'] = rid
|
|
|
|
# 2. Map & Resolve Relational IDs
|
|
id_map = [
|
|
('account_id', 'account'),
|
|
('person_id', 'person'),
|
|
('user_id', 'user'),
|
|
]
|
|
|
|
for field, table in id_map:
|
|
r_val = values.get(f'{field}_random')
|
|
if r_val and isinstance(r_val, str):
|
|
values[field] = r_val
|
|
elif values.get(field) and isinstance(values[field], (int, str)):
|
|
# If it's a string but doesn't look like a random ID (e.g. integer string), resolve it
|
|
is_random = isinstance(values[field], str) and len(values[field]) >= 11
|
|
if not is_random:
|
|
resolved_rid = get_id_random(values[field], table)
|
|
if resolved_rid:
|
|
values[field] = resolved_rid
|
|
values[f'{field}_random'] = resolved_rid
|
|
|
|
# 3. Handle Polymorphic for_id
|
|
if f_rid := values.get('for_id_random'):
|
|
values['for_id'] = f_rid
|
|
elif values.get('for_id') and values.get('for_type'):
|
|
# Resolve based on the for_type
|
|
is_random = isinstance(values['for_id'], str) and len(values['for_id']) >= 11
|
|
if not is_random:
|
|
resolved_for_rid = get_id_random(values['for_id'], values['for_type'])
|
|
if resolved_for_rid:
|
|
values['for_id'] = resolved_for_rid
|
|
values['for_id_random'] = resolved_for_rid
|
|
|
|
# 4. Final Vision Enforcement: Strip internal integers from public fields
|
|
for k in ['id', 'data_store_id', 'account_id', 'person_id', 'user_id', 'for_id']:
|
|
val = values.get(k)
|
|
# If value is present but not a valid random string ID
|
|
if val is not None:
|
|
if not isinstance(val, str) or len(val) < 11:
|
|
values[k] = None
|
|
|
|
return values
|
|
|
|
class Config:
|
|
underscore_attrs_are_private = True
|
|
allow_population_by_field_name = False
|
|
fields = base_fields
|
|
# ### END ### API Data Store Models ### Data_Store_Base() ###
|