dice icon indicating copy to clipboard operation
dice copied to clipboard

Add error message for using JSON.OBJLEN on non-existant key

Open paulwalrath opened this issue 1 year ago • 5 comments

This fixes the missing error message that happens when you run the JSON.OBJLEN command on a key that doesn't exist. (Presently, it returns a blank message.)

I discovered this inconsistency while writing the JSON.OBJLEN page.

Ref: #1353

paulwalrath avatar Dec 03 '24 00:12 paulwalrath

Investigating test failure...

paulwalrath avatar Dec 03 '24 02:12 paulwalrath

I'm seeing the tests fail on the current master branch too.

paulwalrath avatar Dec 03 '24 02:12 paulwalrath

@paulwalrath can you please validate --- FAIL: TestJSONSetWithNXAndXX (0.01s) seems to be failing with your changes.

lucifercr07 avatar Dec 03 '24 10:12 lucifercr07

@paulwalrath can you please validate --- FAIL: TestJSONSetWithNXAndXX (0.01s) seems to be failing with your changes.

Investigating now.

paulwalrath avatar Dec 03 '24 18:12 paulwalrath

Sorry about that. Corrected now.

paulwalrath avatar Dec 03 '24 20:12 paulwalrath

FYI, this PR is ready to be merged.

paulwalrath avatar Dec 17 '24 02:12 paulwalrath