openlp-core team mailing list archive

[Andreas Preikschat <andreaspreikschat@xxxxxxxxxx>]

Andreas Preikschat has proposed merging lp:~googol/openlp/lo-fix into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~googol/openlp/lo-fix/+merge/170676

Hello,

- removed spaces to fix LO (partly!)

I was trying to get LO to work with python 3. Somewhere I read that there should not be any space.

http://pastebin.com/VyGT084h

As you can see the spaces causes "UNO URL contains bad descriptor name".
-- 
https://code.launchpad.net/~googol/openlp/lo-fix/+merge/170676
Your team OpenLP Core is requested to review the proposed merge of lp:~googol/openlp/lo-fix into lp:openlp.

=== modified file 'openlp/core/utils/__init__.py'
--- openlp/core/utils/__init__.py	2013-06-06 06:46:11 +0000
+++ openlp/core/utils/__init__.py	2013-06-20 18:12:26 +0000
@@ -346,9 +346,9 @@
     """
     log.debug(u'get UNO Desktop Openoffice - resolve')
     if UNO_CONNECTION_TYPE == u'pipe':
-        return resolver.resolve(u'uno:pipe,name=openlp_pipe; urp;StarOffice.ComponentContext')
+        return resolver.resolve(u'uno:pipe,name=openlp_pipe;urp;StarOffice.ComponentContext')
     else:
-        return resolver.resolve(u'uno:socket,host=localhost,port=2002; urp;StarOffice.ComponentContext')
+        return resolver.resolve(u'uno:socket,host=localhost,port=2002;urp;StarOffice.ComponentContext')
 
 
 def format_time(text, local_time):