Cannot override rules with dependsOn field in action

[simpsons310]

• Laravel Version: 2.21.0
• Nova Version: 4.12.13
• PHP Version: 8.1.8
• Database Driver & Version:
• Operating System and Version: mysql:8.0
• Browser type and version: Google Chrome 104.0.5112.101 (Official Build) (64-bit)

Description:

I have an action, and this is my list fields definition:

public function fields(NovaRequest $request)
{
    $commands = [
        0 => 'Close',
        1 => 'Open
    ];
    return [
         Select::make(__('Control Mode'), 'command_mode')
                ->rules('required')
                ->options($commands)
                ->displayUsingLabels(),
            Number::make(__('Timeout'), 'session_timeout')
                ->rules('integer', 'min:1', 'max:1000')
                ->default(10)
                ->hide()
                ->dependsOn(['command_mode'], function (Number $field, NovaRequest $request, FormData $formData) {
                    if ($formData->command_mode === 1) {
                        $field->rules('required', 'min:10')->show();
                    }
                }),
        ];
    }

When I input value 1 into Timeout field then execute action, action still execute successfully.

This doesn't work required rule too, I have to use required() method instead. I think this should have been fixed in issue 4170

[crynobone]

Sorry, I can't follow the description. Can you show a video or create a full reproducing repository to demonstrate the issue?

[simpsons310]

@crynobone

With above code, action should throw validation exception for Access Entry Timeout field. Access Entry Timeout's value must be greater or equal 10. But action execute successfully with value 1

[crynobone]

There still missing information here. Does $formData->command_code exactly match the configuration?

This is why it always good to add reproducing repository when reporting a bug.

[simpsons310]

Does $formData->command_code exactly match the configuration?

Yes, it does. Timeout field will show when command_code = 1.

Actually, initially I wanted to test with the required rule, but it didn't work so I wanted to try testing with other rules. So in the code above I put the rule min:10. I've edited code above, please check again.

This is why it always good to add reproducing repository when reporting a bug.
Sorry, but i cannot create another laravel nova project because i don't have license key