Vortex progbutton.m

[ariszlo]

(Continued from Why does this crash Winamp?)

The programmable buttons in the left drawer of Vortex used to work fine when Vortex was released for Winamp 5.2 but are broken in recent versions of Winamp.

You can still bookmark the currently played song by right-clicking on a button and selecting the option but then you cannot play it by left-clicking on the button.

However, the same feature works well in KameleonDUI.

I looked inside studio.xnf to see the difference between how Vortex and KameleonDUI save the currently played song and I found that Vortex starts the path with

code:
PATH|file://C:\

while KameleonDUI starts it with

code:
PATH|C:\

After adding file:// between PATH| and C:\ the bookmarked song could be played by left-clicking on the programmed button.

My question is how shall I modify progbutton.m to make it save the path of the currently played song with file:// between PATH| and C:\ ?

Plague has a script that has something to do with the file:// part but I cannot figure out how it could help with this issue.

[PeterK.]

Originally Posted by ariszlo

...

Do both skins use exactly the same command? Please copy and paste the lines from both scripts.

[ariszlo]

It was enough to ask the question to find the answer. All I had to do was replace "PATH|" with "PATH|file://" in two lines of progbutton.m.

Now another question. Right-clicking on a programmable button and left-clicking on "type path/url" does nothing in Vortex. In KameleonDUI, it pops up a dialog.

What does Vortex do wrong? How shall I modify progbutton.m?

[ariszlo]

Originally Posted by PeterK.

Do both skins use exactly the same command? Please copy and paste the lines from both scripts.

KameleonDUI does not provide the source of progbutton.maki.

[PeterK.]

Originally Posted by ariszlo

KameleonDUI does not provide the source of progbutton.maki.

Really?

Originally Posted by ariszlo

...Right-clicking on a programmable button and left-clicking on "type path/url" does nothing in Vortex....

It seems there are a lot of things that do nothing in Vortex. Like those Open buttons in Skin Preferences.

[ariszlo]

Originally Posted by PeterK.

Really?

Oh, yes, it does. I was looking inside the wrong .wal file.

[PeterK.]

Originally Posted by ariszlo

... How shall I modify progbutton.m?

Are you looking for this?

[ariszlo]

Yes. How did you fix it?

[PeterK.]

Originally Posted by ariszlo

Yes. How did you fix it?

Add <include file="xml/popup-container.xml"/> to skin.xml.

[ariszlo]

Thanks a lot!

Two more remaining issues:

1. Replacing "PATH|" with "PATH|file://" fixes bookmarking but it is a bit ugly (although it seems to work) that urls are stored in studio.xnf as "PATH|file://http://..."
2. Albumart is not shown if I play a song by left-clicking on a programmed button.

[PeterK.]

Originally Posted by ariszlo

Thanks a lot!

Two more remaining issues:
Replacing "PATH|" with "PATH|file://" fixes bookmarking but it is a bit ugly (although it seems to work) that urls are stored in studio.xnf as "PATH|file://http://..."

Where would you like to store them otherwise? And why is it ugly? No user will ever see it.

Originally Posted by ariszlo

Albumart is not shown if I play a song by left-clicking on a programmed button.

Which button? One of those 5 on the right side? Please provide a screenshot showing what you click and where the Albumart is missing.

[ariszlo]

Originally Posted by PeterK.

Please provide a screenshot showing what you click and where the Albumart is missing.

I programmed prog.button.1 in the left drawer to play "When a Blind Man Cries" by right-clicking on prog.button.1 and selecting "Bookmark current song". When I play the song from a playlist or open it from the disk then the album art is shown:



However, if I start playing the song by left-clicking on prog.button.1 then the album art is not shown:

Originally Posted by PeterK.

Which button? One of those 5 on the right side?

Ah, those. They do not work for me. They used to work in Winamp 5.2 but not in recent versions.

[PeterK.]

First of all I managed to get those buttons in the left drawer to work properly - without adding any "file://" to the PATH string.

This is what you should add to progbutton.m script:

PHP Code:

Function string pathToURL(string path); 


PHP Code:

