Catch exceptions of XMLEncoder

Author: car031

logicaldoc-gui/src/main/java/com/logicaldoc/gui/frontend/client/metadata/form/FormDetailsPanel.java

@@ -141,7 +141,6 @@ private void preparePropertiesTab(ChangedHandler changedHandler) {
 		TextItem name = ItemFactory.newSimpleTextItem("name", form.getName());
 		name.addChangedHandler(changedHandler);
 		name.setRequired(true);
-		name.setDisabled(form.getId() != 0L);
 
 		SelectItem template = ItemFactory.newTemplateSelector(true, null);
 		template.addChangedHandler(changedHandler);

logicaldoc-gui/src/main/java/com/logicaldoc/gui/frontend/client/reports/custom/ReportStandardProperties.java

@@ -13,6 +13,7 @@
 import com.smartgwt.client.widgets.form.fields.SelectItem;
 import com.smartgwt.client.widgets.form.fields.StaticTextItem;
 import com.smartgwt.client.widgets.form.fields.TextAreaItem;
+import com.smartgwt.client.widgets.form.fields.TextItem;
 import com.smartgwt.client.widgets.form.fields.events.ChangedHandler;
 import com.smartgwt.client.widgets.layout.HLayout;
 import com.smartgwt.client.widgets.layout.Layout;
@@ -70,7 +71,8 @@ private void refresh() {
 
 		StaticTextItem id = ItemFactory.newStaticTextItem("id", Long.toString(report.getId()));
 
-		StaticTextItem name = ItemFactory.newStaticTextItem("name", report.getName());
+		TextItem name = ItemFactory.newSimpleTextItem("name", report.getName());
+		name.addChangedHandler(changedHandler);
 
 		TextAreaItem description = ItemFactory.newTextAreaItem("description", report.getDescription());
 		description.setWidth(250);

logicaldoc-util/src/main/java/com/logicaldoc/util/io/IOUtil.java

@@ -22,10 +22,15 @@
 
 import org.apache.commons.io.IOUtils;
 import org.apache.commons.lang.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class IOUtil {
+
 	private static final int DEFAULT_BUFFER_SIZE = 10240; // ..bytes = 10KB.
 
+	private static final Logger log = LoggerFactory.getLogger(IOUtil.class);
+
 	private IOUtil() {
 		throw new IllegalStateException("Utility class");
 	}
@@ -101,6 +106,7 @@ public static byte[] getBytesOfStream(InputStream stream) throws IOException {
 	public static String serialize(Serializable object, String charset) {
 		ByteArrayOutputStream baos = new ByteArrayOutputStream();
 		try (XMLEncoder encoder = new XMLEncoder(baos, StringUtils.defaultString(charset, "UTF-8"), false, 0);) {
+			encoder.setExceptionListener(e -> log.warn(e.getMessage(), e));
 			encoder.writeObject(object);
 		}
 		return baos.toString().trim().replace("\n", "");
@@ -129,7 +135,8 @@ public static Object deserialize(String xml) {
 	 * @throws IOException I/O error
 	 * @throws URISyntaxException The URL is invalid
 	 */
-	public static void download(final String url, File dest, int timeout, int bufferSize) throws IOException, URISyntaxException {
+	public static void download(final String url, File dest, int timeout, int bufferSize)
+			throws IOException, URISyntaxException {
 		if (url == null || url.isEmpty()) {
 			throw new IOException("Url argument is not specified"); // URL
 																	// isn't

logicaldoc-util/src/main/java/com/logicaldoc/util/io/TarUtil.java

@@ -45,21 +45,20 @@ public List<String> listEntries(File tarFile) throws IOException {
 		try {
 			try (FileInputStream fis = new FileInputStream(tarFile);
 					BufferedInputStream bis = new BufferedInputStream(fis);
-					ArchiveInputStream input = new ArchiveStreamFactory().createArchiveInputStream(bis);) {
-
-				if (input instanceof TarArchiveInputStream tarInput) {
-					try {
-						TarArchiveEntry entry = tarInput.getNextTarEntry();
-						while (entry != null) {
-							String name = entry.getName();
-							if (name.endsWith("/"))
-								name = name.substring(0, name.lastIndexOf('/'));
-							entries.add(name);
-							entry = tarInput.getNextTarEntry();
-						}
-					} finally {
-						tarInput.close();
+					ArchiveInputStream<TarArchiveEntry> tarInput = new ArchiveStreamFactory()
+							.createArchiveInputStream(bis);) {
+
+				try {
+					TarArchiveEntry entry = tarInput.getNextEntry();
+					while (entry != null) {
+						String name = entry.getName();
+						if (name.endsWith("/"))
+							name = name.substring(0, name.lastIndexOf('/'));
+						entries.add(name);
+						entry = tarInput.getNextEntry();
 					}
+				} finally {
+					tarInput.close();
 				}
 			}
 		} catch (ArchiveException e) {