Hi,
just installed latest cvs version of og and og_roles.
When no custom roles are defined og_roles_settings displays a message refering to admin/access/roles.
It seems that the % placeholder causes the url to contain em-tags. Switching to @ placeholder fixes this problem.
Patch attached.

CommentFileSizeAuthor
#2 og_roles_url_fix_1.txt1.39 KBknseibert
#1 og_roles_url_fix_0.txt959 bytesknseibert
og_roles_url_fix.txt963 bytesknseibert

Comments

knseibert’s picture

knseibert commented
StatusFileSize
new og_roles_url_fix_0.txt959 bytes

Doh. Just noticed that it is the wrong url after all. Roles are now to be found under /admin/user/roles.
I have kept the placeholder name @access although it has not much to do with "access" anymore.
Anyway... patch attached.

knseibert’s picture

knseibert commented
StatusFileSize
new og_roles_url_fix_1.txt1.39 KB

Now i have added some custom roles. The fieldset title html reads:

<legend><em>&lt;em&gt;Group&lt;/em&gt;</em> role options</legend>

Which looks like this:

<em>Group</em> role options

This is due to:
1) % placeholder iso @
2) theme('placeholder') function, which IMHO is not necessary here.
Again, patch attached.