New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

accounts-ui Uncaught TypeError: Cannot read property 'parentNode' of null #4485

Closed
udiedrichsen opened this Issue May 31, 2015 · 2 comments

Comments

Projects
None yet
3 participants
@udiedrichsen

udiedrichsen commented May 31, 2015

Hi,

I made a very simple login:

<body>
{{> loginButtons}}
</body>

Packages used:

accounts-base
accounts-password
accounts-ui

My Account.config:

  Accounts.config({
    sendVerificationEmail: false,
    forbidClientAccountCreation: true,
    loginExpirationInDays: null  // Expires never
  });
  Accounts.ui.config({
    passwordSignupFields: 'USERNAME_ONLY'
  });

I got an exception in console:

Uncaught TypeError: Cannot read property 'parentNode' of null accounts-ui-unstyled.js?14a3fc68e1a01e2a5a37fa082ac84346e46ba1de:1777
correctDropdownZIndexes accounts-ui-unstyled.js?14a3fc68e1a01e2a5a37fa082ac84346e46ba1de:1777
Template.loginButtons.events.click #login-name-link, click #login-sign-in-link accounts-ui-unstyled.js?14a3fc68e1a01e2a5a37fa082ac84346e46ba1de:1280
(anonymous function) blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:3502
Template._withTemplateInstanceFunc blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:3476
(anonymous function) blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:3501
(anonymous function) blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2543
Blaze._withCurrentView blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2197
(anonymous function) blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:2542
(anonymous function) blaze.js?a5c324925e5f6e800a4c618d71caf2848b53bf51:839
jQuery.event.dispatch jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:4698
elemData.handle jquery.js?dd8bac56f8fd3666d433d2285ae01e52597cc51a:4366

Maybe my error, but I made a very simple account handling (just a tester app for local packages).

I got this error only on my Cordova APP using meteor run android-device --mobile-server ...
Device: Huawei Mate 7
Android 4.4.2

glasser added a commit that referenced this issue Jun 2, 2015

Remove IE7-specific workaround
We no longer support IE7, and a user reported that this function threw
in some context.  Might as well just delete the code.  See #4485.
@glasser

This comment has been minimized.

Member

glasser commented Jun 2, 2015

Please provide a full reproduction as described in: https://github.com/meteor/meteor/wiki/Contributing-to-Meteor#reporting-a-bug-in-meteor Without a way to replicate your problem, we can't debug and fix it. In addition to not providing a cloneable repo, you didn't actually say what action you did (clicking a link?) that caused the exception to occur.

That said, it looks like the error happened in some IE7 workaround code, which I've taken the liberty of deleting as we no longer support IE7.

@udiedrichsen

This comment has been minimized.

udiedrichsen commented Jun 3, 2015

I tried to reproduce it, but without success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment