Issue 26133 - Wizards: WebWizard fails to open on PPC Plattforms.
Summary: Wizards: WebWizard fails to open on PPC Plattforms.
Status: CLOSED FIXED
Alias: None
Product: Installation
Classification: Application
Component: code (show other issues)
Version: OOo 1.1.1RC
Hardware: PowerPC (PPC) All
: P2 Trivial (vote)
Target Milestone: OOo 1.1.1
Assignee: Unknown
QA Contact: issues@installation
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-04 19:27 UTC by eric_openoffice
Modified: 2004-09-27 11:21 UTC (History)
4 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
Patch for WebWizard to launch on PPC Plattform (394 bytes, patch)
2004-03-04 19:28 UTC, eric_openoffice
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description eric_openoffice 2004-03-04 19:27:43 UTC
On the PowerPC Plattforms using Mac OS X or Linux the WebWizards fails to
launch. There is only a blank box with an exclamation mark and an OK Button.

Kevin B. Hendricks created an patch which works fine with his build and with my
build too.
Comment 1 eric_openoffice 2004-03-04 19:28:29 UTC
Created attachment 13577 [details]
Patch for WebWizard to launch on PPC Plattform
Comment 2 Olaf Felka 2004-03-05 08:46:21 UTC
Please have a look.
Comment 3 khendricks 2004-03-05 12:29:26 UTC
Hi, 
 
I can confirm this issue. 
 
The problem seems to be that the HtmlAutoPilotBasic.Main passes a NoArgs() in the arguments 
to create the private:factory/swriter/web but since NoArgs() was never dimensioned this plays 
havoc with things generating an error which is cuaght and all you see is an empty error block. 
 
This same problem appears with Linux Sparc in the 680 series as well. 
 
Once I add the Dim NoArgs() the private:factory/swriter/web URL opens fine and it all works well. 
 
This fix is needed for MacOSX and PPC Linux in OOo 1.1.1 and it appears to be needed for 
Sparc as well.  In fact, it should be needed for all platforms.  I can't figure out why x86 does not 
have the same issue. 
 
Kevin 
 
 
 
Comment 4 khendricks 2004-03-05 16:49:04 UTC
Hi, 
 
Adding myself to CC on this to keep track of its progress and possible inclusion in OOo 1.1.1 
final. 
 
Kevin 
 
Comment 5 Martin Hollmichel 2004-03-09 16:25:14 UTC
this patch will not work for Linuxx86
Comment 6 khendricks 2004-03-09 17:40:06 UTC
Hi Martin,

I am confused.

How can it not work?  It is simply doing what the other xba files do and that is Dim NoArgs() before 
using them.

What error message are you getting from Dimensioning NoArgs?

