Fixing bug by adding WINDOWS 8 a a new Windows version in FinderFactory which provides the correct finder implementation based on the OS.

Author: Emmanuel Hugonnet

src/main/java/org/silverpeas/openoffice/OfficeLauncher.java

@@ -24,8 +24,6 @@
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
-import org.apache.commons.httpclient.HttpException;
-
 import org.silverpeas.openoffice.util.FinderFactory;
 import org.silverpeas.openoffice.util.MessageDisplayer;
 import org.silverpeas.openoffice.util.MessageUtil;
@@ -34,6 +32,8 @@
 import org.silverpeas.openoffice.windows.FileWebDavAccessManager;
 import org.silverpeas.openoffice.windows.MsOfficeVersion;
 
+import org.apache.commons.httpclient.HttpException;
+
 /**
  * @author Emmanuel Hugonnet
  */

src/main/java/org/silverpeas/openoffice/util/FinderFactory.java

@@ -1,25 +1,22 @@
 /**
  * Copyright (C) 2000 - 2009 Silverpeas
  *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License as
- * published by the Free Software Foundation, either version 3 of the
- * License, or (at your option) any later version.
+ * This program is free software: you can redistribute it and/or modify it under the terms of the
+ * GNU Affero General Public License as published by the Free Software Foundation, either version 3
+ * of the License, or (at your option) any later version.
  *
- * As a special exception to the terms and conditions of version 3.0 of
- * the GPL, you may redistribute this Program in connection with Free/Libre
- * Open Source Software ("FLOSS") applications as described in Silverpeas's
- * FLOSS exception.  You should have received a copy of the text describing
- * the FLOSS exception, and it is also available here:
+ * As a special exception to the terms and conditions of version 3.0 of the GPL, you may
+ * redistribute this Program in connection with Free/Libre Open Source Software ("FLOSS")
+ * applications as described in Silverpeas's FLOSS exception. You should have received a copy of the
+ * text describing the FLOSS exception, and it is also available here:
  * "http://repository.silverpeas.com/legal/licensing"
  *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Affero General Public License for more details.
+ * This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without
+ * even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Affero General Public License for more details.
  *
- * You should have received a copy of the GNU Affero General Public License
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ * You should have received a copy of the GNU Affero General Public License along with this program.
+ * If not, see <http://www.gnu.org/licenses/>.
  */
 
 /*
@@ -45,9 +42,10 @@ public static OfficeFinder getFinder(final MsOfficeType contentType) {
       case WINDOWS_XP:
       case WINDOWS_VISTA:
       case WINDOWS_SEVEN:
+      case WINDOWS_8:
         if (contentType.isMsOfficeCompatible()) {
-          return new MsOfficeRegistryHelper();
-        }
+        return new MsOfficeRegistryHelper();
+      }
         return new WindowsOpenOfficeFinder();
       case LINUX:
         return new WhereisHelper();