keystonejs/keystone-classic

Code format lost if using dependsOn

Open

#4.608 geöffnet am 4. Apr. 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (14.656 Stars) (2.288 Forks)batch import
bughelp wanted

Beschreibung

Expected behavior

When using dependsOn in a Type.Code field, it should keep its code formating.

Actual/Current behavior

When using dependOn in Type.Code field, the format defaults to a plain text field.

Steps to reproduce the actual/current behavior

code: {type: Types.Code, dependsOn: {type: "code"}"}

Environment

Software Version
Keystone 4.0.0-beta.5
Node v8.10.0

Contributor Guide

Tech Stack
javascriptnodejs
Domain
backend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic understanding of Keystone field typesFamiliarity with dependsOn
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35
Research-Richtung
Examine the Code field type implementation, likely in `fields/types/Code.js`. Investigate how `dependsOn` affects the field's rendering: the issue reports that the code format (editor) is lost and defaults to a plain text field when `dependsOn` is used. Look at the field definition and any conditional logic that might disable the code editor. Since no comments or PRs exist and the issue is from 2018, first verify if the bug persists in the current version of keystone classic. If reproducing, check the related field type registration and how `dependsOn` is processed during field initialization.