Strip whitespace from AR field names and labels to prevent image lookup issues
- Add .strip() to field name and label inputs in AR field management - Implement defensive stripping at model layer for extra safety - Prevents spacing issues that cause image serving routes to fail
This commit is contained in:
@@ -321,8 +321,8 @@ def manage_ar_fields(tenant_id):
|
|||||||
|
|
||||||
if action == 'add':
|
if action == 'add':
|
||||||
field_data = {
|
field_data = {
|
||||||
'fieldName': request.form.get('fieldName'),
|
'fieldName': request.form.get('fieldName', '').strip(),
|
||||||
'label': request.form.get('label'),
|
'label': request.form.get('label', '').strip(),
|
||||||
'fieldType': request.form.get('fieldType'),
|
'fieldType': request.form.get('fieldType'),
|
||||||
'editable': request.form.get('editable', 'true'),
|
'editable': request.form.get('editable', 'true'),
|
||||||
'displayOrder': int(request.form.get('displayOrder', 0))
|
'displayOrder': int(request.form.get('displayOrder', 0))
|
||||||
@@ -343,8 +343,8 @@ def manage_ar_fields(tenant_id):
|
|||||||
elif action == 'update':
|
elif action == 'update':
|
||||||
field_data = {
|
field_data = {
|
||||||
'id': int(request.form.get('field_id')),
|
'id': int(request.form.get('field_id')),
|
||||||
'fieldName': request.form.get('fieldName'),
|
'fieldName': request.form.get('fieldName', '').strip(),
|
||||||
'label': request.form.get('label'),
|
'label': request.form.get('label', '').strip(),
|
||||||
'fieldType': request.form.get('fieldType'),
|
'fieldType': request.form.get('fieldType'),
|
||||||
'editable': request.form.get('editable', 'true'),
|
'editable': request.form.get('editable', 'true'),
|
||||||
'displayOrder': int(request.form.get('displayOrder', 0))
|
'displayOrder': int(request.form.get('displayOrder', 0))
|
||||||
|
|||||||
@@ -36,6 +36,10 @@ class ARFieldModel:
|
|||||||
"""Save or update a custom AR field"""
|
"""Save or update a custom AR field"""
|
||||||
tenant_id = tenant_id.lower()
|
tenant_id = tenant_id.lower()
|
||||||
|
|
||||||
|
# Strip whitespace from field name and label to prevent spacing issues
|
||||||
|
field_name = field_data['fieldName'].strip() if field_data.get('fieldName') else ''
|
||||||
|
label = field_data['label'].strip() if field_data.get('label') else ''
|
||||||
|
|
||||||
with get_db() as conn:
|
with get_db() as conn:
|
||||||
cursor = conn.cursor()
|
cursor = conn.cursor()
|
||||||
|
|
||||||
@@ -47,8 +51,8 @@ class ARFieldModel:
|
|||||||
display_order = ?, updated_at = CURRENT_TIMESTAMP
|
display_order = ?, updated_at = CURRENT_TIMESTAMP
|
||||||
WHERE id = ? AND tenant_id = ?
|
WHERE id = ? AND tenant_id = ?
|
||||||
''', (
|
''', (
|
||||||
field_data['fieldName'],
|
field_name,
|
||||||
field_data['label'],
|
label,
|
||||||
field_data['fieldType'],
|
field_data['fieldType'],
|
||||||
field_data['editable'],
|
field_data['editable'],
|
||||||
field_data['displayOrder'],
|
field_data['displayOrder'],
|
||||||
@@ -63,8 +67,8 @@ class ARFieldModel:
|
|||||||
VALUES (?, ?, ?, ?, ?, ?)
|
VALUES (?, ?, ?, ?, ?, ?)
|
||||||
''', (
|
''', (
|
||||||
tenant_id,
|
tenant_id,
|
||||||
field_data['fieldName'],
|
field_name,
|
||||||
field_data['label'],
|
label,
|
||||||
field_data['fieldType'],
|
field_data['fieldType'],
|
||||||
field_data['editable'],
|
field_data['editable'],
|
||||||
field_data['displayOrder']
|
field_data['displayOrder']
|
||||||
|
|||||||
Reference in New Issue
Block a user