Hey,

I copied new icons sets as advised on admin/config/media/socialmedia to the directories listed there, so that for example I got this:

sites/all/modules/socialmedia/icons/icondock/16px
sites/all/modules/socialmedia/icons/icondock/24px
sites/all/modules/socialmedia/icons/icondock/32px

and .png files reside in those directories correctly. However, when I refresh the configuration page it says that no icons are installed and I can't enable them.

What's the problem here?
Thanks

CommentFileSizeAuthor
#5 socialmedia-1785422-5-spelling-error-in-vector-icon-path.patch810 bytesohthehugemanatee
#3 Patched socialmedia.icons.inc11.23 KBthe_loeki

Comments

ProfX’s picture

ProfX commented

I'm getting the same issue. On my development server it works just fine, but on my production server it will not work.

I've tried..

  • Uninstalling and reinstalling the modules + dependencies
  • Reverting to earlier builds
  • Resetting all file permissions

Nothing seems to be working at all. I really don't want to re-size the Glossy icons to 24x24 via css.... 32 is too big for my site, and 16 is too small. Any help on this would be greatly appreciated.

the_loeki’s picture

the_loeki commented
Title: Can't install new icon sets » Typo in IconDock set

The problem is that the paths are 'wrong' in the configs due to incorrect capitalization. Depending on what platforms you're using for testing/development, this might not come up on at least Windows paths (since those are case insensitive)

Suggested corrections to socialmedia.icons.inc:

For IconDock:
You're instructed to unzip the download into {module}/icons/icondock.

When you do that, the ZIP file unzips the files into icons/icondock/Vector Social Media Icons/PNG/.
The module expects them in Vector Social Media Icons/png/.

For DryIcons the issue is the other way around; it unzips to png/ but the expected path is PNG/.

The InstaShift set is a MeSs capitalization-wise, with for example 32/google.png and 64/Google.png. The code can't handle that, and rightly so. Either we should expand the array function in socialmedia_icon_path_instashift or instruct designers/devs to lowercase the entire dir after unzipping.

That's all the cap-errors I could find for now, I'll post more as I find them.

the_loeki’s picture

the_loeki commented
StatusFileSize
new Patched socialmedia.icons.inc11.23 KB
the_loeki’s picture

the_loeki commented
Title: Typo in IconDock set » Capitalization typos in icon sets
Component: Miscellaneous » Code
Category: support » bug
ohthehugemanatee’s picture

ohthehugemanatee commented
Status: Active » Needs review
StatusFileSize
new socialmedia-1785422-5-spelling-error-in-vector-icon-path.patch810 bytes

trivial patch attached. Works for me, please review.