Sweet module. I've implemented your module in some work that I'm currently going. The integration was a piece of cake and the results stunning.

One thing I noticed is that the custom piclense logo should have an absolute path rather than a relative one in the atom feed. Attached is a one line patch to fix this.

CommentFileSizeAuthor
piclens.module.patch591 bytesaaron1234nz

Comments

swentel’s picture

swentel commented
Status: Needs review » Fixed

Hmm, good catch. I did it a bit different though. The theme setting allready returns the absolute path of the default feed icon, cf underneath.
I also added a remark on the settings page to make sure that you use an absolute URL, if you want to use a custom icon.

  global $base_url;

  if (empty($logo)) {
    $theme_fav = theme_get_setting('favicon');
    if (!empty($theme_fav)) $logo = $base_url . $theme_fav;
  }
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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