string pathToURL(string path) {
    int c, len = strlen(path);
    string ret = "", char;

    
    c=0;
    while (c < len) {
        char = strmid(path,c,1);
        if (char==chr(92)) {
            ret = ret + chr(47);
            c ++;
        } else if (char==":") { //for ":/"
            ret = ret + ":";
            c ++;
            
            char = strmid(path,c,1);
            if (char==chr(92)) {
                ret = ret + chr(47) + chr(47);
                c ++;
            }
        } else {
            ret = ret + char;
            c++;
        }
    }
    
    return ret;
} 


(It does not matter where in the script you place those lines from above)

Change the line 237 from

PHP Code:

string currPath = getPlayItemMetaDataString("filename"); 


to

PHP Code:

string currPath = System.getPlayItemString(); 


Change the line 156 from

PHP Code:

playFile(path); 


to

PHP Code:

playFile(pathToURL(path)); 


You can find those lines in progbutton.m in leechbite's Kameleon skin.

Furthermore: Kameleon seems to have exactly the same problem with AlbumArt not being shown after left clicking on a button which was programmed to play a bookmarked song:



So for now I cannot promise any working solutions.

[PeterK.]

The prefix file:// is the reason why there is no Albumart shown, but when we delete it Winamp refuses to play the bookmarked song.

[PeterK.]

I do remember my struggle with the System.playFile() command which would not play my saved bookmarks while adding Track Bookmarks feature to my skin. It turned out that this particular line in std.mi library is buggy. So Victhor had an idea to choose another approach in order to obtain files's location on the hard drive and being able to play it. I, on the other side, found out that using the plDir.mi library we could play a bookmarked song, but first we had to remove the file:// prefix. So this is what I did and now you do have AlbumArt in the window.

Please delete line 128 - playFile(path) - and paste those:

PHP Code:

int i=strlen(pathToURL(path))-7; 
string s=strright(pathToURL(path),i);
plEdit.clear(); 
plEdit.enqueueFile(s);
int playThisFile=plEdit.getNumTracks();
plEdit.playTrack(playThisFile); 


Additional you need to add the plDir.mi library:

PHP Code:

#include <lib/pldir.mi> 


[PeterK.]

And when you open the file quicksongpick.m and replace the line System.playFile(filename);

with those following lines ...

PHP Code:

plEdit.clear(); 
plEdit.enqueueFile(filename);
int playThisFile=plEdit.getNumTracks();
plEdit.playTrack(playThisFile); 


... you also can use those memory buttons on the right side of the Main Window.

Wait! You also have to add the plDir.mi library to the script file.

Kinda cool, huh?

[ariszlo]

Originally Posted by PeterK.

First of all I managed to get those buttons in the left drawer to work properly - without adding any "file://" to the PATH string.

This is more elegant than my solution. Thank you!

Originally Posted by PeterK.

You can find those lines in progbutton.m in leechbite's Kameleon skin.

Where did you get KameleonDUI? Versions 1.0 and 1.1 hosted at the Nullsoft archive have the maki sources but bookmarking with buttons is broken with them. Bookmarking works well with Version 1.5 hosted at DeviantArt but it has no maki sources.

Originally Posted by PeterK.

Please delete line 128 - playFile(path) - and paste those:

PHP Code:

int i=strlen(pathToURL(path))-7; 
string s=strright(pathToURL(path),i);
plEdit.clear(); 
plEdit.enqueueFile(s);
int playThisFile=plEdit.getNumTracks();
plEdit.playTrack(playThisFile); 


Additional you need to add the plDir.mi library:

Now, this fixes Album Art but breaks playing bookmarked streams or songs added by the "Type path/url" dialog.

[ariszlo]

Originally Posted by PeterK.

And when you open the file quicksongpick.m and replace the line System.playFile(filename);

I don't have that. I only have playfile(filename);

Originally Posted by PeterK.

with those following lines ...

PHP Code:

plEdit.clear(); 
plEdit.enqueueFile(filename);
int playThisFile=plEdit.getNumTracks();
plEdit.playTrack(playThisFile); 


