Pull request #6685: REPORT-62623 && REPORT-62399 && REPORT-62547 && REPORT-62626 && REPORT-62400 && REPORT-62542 && REPORT-62544 bug 修复

Merge in DESIGN/design from ~KERRY/design_10.0:feature/x to feature/x

* commit 'fd7a89abd349ebb97eae2bd0067f4a5c5f82b883':
  REPORT-62623 && REPORT-62399 && REPORT-62547 && REPORT-62626 && REPORT-62400 && REPORT-62542 && REPORT-62544
  bug修复
  REPORT-62400 && REPORT-62399 && REPORT-62547【接触点优化】拖拽动效出现时，预览图不消失；【接触点优化】轮播的同时，鼠标悬浮在组件上，轮播图不消失，与组件预览图同时出现；【接触点优化】轮播的图片固定大小，导致预览图显示不全

designer-base/src/main/java/com/fr/design/DesignerCloudURLManager.java

@@ -63,7 +63,7 @@ public class DesignerCloudURLManager implements XMLable {
             executorService.submit(() -> {
                 updateURLXMLFile(key, latestUrl);
             });
-            return url;
+            return latestUrl;
         }
         //本地缓存不为空时，直接返回对应 url，同时异步更新
         executorService.submit(() -> {
@@ -74,7 +74,7 @@ public class DesignerCloudURLManager implements XMLable {
     }
 
     private synchronized void updateURLXMLFile(String key, String url) {
-        if (!urlMap.containsKey(key) || !url.equals(urlMap.get(key))) {
+        if (StringUtils.isNotEmpty(url) && (!urlMap.containsKey(key) || !url.equals(urlMap.get(key)))) {
             urlMap.put(key, url);
             saveURLXMLFile();
         }

designer-base/src/main/java/com/fr/design/mainframe/share/ComponentShareUtil.java

@@ -66,7 +66,7 @@ public class ComponentShareUtil {
      * @return boolean
      */
     public static boolean needShowFirstDragAnimate() {
-        return ComponentReuseNotificationInfo.getInstance().isFirstDrag() && !hasTouched();
+        return ComponentReuseNotificationInfo.getInstance().isFirstDrag();
     }
 
     /**

designer-form/src/main/java/com/fr/design/mainframe/FormWidgetDetailPane.java

@@ -13,7 +13,6 @@ import com.fr.design.mainframe.share.ui.local.LocalWidgetRepoPane;
 import com.fr.design.mainframe.share.ui.online.OnlineWidgetRepoPane;
 import com.fr.general.locale.LocaleCenter;
 import com.fr.general.locale.LocaleMark;
-
 import javax.swing.BorderFactory;
 import javax.swing.Icon;
 import javax.swing.JPanel;
@@ -33,13 +32,15 @@ import java.util.List;
  * Date: 14-7-8
  * Time: 下午8:18
  */
-public class FormWidgetDetailPane extends FormDockView{
+public class FormWidgetDetailPane extends FormDockView {
     private static final int ONLINE_TAB = 1;
 
     private JPanel centerPane;
     private UIHeadGroup headGroup;
     private List<BasicPane> paneList;
     private CardLayout cardLayout;
+    //用来标记当前组件库界面是否处于已触达状态
+    private boolean hasTouched = false;
 
     private boolean isEmptyPane = false;
 
@@ -50,7 +51,7 @@ public class FormWidgetDetailPane extends FormDockView{
         return HOLDER.singleton;
     }
 
-    private  FormWidgetDetailPane(){
+    private FormWidgetDetailPane() {
         setLayout(FRGUIPaneFactory.createBorderLayout());
     }
 
@@ -77,7 +78,7 @@ public class FormWidgetDetailPane extends FormDockView{
     /**
      * 初始化
      */
-    public void refreshDockingView(){
+    public void refreshDockingView() {
         if (isEmptyPane) {
             return;
         }
@@ -87,7 +88,7 @@ public class FormWidgetDetailPane extends FormDockView{
             clearDockingView();
             return;
         }
-
+        hasTouched = ComponentShareUtil.hasTouched();
         initPaneList();
         this.setBorder(null);
         cardLayout = new CardLayout();
@@ -113,13 +114,21 @@ public class FormWidgetDetailPane extends FormDockView{
     }
 
 
+    /**
+     * 判断是否可触达
+     *
+     * @return boolean
+     */
+    public boolean hasTouched() {
+        return hasTouched;
+    }
+
 
-    public void resetEmptyPane(){
+    public void resetEmptyPane() {
         this.isEmptyPane = false;
     }
 
 
-
     /**
      * 清除数据
      */
@@ -129,7 +138,7 @@ public class FormWidgetDetailPane extends FormDockView{
         this.add(psp, BorderLayout.CENTER);
     }
 
-    public void switch2Empty(){
+    public void switch2Empty() {
         isEmptyPane = true;
         this.removeAll();
         JPanel panel = FRGUIPaneFactory.createVerticalFlowLayout_Pane(true, FlowLayout.LEADING, 0, 5);

designer-form/src/main/java/com/fr/design/mainframe/share/ui/block/PreviewWidgetBlock.java

@@ -5,6 +5,7 @@ import com.fr.design.DesignerEnvManager;
 import com.fr.design.gui.ilable.UILabel;
 import com.fr.design.layout.FRGUIPaneFactory;
 import com.fr.design.mainframe.EastRegionContainerPane;
+import com.fr.design.mainframe.FormWidgetDetailPane;
 import com.fr.design.mainframe.reuse.ComponentReuseNotificationInfo;
 import com.fr.design.mainframe.share.ComponentShareUtil;
 import com.fr.design.mainframe.share.collect.ComponentCollector;
@@ -147,7 +148,7 @@ public abstract class PreviewWidgetBlock<T> extends JPanel implements MouseListe
     @Override
     public void mouseEntered(MouseEvent e) {
         hover = true;
-        if (ComponentShareUtil.needShowFirstDragAnimate() && checkWidget()) {
+        if (ComponentShareUtil.needShowFirstDragAnimate() && !FormWidgetDetailPane.getInstance().hasTouched() && checkWidget()) {
             schedule(ANIMATE_START_TIME);
             awtEventListener = event -> {
                 if (!this.isShowing()) {

designer-form/src/main/java/com/fr/design/mainframe/share/ui/online/CarouselStateManger.java

@@ -4,14 +4,11 @@ package com.fr.design.mainframe.share.ui.online;
 import com.fr.general.ComparatorUtils;
 import com.fr.stable.StringUtils;
 
-import java.util.concurrent.CountDownLatch;
-
 /**
  * Created by kerry on 2021/10/22
  */
 public class CarouselStateManger {
 
-    private static final String START_CAROUSEL ="START_CAROUSEL";
     public static final String RIGHT_CLICK ="RIGHT_CLICK";
     public static final String DOWNLOAD_COMPONENT ="DOWNLOAD_COMPONENT";
     public static final String FIRST_DRAG_ANIMATE ="FIRST_DRAG_ANIMATE";
@@ -19,7 +16,6 @@ public class CarouselStateManger {
 
     private CarouseState state;
 
-    private final CountDownLatch countDownLatch = new CountDownLatch(1);
 
     private String suspendEvent;
 
@@ -36,14 +32,6 @@ public class CarouselStateManger {
 
     }
 
-    public void countDown() {
-        countDownLatch.countDown();
-    }
-
-    public void countDownLatchAwait() throws InterruptedException {
-        countDownLatch.await();
-    }
-
     public void start() {
         this.state = CarouseState.RUNNING;
     }

designer-form/src/main/java/com/fr/design/mainframe/share/ui/online/embed/OnlineEmbedFilterSelectPane.java

@@ -7,11 +7,9 @@ import com.fr.design.mainframe.share.ui.online.CarouselStateManger;
 import com.fr.design.mainframe.share.ui.widgetfilter.FilterPane;
 import com.fr.form.share.base.DataLoad;
 import com.fr.form.share.bean.OnlineShareWidget;
-import com.fr.form.share.constants.ShareComponentConstants;
-import com.fr.log.FineLoggerFactory;
+import com.fr.form.share.exception.NetWorkFailedException;
+import com.fr.general.http.HttpClient;
 import com.fr.module.ModuleContext;
-import com.fr.stable.EncodeConstants;
-import com.fr.third.springframework.web.util.UriUtils;
 
 import javax.imageio.ImageIO;
 import javax.swing.JPanel;
@@ -24,7 +22,8 @@ import java.awt.Rectangle;
 import java.awt.event.AWTEventListener;
 import java.awt.event.MouseEvent;
 import java.io.IOException;
-import java.net.URL;
+import java.net.HttpURLConnection;
+import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicInteger;
@@ -34,10 +33,12 @@ import java.util.concurrent.atomic.AtomicInteger;
  */
 public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane {
     private static final String CAROUSEL_PREVIEW = "carousel_preview";
+    private static final int CAROUSE_IMAGE_LOAD_TIMEOUT = 2000;
     private OnlineShareWidget[] showWidgets;
 
     private PreviewDialog previewDialog;
     private JPanel widgetPane;
+    private final CountDownLatch countDownLatch = new CountDownLatch(1);
 
     private final AWTEventListener awtEventListener;
 
@@ -46,6 +47,7 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
         awtEventListener = event -> {
             if (event instanceof MouseEvent) {
                 if (((MouseEvent) event).getClickCount() > 0) {
+                    try {
                         Point selectPanePoint = OnlineEmbedFilterSelectPane.this.getLocationOnScreen();
                         Dimension selectPaneDimension = OnlineEmbedFilterSelectPane.this.getSize();
                         Rectangle selectPaneRec = new Rectangle(selectPanePoint.x, selectPanePoint.y, selectPaneDimension.width, selectPaneDimension.height);
@@ -53,6 +55,10 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
                                 !selectPaneRec.contains(((MouseEvent) event).getLocationOnScreen())) {
                             CarouselStateManger.getInstance().stop();
                         }
+                    } catch (Exception e) {
+                        //忽略
+                    }
+
                 }
 
             }
@@ -77,17 +83,34 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
 
     protected void fireAfterDataLoad() {
         super.fireAfterDataLoad();
-        CarouselStateManger.getInstance().countDown();
+        countDownLatch.countDown();
     }
 
     public void animate() throws InterruptedException {
-        CarouselStateManger.getInstance().countDownLatchAwait();
-        AtomicInteger integer = new AtomicInteger(showWidgets.length-1);
+        countDownLatch.await();
+        AtomicInteger integer = new AtomicInteger(showWidgets.length - 1);
         showCurrentLoadBlock(integer, widgetPane);
         this.repaint();
+        CarouselStateManger.getInstance().start();
         previewDialog.setVisible(true);
     }
 
+    private Image getPreviewImage(String url) throws NetWorkFailedException {
+        HttpClient httpClient = new HttpClient(url);
+        httpClient.setTimeout(CAROUSE_IMAGE_LOAD_TIMEOUT);
+        httpClient.asGet();
+        int responseCode = httpClient.getResponseCode();
+        if (responseCode != HttpURLConnection.HTTP_OK) {
+            throw new NetWorkFailedException();
+        }
+        try {
+            return ImageIO.read(httpClient.getResponseStream());
+        } catch (IOException e) {
+            throw new NetWorkFailedException();
+        }
+
+    }
+
     private void showCurrentLoadBlock(AtomicInteger integer, JPanel widgetPane) {
         ScheduledExecutorService service = createToastScheduleExecutorService();
         OnlineShareWidget shareWidget = showWidgets[integer.get()];
@@ -95,7 +118,13 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
         this.doLayout();
         this.validate();
         this.repaint();
-        previewDialog.setImage(getPreviewImage(shareWidget));
+        try {
+            previewDialog.setImage(getPreviewImage(shareWidget.getPicPath()), widgetPane.getLocationOnScreen());
+        } catch (NetWorkFailedException e) {
+            this.stopCarouse(integer, false);
+            this.switchPane(PaneStatue.DISCONNECTED);
+            return;
+        }
         //展示弹出框
         service.schedule(new Runnable() {
             @Override
@@ -114,6 +143,7 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
                 if (!CarouselStateManger.getInstance().isSuspend()) {
                     showCurrentLoadBlock(integer, widgetPane);
                 } else {
+                    previewDialog.setVisible(false);
                     pollingCarouselState(integer, widgetPane);
                 }
             }
@@ -130,6 +160,7 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
                     return;
                 }
                 if (!CarouselStateManger.getInstance().isSuspend()) {
+                    previewDialog.setVisible(true);
                     showCurrentLoadBlock(integer, widgetPane);
                     service.shutdown();
                 }
@@ -138,11 +169,18 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
     }
 
     private void stopCarouse(AtomicInteger integer) {
+        this.stopCarouse(integer, true);
+    }
+
+    private void stopCarouse(AtomicInteger integer, boolean showExtra) {
         previewDialog.setVisible(false);
+        if (showExtra) {
             loadRestShowWidgets(integer.get() - 1);
+        }
         java.awt.Toolkit.getDefaultToolkit().removeAWTEventListener(awtEventListener);
     }
 
+
     private void loadRestShowWidgets(int startIndex) {
         for (int i = startIndex; i >= 0; i--) {
             OnlineShareWidget shareWidget = showWidgets[i];
@@ -153,15 +191,6 @@ public class OnlineEmbedFilterSelectPane extends AbstractOnlineWidgetSelectPane
         this.repaint();
     }
 
-    public Image getPreviewImage(OnlineShareWidget widget) {
-        try {
-            return ImageIO.read(new URL(UriUtils.encodePath(widget.getPicPath(), EncodeConstants.ENCODING_UTF_8)));
-        } catch (IOException e) {
-            FineLoggerFactory.getLogger().error(e.getMessage(), e);
-            return ShareComponentConstants.DEFAULT_COVER;
-        }
-    }
-
     private ScheduledExecutorService createToastScheduleExecutorService() {
         return ModuleContext.getExecutor().newSingleThreadScheduledExecutor(new NamedThreadFactory(CAROUSEL_PREVIEW));
     }

designer-form/src/main/java/com/fr/design/mainframe/share/ui/online/embed/PreviewDialog.java

@@ -6,35 +6,33 @@ import com.fr.design.mainframe.EastRegionContainerPane;
 
 import javax.swing.ImageIcon;
 import javax.swing.JDialog;
+import java.awt.Dimension;
 import java.awt.Image;
+import java.awt.Point;
 
 
 /**
  * Created by kerry on 2021/10/22
  */
 public class PreviewDialog extends JDialog {
+    private static final int OFFSET_Y = 9;
 
     public PreviewDialog() {
         super(DesignerContext.getDesignerFrame());
         setUndecorated(true);
-        setSize(300, 300);
-        adjustLocation();
         this.setVisible(false);
     }
 
-    public void setImage(Image image) {
+    public void setImage(Image image, Point point) {
         this.getContentPane().removeAll();
         this.getContentPane().add(new UILabel(new ImageIcon(image)));
+        int width = image.getWidth(null);
+        this.setSize(width, image.getHeight(null));
+        this.setLocation(point.x - width, point.y + OFFSET_Y);
         this.doLayout();
+        
         this.validate();
         this.repaint();
     }
 
-    public void adjustLocation() {
-
-        this.setLocation(
-                EastRegionContainerPane.getInstance().getX() - 300,
-                20
-        );
-    }
 }