Topic-icon Bug in SC Login 3.2

Active Subscriptions:

None
10 years 5 months ago #38673 by drjjw
Bug in SC Login 3.2 was created by drjjw
Hi Alex,

In the new SC Login module, there is a bug in the Twitter integration.

In the module, if one supplies a custom image for Twitter, it loads Google twice.

In the helper.php file, this variable is duplicated:

$params = $goLoginButtonLinkImage;

(As an aside, I notice that the download package does not seem to contain the module. Does the installation script grab the latest files from Sourcoast?).
The topic has been locked.
Support Specialist
10 years 5 months ago #38678 by alzander
Replied by alzander on topic Bug in SC Login 3.2
Jordan,
Thanks for the bug report, and sorry for the silliness you encountered. We'll have that fixed in the next release.

As an aside, I notice that the download package does not seem to contain the module. Does the installation script grab the latest files from Sourcecoast.

The component zip-file you download from here should be installing the SCLogin module as well. The component zip installs all the plugins, modules and other things required for JFBConnect. There is no internet communication required to install our stuff.

I'm not sure why you say it's not included, but it may just be tucked away. The SCLogin module file is in the /admin/install/packages/ directory.

I hope that helps,
Alex
The topic has been locked.
Active Subscriptions:

None
10 years 5 months ago #38693 by drjjw
Replied by drjjw on topic Bug in SC Login 3.2
Ah, that's what it is, tucked away :)
The topic has been locked.
Support Specialist
10 years 5 months ago #38705 by alzander
Replied by alzander on topic Bug in SC Login 3.2
Glad to hear that helped. The new unified installer (introduced in v5.1, back in August) was meant to simplify the installation process a bit and remove the need to individually install everything. It can make things confusing for those who like to do so. Overall though, it's been well received.

As always, if you have any suggestions for improvement or clarification, just let us know!

Thanks,
Alex
The topic has been locked.