Bug found in the new api code reported in chan

[g1itch]

Inconsistent query in HandleGetInboxMessageById():

The current head of the master branch has a broken API, specifically when calling getInboxMessageById() the following error is produced:

Unexpected API Failure - _dump_inbox_message() takes exactly 8 arguments (7 given)

This is the result of the SQL query returning 7 values instead of 8. It is missing the msgid as the first value. See the following line that is the issue:

PyBitmessage/src/api.py

Line 933 in 380530c

"SELECT toaddress, fromaddress, subject, received, message,"

The following code:

    queryreturn = sqlQuery(
        "SELECT toaddress, fromaddress, subject, received, message,"
        " encodingtype, read FROM inbox WHERE msgid=?", msgid
    )

Should be changed to:

    queryreturn = sqlQuery(
        "SELECT msgid, toaddress, fromaddress, subject, received, message,"
        " encodingtype, read FROM inbox WHERE msgid=?", msgid
    )