kdetv: bug in dcop function "navigate" from teletext interface

[BongoMan]

Binary package hint: kdetv

Hello,

I'm running kubuntu 7.10 with kdetv 0.8.9-1 (you can find the sources here: https://launchpad.net/ubuntu/+source/kdetv/0.8.9-1)

The dcop function "TelexIface navigate(int page)" in kdetv doesn't accept the argument "100", which is actually the most important page to navigate to in teletext. I found the error in kdetv/plugins/misc/telex/telex.cpp:

void Plugin::navigate( int page )
    {
        if ( page > 100 )
            navigate( Page( page ) );
    }

The if-condition should be: if ( page > 99 )

I attached a patch which changes the 100 to 99.

ciao,
Pascal Pollet

[Daniel T Chen (crimsun)]

Is this symptom still reproducible in 8.10 or 9.04?

[BongoMan (pascal-bongosoft)]

yes, it is in 8.10. Im am now running Intrepid and had to patch and compile the sources again for this to work. For 9.04, I don't know...

[BongoMan (pascal-bongosoft)]

Hey, sometimes I don't understand how things work in Launchpad. This bug is now older than one year, it has changed it's status twice and is now marked as "new". What is new about that? It is a clear bug and I provided an exact description and the solution. It is one line to change in the sources. Why does someone package this for new distribution releases (already 4 times since initial report) and does not correct the bug before? It would take ~30 seconds. Sorry, I don't get that, what's reporting bugs good for?

[Thomas Hotz (thotz-deactivatedaccount)]

Setting to confirmed.