... you also can use those memory buttons on the right side of the Main Window.

Wait! You also have to add the plDir.mi library to the script file.

Replaced playfile(filename); with those lines and added the plDir.mi library:

code:
#include "../../../lib/pldir.mi"

but it still does not work for me.

[PeterK.]

Originally Posted by ariszlo

... Where did you get KameleonDUI? ...

As I remember correctly, I started once the skin and got a notification about an update. So I downlaoded the 1.6 Beta WITH m-files.

Originally Posted by ariszlo

Now, this fixes Album Art but breaks playing bookmarked streams or songs added by the "Type path/url" dialog.

A typical scenario while fixing a code: you get rid of one bug but create two more.

[ariszlo]

Originally Posted by PeterK.

As I remember correctly, I started once the skin and got a notification about an update. So I downlaoded the 1.6 Beta WITH m-files.

Yes! Thank you.

[ariszlo]

Trying to apply your pathToURL solution for quicksongpick.m, I added these lines:

Originally Posted by PeterK.

Function string pathToURL(string path);

string pathToURL(string path) { ... }

Then replaced this line (line 69 in the original file):

PHP Code:

playfile(filename); 


with this:

PHP Code:

playFile(pathToURL(filename)); 


And then this (line 83):

PHP Code:

setPrivateString(getSkinName(), buttonID, filename); 


with this:

PHP Code:

setPrivateString(getSkinName(), buttonID, "PATH|"+filename); 


The result is partial success: the first memory button can save the currently played song but the other four buttons cannot. And once the first button has saved a song, it will never forget it. I cannot save another song.

[PeterK.]

Originally Posted by ariszlo

I don't have that. I only have playfile(filename);

It is the same.

Originally Posted by ariszlo

Replaced playfile(filename); with those lines and added the plDir.mi library:

code:
#include "../../../lib/pldir.mi"

but it still does not work for me.

Memory Buttons section should look like this:

PHP Code:

MemoryButtons.onLeftButtonUp(int x, int y)
{
    buttonDown = 0;
    buttondelay.stop();
    
    if (songrecorded) return;
    
    string filename = getPrivateString(getSkinName(), buttonID, "");
    if (filename!="")
    {
        plEdit.clear();
        plEdit.enqueueFile(filename);
        int playThisFile=plEdit.getNumTracks();
        plEdit.playTrack(playThisFile);
        
        tempText("Play file: "+filename);
    }
} 


[PeterK.]

You know that you have to hold the memory button for 2 seconds untill you see the text "Song recorded" on the display?

[ariszlo]

The good news: The plDir solution for quicksongpick.m works fine with songs on disk. I'm just not dexterous enough with small buttons. I must have held the mouse button down a couple of pixels away...

The bad news: it hangs Winamp if I want to save a stream.

So it seems the plDir solution does not like streams. Both the left-drawer progbuttons and the right-side memory buttons are broken with streams if we use the plDir solution.

As for the left-drawer progbuttons I do not mind losing albumart if that is the price of playing streams or songs opened with the "Type path/url" dialog.

[ariszlo]

Fixed progbutton.m!

The solution is what you posted in #13 with one difference. Do not change this line:

PHP Code:

string currPath = getPlayItemMetaDataString("filename"); 


[PeterK.]

Originally Posted by ariszlo

Fixed progbutton.m!

Great news...
And those memory buttons?

[ariszlo]

Originally Posted by PeterK.

Great news...

A huge THANK YOU to you, PeterK.!

Originally Posted by PeterK.

And those memory buttons?

That's tough. I will start a new topic about them.

[PeterK.]

Off topic: Seeing your new signature one might think we pay you for this.

[ariszlo]

Originally Posted by PeterK.

Off topic: Seeing your new signature one might think we pay you for this.

I'm just ashamed that my Vortex topics are pushing down your release announcements from the main page.

[PeterK.]

Originally Posted by ariszlo

I'm just ashamed that my Vortex topics are pushing down your release announcements from the main page.

As long as they are on the Modern Skins page visible you do not have to worry about it. This section is not Quinto's section but ours. Right?