1400404 - React should expect a boolean or a number as the JSON data

Status: RESOLVED FIXED

(DevTools :: JSON Viewer, enhancement, P3)

Description

[Oriol Brufau [:Oriol]]

http://searchfox.org/mozilla-central/rev/2c9a5993ac40ec1db8450e3e0a85702fa291b9e2/devtools/client/jsonview/components/main-tabbed-area.js#31

      json: PropTypes.oneOfType([
        PropTypes.string,
        PropTypes.object,
        PropTypes.array
      ])

http://searchfox.org/mozilla-central/rev/2c9a5993ac40ec1db8450e3e0a85702fa291b9e2/devtools/client/jsonview/components/json-panel.js#38

      data: PropTypes.oneOfType([
        PropTypes.string,
        PropTypes.array,
        PropTypes.object
      ])

But the JSON data can also be a boolean or a number, e.g.

    data:application/json,true
    data:application/json,1234

With --enable-debug-js-modules, they produce a warning in the console.

There is also null, but it seems that's already covered, probably by PropTypes.object.

Comment 1

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Yep, good catch. Thanks for the report!

Honza

Comment 2

[Abhinav Koppula]

Attachment: Bug 1400404 - Adding bool and number to list of PropTypes in Json-Panel.

Review commit: https://reviewboard.mozilla.org/r/182542/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/182542/

Comment 3

[Abhinav Koppula]

Hi Honza,
I was able to see that the following warnings go away with my patch applied.
Warning: Failed prop type: Invalid prop `json` supplied to `MainTabbedArea`.
    in MainTabbedArea  react-dev.js:22807
Warning: Failed prop type: Invalid prop `data` supplied to `JsonPanel`.
    in JsonPanel (created by MainTabbedArea)
    in MainTabbedArea

Can you take a look at my changes?

Comment 4

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8911072 [details]
Bug 1400404 - Adding bool and number to list of PropTypes in Json-Panel.

https://reviewboard.mozilla.org/r/182542/#review188762

Works for me, thanks Abhinav!

Honza

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5d5d1d227104
Adding bool and number to list of PropTypes in Json-Panel. r=Honza

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/5d5d1d227104