Bug 46339

Created attachment 57366 [details]
Java stacktrace

I reproduced this problem from a new odb file on pc Debian 86-64 with 3.5 branch (updated yesterday)
java version "1.6.0_24"
OpenJDK Runtime Environment (IcedTea6 1.11pre) (6b24~pre2-1)
OpenJDK 64-Bit Server VM (build 20.0-b12, mixed mode)

I attached the Java stacktrace. I haven't took a look for the moment, just give these info first.

Oups, I didn't put the first line, the main info, type of exception : com.sun.star.container.ElementExistException

Created attachment 57367 [details]
Searching the element in double

I added a System.out.prinln to find which element is in double.

Keeping on investigation, it seems in the file wizards/com/sun/star/wizards/ui/ControlScroller.java, the line 114 returns empty :
sIncSuffix = com.sun.star.wizards.common.Desktop.getIncrementSuffix(CurUnoDialog.getDlgNameAccess(), "imgBackground");

So line 115 :
CurUnoDialog.insertScrollBar("TitleScrollBar" + sIncSuffix, ....
always uses "TitleScrollBar" as first parameter instead of TitleScrollBar<n>

Must keep on :-)

By putting some printstacktrace, I just noticed that the constructor ControlScroller is called twice :
- once by AggregateComponent (from QueryWizard.java:209)
- once by TitlesComponent (from QueryWizard.java:342)

Then I don't know if there's something in ControlScroller.java with sIncSuffix (line 114) or the call to CurUnoDialog.insertScrollBar (line 115) or if it's something else. All the files seem have been modified 1 year ago.

I succeeded to make this work by 2 options :
1) Either
Replace the line 114 :
sIncSuffix = com.sun.star.wizards.common.Desktop.getIncrementSuffix(CurUnoDialog.getDlgNameAccess(), "imgBackground");

by this :
sIncSuffix = com.sun.star.wizards.common.Desktop.getIncrementSuffix(CurUnoDialog.getDlgNameAccess(), "TitleScrollBar");

2) Or
Replace this line 115 :
xScrollBar = CurUnoDialog.insertScrollBar("TitleScrollBar" + sIncSuffix, 0,

by this :
xScrollBar = CurUnoDialog.insertScrollBar("imgBackground" + sIncSuffix, 0,

(In reply to comment #6)
> I succeeded to make this work by 2 options :
> 1) Either
> Replace the line 114 :
> sIncSuffix =
> com.sun.star.wizards.common.Desktop.getIncrementSuffix(CurUnoDialog.getDlgNameAccess(),
> "imgBackground");
> 
> by this :
> sIncSuffix =
> com.sun.star.wizards.common.Desktop.getIncrementSuffix(CurUnoDialog.getDlgNameAccess(),
> "TitleScrollBar");
> 
> 2) Or
> Replace this line 115 :
> xScrollBar = CurUnoDialog.insertScrollBar("TitleScrollBar" + sIncSuffix, 0,
> 
> by this :
> xScrollBar = CurUnoDialog.insertScrollBar("imgBackground" + sIncSuffix, 0,
Just forgot to precise :
- these lines are in wizards/com/sun/star/wizards/ui/ControlScroller.java
- "TitleScrollBar" is used too line 279 in the same file. Does it indicate something about which option to choose ? (if it's one of them of course :-) )

*Presumably* this arises because of the fix
http://cgit.freedesktop.org/libreoffice/core/commit/?id=03e17a141fbb4e1242de9d9979b5b699e6840454
for fdo#42543 ?

Given that the above removed inserting a imgBackground control, my guess would be that your version 1 makes more sense, but I'm just guessing.

Julien Nabet committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=bad98a1aa71e1a76c6ca4033f82168fe5b098b64

fdo#46339: Fix Query Wizard by putting the right ElementName

Julien Nabet committed a patch related to this issue.
It has been pushed to "libreoffice-3-5":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=b3a326c16127c9f696e6e4e264729991c4ef4981&g=libreoffice-3-5

fdo#46339: Fix Query Wizard by putting the right ElementName


It will be available in LibreOffice 3.5.2.

Julien Nabet committed a patch related to this issue.
It has been pushed to "libreoffice-3-5-1":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=83ba2fb18f3ec7835591c6ffeda6e713bc09a9f9&g=libreoffice-3-5-1

fdo#46339: Fix Query Wizard by putting the right ElementName


It will be available already in LibreOffice 3.5.1.

I added Fix submitter as assignee because this will ease queries and bub tracking.