[dbus] text objects created with dbus are weird

- the xml editor view does not show the newly created text object either
- the text object does not survive save/load.

I have a patch that improves the issue. The created text now behaves much more like text created through the UI.
Bugs certainly remain though.

gboolean
document_interface_text (DocumentInterface *object, int x, int y, gchar *text, GError **error)
{

  Inkscape::XML::Node *text_node = dbus_create_node(object->desk, "svg:text");
    Inkscape::XML::Node *tspan_node = NULL;

    tspan_node = dbus_create_node(object->desk, "svg:tspan");
    sp_repr_set_svg_double(tspan_node, "x", x);
    sp_repr_set_svg_double(tspan_node, "y", y);

    // Add text content node to tspan
    SPDocument * doc = sp_desktop_document (object->desk);
    Inkscape::XML::Document *xml_doc = doc->getReprDoc();

    Inkscape::XML::Node *text_content = xml_doc->createTextNode(text);
    tspan_node->appendChild(text_content);
    text_node->appendChild(tspan_node);
    // Inkscape::GC::release(text_content);
     finish_create_shape (object, error, text_node, (gchar *)"create text");
    return TRUE;
}

Solves most of the issues in this bug.

Please note that I changed the interface to return the object id.
I think this correct and the previous behaviour a bug.

I have a much better patch now below. It is simpler and supports linefeeds.

gchar*
document_interface_text (DocumentInterface *object, int x, int y, gchar *text, GError **error)
{

  Inkscape::XML::Node *text_node = dbus_create_node(object->desk, "svg:text");
    sp_repr_set_int(text_node, "x", x);
    sp_repr_set_int(text_node, "y", y);
    //just a workaround so i can get an spitem from the name
    gchar *name = finish_create_shape (object, error, text_node, (gchar *)"create text");

    SPItem* text_obj=(SPItem* )get_object_by_name(object->desk, name, error);
    sp_te_set_repr_text_multiline(text_obj, text);

    return name;
}

I will instead publish my dbus fixes in a branch:
https://code.launchpad.net/~joakim-verona/inkscape/dbus-fixes

<off-topic>
Sorry for yet another off-topic comment. Your work on testing and improving the dbus API and sharing your patches is highly appreciated!

> I have a much better patch now below. It is simpler and supports linefeeds.

Could you attach the patch as unified diffs? (e.g. created with 'bzr diff [FILE…]')?

See also
<http://thread.gmane.org/gmane.comp.graphics.inkscape.devel/32992/focus=32998>
<http://doc.bazaar.canonical.com/latest/en/user-reference/diff-help.html>
</off-topic>

oops - I definitely need to reload a page before posting a comment... ;)

Please disregard my previous off-topic comment!

Fix committed in the trunk, revision 10114.
Thanks Joakim!