Cannot access functions inside Javascript function wrapper?

Question

All,

My code was failing JSLint for the "use strict" issue. As suggested here : Disable "use the function form of use strict" but keep the "Missing 'use strict' statement" warning

If you wrap all the functions in the javascript file with another function wrapper, it will solve this issue.

However now after doing this all my functions are undefined when I call them from the page?

(function () {
"use strict";

/**
 * Update the page with a message informing the user
 * an email has been sent to their email address
 * @param details user details (email address)
 */
function shareEmailSent(details) {

    // unhide confirmation message
    $('strong#emailAddress').text(details.email);
    $('#confirmationMessage').removeClass('hide');
}

/**
 * Handle error
 */
function showError() {
    return false;
}

/**
 * Makes a POST request to send the current user
 * an email containing their unqiue share url
 * @param emailList
 */
function sendEmail(emailList) {
    var data = {};
    data.formToken = $('#formToken').val();
    data.emails = emailList;
    $.mooAjax({
        url: "/ajax/share/refer_a_friend_email.php",
        type: "POST",
        data: data,
        dataType: "json",
        error: showError,
        success: function (response) {
            shareEmailSent(response);
        }
    });
}
}());

e.g.

shareEmailSent is not defined

How can I fix this but also pass the JSLint issue as well?

Thanks

Community · Accepted Answer · 2017-05-23 12:30:19Z

1

There are several approaches you can take. In descending order of niceness:

Don't call the functions from outside. Bind all your event handlers using JavaScript (e.g. with addEventListener)
Use the revealing module pattern
Explicitly make functions globals (e.g. with window.someFunction = someFunction).

Obviously, since sendEmail calls the other functions itself, it is probably the only one that would need to be exposed if you used the second or third approach.

edited May 23, 2017 at 12:30

CommunityBot

11 silver badge

answered Dec 10, 2015 at 10:47

Quentin

949k136 gold badges1.3k silver badges1.4k bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

tomaytotomato Over a year ago

Thanks, for the help. I went with option 3. However now it breaks my functions validation for some reason, any idea why assigning a function to a global variable would affect the logic?

Quentin Over a year ago

It shouldn't do, and I've no idea what you mean by "breaks my functions validation".

Collectives™ on Stack Overflow

Cannot access functions inside Javascript function wrapper?

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related