tdf#94351: Web Wizard multiselection doesn't work

Call getSelectedFiles instead of getFiles+adapt behavior so it deals
with an array of files with their full path

Change-Id: I21b6c32dea85460461925a9b44dac195d79b7eac
Reviewed-on: https://gerrit.libreoffice.org/18714
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

2 files changed, 7 insertions, 23 deletions

diff --git a/wizards/com/sun/star/wizards/web/WWD_Events.py b/wizards/com/sun/star/wizards/web/WWD_Events.py
index c5dc75e8566b..03f2e12d712f 100644
--- a/wizards/com/sun/star/wizards/web/WWD_Events.py
+++ b/wizards/com/sun/star/wizards/web/WWD_Events.py
@@ -894,39 +894,23 @@ class WWD_Events(WWD_Startup):
         def loadDocuments(self):
             offset = self.parent.selectedDoc[0] + 1 if (len(self.parent.selectedDoc) > 0) else self.parent.getDocsCount()
 
-            '''
-            if the user chose one file, the list starts at 0,
-            if he chose more than one, the first entry is a directory name,
-            all the others are filenames.
-            '''
-            if len(self.files) > 1:
-                start = 1
-            else:
-                start = 0
-
             #Number of documents failed to validate.
             failed = 0
 
             # store the directory
-            if start == 1:
-                self.parent.settings.cp_DefaultSession.cp_InDirectory = self.files[0]
-            else:
-                self.parent.settings.cp_DefaultSession.cp_InDirectory = \
-                    FileAccess.getParentDir(self.files[0])
+            self.parent.settings.cp_DefaultSession.cp_InDirectory = \
+                FileAccess.getParentDir(self.files[0])
 
             '''
             Here i go through each file, and validate it.
             If its ok, I add it to the ListModel/ConfigSet
             '''
 
-            for i in range(start, len(self.files)):
+            for i in range(0, len(self.files)):
                 doc = CGDocument()
                 doc.setRoot(self.parent.settings)
 
-                if start == 0:
-                    doc.cp_URL = self.files[i]
-                else:
-                    doc.cp_URL = FileAccess.connectURLs(self.files[0], self.files[i])
+                doc.cp_URL = self.files[i]
 
                 '''
                 so - i check each document and if it is ok I add it.
@@ -935,14 +919,14 @@ class WWD_Events(WWD_Startup):
                 done in the checkDocument(...) method
                 '''
                 if self.parent.checkDocument(doc, self.task, self.xC):
-                    index = offset + i - failed - start
+                    index = offset + i - failed
                     self.parent.settings.cp_DefaultSession.cp_Content.cp_Documents.add(index, doc)
                 else:
                     failed += 1
 
             # if any documents where added,
             # set the first one to be the current-selected document.
-            if len(self.files) > start + failed:
+            if len(self.files) > failed:
                 self.parent.setSelectedDoc([offset])
 
             # update the ui...
diff --git a/wizards/com/sun/star/wizards/web/WebWizardConst.py b/wizards/com/sun/star/wizards/web/WebWizardConst.py
index f6a2282dc9f7..363ecd80d05a 100644
--- a/wizards/com/sun/star/wizards/web/WebWizardConst.py
+++ b/wizards/com/sun/star/wizards/web/WebWizardConst.py
@@ -55,4 +55,4 @@ TASK_FINISH = "t_fin"
 when the user adds more than this number
 of documents to the list, a status dialog opens.
 '''
-MIN_ADD_FILES_FOR_DIALOG = 2
+MIN_ADD_FILES_FOR_DIALOG = 1