search

orlyapps / nova-belongsto-depend

Larave Nova BelongsTo Field with Dependcy
MIT License
182 stars 65 forks source link

Fix key name to be lowercase #7

Closed CasperLaiTW closed 6 years ago

CasperLaiTW commented 6 years ago

dependsOn method in package will change the key to lowercase. https://github.com/orlyapps/nova-belongsto-depend/blob/master/src/NovaBelongsToDepend.php#L67

If custom the field name and set attribute, package will not match the event key.

key mismatch 
$fields = [
    NovaBelongsToDepend::make('Level 1', 'coursePropertyLevel1', CoursePropertyLevel1::class)
                ->options(\App\CourseProperty::root()->get())
                ->rules('required')
                ->hideFromIndex(),
    NovaBelongsToDepend::make('Level 2', 'coursePropertyLevel2', CoursePropertyLevel2::class)
                ->options(\App\CourseProperty::root()->get())
                ->dependsOn('coursePropertyLevel1')
                ->rules('required')
                ->hideFromIndex(),
];