Bug #2296

duktape is pesky on null values in appendItem

Added by Leonid Protasov over 6 years ago. Updated over 6 years ago.

Status:FixedStart date:10/29/2014
Priority:NormalDue date:
Assignee:Andreas Smas% Done:

100%

Category:API
Target version:4.8
Found in version:last BE Platform:Linux

Description

tried to set null in timestamp and got:

ECMASCRIPT [ERROR]: TypeError (unexpected type) at duk_bi_global.c:36
ECMASCRIPT [ERROR]: STACK DUMP: TypeError: unexpected type
                        duk_api_stack.c:1272
                        anon duk_bi_global.c:36 preventsyield
                        anon duk_bi_global.c:47 preventsyield
                        anon duk_bi_global.c:82
                        anon file:///root/showtime-plugins/baskino/baskino.js:525 preventsyield
                        apply  native strict preventsyield
                        anon duk_bi_global.c:108 preventsyield

Associated revisions

Revision c0b8c858
Added by Andreas Smas over 6 years ago

prop.js: Need to explicitly check that object isn't null

Fixes #2296

Change included in version 4.7.402

History

#1 Updated by Leonid Protasov over 6 years ago

Problematic code:

                page.appendItem(link, 'video', {
                    title: 'test',
                    timestamp: null
                });

Fixed code:
                page.appendItem(link, 'video', {
                    title: 'test',
                    timestamp: timestamp ? timestamp : ''
                });

#2 Updated by Andreas Smas over 6 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Also available in: Atom PDF