Select the cloned template right after creation

Author: car031

logicaldoc-gui/src/main/java/com/logicaldoc/gui/common/client/util/GridUtil.java

@@ -60,6 +60,47 @@ public void run() {
 			timer.schedule(100);
 		}
 	}
+	
+	/**
+	 * Scrolls the grid all down
+	 * 
+	 * @param listGrid the grid to process
+	 * @param listener optional listener inoked at the end of the scroll
+	 */
+	public static void scrollDownGrid(ListGrid listGrid, EndScrollListener listener) {
+		if (listGrid.getTotalRows() > 0) {
+			LD.contactingServer();
+
+			listGrid.scrollToRow(0);
+			listGrid.draw();
+
+			if (listGrid.getVisibleRows()[0] == -1) {
+				LD.clearPrompt();
+				return;
+			}
+
+			/*
+			 * With a timer we scroll the grid in order to fetch all the data
+			 */
+			final Timer timer = new Timer() {
+				public void run() {
+					Integer[] visibleRows = listGrid.getVisibleRows();
+					if (visibleRows[1] >= listGrid.getTotalRows() - 1) {
+						try {
+							if (listener != null)
+								listener.endScroll(listGrid);
+						} finally {
+							LD.clearPrompt();
+						}
+					} else if (visibleRows[0] != -1 && visibleRows[1] < listGrid.getTotalRows() - 1) {
+						listGrid.scrollToRow(visibleRows[1] + 1);
+						schedule(100);
+					}
+				}
+			};
+			timer.schedule(100);
+		}
+	}
 
 	/**
 	 * Prints a grid

logicaldoc-gui/src/main/java/com/logicaldoc/gui/frontend/client/document/DocumentToolbar.java

@@ -481,8 +481,7 @@ private void addScan() {
 
 	private void addUpload() {
 		add.addClickHandler(event -> {
-			DocumentsUploader uploader = new DocumentsUploader();
-			uploader.show();
+			new DocumentsUploader().show();
 			event.cancel();
 		});
 		addButton(add);

logicaldoc-gui/src/main/java/com/logicaldoc/gui/frontend/client/metadata/template/TemplatesPanel.java

@@ -19,7 +19,6 @@
 import com.smartgwt.client.widgets.grid.ListGrid;
 import com.smartgwt.client.widgets.grid.ListGridField;
 import com.smartgwt.client.widgets.grid.ListGridRecord;
-import com.smartgwt.client.widgets.grid.events.DataArrivedEvent;
 import com.smartgwt.client.widgets.layout.Layout;
 import com.smartgwt.client.widgets.layout.VLayout;
 import com.smartgwt.client.widgets.menu.Menu;
@@ -46,6 +45,8 @@ public class TemplatesPanel extends VLayout {
 
 	static final Canvas SELECT_TEMPLATE = new HTMLPanel(" " + I18N.message("selecttemplate"));
 
+	protected Long templateIdToSelect;
+
 	public TemplatesPanel() {
 		setWidth100();
 	}
@@ -152,8 +153,18 @@ public void onSuccess(GUITemplate template) {
 						});
 		});
 
-		list.addDataArrivedHandler((DataArrivedEvent event) -> infoPanel
-				.setMessage(I18N.message("showtemplates", Integer.toString(list.getTotalRows()))));
+		list.addDataArrivedHandler(event -> {
+			infoPanel.setMessage(I18N.message("showtemplates", Integer.toString(list.getTotalRows())));
+			if (templateIdToSelect != null)
+				for (ListGridRecord rec : list.getRecords()) {
+					if (templateIdToSelect.longValue() == rec.getAttributeAsDouble("id").longValue()) {
+						list.scrollToRow(list.getRowNum(rec));
+						list.selectRecord(rec);
+						break;
+					}
+				}
+			templateIdToSelect = null;
+		});
 
 		detailsContainer.setAlign(Alignment.CENTER);
 		detailsContainer.addMember(details);
@@ -191,6 +202,7 @@ public void onSuccess(Void result) {
 		clone.setTitle(I18N.message("clone"));
 		clone.addClickHandler(event -> TemplateService.Instance.get().clone(selectedTemplateId,
 				selectedRecord.getAttribute("name") + "-Clone", new AsyncCallback<GUITemplate>() {
+
 					@Override
 					public void onFailure(Throwable caught) {
 						GuiLog.serverError(caught);
@@ -199,18 +211,11 @@ public void onFailure(Throwable caught) {
 					@Override
 					public void onSuccess(GUITemplate templateClone) {
 						list.deselectAllRecords();
-						ListGridRecord newRecord = new ListGridRecord();
-						newRecord.setAttribute("id", templateClone.getId());
-						newRecord.setAttribute("readonly", "" + templateClone.isReadonly());
-						newRecord.setAttribute("name", templateClone.getName());
-						newRecord.setAttribute(LABEL,
-								templateClone.getLabel() != null ? templateClone.getLabel() : templateClone.getName());
-						newRecord.setAttribute(DESCRIPTION, templateClone.getDescription());
-						list.addData(newRecord);
-						list.selectRecord(newRecord);
-						list.scrollToRow(list.getRowNum(newRecord));
+						list.refresh(new TemplatesDS(false, null, GUITemplate.TYPE_DEFAULT));
+						templateIdToSelect = templateClone.getId();
 						showTemplateDetails(templateClone);
 					}
+
 				}));
 
 		contextMenu.setItems(clone, delete);