We patched the module for our site, as we run our site on HTTPS, and we needed to load the CC images over HTTPS as well to avoid certificate errors. I'm attaching the patch that we used.

CommentFileSizeAuthor
#2 creative_commons.module.patch863 bytesbkat
creative_commons.module.patch4.05 KBrael9
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bkat’s picture

bkat CreditAttribution: bkat commented

A better solution for the image uri is to remove the protocol from the URI so that uses whatever URI the page is being viewed with:

define('CC_IMAGE_URI', '//i.creativecommons.org/l');

This way if the page displaying the license image is http, it will use http://i.creativecommons.org/l and if the page is being viewed via https, it will use https to retrieve the image: https://i.creativecommons.org/l

bkat’s picture

bkat CreditAttribution: bkat commented
FileSize
creative_commons.module.patch863 bytes

Here's a patch based on my changes.

stupiddingo’s picture

stupiddingo CreditAttribution: stupiddingo commented
Status: Active » Reviewed & tested by the community

RTBC #2 works well in mixed mode testing.

  • sdrycroft committed d32eccc on 7.x-1.x authored by bkat 
    Issue #2290017 by bkat, rael9: HTTPS support for images
sdrycroft’s picture

sdrycroft CreditAttribution: sdrycroft commented
Status: Reviewed & tested by the community » Fixed

Thanks bkat and rael9, I've committed this change.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.