fix an issue with the offset meshing plugin (#8779)

Author: sloriot

Lab/demo/Lab/CGALlab_macros.cmake

@@ -63,30 +63,20 @@ add_dependencies(CGALlab_compile_all_plugins CGALlab_all_plugins)
     STRING(TOLOWER "${plugin_implementation_base_name}.json" base_name)
     SET(filename "${CMAKE_CURRENT_BINARY_DIR}/${base_name}")
     LIST(LENGTH ARG_KEYWORDS size)
+    SET(keywords "")
     if(${size} GREATER 0)
-      SET(keywords )
-      FILE(WRITE ${filename} "{ \"Keywords\" : [")
       foreach(keyword ${ARG_KEYWORDS})
-        LIST(APPEND keywords "\"${keyword}\", ")
+        SET(keywords "${keywords}\"${keyword}\", ")
         if(NOT TARGET ${keyword})
           add_custom_target(${keyword})
         endif()
         add_dependencies( ${keyword} ${plugin_name})
       endforeach()
-      LIST(LENGTH keywords size)
-      math(EXPR size "${size} - 1")
-      LIST(GET keywords -1 last_element)
-      LIST(REMOVE_AT keywords ${size})
-      STRING(LENGTH ${last_element} size)
-      math(EXPR size "${size} - 2")
-      STRING(SUBSTRING ${last_element} 0 ${size} last_element)
-      LIST(APPEND keywords ${last_element})
-      foreach(keyword ${keywords})
-        file(APPEND ${filename} ${keyword})
-      endforeach()
-      file(APPEND ${filename} "], \n")
-      string(TIMESTAMP VERSION "%Y-%m-%d %H:%M")
-      file(APPEND ${filename} "\"ConfigDate\" : \"${VERSION}\" }")
+      # Remove the last comma and space
+      string(REGEX REPLACE ", $" "" keywords "${keywords}")
     endif()
+    file(WRITE ${filename} "{\n  \"Keywords\" : [ ${keywords} ],\n")
+    string(TIMESTAMP VERSION "%Y-%m-%d %H:%M")
+    file(APPEND ${filename} "  \"ConfigDate\" : \"${VERSION}\"\n}\n")
     CGAL_install_hooks()
   endmacro(cgal_lab_plugin)

Lab/demo/Lab/MainWindow.cpp

@@ -565,7 +565,7 @@ bool MainWindow::load_plugin(QString fileName, bool blacklisted)
     }
     QDebug qdebug = qDebug();
     if(verbose)
-      qdebug << "### Loading \"" << fileName.toUtf8().data() << "\"... ";
+      qdebug << "### Loading" << fileName << "... ";
     QPluginLoader loader;
     loader.setFileName(fileinfo.absoluteFilePath());
     QJsonArray keywords = loader.metaData().value("MetaData").toObject().value("Keywords").toArray();
@@ -612,6 +612,8 @@ bool MainWindow::load_plugin(QString fileName, bool blacklisted)
     }
     else{
       pluginsStatus_map[name] = loader.errorString();
+      if(verbose)
+        qdebug << "\n#### Error: " << loader.errorString();
 
     }
     PathNames_map[name].push_back(fileinfo.absoluteDir().absolutePath());

Lab/demo/Lab/Plugins/Mesh_3/CMakeLists.txt

@@ -2,6 +2,8 @@ include(CGALlab_macros)
 
 remove_definitions(-DQT_STATICPLUGIN)
 
+set(CMAKE_AUTOMOC ON)
+
 qt6_wrap_cpp(VOLUME_MOC_OUTFILES
              ${CMAKE_CURRENT_SOURCE_DIR}/Volume_plane_thread.h)
 qt6_wrap_cpp(VOLUME_MOC_OUTFILES

Lab/demo/Lab/Plugins/Mesh_3/Offset_meshing_plugin.cpp

@@ -1,7 +1,5 @@
 #include "config.h"
 
-#ifdef CGAL_LAB_DEMO_USE_SURFACE_MESHER
-
 #include <CGAL/Three/CGAL_Lab_plugin_interface.h>
 #include "ui_Offset_meshing_dialog.h"
 
@@ -761,5 +759,3 @@ inflate_mesh()
 }
 
 #include "Offset_meshing_plugin.moc"
-
-#endif // CGAL_LAB_DEMO_USE_SURFACE_MESHER