Launching PAVUControl works from Xubuntu, but not from Xfce

Yup, that is the case. So far, only Xubuntu and Ubuntu Studio are taken into account.
http://bazaar.launchpad.net/~indicator-applet-developers/indicator-sound/trunk.14.04/view/head:/src/service.vala#L209

I guess the reason the Xfce session isn't there is because nobody complained about it before.

Reading this code, shouldn't gnome-control-center sound launch in the case where Xfce is the session and it is installed? Because I tried that, and it launches gnome-control-center with only about two or three icons visible, workaround being to launch it with gksu. Which I've filed under bug #1359248

And as an aside, wouldn't a setting, even something in dconf, be a better fit anyway?

Wanted to update, this bug is still in effect, the other bug I have figured out how to solve, will update that thread.

That's also the reason it had to be patched in Linux Mint Xfce edition.

Roman, what would these commands say in this "other" Xfce session?

echo $XDG_CURRENT_DESKTOP
echo $DESKTOP_SESSION

Status changed to 'Confirmed' because the bug affects multiple users.

Ok, I've tested it myself in the same "other" Xfce session.

monsta@xubuntu:~$ echo $XDG_CURRENT_DESKTOP
XFCE
monsta@xubuntu:~$ echo $DESKTOP_SESSION
xfce

So it's clear that the condition in indicator-application source code should be changed.

Sorry, *indicator-sound* of course, damn those unchangeable comments :)

roman@Roman-PC:~$ echo $XDG_CURRENT_DESKTOP
XFCE
roman@Roman-PC:~$ echo $DESKTOP_SESSION
xfce

The thing is, I would rather have the gnome audio manager over PAVUControl (I consider it more user-friendly) and certainly o.ver the Unity one, especially since Unity is planning on switching to Qt-based one anyway.

I doubt the maintainers would accept gnome-control-center in Xubuntu just for that... We can suggest making the command configurable though.

same as #1443229.

I investigate the source code of indicator-sound, and find that in src/service.vala:

var env = Environment.get_variable ("DESKTOP_SESSION");
string cmd;
if (env == "xubuntu" || env == "ubuntustudio")
 cmd = "pavucontrol";
else

I think you would support more desktop environment such as XFCE by adding " || env == "xfce""