Skip to content
This repository has been archived by the owner on Apr 11, 2024. It is now read-only.

Cannot read property '0' of null #109

Closed
sakulstra opened this issue Oct 24, 2016 · 2 comments
Closed

Cannot read property '0' of null #109

sakulstra opened this issue Oct 24, 2016 · 2 comments
Labels
bug

Comments

@sakulstra
Copy link
Contributor

Hello, when trying to use this library inside meteor I get the following error:
platform.js:734 Uncaught TypeError: Cannot read property '0' of null as reported here: meteor/meteor#7839

Would the solution at the end of the ticket be accepted, or do you guys have another suggestion?

Thanks for your efforts,
regards,
lukas
@jdalton
Copy link
Member

jdalton commented Oct 24, 2016

Hi @sakulstra!

Sure, a PR would be great!

sakulstra added a commit to sakulstra/platform.js that referenced this issue Oct 25, 2016
@sakulstra
make it usable with meteor
dc6e1ce 
- closes bestiejs#109
This is occurring because of the platform dependency of the react-share NPM. The platform package does the following to check for Node but since process is defined as an object because of the process module (a module that I believe is typical of all Meteor/Browserify/Webpack setups) it's inadvertently classifying itself as Node in this step and then failing when it tries to grab the numeric version of the process.version (which is just an empty string).
@sakulstra sakulstra mentioned this issue Oct 25, 2016
Closed
@jdalton jdalton added the bug label Oct 25, 2016
@jdalton
Copy link
Member

jdalton commented Oct 25, 2016

Moved to #111.

@jdalton jdalton closed this as completed Oct 25, 2016
@sakulstra sakulstra mentioned this issue Oct 27, 2016
Closed
bnjmnt4n pushed a commit that referenced this issue Oct 31, 2016
@sakulstra @bnjmnt4n
Ensure that platform.js does not detect Node.js when used in a browse…
56a49d6 
…rified module.

Closes #109.
Closes #111.
bnjmnt4n added a commit that referenced this issue Nov 2, 2016
@bnjmnt4n
Fix test for browserified Node modules.
a02e5fa 
Ref. #109.
Ref. #111.
bnjmnt4n added a commit that referenced this issue Nov 10, 2016
@bnjmnt4n
Fix test for browserified Node modules.
b1ef2e1 
Ref. #109.
Ref. #111.
@abernix abernix mentioned this issue Nov 17, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

make it usable with meteor sakulstra/platform.js
2 participants
@jdalton @sakulstra