Skip to content

Long Operations Improvements #44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
marioserrano09 merged 10 commits into from
Dec 28, 2025
Merged

Long Operations Improvements #44

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
b96af0e
add utility methods for scheduling tasks with ZK Timer component
marioserrano09 Dec 4, 2025
cb4714c
bump versions to 5.4.12 for tools.dynamia dependencies
marioserrano09 Dec 4, 2025
a8327e3
add WebSocket support with DemoWebSocketConfigurer; update pom.xml fo…
marioserrano09 Dec 8, 2025
edf15dd
enhance LongOperation and LongOperationMonitorWindow to support multi…
marioserrano09 Dec 24, 2025
79349df
add Randoms utility class for generating random numbers and strings
marioserrano09 Dec 24, 2025
0e9c959
update dependencies in pom.xml; add failureaccess library and bump sp…
marioserrano09 Dec 24, 2025
7ce5683
refactor: rename and enhance dynamia-tools.js with additional utility…
marioserrano09 Dec 28, 2025
2f778a9
refactor: replace ObjectMapper with JsonMapper for improved JSON hand…
marioserrano09 Dec 28, 2025
6f68f4c
update pom.xml to set tools.dynamia.commons and tools.dynamia.zk vers…
marioserrano09 Dec 28, 2025
8976cc3
add null check and trim for navigation path in BaseNavigationManager
marioserrano09 Dec 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions actions/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@
<parent>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.parent</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</parent>

<artifactId>tools.dynamia.actions</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
<name>DynamiaTools - Actions</name>
<url>https://dynamia.tools/docs/actions</url>

Expand Down Expand Up @@ -65,12 +65,12 @@
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.integration</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.commons</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>

</dependencies>
Expand Down
28 changes: 14 additions & 14 deletions app/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@
<parent>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.parent</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</parent>

<artifactId>tools.dynamia.app</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
<name>DynamiaTools - App</name>
<url>https://dynamia.tools/docs/app</url>

Expand Down Expand Up @@ -74,58 +74,58 @@
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.actions</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.commons</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.crud</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.domain</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.integration</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.io</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.navigation</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.reports</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.templates</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>

<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.viewers</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.web</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</dependency>
<dependency>
<groupId>org.springframework.data</groupId>
Expand Down Expand Up @@ -208,7 +208,7 @@
<dependency>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.domain.jpa</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
<scope>test</scope>
</dependency>
<dependency>
Expand Down
4 changes: 2 additions & 2 deletions app/src/main/java/tools/dynamia/app/controllers/CrudServiceRestController.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package tools.dynamia.app.controllers;

