I started doing some testing with the latest Drupal 7.x-dev builds and I see no icons in any admin page no matter what option is chosen (before/after/replace link) :/
The icons do show up when viewing individual nodes though.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 904948-07-no_icons-d7.patch | 1.82 KB | Freso |
Comments
Comment #1
klonos...now using latest D7 dev and latest 7.x-1.x-dev of languageicons (2010-Dec-04). I am not able to see any icons at all. What might I be doing wrong?
Comment #2
Freso commentedYou are quite likely not doing anything wrong. The current version for 7.x was a quick-ish port for a pre-RC edition of Drupal 7 (which didn't even have fully updated module porting docs at the time). I'm about to upload an updated port and I was already thinking pinging you here when that is done, so stay tuned to let me know how stuff works out. :)
ps. Thanks for testing!
Comment #3
klonosOk, I'll be standing by for the new upload Frederik and I'll report back once I test it. I just saw the available new 2010-Dec-04 dev and thought that there was something ready for testing, so I gave it a go. Besides, I had no reply here since my initial post back in September, so I wasn't aware you were actually 'cooking' something ;)
Comment #4
Freso commentedOkay, I haven't committed the code yet, as I still have one wee outstanding issue to look into. But feel free to test the patch at #995446: Update the 7.x porting guided by upgrade.boombatower.com and see if that solves the issue. :)
Comment #5
klonosI tested #2, but still no magic :(
Comment #6
klonos...I mean #1 (it was the second patch in the issue).
Comment #7
Freso commentedAlright, I've fiddled around a bit. Check this one out.
I'm currently in a bit of a pinch though, as the node links now get two language icons... but, hey, it's better than none, right? :) Any ideas would be appreciated.
Comment #8
klonosDid you clear caches and all? I'm saying because in my setup it works as advertised (1 single icon).
Comment #9
klonos+ a minor title change to see if others are having your issue as well.
Comment #10
Freso commentedLet's keep the double icon out of this issue, as it's an issue with the patch, and not with code currently in CVS. :)
Under settings, have you ticked off both node links and language switcher block links?
Comment #11
Freso commentedAlright, I've committed the patch. I've also created a follow-up issue at #1005144: Doesn't differentiate between node and language switcher blocks.
Comment #12
klonosThanx Frederik ;)
Comment #14
bavarian commentedIssue not resolved, no flag-icons showing unfortunately. using 7.x-1.0-alpha1 (which obviously has the patch mentioned here already applied, right?)
Notice: Undefined index: language i languageicons_link_add() (rad 102 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Trying to get property of non-object i theme_languageicons_icon() (rad 132 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Undefined index: language i languageicons_link_add() (rad 102 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Trying to get property of non-object i theme_languageicons_icon() (rad 132 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Undefined index: language i languageicons_link_add() (rad 102 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Trying to get property of non-object i theme_languageicons_icon() (rad 132 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Undefined index: language i languageicons_link_add() (rad 102 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Trying to get property of non-object i theme_languageicons_icon() (rad 132 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Undefined index: language i languageicons_link_add() (rad 102 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Trying to get property of non-object i theme_languageicons_icon() (rad 132 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Undefined index: language i languageicons_link_add() (rad 102 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.module).
Notice: Trying to get property of non-object i theme_languageicons_icon() (rad 132 av/home/xxxxxxxxx/public_html/yyyyyyy/gtm_0.0_D7.0/sites/all/modules/languageicons/languageicons.modul
Comment #15
Freso commentedThis issue was fixed and closed, please open a new issue if you experience something similar.