Here is a greb of share/basic/*/*.xba that show other basic programs doing the same.

.//ImportWizard/FilesModul.xba:Dim NoArgs()
.//ImportWizard/FilesModul.xba:         oLogDocument.StoreAsUrl(sLogUrl, NoArgs())
.//ImportWizard/FilesModul.xba:Dim NoArgs() as New com.sun.star.beans.PropertyVal...


So should I instead have done Dim NoArgs() as New blah?

What am I missing here?

Kevin
Comment 7 Martin Hollmichel 2004-03-10 09:18:30 UTC
mh->tv: can you please answer as you did the test ?
Comment 8 tom 2004-03-10 13:41:55 UTC
Hi Kevin,

I am not quite sure why ther is an Error on Linux PPC. The correct revision of
the file <b>wizards/source/webwizard/HtmlAutoPilotBasic.xba</b> is <b>1.31</b>.
This revision includes the line <br>
<i>Public NoArgs as New com.sun.star.beans.PropertyValue</i>

This is also the reason, why another <i>Dim NoArgs</i> leads to an error.<br>
Please double-check if you are working with the correct revision of the file...
Comment 9 khendricks 2004-03-10 14:02:47 UTC
Hi,

Is there any difference between:

Public NoArgs()as New com.sun.star.beans.PropertyValue

as used in other xba files 

and 

Public NoArgs as New com.sun.star.beans.PropertyValue

Notice the version you are using has "NoArgs" not "NoArgs()" in the public statement yet passes 
NoArgs() not NoArgs.

To test this I removed my added Dim NoArgs() but then chenged your Public line to use "NoArgs()" and 
not "NoArg" and everything works agin on both MacOSX and Linux PowerPC.

So the Pulbic NoArg is notbeing considered the same thigns as Public NoArgs() for some reason.

What happnes on x86 Linux if you change that line from NoArgs to NoArgs()?

Thanks,

Kevin



Comment 10 tom 2004-03-10 14:19:44 UTC
I just tested it on X86 Linux

both
Public NoArgs as New com.sun.star.beans.PropertyValue
and
Public NoArgs() as New com.sun.star.beans.PropertyValue

work, althouth the latter one is more correct because it is defining the
Variable as an Array. Since we are passing an empty Array anyways it both works.

To achieve code level compatibility, you should use the same version of the file
in Linux PPC (that is: the first version).
Comment 11 Martin Hollmichel 2004-03-10 14:27:53 UTC
so the NoArgs() variant works for all platforms now ?
Comment 12 khendricks 2004-03-10 14:48:07 UTC
Hi,

Both do not work.  Passing an array is not the same as passing a non-array under PPC abi.
Arrays are passed by reference (i.e. indirected) while non-arrays are not.

The "more correct" way is the only way it will work on PPC Linux and MacOSX.

I am confused by your "first versus second" reference.

The correct cross-platform code should be:

Public NoArgs() as New com.sun.star.beans.PropertyValue

to match the use of NoArgs() in the method call.

So please make change in OOo 1.1.1 final so that it will work for all platforms.

Thank you,

Kevin

Comment 13 khendricks 2004-03-10 14:50:46 UTC
Hi Martin,

I can confirm that using the "more correct" NoArgs() works properly on both PPC Linux and MacOSX.  tv 
confirmed it works on Linux x86.  I do not have access to a 680 Sparc build to test the same thing for 
Linux Sparc.

Kevin
Comment 14 tom 2004-03-10 14:55:39 UTC
Kevin!

You are absolutely correct. It needs to be NoArgs(). We still do not understand
why NoArgs -whithout paranthesis works under X86 (Windows, Linux)

So the fix needs to be:
RCS file: /cvs/installation/wizards/source/webwizard/HtmlAutoPilotBasic.xba,v
retrieving revision 1.31
diff -u -r1.31 HtmlAutoPilotBasic.xba
--- HtmlAutoPilotBasic.xba      27 Mar 2003 17:58:56 -0000      1.31
+++ HtmlAutoPilotBasic.xba      10 Mar 2004 14:53:35 -0000
@@ -35,7 +35,7 @@
 Public oBaseDocument as Object
 Public oViewCursor as Object
 Public oViewSettings as Object
-Public NoArgs as New com.sun.star.beans.PropertyValue
+Public NoArgs() as New com.sun.star.beans.PropertyValue

 Public oCursor as Object
 Public oBookmarks as Object

Comment 15 khendricks 2004-03-10 15:06:27 UTC
Hi,

Thanks for confirming that.  I think the true power of having a product that is built on multiple 
platforms with multiple abis (x86 Linux, Win and Solaris, PPC on Linux and MacOSX, Sparc on Linux and 
Solaris) etc is that it helps find lots of little nits that would otherwise stick around longer without 
testing on multiple platforms.

Thanks again for looking at this!

Kevin
 
Comment 16 jens-heiner.rechtien 2004-03-10 15:17:40 UTC
Fixed on ooo111regression4
Comment 17 jens-heiner.rechtien 2004-03-10 15:18:03 UTC
Fixed.
Comment 18 Martin Hollmichel 2004-09-27 11:21:58 UTC
close issue.