View Single Post
  #121  
Old 11-13-2014, 02:33 PM
 
Kman Kman is offline
 

Senior Member
  
Join Date: Oct 2008
Posts: 124
 

Default Re: Pop-up Anywhere module released

We've used this module at times, and have it wired as far as the functionality is concerned. Very useful. But . . .

In modules/Popup_Anwhere/js/popups.js there is this code:

window.fbAsyncInit = function() {
FB.init({
status: true, // check login status
cookie: true, // enable cookies to allow the server to access the session
xfbml: true // parse XFBML
});
};

(function(d){
var js, id = 'facebook-jssdk'; if (d.getElementById(id)) {return;}
js = d.createElement('script'); js.id = id; js.async = true;
var host = document.location.protocol == "https:" ? "https:" : "http:";
js.src = host + "//connect.facebook.net/en_US/all.js";
d.getElementsByTagName('head')[0].appendChild(js);
}(document));


which among other things loads Facebook's all.js

With this loaded, these browser warnings occur:

Invalid App Id: Must be a number or numeric string representing the application id. all.js:58
The "fb-root" div has not been created, auto-creating all.js:58
FB.getLoginStatus() called before calling FB.init().


A little research shows that this is an error that may trigger when FB.init is not called before FB.login, which is not the case here.

Since we have not changed any of the basic templates or js files, I have to assume that we are not the only ones who have this issue.

Qualiteam - what's up with this?
__________________
4.6.1 Gold Plus, working on moving to the latest 4.x soon
reBOOT Template (the best!)


AlteredCart Checkout One, Checkout One Tools & Checkout One Payments
AlteredCart SmartSearch & Search by Filters
AlteredCart On Sale
CDSEO
xCMS
BCS Point of Sale, Pre-order / Back Order
Magnetic One's Autoresponder

Heavily modified
Reply With Quote