songinfo redirection in 7.1ff?

[pippin]

I feel that my own songinfo.html (iPeng skin) does not get called anymore in 7.1 and 7.2 but some other page.
Is there a change in these versions?

[erland]

I'm not sure which page you are supposed to override in 7.1, hopefully someone that actually knows can give you more information.

However I had a quick look in the code and the "Song Info" page seems to be handled by xmlbrowser.html where the upper part seems to be delegated to songinfo_header.html.

[pippin]

I'm not sure which page you are supposed to override in 7.1, hopefully someone that actually knows can give you more information.

However I had a quick look in the code and the "Song Info" page seems to be handled by xmlbrowser.html where the upper part seems to be delegated to songinfo_header.html.

that fits what I see. The output looks like output from xmlbrowser.
Until 7.0.1 it was loading a file called songinfo.html.
Actually I have no idea what this damn redirection is all about. It's now the second time it breaks my skin (the first one being playlist management, which I'm not going to fix) and I can see about zero useful reasons to do this since all you would have to do this to switch to another page in other skins is to change a single link in hreftemplates.

[andyg]

On Jul 20, 2008, at 6:35 AM, pippin wrote:

>
> erland;321427 Wrote:
>> I'm not sure which page you are supposed to override in 7.1,
>> hopefully
>> someone that actually knows can give you more information.
>>
>> However I had a quick look in the code and the "Song Info" page seems
>> to be handled by xmlbrowser.html where the upper part seems to be
>> delegated to songinfo_header.html.
>
> that fits what I see. The output looks like output from xmlbrowser.
> Until 7.0.1 it was loading a file called songinfo.html.
> Actually I have no idea what this damn redirection is all about. It's
> now the second time it breaks my skin (the first one being playlist
> management, which I'm not going to fix) and I can see about zero
> useful
> reasons to do this since all you would have to do this to switch to
> another page in other skins is to change a single link in
> hreftemplates.

songinfo has been replaced by trackinfo. You should still be able to
use songinfo.html though, it's an alias for the same page. But yes
under the hood that page is now running based on XMLBrowser.

This came about as a result of http://bugs.slimdevices.com/show_bug.cgi?id=6930
and a need to get rid of the 3 separate implementations of songinfo.

[mherger]

> songinfo has been replaced by trackinfo. You should still be able to
> use songinfo.html though, it's an alias for the same page.

Which means: you can still open songinfo.html, but your template isn't
used any more. You'll have to modify xmlbrowser.html

Michael

[pippin]

That's all fine but I still don't get why you need the redirections for this.
What is the problem in just calling trackinfo.html instead of songinfo.html and let skins which have own code just use that.
The fundamental issue in all this stems from the fact that you are not free to load any *.html page through the server but are bound to the pages that are being processed by the server.
A good alternative - in my view - would be to be able to load any page and then get the parameters like - say - status.html.
Currently I branch out status.html through parameters but this has the big drawback to break caching.