import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.databind.json.JsonMapper;
import io.swagger.v3.oas.annotations.tags.Tag;
import org.springframework.http.ResponseEntity;
import org.springframework.web.bind.annotation.*;
Expand Down Expand Up @@ -43,7 +43,7 @@ public class CrudServiceRestController {
/**
* JSON object mapper for entity serialization/deserialization.
*/
private final ObjectMapper mapper = StringPojoParser.createJsonMapper();
private final JsonMapper mapper = StringPojoParser.createJsonMapper();

/**
* Constructs a new {@code CrudServiceRestController} with the given CRUD service.
Expand Down
3 changes: 1 addition & 2 deletions commons/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,10 @@

<artifactId>tools.dynamia.commons</artifactId>
Copy link

Copilot AI Dec 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version number is missing from commons/pom.xml. While it may be inherited from the parent, it's inconsistent with the pattern used in other module pom.xml files where the version is explicitly declared. Consider adding the version tag for consistency.

Suggested change
<artifactId>tools.dynamia.commons</artifactId>
<artifactId>tools.dynamia.commons</artifactId>
<version>5.4.12</version>

Copilot uses AI. Check for mistakes.
<packaging>jar</packaging>
<version>5.4.11</version>
<parent>
<groupId>tools.dynamia</groupId>
<artifactId>tools.dynamia.parent</artifactId>
<version>5.4.11</version>
<version>5.4.12</version>
</parent>
<name>DynamiaTools - Commons</name>
<url>https://dynamia.tools/docs/common</url>
Expand Down
21 changes: 10 additions & 11 deletions commons/src/main/java/tools/dynamia/commons/StringPojoParser.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.core.type.TypeReference;
import com.fasterxml.jackson.databind.JavaType;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.fasterxml.jackson.databind.SerializationFeature;
import com.fasterxml.jackson.databind.json.JsonMapper;
import com.fasterxml.jackson.dataformat.xml.XmlMapper;
Expand Down Expand Up @@ -53,19 +52,19 @@ public static String convertMapToJson(Map map) {
if (map == null || map.isEmpty()) {
return "";
}
ObjectMapper jsonMapper = createJsonMapper();
var jsonMapper = createJsonMapper();
return jsonMapper.writeValueAsString(map);
} catch (JsonProcessingException e) {
throw new JsonParsingException(e);
}
}

/**
* Creates a configured JSON {@link ObjectMapper} with indentation, disabled empty beans, and JavaTimeModule support.
* Creates a configured JSON {@link JsonMapper} with indentation, disabled empty beans, and JavaTimeModule support.
*
* @return the configured JSON ObjectMapper
Copy link

Copilot AI Dec 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Javadoc still references 'ObjectMapper' when the return type has been changed to 'JsonMapper'. Update the Javadoc comment to accurately reflect the return type.

Suggested change
* @return the configured JSON ObjectMapper
* @return the configured JSON {@link JsonMapper}

Copilot uses AI. Check for mistakes.
*/
public static ObjectMapper createJsonMapper() {
public static JsonMapper createJsonMapper() {
return JsonMapper.builder()
.enable(SerializationFeature.INDENT_OUTPUT)
.disable(SerializationFeature.FAIL_ON_EMPTY_BEANS)
Expand All @@ -85,7 +84,7 @@ public static String convertPojoToJson(Object pojo) {
if (pojo == null) {
return "";
}
ObjectMapper jsonMapper = createJsonMapper();
var jsonMapper = createJsonMapper();
return jsonMapper.writeValueAsString(pojo);
} catch (JsonProcessingException e) {
throw new JsonParsingException(e);
Expand Down Expand Up @@ -141,7 +140,7 @@ public static <T> T parseJsonToPojo(String json, Class<T> pojoType) {
return null;
}

ObjectMapper jsonMapper = createJsonMapper();
var jsonMapper = createJsonMapper();
return jsonMapper.readerFor(pojoType).readValue(json);
} catch (IOException e) {
throw new JsonParsingException(e);
Expand All @@ -161,7 +160,7 @@ public static <T> T parseJsonToPojo(Map map, Class<T> pojoType) {
return null;
}

ObjectMapper jsonMapper = createJsonMapper();
var jsonMapper = createJsonMapper();
return jsonMapper.convertValue(map, pojoType);
} catch (IllegalArgumentException e) {
throw new JsonParsingException(e);
Expand All @@ -184,12 +183,12 @@ public static String convertPojoToXml(Object pojo) {
}

/**
* Create a xml {@link ObjectMapper} with enable IDENT_OUTPUT and disabled FAIL_ON_EMPTY_BEANS. Also add support
* Create a xml {@link XmlMapper} with enable IDENT_OUTPUT and disabled FAIL_ON_EMPTY_BEANS. Also add support
* to {@link JavaTimeModule} from JSR310 dependency
*
* @return xml ObjectMapper
* @return xml mapper
Copy link

Copilot AI Dec 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Javadoc incorrectly describes the return type as 'xml ObjectMapper' when it should be 'XmlMapper'. The return type in the method signature has been updated to XmlMapper, but the Javadoc comment should reflect this more accurately.

Suggested change
* @return xml mapper
* @return configured {@link XmlMapper} instance

Copilot uses AI. Check for mistakes.
*/
public static ObjectMapper createXmlMapper() {
public static XmlMapper createXmlMapper() {
return XmlMapper.builder()
.enable(SerializationFeature.INDENT_OUTPUT)
.disable(SerializationFeature.FAIL_ON_EMPTY_BEANS)
Expand Down Expand Up @@ -244,7 +243,7 @@ public static <T> String convertListToJson(List<T> list) {
if (list == null || list.isEmpty()) {
return "";
}
ObjectMapper jsonMapper = createJsonMapper();
var jsonMapper = createJsonMapper();
return jsonMapper.writeValueAsString(list);
} catch (JsonProcessingException e) {
throw new JsonParsingException(e);
Expand Down
Loading