[Konversation-devel] [Bug 192512] New: Regression in script argument order in script launcher

[Eike Hein]

https://bugs.kde.org/show_bug.cgi?id=192512

           Summary: Regression in script argument order in script launcher
           Product: konversation
           Version: SVN
          Platform: Unlisted Binaries
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: konversation-devel at kde.org
        ReportedBy: hein at kde.org
            Blocks: 187307


The signature for the D-Bus 'say' interface used by scripts is currently as
follows:

method void org.kde.konversation.say(QString server, QString target, QString
command)

However, the script launcher code currently passes the arguments in 'target
connectionId' order when it runs a script. This is problematic:

- It would be more intuitive for script authors if the positions of the
arguments handed to the script would correspond to the positions they need to
be used in in the D-Bus call.

- It's a change from Konversation/KDE3, where the arguments passed to scripts
were 'port connectionId target'. "Port" no longer being relevant to us with
D-Bus, this leaves 'connectionId target', i.e. exactly the opposite of what the
KDE 4 port does right now.

This should be changed before we make any public release of the KDE 4 port so
we don't annoy script authors too much by changing it later. It also means we
need to edit all the bundled scripts again.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.